Summary: | app-admin/realpath-1.14 - ridiculously heavy dependencies only to get translations for manuals | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Maciej Mrozowski <reavertm> |
Component: | New packages | Assignee: | Gentoo Shell Tools project <shell-tools> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | eselect, loki_val, ssuominen |
Priority: | High | ||
Version: | 2008.0 | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
Proposed quick fix
Proposed changes for realpath-1.14-r1.ebuild Proposed changes for realpath-1.14-r1.ebuild |
Description
Maciej Mrozowski
2009-05-15 00:59:31 UTC
Created attachment 191296 [details, diff]
Proposed quick fix
Only make/install targets in src/ and man/ subdirectories (effectively skipping po/).
Samuli or Peter can fix this. They have been working on the package lately and know more about it than the shell-tools team does ;) BTW, a proper fix would be a USE flag, I think. USE=minimal or so. (In reply to comment #3) > BTW, a proper fix would be a USE flag, I think. USE=minimal or so. Looks like an overkill to me. I've committed a fix that drops the translated man pages, but keeps the MO files. Thanks for reporting the issue. IMHO, manpages should always be installed. PERL modules are so small files, I can't see why it's a problem at all. We don't stop intltool, or gettext installing either. This should be USE="nls" if any Created attachment 191907 [details, diff] Proposed changes for realpath-1.14-r1.ebuild <ulm> ssuominen: I had discussed the issue of bug 269905 wit loki_val and with darkside... <Willikins> ulm: https://bugs.gentoo.org/269905 "app-admin/realpath-1.14 - ridiculously heavy dependencies only to get translations for manuals"; Gentoo Linux, Ebuilds; REOP; reavertm@poczta.fm:shell-tools@g.o <ssuominen> ulm: You can reassign it to me, and leave the ebuild as in now tree but I'm still going to write a use NLS patch for the build system <ssuominen> ulm: I assume that won't be a problem? Disabled until optional patch provided <ulm> ssuominen: USE=nls is turned on by default in profiles I think <ssuominen> We've allways installed manpages.. same as basic docs.. few perl deps just don't change my view on it :-) <ulm> ssuominen: well, it installs a man page <ssuominen> It is, that's what the user requsted, to build translations <ssuominen> == USE nls <ulm> ssuominen: hm, another option would be to prepackage the translated man pages <ssuominen> that would fine <ulm> ssuominen: i'll look at it and attach a patch to the bug for your approval ;) <ssuominen> it could download the matching ${PV} from the .deb since the source is as well debian based <ulm> or that Are you O.K. with attached changes to the ebuild? Created attachment 191992 [details, diff]
Proposed changes for realpath-1.14-r1.ebuild
Small update: If the user builds with USE=-nls, then we also shouldn't install any MO files.
(In reply to comment #7) > Created an attachment (id=191992) [edit] > Proposed changes for realpath-1.14-r1.ebuild I hope you aren't waiting for me, I thought we talked about this in irc. Have at it. Committed then, and bumped to 1.15. |