Re: [PATCH v2] intltool: include intltool.m4 and add missing rdepends

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Wed, 2012-09-19 at 06:23 -0700, Khem Raj wrote:
> On Wed, Sep 19, 2012 at 2:59 AM, Constantin Musca
> <constantinx.musca@intel.com> wrote:
> > - include /usr/share/aclocal/intltool.m4 into the intltool
> > package (the files from intltool-dev must be included into the
> > main package, as intltool is a development tool)
> > - add missing rdepends: gettext-dev, libxml-parser-perl
> >
> > [YOCTO #2597]
> >
> > Signed-off-by: Constantin Musca <constantinx.musca@intel.com>
> > ---
> >  meta/recipes-devtools/intltool/intltool.inc       |    9 ++++++++-
> >  meta/recipes-devtools/intltool/intltool_0.50.0.bb |    2 +-
> >  2 files changed, 9 insertions(+), 2 deletions(-)
> >
> > diff --git a/meta/recipes-devtools/intltool/intltool.inc b/meta/recipes-devtools/intltool/intltool.inc
> > index 1c41f0f..59974d8 100644
> > --- a/meta/recipes-devtools/intltool/intltool.inc
> > +++ b/meta/recipes-devtools/intltool/intltool.inc
> > @@ -7,11 +7,18 @@ SRC_URI = "http://launchpad.net/${BPN}/trunk/${PV}/+download/${BP}.tar.gz"
> >  S = "${WORKDIR}/intltool-${PV}"
> >
> >  DEPENDS = "libxml-parser-perl-native"
> > +RDEPENDS_${PN} = "gettext-dev libxml-parser-perl"
> 
> creating a runtime dependency on a dev package seems troublesome to me.
> Can you explain why is this being done ?

Constantin has posted an update about this. The issue is the intltool.m4
macros references macros that come from gettext-dev.

I think that whilst this unusual, developer tools can need other
development tools. We need to remember that intltool is only ever used
for development.

Cheers,

Richard