-
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
net-libs/grpc: Fix files installed outside prefix #11916
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 assignmentSubmitter: @indocomsoft net-libs/grpc: @perfinion, @gyakovlev Linked bugsBugs linked: 685198 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 |
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 catching that.
Please edit all EAPI6 ebuild to be double-slash-less.
net-libs/grpc/grpc-1.12.0.ebuild
Outdated
@@ -59,7 +59,7 @@ src_compile() { | |||
|
|||
src_install() { | |||
emake \ | |||
prefix="${D}"/usr \ | |||
prefix="${ED}"/usr \ |
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.
in EAPI6 ebuild this should be ${ED%/}
( or ${ED}"usr
) as this variable does not have a trailing slash in EAPI7 but does have it in EAPI6
net-libs/grpc/grpc-1.12.1-r1.ebuild
Outdated
@@ -134,7 +134,7 @@ python_compile_all() { | |||
|
|||
src_install() { | |||
emake \ | |||
prefix="${D}"/usr \ | |||
prefix="${ED}"/usr \ |
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.
ditto
net-libs/grpc/grpc-1.13.0-r1.ebuild
Outdated
@@ -138,7 +138,7 @@ python_compile_all() { | |||
|
|||
src_install() { | |||
emake \ | |||
prefix="${D}"/usr \ | |||
prefix="${ED}"/usr \ |
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.
this as well
net-libs/grpc/grpc-1.13.1.ebuild
Outdated
@@ -137,7 +137,7 @@ python_compile_all() { | |||
|
|||
src_install() { | |||
emake \ | |||
prefix="${D}"/usr \ | |||
prefix="${ED}"/usr \ |
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.
and again
net-libs/grpc/grpc-1.15.0.ebuild
Outdated
@@ -137,7 +137,7 @@ python_compile_all() { | |||
|
|||
src_install() { | |||
emake \ | |||
prefix="${D}"/usr \ | |||
prefix="${ED}"/usr \ |
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.
and the last one.
Signed-off-by: Julius Putra Tanu Setiaji <indocomsoft@gmail.com> Closes: https://bugs.gentoo.org/685198 Package-Manager: Portage-2.3.66, Repoman-2.3.12
@gyakovlev Fixed! |
Pull request CI reportReport generated at: 2019-05-07 05:37 UTC No issues found |
Signed-off-by: Julius Putra Tanu Setiaji <indocomsoft@gmail.com> Closes: https://bugs.gentoo.org/685198 Package-Manager: Portage-2.3.66, Repoman-2.3.12 Closes: gentoo#11916 Signed-off-by: Georgy Yakovlev <gyakovlev@gentoo.org>
EAPI7 ebuilds do not need stripping trailing slash, but I fixed it in my own commit right after yours. |
closing manually, as CI bot is down. |
I was getting
Aborting due to QA concerns: there are files installed outside the prefix
when emergingnet-libs/grpc
.This PR should fix this.