Skip to content
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

Only add subdirectories if necessary #101

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

Conversation

skitt
Copy link
Contributor

@skitt skitt commented Jul 8, 2020

If the build is using system libraries (for bzip2, lua, ode, or
xdgbasedir), there's no need to add the corresponding subdirectories.
This reduces build time and avoids potential mix-ups.

Signed-off-by: Stephen Kitt steve@sk2.org

@skitt skitt marked this pull request as draft July 8, 2020 18:07
@Nikekson
Copy link
Member

Nikekson commented Jul 8, 2020

I don't think this will work since the the USE_* variables use generator expressions and they get evaluated at build time, whereas the add_subdirectory calls are run at configure time

@skitt skitt force-pushed the conditional-subdirectories branch from 4c231d1 to 0baed68 Compare July 8, 2020 19:25
@skitt
Copy link
Contributor Author

skitt commented Jul 8, 2020

I don't think this will work since the the USE_* variables use generator expressions and they get evaluated at build time, whereas the add_subdirectory calls are run at configure time

It turns out that using explicit variable evaluations (${}) works :-).

@skitt skitt marked this pull request as ready for review July 8, 2020 19:31
@skitt skitt force-pushed the conditional-subdirectories branch from 0baed68 to a822826 Compare July 8, 2020 19:37
If the build is using system libraries (for bzip2, lua, ode, or
xdgbasedir), there's no need to add the corresponding subdirectories.
This reduces build time and avoids potential mix-ups.

Signed-off-by: Stephen Kitt <steve@sk2.org>
@skitt skitt force-pushed the conditional-subdirectories branch from a822826 to 112e33f Compare July 20, 2020 07:20
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.

None yet

2 participants