build: Solve SmartOS FD_ZERO build issue #15146

1. 
   Empact commented at 2:40 am on January 11, 2019: member

   SmartOS FD_ZERO is implemented in a way that requires an external declaration of memcpy. We can not simply include cstring in the existing file because sanity_test_memcpy is attempting to replace memcpy.

   Instead split glibc_sanity into fdelt and memcpy files, and include in glibc_sanity/fdelt.cpp.

   Fixes #13581, see also #13619

2. fanquake added the label Build system on Jan 11, 2019
3. Empact force-pushed on Jan 11, 2019
4. Empact force-pushed on Jan 11, 2019
5. 
   laanwj commented at 10:31 am on January 11, 2019: member

   Concept ACK
6. MarcoFalke added the label Needs gitian build on Jan 11, 2019
7. DrahtBot removed the label Needs gitian build on Jan 12, 2019
8. 
   laanwj commented at 11:47 am on January 16, 2019: member

   @theuni can you take a look please
9. Empact force-pushed on Jan 16, 2019
10. MarcoFalke deleted a comment on Feb 14, 2019
11. 
    DrahtBot commented at 4:57 pm on February 14, 2019: member

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Conflicts

    No conflicts as of last run.

12. fanquake added the label Needs rebase on Feb 15, 2019
13. 
    fanquake commented at 3:08 pm on February 15, 2019: member

    Concept ACK
14. Empact force-pushed on Feb 25, 2019
15. DrahtBot removed the label Needs rebase on Feb 25, 2019
16. Empact force-pushed on Mar 5, 2019
17. Empact force-pushed on Mar 9, 2019
18. Empact force-pushed on Mar 9, 2019
19. 
    Empact commented at 5:43 pm on March 9, 2019: member

    Made an effort to pare this down significantly. Now only one new file, and the necessity of the cstring include is determined in configure.
20. Empact force-pushed on Mar 9, 2019
21. Empact force-pushed on Mar 9, 2019
22. Empact force-pushed on Mar 9, 2019
23. Empact force-pushed on Apr 3, 2019
24. MarcoFalke added the label Needs gitian build on Apr 8, 2019
25. DrahtBot removed the label Needs gitian build on Apr 9, 2019
26. [moveonly] Split glibc sanity_test_fdelt out 7fb886b1b1
27. Include cstring for sanity_test_fdelt if required

    SmartOS FD_ZERO is implemented in a way that requires
    an external declaration of memcpy. We can not simply
    include cstring in the existing file because
    sanity_test_memcpy is attempting to replace memcpy, but we can do
    so here, now that the fdelt test is split out.

    b4fd0ca9be
28. Empact force-pushed on Apr 14, 2019
29. 
    Empact commented at 3:25 am on April 14, 2019: member

    @MarcoFalke realized that in attempting to rebase earlier I unintentionally overwrote my later implementation (https://github.com/bitcoin/bitcoin/pull/15146#issuecomment-471204901) with the earlier. Restored the earlier implementation - would need gitian again.
30. MarcoFalke deleted a comment on Apr 15, 2019
31. 
    laanwj commented at 11:50 am on June 7, 2019: member

    Code review an lightly tested (but not on SmartOS) ACK b4fd0ca9be14c81023db759c405c0f67cfa78166
32. MarcoFalke added the label Needs gitian build on Jun 7, 2019
33. MarcoFalke added this to the milestone 0.19.0 on Jun 7, 2019
34. 
    DrahtBot commented at 5:26 am on June 9, 2019: member

    Gitian builds for commit 5d2ccf0ce9ca1571c650a69319fb9c1e0b626ecb (master):

    Gitian builds for commit bb4d0dd694e6a0847e4c2c40d8c554bd4be9c6b1 (master and this pull):

35. DrahtBot removed the label Needs gitian build on Jun 9, 2019
36. MarcoFalke added the label Needs gitian build on Jun 11, 2019
37. 
    DrahtBot commented at 10:06 pm on June 11, 2019: member

    Gitian builds for commit 8a503a6c6dd419921373f45d7aa6f1787c9b8884 (master):

    Gitian builds for commit 6ba19505d59dd0d06ea33a15daf7f86043c9f53a (master and this pull):

38. DrahtBot removed the label Needs gitian build on Jun 11, 2019
39. MarcoFalke added the label Needs gitian build on Jun 11, 2019
40. DrahtBot removed the label Needs gitian build on Jun 13, 2019
41. laanwj renamed this:
    Solve SmartOS FD_ZERO build issue
    build: Solve SmartOS FD_ZERO build issue
    on Sep 18, 2019
42. laanwj referenced this in commit dd8cf82e96 on Sep 18, 2019
43. laanwj merged this on Sep 18, 2019
44. laanwj closed this on Sep 18, 2019

    ---

45. Empact deleted the branch on Sep 18, 2019
46. sidhujag referenced this in commit f5f7b1e08e on Sep 23, 2019
47. MarcoFalke deleted a comment on Oct 13, 2019
48. deadalnix referenced this in commit 5c0e43d97c on Jun 4, 2020
49. kittywhiskers referenced this in commit 2f79e04a8e on Dec 4, 2021
50. kittywhiskers referenced this in commit c53856c984 on Dec 8, 2021
51. kittywhiskers referenced this in commit 79c7766095 on Dec 12, 2021
52. kittywhiskers referenced this in commit 25468188bf on Dec 13, 2021
53. kittywhiskers referenced this in commit 77021fadff on Dec 13, 2021
54. MarcoFalke locked this on Dec 16, 2021

Contributors
Empact laanwj DrahtBot fanquake

Labels
Build system

Milestone
0.19.0