-
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
dev-lang/nasm: bump to 2.13.01, EAPI=6, fix doc dependencies #4938
Conversation
Package-Manager: Portage-2.3.6, Repoman-2.3.2
Pull Request assignment Areas affected: ebuilds dev-lang/nasm: At least one of the listed packages is maintained entirely by non-GitHub developers! |
dev-lang/nasm/metadata.xml
Outdated
@@ -15,5 +15,6 @@ Pentium, P6, MMX, 3DNow!, SSE and SSE2 opcodes, and has macro capability. | |||
</longdescription> | |||
<upstream> | |||
<remote-id type="sourceforge">nasm</remote-id> | |||
<bugs-to>https://bugzilla.nasm.us/</bugs-to> |
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.
did you run xmllint
on this?
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 blame vim! (I didn't see that the original had spaces and my vim config uses tabs for those files, which is Gentoo's default I guess).
dev-lang/nasm/nasm-2.13.01.ebuild
Outdated
" | ||
|
||
S=${WORKDIR}/${P/_} | ||
DOCS=( AUTHORS CHANGES ChangeLog README TODO ) |
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.
all of these are part of the default set?
dev-lang/nasm/nasm-2.13.01.ebuild
Outdated
src_install() { | ||
emake INSTALLROOT="${D}" install install_rdf | ||
if use doc; then | ||
DOCS+=( doc/nasmdoc.{pdf,txt} ) |
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.
rather just do dodoc doc/nasmdoc.{pdf,txt}
here and nix the prior DOCS
definition
Gentoo-Bug: 619360 Package-Manager: Portage-2.3.6, Repoman-2.3.2
Related bug: https://bugs.gentoo.org/show_bug.cgi?id=619360
I added some missing dependencies and removed texinfo which isn't required.
I will make a rekeywording bug for the arches that do not have the new deps once this is merged.