Bugzilla Bug 122891

QA checks for /usr/portage/media-gfx/gqview:
media-gfx/gqview-2.0.1: dep flags:
  major:  Conditional flag 'nls' in DEPEND not in IUSE
  major:  Conditional flag 'nls' in RDEPEND not in IUSE
media-gfx/gqview-2.1.1: dep flags:
  major:  Conditional flag 'nls' in DEPEND not in IUSE
  major:  Conditional flag 'nls' in RDEPEND not in IUSE


Looks like USE=nls is being used purely to pull in a package, and isn't
altering configure in any way. Is this a bogus dep?

Reading the ebuild shows the package being pulled in is gettext. So, I'm
copying+pasting what Diego wrote for other similar bugs:

"The ebuild in summary has a runtime dependency over gettext.

gettext is normally used only as build-time dependency to create the po files
in nls-enabled packages.
A part from development tools, the only case when gettext is needed as runtime
dependency is when the system c library does not provide intl functions.
For this reason the right way to state the dependencies for nls-enabled
packages is:

RDEPEND="nls? ( virtual/libintl )
   ...."

DEPEND="${RDEPEND}
   nls? ( sys-devel/gettext )"

this way libintl takes care of adding the dependency when needed and gettext is
required only at build time.

Note: although usually packages using gettext and autotools are supposed to
honour --disable-nls at configure time, that might not always be true, so
gettext and libintl might be an hard dependency. Also, gettext (but not
libintl) might be an hard dependency if the sources needs to be bootstrapped
for autotools from scratch (and thus autopoint has to be run).

Thanks,
Diego"

It's being pulled in and not even used... the ebuild should either pass
$(use_enable nls) to econf, pull in gettext as an hard dep, or not at all.
Just a sec and I'll take a look...

As it is, gettext should be an hard dependency as nls is always enabled
(bindtextdomain is called unconditionally). I'm not sure how gtk handles those
stuff as I can't see a setlocale() call but rather a gtk_set_locale() one.
Passing $(use_enable nls) is likely to break the package on *BSD and uClibc
systems.

IIRC most of GTK stuff simply can't disable nls at all, so gettext should be an
hard dependency (and virtual/libintl, too).

make_it_a_hard_dep++;
->gettext in DEPEND
->virtual/libintl in RDEPEND

fixed in CVS.  Check it out and make sure I didn't dork it up too bad.

Looks good, thanks.