-
Notifications
You must be signed in to change notification settings - Fork 2k
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
option to disable building libraries (zcutil/build.sh) #2420
Conversation
👍 My nightly build has at least a thousand |
@aayanl That's not what this patch does; it only affects building output libraries, not dependencies.
utACK. @zkbot try |
option to disable building libraries (zcutil/build.sh) option to disable building libraries (zcutil/build.sh)
☀️ Test successful - pr-try |
Needs another ACK + smoke-testing the case where the option is set. |
Note that this is just exposing an existing option in the Makefiles via build.sh. |
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.
utACK. I'm personally against adding yet more flags to build.sh
; I'd prefer the approach I take in #2120 where we use an environment variable for configure flags vs. program arguments for make. But for now I'm happy to enable better use cases for building.
@zkbot r+ |
📌 Commit c63417b has been approved by |
option to disable building libraries (zcutil/build.sh) option to disable building libraries (zcutil/build.sh)
option to disable building libraries (zcutil/build.sh)