💬 sarthak13gupta commented on issue "Not able to build and compile on macos":
(https://github.com/bitcoin/bitcoin/issues/29083#issuecomment-1856510625)
Yes I have tried that.

💬 maflcko commented on issue "Not able to build and compile on macos":
(https://github.com/bitcoin/bitcoin/issues/29083#issuecomment-1856514195)
What is your operating system and architecture? Where did you get BDB from?

💬 maflcko commented on issue "Not able to build and compile on macos":
(https://github.com/bitcoin/bitcoin/issues/29083#issuecomment-1856514918)
Do you need BDB? If not, you can disable it with `--without-bdb`.

💬 hebasto commented on issue "Not able to build and compile on macos":
(https://github.com/bitcoin/bitcoin/issues/29083#issuecomment-1856517075)
What is the `./configure` script summary?

💬 maflcko commented on pull request "fuzz: Improve fuzzing stability for minisketch harness":
(https://github.com/bitcoin/bitcoin/pull/29064#issuecomment-1856517924)
review ACK b2fc7a2eda103724ac8cbeaf99df3ce6f5b7d974

🚀 achow101 merged a pull request: "Get rid of shutdown.cpp/shutdown.h, use SignalInterrupt directly"
(https://github.com/bitcoin/bitcoin/pull/28051)

👋 dergoegge's pull request is ready for review: "fuzz: Improve fuzzing stability for minisketch harness"
(https://github.com/bitcoin/bitcoin/pull/29064)

✅ LeoSpyke closed an issue: "test: `feature_index_prune.py` timeout failure"
(https://github.com/bitcoin/bitcoin/issues/29082)

💬 LeoSpyke commented on issue "test: `feature_index_prune.py` timeout failure":
(https://github.com/bitcoin/bitcoin/issues/29082#issuecomment-1856519929)
Setting `--timeout-factor=2` did the trick.
Thank you @maflcko.

⚠️ hebasto opened an issue: "`security-check.py` produces false positive result for stack smashing protection"
(https://github.com/bitcoin/bitcoin/issues/29084)
I'm aware that `test-security-check.py` tests both cases `-fno-stack-protector` and `-fstack-protector-all`.

However, the `security-check.py` passes in the master branch @ 986047170892c9482ccbc21f05bf4f1499b3089d with the diff as follows:
```diff
--- a/configure.ac
+++ b/configure.ac
@@ -935,7 +935,7 @@ fi
if test "$use_hardening" != "no"; then
use_hardening=yes
AX_CHECK_COMPILE_FLAG([-Wstack-protector], [HARDENED_CXXFLAGS="$HARDENED_CXXFLAGS -Wstack-protector"])
- AX_CHECK_CO
...

💬 fanquake commented on issue "`security-check.py` produces false positive result for stack smashing protection":
(https://github.com/bitcoin/bitcoin/issues/29084#issuecomment-1856530747)
Probably because our GCC in guix is passing stack protection flags by default?

💬 hebasto commented on issue "`security-check.py` produces false positive result for stack smashing protection":
(https://github.com/bitcoin/bitcoin/issues/29084#issuecomment-1856532058)
> Probably because our GCC in guix is passing stack protection flags by default?

Shouldn't `-fno-stack-protector` override compiler's defaults?

💬 hebasto commented on issue "`security-check.py` produces false positive result for stack smashing protection":
(https://github.com/bitcoin/bitcoin/issues/29084#issuecomment-1856533170)
> Probably because our GCC in guix is passing stack protection flags by default?

Why does the `test-security-check.py` pass then?

💬 jonatack commented on issue "Not able to build and compile on macos":
(https://github.com/bitcoin/bitcoin/issues/29083#issuecomment-1856534079)
Can also try `make distclean` and then rebuild.

If that doesn't work, or if you are seeing linker issues, the header and compilation might be out of sync and you may want to ensure that you are building from a clean tree. If you are using ccache, try clearing the cache by running `ccache -C` (see ccache -h for help) and then rebuilding.

If that doesn't solve it, you can also run `git clean -f -x -d` and rebuild. This wipes everything in the tree that doesn't belong to git, so be careful th
...

💬 fanquake commented on issue "`security-check.py` produces false positive result for stack smashing protection":
(https://github.com/bitcoin/bitcoin/issues/29084#issuecomment-1856538009)
Probably because the toolchain is built hardened, so any test bin still has the stack protection symbol coming from glibc or similar.

💬 sarthak13gupta commented on issue "Not able to build and compile on macos":
(https://github.com/bitcoin/bitcoin/issues/29083#issuecomment-1856542633)
> What is your operating system and architecture? Where did you get BDB from?

MacOS 13.0 Arm64 architechture . I tried installing BDB using homebrew while following the docs -> https://github.com/bitcoin/bitcoin/blob/master/doc/build-osx.md

👍 stickies-v approved a pull request: "refactor: Remove pre-C++20 code, fs::path cleanup"
(https://github.com/bitcoin/bitcoin/pull/29040#pullrequestreview-1782682980)
re-ACK 66667130416b86208e01a0eb5541a15ea805ac26

💬 maflcko commented on issue "Not able to build and compile on macos":
(https://github.com/bitcoin/bitcoin/issues/29083#issuecomment-1856548573)
> MacOS 13.0 Arm64 architechture . I tried installing BDB using homebrew while following the docs -> https://github.com/bitcoin/bitcoin/blob/master/doc/build-osx.md

Can you try to re-install it, and check that it installs it for arm64? Otherwise, if you don't need it, you can disable it by passing `--without-bdb` to `./configure`.

💬 sarthak13gupta commented on issue "Not able to build and compile on macos":
(https://github.com/bitcoin/bitcoin/issues/29083#issuecomment-1856555215)
> What is the `./configure` script summary?

I don't exactly understand it but this is what i got after running the ./configure command.
https://gist.github.com/sarthak13gupta/6ab7cf9b402a3695b20da574a5957804

💬 murchandamus commented on pull request "fuzz: coinselection, improve `min_viable_change`/`change_output_size`":
(https://github.com/bitcoin/bitcoin/pull/28372#issuecomment-1856559034)
> ```diff
> @@ -128,7 +128,7 @@ FUZZ_TARGET(coinselection)
> auto result_bnb = coin_params.m_subtract_fee_outputs ? util::Error{Untranslated("BnB disabled when SFFO is enabled")} :
> SelectCoinsBnB(group_pos, target, coin_params.m_cost_of_change, MAX_STANDARD_TX_WEIGHT);
> if (result_bnb) {
> - assert(result_bnb->GetChange(coin_params.m_cost_of_change, CAmount{0}) == 0);
> + assert(result_bnb->GetChange(coin_params.min_viable_change, coin_para
...