-
Notifications
You must be signed in to change notification settings - Fork 6
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
cmake: -logsourcelocations file name differences #297
Comments
Looks like that is also introducing a source of non-determinism (for non-guix build binaries)? |
Note: I haven't tried guix. Those are the CI-generated binaries. I'll try guix next week, but I expect the result to be the same. |
cmake is probably missing the prefix maps: https://stackoverflow.com/a/53848526/2084795 |
Seems our usage in configure (when using ccache) just wasn't ported over. |
Thanks! Fixed in #298. |
Testing now, but it still fails. Please re-open. |
Master:
Cmake:
|
Re-opened. |
Fixed in #315. |
Checked that guix builds work fine now:
|
Is this intentional?
autotools:
[init] [wallet/init.cpp:129] [Construct] Wallet disabled!
[init] [wallet/init.cpp:129] [Construct] Wallet disabled!
[init] [D:\a\bitcoin-core-with-ci\bitcoin-core-with-ci\src\wallet\init.cpp:129] [Construct] Wallet disabled!
Cmake:
[init] [/home/runner/work/_temp/src/wallet/init.cpp:129] [Construct] Wallet disabled!
[init] [/Users/runner/work/bitcoin-core-with-ci/bitcoin-core-with-ci/src/wallet/init.cpp:129] [Construct] Wallet disabled!
[init] [D:\a\bitcoin-core-with-ci\bitcoin-core-with-ci\src\wallet\init.cpp:129] [Construct] Wallet disabled!
The text was updated successfully, but these errors were encountered: