build: Ensure source tarball has leading directory name (0.20)#18945
Merged
laanwj merged 1 commit intobitcoin:0.20from May 12, 2020
Merged
build: Ensure source tarball has leading directory name (0.20)#18945laanwj merged 1 commit intobitcoin:0.20from
laanwj merged 1 commit intobitcoin:0.20from
Conversation
laanwj
changed the title
Member
|
I think DrahtBot can't build non-master branches, sorry 😅 Maybe @jonasschnelli's build farm can do? |
Member
Author
I'll try to run my own gitian build.
Yes, thank you. |
Member
Author
|
Gitian output: |
Member
|
tACK 59d57f6 This extracts to |
Member
Author
|
Yes, this is a fix for 0.20.0, it's supposed to do that. We should do a similar thing on master, but the gitian descriptors have been reorganized there (so the solution needs to be slightly different) and there's not as much hurry. |
laanwj
added a commit
that referenced
this pull request
Nov 9, 2020
faa2f06 scripted-diff: [build] Ensure source tarball has leading directory name (MarcoFalke) Pull request description: This has been fixed in 0.20, so it needs to be fixed on master as well to avoid a regression #18945 ACKs for top commit: laanwj: ACK faa2f06 hebasto: ACK faa2f06, tested gitian builds only. promag: ACK faa2f06. Tree-SHA512: e3b025c29c45b025002abc35262bb5d771f6cbd807f1c256c477c243685e93cd43ad9f642b38e3cf218590912abe6ea0ddfec3bfbef36f99080aad74ed6cc0af
sidhujag
pushed a commit
to syscoin/syscoin
that referenced
this pull request
Nov 9, 2020
…ry name faa2f06 scripted-diff: [build] Ensure source tarball has leading directory name (MarcoFalke) Pull request description: This has been fixed in 0.20, so it needs to be fixed on master as well to avoid a regression bitcoin#18945 ACKs for top commit: laanwj: ACK faa2f06 hebasto: ACK faa2f06, tested gitian builds only. promag: ACK faa2f06. Tree-SHA512: e3b025c29c45b025002abc35262bb5d771f6cbd807f1c256c477c243685e93cd43ad9f642b38e3cf218590912abe6ea0ddfec3bfbef36f99080aad74ed6cc0af
UdjinM6
pushed a commit
to UdjinM6/dash
that referenced
this pull request
Oct 23, 2021
… directory name faa2f06 scripted-diff: [build] Ensure source tarball has leading directory name (MarcoFalke) Pull request description: This has been fixed in 0.20, so it needs to be fixed on master as well to avoid a regression bitcoin#18945 ACKs for top commit: laanwj: ACK faa2f06 hebasto: ACK faa2f06, tested gitian builds only. promag: ACK faa2f06. Tree-SHA512: e3b025c29c45b025002abc35262bb5d771f6cbd807f1c256c477c243685e93cd43ad9f642b38e3cf218590912abe6ea0ddfec3bfbef36f99080aad74ed6cc0af
UdjinM6
pushed a commit
to UdjinM6/dash
that referenced
this pull request
Dec 4, 2021
… directory name faa2f06 scripted-diff: [build] Ensure source tarball has leading directory name (MarcoFalke) Pull request description: This has been fixed in 0.20, so it needs to be fixed on master as well to avoid a regression bitcoin#18945 ACKs for top commit: laanwj: ACK faa2f06 hebasto: ACK faa2f06, tested gitian builds only. promag: ACK faa2f06. Tree-SHA512: e3b025c29c45b025002abc35262bb5d771f6cbd807f1c256c477c243685e93cd43ad9f642b38e3cf218590912abe6ea0ddfec3bfbef36f99080aad74ed6cc0af
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
In the interest of moving 0.20.0 forward and being able to do rc2, extract and backport the non-controversial part from #18818.: ensure that the source tarball has leading directory name.