This has been fixed in 0.20, so it needs to be fixed on master as well to avoid a regression
build: Ensure source tarball has leading directory name #20318
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2011-buildTar changing 4 files +8 −8-
MarcoFalke commented at 4:28 pm on November 5, 2020: member
-
MarcoFalke added the label Needs gitian build on Nov 5, 2020
-
MarcoFalke added the label Needs Guix build on Nov 5, 2020
-
MarcoFalke added this to the milestone 0.21.0 on Nov 5, 2020
-
MarcoFalke force-pushed on Nov 5, 2020
-
scripted-diff: [build] Ensure source tarball has leading directory name
-BEGIN VERIFY SCRIPT- sed -i 's|git archive --|git archive --prefix="${DISTNAME}/" --|g' $(git grep -l 'git archive' ./contrib) sed -i 's|tar -xf "\?${\?GIT_ARCHIVE}\?"\?|tar --strip-components=1 -xf "${GIT_ARCHIVE}"|g' $(git grep -l 'tar -xf' ./contrib) -END VERIFY SCRIPT-
-
MarcoFalke force-pushed on Nov 5, 2020
-
hebasto approved
-
hebasto commented at 8:53 pm on November 5, 2020: memberACK faa2f06f5eaf8578873495f44603ee74d7a1abf4, tested gitian builds only.
-
DrahtBot commented at 9:18 pm on November 5, 2020: 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:
- #18902 (Bugfix: Only use git for build info if the repository is actually the right one by luke-jr)
- #18818 (Fix release tarball generated by gitian by luke-jr)
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.
-
DrahtBot commented at 8:29 am on November 7, 2020: member
Guix builds
-
DrahtBot removed the label Needs Guix build on Nov 7, 2020
-
DrahtBot removed the label Needs gitian build on Nov 8, 2020
-
DrahtBot added the label Build system on Nov 8, 2020
-
DrahtBot added the label Scripts and tools on Nov 8, 2020
-
MarcoFalke removed the label Scripts and tools on Nov 8, 2020
-
MarcoFalke added the label Needs gitian build on Nov 8, 2020
-
MarcoFalke deleted a comment on Nov 8, 2020
-
laanwj commented at 2:06 pm on November 9, 2020: memberACK faa2f06f5eaf8578873495f44603ee74d7a1abf4
-
laanwj merged this on Nov 9, 2020
-
laanwj closed this on Nov 9, 2020
-
sidhujag referenced this in commit a59fbc36ff on Nov 9, 2020
-
DrahtBot commented at 11:14 pm on November 10, 2020: member
Gitian builds
-
DrahtBot removed the label Needs gitian build on Nov 10, 2020
-
MarcoFalke deleted the branch on Nov 12, 2020
-
UdjinM6 referenced this in commit 5242773943 on Oct 23, 2021
-
UdjinM6 referenced this in commit 94bf922354 on Oct 23, 2021
-
UdjinM6 referenced this in commit 09199e8415 on Dec 4, 2021
-
UdjinM6 referenced this in commit 632c759352 on Dec 5, 2021
-
DrahtBot locked this on Feb 15, 2022