build: Add README.md to DIST target #17097

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:1909-buildDocDist changing 1 files +4 −1
  1. MarcoFalke commented at 2:51 pm on October 10, 2019: member

    This is required because our release tarball is generated by listing each needed file. See: #16734

    Should fix the failing builds after commit 9b4dfec831e6c20ce06c40412a692315b482e456

  2. MarcoFalke added the label Build system on Oct 10, 2019
  3. MarcoFalke added the label Needs gitian build on Oct 10, 2019
  4. MarcoFalke commented at 2:52 pm on October 10, 2019: member
    cc @dongcarl
  5. build: Add README.md to DIST target fa91590a74
  6. MarcoFalke force-pushed on Oct 10, 2019
  7. laanwj commented at 5:30 pm on October 10, 2019: member
    We didn’t even include README.md?!? I guess it needs doc/README_windows.txt as well (or isn’t this needed for windows)
  8. MarcoFalke commented at 5:34 pm on October 10, 2019: member
    This whole concept of explicitly listing each and every file manually (or with a fragile wildcard) is an obvious sisyphean task. I’d say all we need to do is run git archive and be done with it forever, see #16734, #6753, #11530 …
  9. laanwj commented at 6:09 pm on October 10, 2019: member
    I agree, I’ve never been a fan of it. I don’t think we have any files in the git repository we don’t want to ship in the source tarball.
  10. DrahtBot commented at 1:07 am on October 11, 2019: member

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

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #17104 ([WIP] build: make dist uses git archive by ch4ot1c)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  11. DrahtBot commented at 3:03 pm on October 11, 2019: member

    Gitian builds for commit d5a770b70d55a5874fbd7796a171de22abbbd342 (master):

    Gitian builds for commit 9bbeb37b1fc067781eafcd4e7d4d6d6ccdc7fd05 (master and this pull):

  12. DrahtBot removed the label Needs gitian build on Oct 11, 2019
  13. MarcoFalke commented at 5:34 pm on October 11, 2019: member
    This has no review, and is my own pull request. Though, it fixes the gitian build, so I am just going to merge this.
  14. MarcoFalke commented at 5:34 pm on October 11, 2019: member
    or maybe @fanquake can take a look?
  15. fanquake approved
  16. fanquake commented at 6:05 pm on October 11, 2019: member
    ACK fa91590a74985ff321000864ab3e8ef7414d84eb - just checked that README.md is present inside the tarball after running make dist.
  17. MarcoFalke commented at 6:22 pm on October 11, 2019: member
    I checked that the readme is also in https://drahtbot.space/gitian/bitcoin/bitcoin/9bbeb37b1fc067781eafcd4e7d4d6d6ccdc7fd05/bitcoin-0.19.99.tar.gz
  18. MarcoFalke referenced this in commit f4d93078dd on Oct 11, 2019
  19. MarcoFalke merged this on Oct 11, 2019
  20. MarcoFalke closed this on Oct 11, 2019
  21. MarcoFalke deleted the branch on Oct 11, 2019
  22. sidhujag referenced this in commit bba1991b59 on Oct 11, 2019
  23. laanwj referenced this in commit baa72cd9a2 on Mar 25, 2020
  24. sidhujag referenced this in commit 56ca0913fc on Mar 28, 2020
  25. MarkLTZ referenced this in commit 04df2f99be on Apr 19, 2020
  26. random-zebra referenced this in commit 3614de632e on Jul 19, 2021
  27. UdjinM6 referenced this in commit 98417ecad2 on Oct 23, 2021
  28. UdjinM6 referenced this in commit 9e85411dc7 on Oct 23, 2021
  29. UdjinM6 referenced this in commit 83cbc3c811 on Dec 4, 2021
  30. DrahtBot locked this on Dec 16, 2021


MarcoFalke laanwj DrahtBot fanquake

Labels
Build system

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 09:12 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me