Bugzilla Bug 123114

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

porthole doesn't have any direct dep on gettext, better use a tool that works.

Yes it does.

* app-portage/porthole (=0.5.0)

    x-/mnt/usbdisc/portage: (0.5.0)KR* 
    Homepage:              http://porthole.sourceforge.net
    Description:           A GTK+-based frontend to Portage
    Build dependencies:   
            (
                >=dev-lang/python-2.3
                >=sys-apps/portage-2.0.51-r3
                >=dev-python/pygtk-2.4.0
                >=dev-python/pyxml-0.8.4
                >=gnome-base/libglade-2.5.0
                nls? (
                    >=sys-devel/gettext-0.14
                )
                !bootstrap? (
                    sys-devel/patch
                )
                virtual/python
            )
    Runtime dependencies: 
            (
                >=dev-lang/python-2.3
                >=sys-apps/portage-2.0.51-r3
                >=dev-python/pygtk-2.4.0
                >=dev-python/pyxml-0.8.4
                >=gnome-base/libglade-2.5.0
                nls? (
                    >=sys-devel/gettext-0.14   <-- *** HERE!!! ***
                )
                debug? (
                    >=dev-python/pycrash-0.4_pre3
                )
            )
    Post dependencies:    
            (
            )

Ok, missed it as it's package.masked.

I've got no issues when making the change, however, when I make the change I go
from one warnings/errors from repoman to the following:

  DEPEND.badmasked               4
   app-portage/porthole/porthole-0.5.0.ebuild: ~x86(default-linux/x86/2005.1)
['virtual/libintl']
   app-portage/porthole/porthole-0.5.0.ebuild:
~amd64(default-linux/amd64/2005.1) ['virtual/libintl']
   app-portage/porthole/porthole-0.5.0.ebuild: ~ppc(default-linux/ppc/2005.0)
['virtual/libintl']
   app-portage/porthole/porthole-0.5.0.ebuild:
~sparc(default-linux/sparc/sparc64/2005.1) ['virtual/libintl']
  RDEPEND.badmasked              4
   app-portage/porthole/porthole-0.5.0.ebuild: ~x86(default-linux/x86/2005.1)
['virtual/libintl']
   app-portage/porthole/porthole-0.5.0.ebuild:
~amd64(default-linux/amd64/2005.1) ['virtual/libintl']
   app-portage/porthole/porthole-0.5.0.ebuild: ~ppc(default-linux/ppc/2005.0)
['virtual/libintl']
   app-portage/porthole/porthole-0.5.0.ebuild:
~sparc(default-linux/sparc/sparc64/2005.1) ['virtual/libintl']
  ebuild.allmasked               1
   app-portage/porthole

Does something else need to be fixed before I commit this change?

Repoman warnings in comment #4 were caused by PEBKAC (i.e not updating my local
CVS tree). Fixed in CVS.