Patchwork [meta-networking,2/2] ebtables, arptables: add perl as runtime dependency

login
register
mail settings
Submitter Khem Raj
Date Jan. 26, 2013, 6:45 a.m.
Message ID <1359182715-3063-2-git-send-email-raj.khem@gmail.com>
Download mbox | patch
Permalink /patch/43453/
State Accepted, archived
Headers show

Comments

Khem Raj - Jan. 26, 2013, 6:45 a.m.
Fixes errors like

| Computing transaction...error: Can't install
arptables-0.0.3+4-r0.0@ppce500v2: no package provides /usr/bin/perl

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 .../recipes-filter/ebtables/ebtables_2.0.10-4.bb   |    2 ++
 .../recipes-support/arptables/arptables_0.0.3-4.bb |    2 ++
 2 files changed, 4 insertions(+)
Martin Jansa - Jan. 27, 2013, 7:13 p.m.
On Fri, Jan 25, 2013 at 10:45:15PM -0800, Khem Raj wrote:
> Fixes errors like

RDEPENDS_${PN}, fixed in jansa/in-test, I'll merge with that fix if Joe
sends his ack.

Cheers,

> 
> | Computing transaction...error: Can't install
> arptables-0.0.3+4-r0.0@ppce500v2: no package provides /usr/bin/perl
> 
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
>  .../recipes-filter/ebtables/ebtables_2.0.10-4.bb   |    2 ++
>  .../recipes-support/arptables/arptables_0.0.3-4.bb |    2 ++
>  2 files changed, 4 insertions(+)
> 
> diff --git a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
> index 2596d2f..4e9b0bc 100644
> --- a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
> +++ b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
> @@ -4,6 +4,8 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=53b4a999993871a28ab1488fdbd2e73e"
>  SECTION = "console/network"
>  PR = "r1"
>  
> +RDEPENDS += "perl"
> +
>  SRC_URI = "${SOURCEFORGE_MIRROR}/ebtables/ebtables-v${PV}.tar.gz \
>             file://installnonroot.patch \
>             file://01debian_defaultconfig.patch \
> diff --git a/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb b/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
> index 71e2600..8da9bb7 100644
> --- a/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
> +++ b/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
> @@ -3,6 +3,8 @@ PRIORITY = "optional"
>  LICENSE = "GPL-2.0"
>  SECTION = "console/network"
>  
> +RDEPENDS += "perl"
> +
>  LIC_FILES_CHKSUM = "file://${COREBASE}/meta/files/common-licenses/GPL-2.0;md5=801f80980d171dd6425610833a22dbe6"
>  SRC_URI = " \
>  	${SOURCEFORGE_MIRROR}/ebtables/arptables-v${PV}.tar.gz;name=arptables \
> -- 
> 1.7.9.5
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
Joe MacDonald - Jan. 27, 2013, 8:20 p.m.
[Re: [oe] [meta-networking][PATCH 2/2] ebtables, arptables: add perl as runtime dependency] On 13.01.27 (Sun 20:13) Martin Jansa wrote:

> On Fri, Jan 25, 2013 at 10:45:15PM -0800, Khem Raj wrote:
> > Fixes errors like
> 
> RDEPENDS_${PN}, fixed in jansa/in-test, I'll merge with that fix if Joe
> sends his ack.

Yes, please.  Thanks!

-J.

> 
> Cheers,
> 
> > 
> > | Computing transaction...error: Can't install
> > arptables-0.0.3+4-r0.0@ppce500v2: no package provides /usr/bin/perl
> > 
> > Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > ---
> >  .../recipes-filter/ebtables/ebtables_2.0.10-4.bb   |    2 ++
> >  .../recipes-support/arptables/arptables_0.0.3-4.bb |    2 ++
> >  2 files changed, 4 insertions(+)
> > 
> > diff --git a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
> > index 2596d2f..4e9b0bc 100644
> > --- a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
> > +++ b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
> > @@ -4,6 +4,8 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=53b4a999993871a28ab1488fdbd2e73e"
> >  SECTION = "console/network"
> >  PR = "r1"
> >  
> > +RDEPENDS += "perl"
> > +
> >  SRC_URI = "${SOURCEFORGE_MIRROR}/ebtables/ebtables-v${PV}.tar.gz \
> >             file://installnonroot.patch \
> >             file://01debian_defaultconfig.patch \
> > diff --git a/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb b/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
> > index 71e2600..8da9bb7 100644
> > --- a/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
> > +++ b/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
> > @@ -3,6 +3,8 @@ PRIORITY = "optional"
> >  LICENSE = "GPL-2.0"
> >  SECTION = "console/network"
> >  
> > +RDEPENDS += "perl"
> > +
> >  LIC_FILES_CHKSUM = "file://${COREBASE}/meta/files/common-licenses/GPL-2.0;md5=801f80980d171dd6425610833a22dbe6"
> >  SRC_URI = " \
> >  	${SOURCEFORGE_MIRROR}/ebtables/arptables-v${PV}.tar.gz;name=arptables \
> > -- 
> > 1.7.9.5
> > 
> > 
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
Koen Kooi - Jan. 27, 2013, 8:40 p.m.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Op 26-01-13 07:45, Khem Raj schreef:
> Fixes errors like
> 
> | Computing transaction...error: Can't install 
> arptables-0.0.3+4-r0.0@ppce500v2: no package provides /usr/bin/perl
> 
> Signed-off-by: Khem Raj <raj.khem@gmail.com> --- 
> .../recipes-filter/ebtables/ebtables_2.0.10-4.bb   |    2 ++ 
> .../recipes-support/arptables/arptables_0.0.3-4.bb |    2 ++ 2 files
> changed, 4 insertions(+)
> 
> diff --git a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
> b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb index
> 2596d2f..4e9b0bc 100644 ---
> a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb +++
> b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb @@ -4,6
> +4,8 @@ LIC_FILES_CHKSUM =
> "file://COPYING;md5=53b4a999993871a28ab1488fdbd2e73e" SECTION =
> "console/network" PR = "r1"
> 
> +RDEPENDS += "perl"

