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
particl-core: 0.16.2.0 -> 0.17.0.2 #48836
Conversation
Update version. Disabled tests and benchmartk to speed up Hydra build.
I agree that running benchmarks during Hydra build doesn't make much sense. But I suggest to keep the tests enabled unless they take so much time that Hydra builds time out. Is that the case here? |
configureFlags = [ "--with-boost-libdir=${boost.out}/lib" ]; | ||
configureFlags = [ | ||
"--disable-bench" | ||
"--enable-tests=no" |
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.
Did we run the tests before?
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.
I think this flag only to build tests, and it was never used (not sure about it).
I trust @tecnovert (particl-core developer) to run all tests before publishing release.
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.
I added doCheck
that run tests, also running tests required python3 which was added: 233dad8
Yes, it takes time. I trying to reduce build time as much as possible, because in parallel PR for particl-qt all builds timed out (even on x86_64): #46760 I have no clue how to speed-up compilation or prevent Hydra from timeout... |
ofborg timeouts (like you see in #46760) and hydra timeouts are two different things. Most builds that time out on ofborg will succeed on hydra, so usually if a build times out on ofborg we merge anyway and see what happens on hydra. I do not see any recent hydra timeouts for But what you can try is |
Build now dependent on Python3 and current version of Boost
Didn't know that, thanks for clarification @xeji. Added this tweak and enable tests. |
@GrahamcOfBorg build particl-core |
Success on aarch64-linux (full log) Attempted: particl-core Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: particl-core Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: particl-core Partial log (click to expand)
|
Motivation for this change
Update version. Disabled tests and benchmartk to speed up Hydra build.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)