-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
configure is slow #634
Comments
Looking at the output, it would appear that it is the recursive Breaking it down on my MacBook Pro, I find:
|
Thanks! Renamed |
@woody-apple is bootstrap.sh in the run referenced above using |
nevermind. got it |
It's 20seconds faster, but still a minute for the 'bootstrap' phase. Any more things we can pull here? |
This also actually breaks the build: |
next biggest chunk of optimization of configure would come from taking nlfriends out of the tree, making them an external dependency like openssl this would also remove nlfriends from |
I agree that running configure of each of the nlfriends is taking the majority of time. And for nlfriends such as nlassert and nlio, which primarily just provide headers (outside of unit tests), this time seems especially wasteful. In a cmake prototype I've been experimenting with, simply hooking in configure of third_party/nlfriends increased initial build time from ~2s to >20s. |
bootstrap takes 56 seconds, vs build takes 17
https://app.circleci.com/pipelines/github/woody-apple/connectedhomeip/159/workflows/4e720f20-aa33-48ca-baa9-84005e6664f5/jobs/846
Originally posted by @woody-apple in #631 (comment)
The text was updated successfully, but these errors were encountered: