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
Added new dependency to CGO usb bindings #105
Conversation
Commit for dev-libs/go-usb package. Please review. |
…fs compile properly
Please leave comment on the bug when you submit a PR, so the assignee knows about it :P. |
src_test() { | ||
go test -ldflags '-extldflags=-fno-PIC' ${GO_PN} || die | ||
} | ||
|
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.
Stray empty line.
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.
I'll fix this and those below during the week. Sorry for late reply, it seems i wasn't getting these email notifications.
@@ -46,7 +46,7 @@ done | |||
|
|||
insinto /usr/lib/go/ | |||
doins -r "${S}/pkg" | |||
insinto "/usr/lib/go/src/pkg/${GO_PN}/" | |||
insinto "/usr/lib/go/src/${GO_PN}/" |
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.
I've pinged go-fuse maintainer about this.
… since I can't reproduce why I used it in the first place.
@ZeroChaos- see my last commit, i removed the added get_libdir and doins loop since it's not even necessary. |
still approved |
Ok, checked this and go itself uses /usr/lib. |
|
||
src_install() { | ||
insinto "${GO_PN}" | ||
doins *.go LICENSE || die "Install failed" |
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.
die unnecessary in EAPI 5 (with doins).
Ok, I'm going to fix that tiny bug and merge it. |
All merged now. Closing. |
… Part of gentoo/gentoo-portage-rsync-mirror#105 by likewhoa. (Portage version: 2.2.18/cvs/Linux x86_64, signed Manifest commit with key EFB4464E!)
…t of gentoo/gentoo-portage-rsync-mirror#105 by likewhoa. (Portage version: 2.2.18/cvs/Linux x86_64, signed Manifest commit with key EFB4464E!)
…t of gentoo/gentoo-portage-rsync-mirror#105 by likewhoa. (Portage version: 2.2.18/cvs/Linux x86_64, signed Manifest commit with key EFB4464E!)
…e-rsync-mirror#105 by likewhoa. (Portage version: 2.2.18/cvs/Linux x86_64, signed Manifest commit with key EFB4464E!)
This is in relation to bug 547204 at b.g.o which bug 523990 needs in order to properly compile. I will send another PR and reference it which will contain the ebuild for dev-libs/go-usb