Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 269905 - app-admin/realpath-1.14 - ridiculously heavy dependencies only to get translations for manuals
Summary: app-admin/realpath-1.14 - ridiculously heavy dependencies only to get transla...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Shell Tools project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-15 00:59 UTC by Maciej Mrozowski
Modified: 2009-06-13 09:38 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Proposed quick fix (realpath-1.14.ebuild.diff,1.12 KB, patch)
2009-05-15 01:04 UTC, Maciej Mrozowski
Details | Diff
Proposed changes for realpath-1.14-r1.ebuild (realpath-1.14-r1.ebuild.diff,1.41 KB, patch)
2009-05-20 15:50 UTC, Ulrich Müller
Details | Diff
Proposed changes for realpath-1.14-r1.ebuild (realpath-1.14-r1.ebuild.diff,1.70 KB, patch)
2009-05-21 06:05 UTC, Ulrich Müller
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Maciej Mrozowski gentoo-dev 2009-05-15 00:59:31 UTC
realpath-1.14, simple Debian-originated tool, brings silly dependencies (some of them have already been worked around), app-text/po4a (+ 20+ new PERL modules) only to translate manual in two languages. That's pretty ridiculous considering realpath is dependency of eselect, core Gentoo tool.

As easiest solution I suggest dropping translated manuals - come on - it's one manual page.
Comment 1 Maciej Mrozowski gentoo-dev 2009-05-15 01:04:38 UTC
Created attachment 191296 [details, diff]
Proposed quick fix

Only make/install targets in src/ and man/ subdirectories (effectively skipping po/).
Comment 2 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2009-05-15 04:38:25 UTC
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 ;)
Comment 3 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2009-05-19 21:24:20 UTC
BTW, a proper fix would be a USE flag, I think. USE=minimal or so.
Comment 4 Ulrich Müller gentoo-dev 2009-05-19 22:24:25 UTC
(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.
Comment 5 Samuli Suominen (RETIRED) gentoo-dev 2009-05-20 12:15:16 UTC
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
Comment 6 Ulrich Müller gentoo-dev 2009-05-20 15:50:41 UTC
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?
Comment 7 Ulrich Müller gentoo-dev 2009-05-21 06:05:53 UTC
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.
Comment 8 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2009-06-13 02:50:54 UTC
(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.

Comment 9 Ulrich Müller gentoo-dev 2009-06-13 09:38:22 UTC
Committed then, and bumped to 1.15.