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
allow to disable apps building #21212
Conversation
Any new option should be documented in INSTALL.md |
Fixed. |
We would need a run-checker target for that no-apps build option and I assume you'll find that many tests will fail without apps so you'll need to fix those failures, which will be a quite tedious patch to do. |
Nope. "no-apps" cascades to "no-tests" so the tests simply don't run at all. |
Ah, right. That makes the option pretty limiting. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This pull request is ready to merge |
Squashed and merged to master branch. Thank you for your contribution. |
Reviewed-by: Matt Caswell <matt@openssl.org> Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from #21212)
This trivial change allows to disable apps building. It's handy for minimization.
The option was already there and adhered to by
apps/build.info
, however Configure did not allow to use it directly, only via cascading.