8000 Allow compiling only the parts you care about by ktf · Pull Request #5888 · alisw/alidist · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Allow compiling only the parts you care about #5888

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ktf
Copy link
Member
@ktf ktf commented May 21, 2025

E.g.

O2PHYSICS_COMPONENTS=Tutorials alibuild build --defaults o2 O2Physics

Will only compile the tutorial. Changing O2PHYSICS_COMPONENTS to something else will also trigger a rebuild if alisw/alibuild#927 is used.

@ktf ktf requested a review from a team as a code owner May 21, 2025 09:33
@ktf
Copy link
Member Author
ktf commented May 21, 2025

@aalkin this is what we discussed, together with alisw/alibuild#927 it should allow people to only compile what they need (and rebuild in case they change their mind). We can also think of some syntactic sugar like:

alibuild build --defaults o2 O2Physics#Tutorials

which expands to correctly. In a future update we could limit what we compile in O2, however at the moment we export everything in O2Targets and then CMake complains if (say) I only try to build the framework and then O2Physics.

@ddobrigk We should probably integrate this feature for the next tutorial, so that people rebuild only a minimal part of O2Physics.

E.g.

O2PHYSICS_COMPONENTS=Tutorials alibuild build --defaults o2 O2Physics

Will only compile the tutorial. Changing O2PHYSICS_COMPONENTS to something else will also trigger a rebuild if alisw/alibuild#927 is used.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0