💬 fanquake commented on issue "./contrib/guix/guix-build does not work on riscv64":
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845149984)
> The guix version is 1.4, but that shouldn't matter, because of the time-machine.
I'm not sure we should make that assumption here. If the version of Guix (1.4), can't properly setup containers on the system (which looks like it could be the issue), and the fixes only landed after 1.4, then builds will still fail. It would be good to get a smaller reproducer of the issue. Does something like `guix shell --container --pure some_package` work?
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845149984)
> The guix version is 1.4, but that shouldn't matter, because of the time-machine.
I'm not sure we should make that assumption here. If the version of Guix (1.4), can't properly setup containers on the system (which looks like it could be the issue), and the fixes only landed after 1.4, then builds will still fail. It would be good to get a smaller reproducer of the issue. Does something like `guix shell --container --pure some_package` work?
💬 maflcko commented on pull request "wip: Split fuzz binary":
(https://github.com/bitcoin/bitcoin/pull/29010#discussion_r1418793132)
> Building and linking just one harness is faster than all of them, maybe useful when reproducing testcases?
Not sure. This isn't supported for any other tests (bench, unit, ...), so maybe leave as a follow-up.
If building takes a long time, you can use more CPUs or a populated `ccache`.
Also, it could be done easier inside the makefile by just skipping over the other files, if needed?
(https://github.com/bitcoin/bitcoin/pull/29010#discussion_r1418793132)
> Building and linking just one harness is faster than all of them, maybe useful when reproducing testcases?
Not sure. This isn't supported for any other tests (bench, unit, ...), so maybe leave as a follow-up.
If building takes a long time, you can use more CPUs or a populated `ccache`.
Also, it could be done easier inside the makefile by just skipping over the other files, if needed?
💬 maflcko commented on issue "./contrib/guix/guix-build does not work on riscv64":
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845156614)
```
# guix shell --container --pure hello
accepted connection from pid 3269077, user root
Backtrace:
19 (apply-smob/0 #<thunk 3f887712e0>)
In ice-9/boot-9.scm:
724:2 18 (call-with-prompt _ _ #<procedure default-prompt-handle?>)
In ice-9/eval.scm:
619:8 17 (_ #(#(#<directory (guile-user) 3f88776c80>)))
In guix/ui.scm:
2275:7 16 (run-guix . _)
2238:10 15 (run-guix-command _ . _)
In ice-9/boot-9.scm:
1752:10 14 (with-exception-handler _ _ #:unwind? _ # _)
1
...
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845156614)
```
# guix shell --container --pure hello
accepted connection from pid 3269077, user root
Backtrace:
19 (apply-smob/0 #<thunk 3f887712e0>)
In ice-9/boot-9.scm:
724:2 18 (call-with-prompt _ _ #<procedure default-prompt-handle?>)
In ice-9/eval.scm:
619:8 17 (_ #(#(#<directory (guile-user) 3f88776c80>)))
In guix/ui.scm:
2275:7 16 (run-guix . _)
2238:10 15 (run-guix-command _ . _)
In ice-9/boot-9.scm:
1752:10 14 (with-exception-handler _ _ #:unwind? _ # _)
1
...
💬 fanquake commented on issue "./contrib/guix/guix-build does not work on riscv64":
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845157887)
Ok. So the issue is a bug in Guix. How many weeks will it take to run a `guix pull`?
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845157887)
Ok. So the issue is a bug in Guix. How many weeks will it take to run a `guix pull`?
💬 maflcko commented on issue "./contrib/guix/guix-build does not work on riscv64":
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845164211)
Happy to try it, but if that works, then there is a bug in the time-machine, which seems unlikely.
```
# guix time-machine --commit=77386bdbfe6b0c649c05ab37f08051d1ab3e5074 -- shell --container --pure bash
accepted connection from pid 3269629, user root
accepted connection from pid 3269629, user root
Backtrace:
18 (primitive-load "/root/.cache/guix/inferiors/4wrulds2lx?")
In guix/ui.scm:
2324:7 17 (run-guix . _)
2287:10 16 (run-guix-command _ . _)
In ice-9/boot-9.scm:
...
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845164211)
Happy to try it, but if that works, then there is a bug in the time-machine, which seems unlikely.
```
# guix time-machine --commit=77386bdbfe6b0c649c05ab37f08051d1ab3e5074 -- shell --container --pure bash
accepted connection from pid 3269629, user root
accepted connection from pid 3269629, user root
Backtrace:
18 (primitive-load "/root/.cache/guix/inferiors/4wrulds2lx?")
In guix/ui.scm:
2324:7 17 (run-guix . _)
2287:10 16 (run-guix-command _ . _)
In ice-9/boot-9.scm:
...
💬 Sjors commented on pull request "wallet: rpc to add automatically generated descriptors":
(https://github.com/bitcoin/bitcoin/pull/25907#issuecomment-1845165287)
You could also test `createwalletdescriptor` by creating a descriptor wallet using a pre-taproot version. You can check the xpub in the resulting descriptor using #22341. That way all the key rotation stuff and the new records can be done in another PR.
(https://github.com/bitcoin/bitcoin/pull/25907#issuecomment-1845165287)
You could also test `createwalletdescriptor` by creating a descriptor wallet using a pre-taproot version. You can check the xpub in the resulting descriptor using #22341. That way all the key rotation stuff and the new records can be done in another PR.
💬 fanquake commented on issue "./contrib/guix/guix-build does not work on riscv64":
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845167698)
> then there is a bug in the time-machine, which seems unlikely.
I don't follow. This is failing in the same way, because time-machine is trying to use a container. So it is the same issue.
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845167698)
> then there is a bug in the time-machine, which seems unlikely.
I don't follow. This is failing in the same way, because time-machine is trying to use a container. So it is the same issue.
📝 kashifs opened a pull request: "Make bitcoin-tx replaceable value optional, #fixes 28638"
(https://github.com/bitcoin/bitcoin/pull/29022)
<!--
*** Please remove the following help text before submitting: ***
Pull requests without a rationale and clear improvement may be closed
immediately.
GUI-related pull requests should be opened against
https://github.com/bitcoin-core/gui
first. See CONTRIBUTING.md
-->
<!--
Please provide clear motivation for your patch and explain how it improves
Bitcoin Core user experience or Bitcoin Core developer experience
significantly:
* Any test improvements or new tests that improv
...
(https://github.com/bitcoin/bitcoin/pull/29022)
<!--
*** Please remove the following help text before submitting: ***
Pull requests without a rationale and clear improvement may be closed
immediately.
GUI-related pull requests should be opened against
https://github.com/bitcoin-core/gui
first. See CONTRIBUTING.md
-->
<!--
Please provide clear motivation for your patch and explain how it improves
Bitcoin Core user experience or Bitcoin Core developer experience
significantly:
* Any test improvements or new tests that improv
...
💬 maflcko commented on issue "./contrib/guix/guix-build does not work on riscv64":
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845171572)
I mean that `guix pull` likely won't fix this issue, because if this issue was fixed in a later version of guix, I could simply use the time-machine to run that version of guix. However, that didn't work.
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845171572)
I mean that `guix pull` likely won't fix this issue, because if this issue was fixed in a later version of guix, I could simply use the time-machine to run that version of guix. However, that didn't work.
💬 fanquake commented on issue "./contrib/guix/guix-build does not work on riscv64":
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845172844)
> I could simply use the time-machine to run that version of guix
You can't though. Because the guix binary itself, that is trying to run the container, for the time-machine, doesn't have the bug fixes from the future.
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845172844)
> I could simply use the time-machine to run that version of guix
You can't though. Because the guix binary itself, that is trying to run the container, for the time-machine, doesn't have the bug fixes from the future.
💬 Sjors commented on pull request "wallet: Have the wallet store the key for automatically generated descriptors":
(https://github.com/bitcoin/bitcoin/pull/26728#issuecomment-1845173418)
As much as I don't like having to go through half the PR again, I do think this is a nice simplification.
(https://github.com/bitcoin/bitcoin/pull/26728#issuecomment-1845173418)
As much as I don't like having to go through half the PR again, I do think this is a nice simplification.
💬 maflcko commented on issue "./contrib/guix/guix-build does not work on riscv64":
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845190697)
Are you sure, because if new guix features are available in the the time-machine, then bug fixes should be as well?
Looking at the code, it seems more likely that someone forgot to add support for riscv64 somewhere, because the code looks mostly unchanged between 1.4 and master.
https://github.com/guix-mirror/guix/blame/v1.4.0/guix/build/syscalls.scm#L1109
https://github.com/guix-mirror/guix/blame/77386bdbfe6b0c649c05ab37f08051d1ab3e5074/guix/build/syscalls.scm#L1114
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845190697)
Are you sure, because if new guix features are available in the the time-machine, then bug fixes should be as well?
Looking at the code, it seems more likely that someone forgot to add support for riscv64 somewhere, because the code looks mostly unchanged between 1.4 and master.
https://github.com/guix-mirror/guix/blame/v1.4.0/guix/build/syscalls.scm#L1109
https://github.com/guix-mirror/guix/blame/77386bdbfe6b0c649c05ab37f08051d1ab3e5074/guix/build/syscalls.scm#L1114
💬 fanquake commented on issue "./contrib/guix/guix-build does not work on riscv64":
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845195847)
> Are you sure, because if new guix features are available in the the time-machine, then bug fixes should be as well?
Yea, new Guix features could be compiled/available inside the time-machine, but if you can't even run a time-machine, because your guix 1.4 binary can't launch a container, you are still stuck.
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845195847)
> Are you sure, because if new guix features are available in the the time-machine, then bug fixes should be as well?
Yea, new Guix features could be compiled/available inside the time-machine, but if you can't even run a time-machine, because your guix 1.4 binary can't launch a container, you are still stuck.
💬 Sjors commented on pull request "wallet: Have the wallet store the key for automatically generated descriptors":
(https://github.com/bitcoin/bitcoin/pull/26728#discussion_r1418839247)
You could create a pre-v22 wallet, upgrade to master (which won't add the `tr()` descriptor) and then continue the test.
Although I don't think we care that deeply about downgrading that far, testing a wallet without a `tr()` descriptor is useful - for followup PR's that enable adding that desciptor.
(https://github.com/bitcoin/bitcoin/pull/26728#discussion_r1418839247)
You could create a pre-v22 wallet, upgrade to master (which won't add the `tr()` descriptor) and then continue the test.
Although I don't think we care that deeply about downgrading that far, testing a wallet without a `tr()` descriptor is useful - for followup PR's that enable adding that desciptor.
💬 maflcko commented on issue "./contrib/guix/guix-build does not work on riscv64":
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845200103)
Ah ok. I forgot to clarify, the time-machine itself is working (so far). For example:
```
# guix time-machine --commit=77386bdbfe6b0c649c05ab37f08051d1ab3e5074 -- shell --pure hello
accepted connection from pid 3274252, user root
hint: Consider passing the `--check' option once to make sure your shell does not clobber environment variables.
# hello
Hello, world!
# exit
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845200103)
Ah ok. I forgot to clarify, the time-machine itself is working (so far). For example:
```
# guix time-machine --commit=77386bdbfe6b0c649c05ab37f08051d1ab3e5074 -- shell --pure hello
accepted connection from pid 3274252, user root
hint: Consider passing the `--check' option once to make sure your shell does not clobber environment variables.
# hello
Hello, world!
# exit
💬 fanquake commented on issue "./contrib/guix/guix-build does not work on riscv64":
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845213666)
Ok, so then this seems to be a bug with just the usage `--container`, (on riscv64 hardware?) that likely still exists in Guix. We should report the minimal reproducer upstream.
(https://github.com/bitcoin/bitcoin/issues/29020#issuecomment-1845213666)
Ok, so then this seems to be a bug with just the usage `--container`, (on riscv64 hardware?) that likely still exists in Guix. We should report the minimal reproducer upstream.
📝 fanquake opened a pull request: "doc: add historical release notes for 26.0"
(https://github.com/bitcoin/bitcoin/pull/29023)
Bins are now up, used for GH release etc.
(https://github.com/bitcoin/bitcoin/pull/29023)
Bins are now up, used for GH release etc.
💬 kristapsk commented on pull request "wallet: No BDB creation, unless -deprecatedrpc=create_bdb":
(https://github.com/bitcoin/bitcoin/pull/28597#issuecomment-1845254045)
This should have had release notes.
(https://github.com/bitcoin/bitcoin/pull/28597#issuecomment-1845254045)
This should have had release notes.
💬 sgs2018 commented on pull request "datacarriersize: Match more datacarrying":
(https://github.com/bitcoin/bitcoin/pull/28408#issuecomment-1845257078)
maybe is hard to define spam ,
it is very fuzzy .
(https://github.com/bitcoin/bitcoin/pull/28408#issuecomment-1845257078)
maybe is hard to define spam ,
it is very fuzzy .
💬 willcl-ark commented on pull request "doc: add historical release notes for 26.0":
(https://github.com/bitcoin/bitcoin/pull/29023#issuecomment-1845259000)
We might want to add a release note for `-deprecatedrpc=create_bdb` as per : https://github.com/bitcoin/bitcoin/pull/28597#issuecomment-1845254045
(https://github.com/bitcoin/bitcoin/pull/29023#issuecomment-1845259000)
We might want to add a release note for `-deprecatedrpc=create_bdb` as per : https://github.com/bitcoin/bitcoin/pull/28597#issuecomment-1845254045