As discussed with @laanwj on IRC. Fixes #5797. This is a hammer approach, but it's the most straightforward.
strnlen is available at build-time but not at runtime, causing a crash.
0.11 drops support for 10.6, so this is not needed in master.
0.10 fix for crashes on OSX 10.6 #5926
pull theuni wants to merge 1 commits into bitcoin:0.10 from theuni:osx-strnlen changing 3 files +5 −8-
theuni commented at 4:27 AM on March 19, 2015: member
-
8752b5c882
0.10 fix for crashes on OSX 10.6
strnlen is available at build-time but not at runtime, causing a crash. 0.11 drops support for 10.6, so this is not needed in master.
-
paveljanik commented at 6:32 AM on March 19, 2015: contributor
utACK
-
fanquake commented at 6:34 AM on March 19, 2015: member
@ericpashman @Pyanne Are you able to test this on 10.6? @jonasschnelli It might help if you spin up some test binaries incase they can't build from source.
utACK for 0.10 branch
-
laanwj commented at 6:37 AM on March 19, 2015: member
utACK for 0.10 branch, I'm glad we don't need it on master
- laanwj merged this on Mar 19, 2015
- laanwj closed this on Mar 19, 2015
- laanwj referenced this in commit 10c09f98b2 on Mar 19, 2015
-
jonasschnelli commented at 7:15 AM on March 19, 2015: contributor
@fanquake currently i have no easy option to build a non-master branch... will update my script soon and try to provide binaries.
-
jonasschnelli commented at 8:45 AM on March 19, 2015: contributor
-
jonasschnelli commented at 8:47 AM on March 19, 2015: contributor
Did a quick test on 10.7 and 10.10. Runs as expected. I don't have a 10.6 VM anymore.
-
jgarzik commented at 9:11 AM on March 19, 2015: contributor
sad face ACK
-
ericpashman commented at 4:58 PM on March 19, 2015: none
Tested on 10.6. All is well.
-
laanwj commented at 4:59 PM on March 19, 2015: member
Thanks for testing!
- DrahtBot locked this on Sep 8, 2021
