Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(Fix) broken tarballs download to inputs dir #16

Merged
merged 2 commits into from
Apr 16, 2021

Conversation

vindard
Copy link
Contributor

@vindard vindard commented Apr 16, 2021

osslsigncode-2.0.tar.gz

The -O flag holds precedence over the -P flag and defaults the location to ./<-O value>. This fix places the 'inputs' dir directly into the -O flag value and removes the -P flag for the osslsigncode-2.0.tar.gz file.

osslsigncode-1.7.1.tar.gz

Original link is now broken but there is now a version hosted on the bitcoin repo (see bitcoin/bitcoin#21416 (comment))

The -O flag holds precedence over the -P flag
and defaults the location to ./<-O value>. This
commit places the 'inputs' dir directly into the
-O flag value and removes the -P flag.
@vindard
Copy link
Contributor Author

vindard commented Apr 16, 2021

Addresses part of the issue flagged in #14 (comment) as well

Replaced the sourceforge location (that now throws a 404 error)
with a version hosted on the bitcoin repo. See this issue for
more details:
https://github.com/bitcoin/bitcoin/issues/21416\#issuecomment-803286481
@vindard vindard changed the title (Fix) download 'osslsigncode-2.0.tar.gz' to inputs dir (Fix) broken tarballs download to inputs dir Apr 16, 2021
Copy link
Owner

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK, thanks! Appreciate the research and the update. Verified the new link. I wouldn't have noticed this as I haven't run these setup instructions since last year.

@jonatack jonatack merged commit f52bfa2 into jonatack:master Apr 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants