Bug 123114 – app-portage/porthole-0.5.0 has gettext as RDEPEND

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Diego Elio Pettenò (RETIRED) 2006-02-16 16:47:47 UTC]

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

[Comment 1 Marius Mauch (RETIRED) 2006-02-16 16:56:40 UTC]

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

[Comment 2 Ciaran McCreesh 2006-02-16 17:09:18 UTC]

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:    
            (
            )

[Comment 3 Marius Mauch (RETIRED) 2006-02-16 17:13:10 UTC]

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

[Comment 4 Paul Varner (RETIRED) 2006-02-17 07:51:06 UTC]

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?

[Comment 5 Paul Varner (RETIRED) 2006-02-27 12:44:22 UTC]

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