Can yo split out out the things that need perl? The meat of ebtables doesn't
need perl AFAIK and it would be a shame to drag in perl only for some helper
apps

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)
Comment: GPGTools - http://gpgtools.org

iD8DBQFRBZDPMkyGM64RGpERAngPAJ9ZXq43eATpg+dYBN9b0M0Sg/LJTgCfZltV
oA6rQGFkTCuvBBEwwOClEMc=
=n1zu
-----END PGP SIGNATURE-----
Martin Jansa - Jan. 28, 2013, 10:41 a.m.
On Fri, Jan 25, 2013 at 10:45:15PM -0800, Khem Raj wrote:
> Fixes errors like
> 
> | Computing transaction...error: Can't install
> arptables-0.0.3+4-r0.0@ppce500v2: no package provides /usr/bin/perl

Applied, please split those perl scripts in followup patch.

Thanks
> 
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
>  .../recipes-filter/ebtables/ebtables_2.0.10-4.bb   |    2 ++
>  .../recipes-support/arptables/arptables_0.0.3-4.bb |    2 ++
>  2 files changed, 4 insertions(+)
> 
> diff --git a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
> index 2596d2f..4e9b0bc 100644
> --- a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
> +++ b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
> @@ -4,6 +4,8 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=53b4a999993871a28ab1488fdbd2e73e"
>  SECTION = "console/network"
>  PR = "r1"
>  
> +RDEPENDS += "perl"
> +
>  SRC_URI = "${SOURCEFORGE_MIRROR}/ebtables/ebtables-v${PV}.tar.gz \
>             file://installnonroot.patch \
>             file://01debian_defaultconfig.patch \
> diff --git a/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb b/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
> index 71e2600..8da9bb7 100644
> --- a/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
> +++ b/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
> @@ -3,6 +3,8 @@ PRIORITY = "optional"
>  LICENSE = "GPL-2.0"
>  SECTION = "console/network"
>  
> +RDEPENDS += "perl"
> +
>  LIC_FILES_CHKSUM = "file://${COREBASE}/meta/files/common-licenses/GPL-2.0;md5=801f80980d171dd6425610833a22dbe6"
>  SRC_URI = " \
>  	${SOURCEFORGE_MIRROR}/ebtables/arptables-v${PV}.tar.gz;name=arptables \
> -- 
> 1.7.9.5
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

Patch

diff --git a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
index 2596d2f..4e9b0bc 100644
--- a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
+++ b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
@@ -4,6 +4,8 @@  LIC_FILES_CHKSUM = "file://COPYING;md5=53b4a999993871a28ab1488fdbd2e73e"
 SECTION = "console/network"
 PR = "r1"
 
+RDEPENDS += "perl"
+
 SRC_URI = "${SOURCEFORGE_MIRROR}/ebtables/ebtables-v${PV}.tar.gz \
            file://installnonroot.patch \
            file://01debian_defaultconfig.patch \
diff --git a/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb b/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
index 71e2600..8da9bb7 100644
--- a/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
+++ b/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
@@ -3,6 +3,8 @@  PRIORITY = "optional"
 LICENSE = "GPL-2.0"
 SECTION = "console/network"
 
+RDEPENDS += "perl"
+
 LIC_FILES_CHKSUM = "file://${COREBASE}/meta/files/common-licenses/GPL-2.0;md5=801f80980d171dd6425610833a22dbe6"
 SRC_URI = " \
 	${SOURCEFORGE_MIRROR}/ebtables/arptables-v${PV}.tar.gz;name=arptables \