-
Notifications
You must be signed in to change notification settings - Fork 2k
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
mail-client/alpine: fix file name #9911
Conversation
Copyright policy changePlease note that on 2018-09-15 Trustees have approved new Gentoo copyright policy. All contributions made to Gentoo need to follow this policy. If you include the Signed-off-by line in your commit message, you indicate that you have read the policy and agree to its terms. For more detailed explanation, please see the new Gentoo copyright policy explained article. Pull Request assignmentAreas affected: ebuilds mail-client/alpine: @robert7k, @gentoo/net-mail, @gentoo/proxy-maint Linked bugsBugs linked: 666490 In order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
mail-client/alpine/Manifest
Outdated
@@ -1,3 +1,4 @@ | |||
DIST 2.21.1-r1.tar.gz 7490140 BLAKE2B 13ac5ef43dd729a7cf21b44dcd6feace3e1813672aec537fd05d5ff90b67bec145ceef71cfcf35b0cb7e250bb307556f5dcf8ebb024bc6366fd0ace0a7c41161 SHA512 106caa7bfe86fd5e82554ba68b9ee579cd8495bcd0dfbf82d9d75da2df3bd292d42d67af8ec737091f2a820b2555c25bbf0058c9c7bb5de5f950b570ffc25c4d |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's not a friendly or safe distfile name. It's asking for accidental collisions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the hint, I fixed that.
@@ -7,8 +7,7 @@ inherit autotools toolchain-funcs vcs-snapshot | |||
DESCRIPTION="An easy to use text-based based mail and news client" | |||
HOMEPAGE="http://www.washington.edu/alpine/ http://repo.or.cz/alpine.git/" | |||
GIT_COMMIT="843b2f16abfd949e09b1c5465387b1b0f724994a" | |||
SRC_URI="http://repo.or.cz/alpine.git/snapshot/${GIT_COMMIT}.tar.gz -> ${P}.tar.gz" | |||
RESTRICT="mirror" | |||
SRC_URI="http://repo.or.cz/alpine.git/snapshot/${GIT_COMMIT}.tar.gz -> ${PF}.tar.gz" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PF is no good here. It means that when someone revbumps it (e.g. due to dep change), he'll suddenly change filename for no reason.
Also, please match the upstream name, i.e. alpine-843b2f1
(grab it off ${GIT_COMMIT::n}
where n is some number), and use that both for ->
name and S
; then remove vcs-snapshot as that eclass is deprecated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for pointing this out, I fixed it accordingly.
Closes: https://bugs.gentoo.org/666490 Signed-off-by: Robert Siebeck <gentoo.bugzilla.2012@r123.de>
Pull request CI reportReport generated at: 2018-09-24 01:38 UTC Issues already there before the PR (double-check them): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot. I'll test just to be sure, and merge.
Closes: https://bugs.gentoo.org/666490