Skip to content

Build recipes from a shared directory [DRAFT] #195

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from

Conversation

agriyakhetarpal
Copy link
Member

No description provided.

@agriyakhetarpal agriyakhetarpal added the do not merge This awaits a decision before it can be merged label Apr 15, 2025
@agriyakhetarpal agriyakhetarpal force-pushed the build-from-shared-directory branch from e224905 to 590672b Compare April 15, 2025 14:16
@agriyakhetarpal
Copy link
Member Author

Ah, never mind – in its current state, it extracts another directory under recipes/numpy_shared_build/numpy-tests-2.2.3/, which evades the caching we need. I will need to revise it a bit.

@ryanking13
Copy link
Member

Thanks @agriyakhetarpal. Please ping me when it is ready.

I wonder if we can utilize source.path field for this, by the way. It is used to building packages using the local files instead of downloading them from URL.

It was used for 1) testing purpose, 2) and when micropip was inside pyodide/pyodide, but I guess we can use it for sharing the build directories as well. For example, numpy packages downloads the archive from URL, and numpy-tests sets the path to the numpy's build directory.

Of course, this is a hack, and I am not sure it is good for the long-term solution, but I think is can be considered as an option.

@agriyakhetarpal
Copy link
Member Author

This will be superseded by pyodide/pyodide-recipes#86, and should no longer be needed.

@agriyakhetarpal agriyakhetarpal deleted the build-from-shared-directory branch June 24, 2025 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge This awaits a decision before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants