[oe] [PATCH meta-networking] quagga: two fixes
Martin Jansa
martin.jansa at gmail.com
Fri Aug 16 19:50:38 UTC 2013
On Fri, Aug 16, 2013 at 04:44:12PM +0800, Rongqing Li wrote:
>
>
> On 08/16/2013 03:19 PM, Martin Jansa wrote:
> > On Fri, Aug 16, 2013 at 10:23:59AM +0800, rongqing.li at windriver.com wrote:
> >> From: "Roy.Li" <rongqing.li at windriver.com>
> >>
> >> 1. quagga should depend on iproute2, since ip from busybox is not sufficient
> >> 2. fix a typOS(should chmod, not chown)
> >
> > As you care about quagga, can you please fix:
> > quagga-0.99.21: The /usr/share/info/dir file is not meant to be shipped
> > in a particular package.
> >
> > ?
>
> I did not see this warning, and this file is shipped to quagga-doc.
>
> If there is difference, it is in oe-core, not quagga
Sorry I don't understand what you mean, the infodir QA test is
relatively new and quagga + 4 more recipes suffer from this issue and
should be fixed, see the check and list of recipes with warning in:
http://patches.openembedded.org/patch/52725/
>
> -Roy
>
>
> >
> > Thanks
> >
> >>
> >> Signed-off-by: Roy.Li <rongqing.li at windriver.com>
> >> ---
> >> meta-networking/recipes-protocols/quagga/quagga.inc | 9 ++++++---
> >> 1 file changed, 6 insertions(+), 3 deletions(-)
> >>
> >> diff --git a/meta-networking/recipes-protocols/quagga/quagga.inc b/meta-networking/recipes-protocols/quagga/quagga.inc
> >> index f5489c6..89b9f7a 100644
> >> --- a/meta-networking/recipes-protocols/quagga/quagga.inc
> >> +++ b/meta-networking/recipes-protocols/quagga/quagga.inc
> >> @@ -5,7 +5,7 @@ Unix platforms, particularly FreeBSD, Linux, Solaris and NetBSD. \
> >> Quagga is a fork of GNU Zebra which was developed by Kunihiro \
> >> Ishiguro. The Quagga tree aims to build a more involved community \
> >> around Quagga than the current centralised model of GNU Zebra."
> >> -HOMEPAGE = "http://www.quagga.net/"
> >> +HOMEPAGE = "http://www.nongnu.org/quagga/"
> >> SECTION = "network"
> >> LICENSE = "GPL-2.0 & LGPL-2.0"
> >> DEPENDS = "readline ncurses perl-native"
> >> @@ -17,6 +17,9 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=8ca43cbc842c2336e835926c2166c28b \
> >>
> >> INC_PR = "r2"
> >>
> >> +# the "ip" command from busybox is not sufficient (flush by protocol flushes all routes)
> >> +RDEPENDS_${PN} += "iproute2"
> >> +
> >> QUAGGASUBDIR = ""
> >> # ${QUAGGASUBDIR} is deal with old versions. Set to "/attic" for old
> >> # versions and leave it empty for recent versions.
> >> @@ -128,11 +131,11 @@ pkg_postinst_${PN} () {
> >> if [ "x$D" != "x" ] ; then
> >> exit 1
> >> fi
> >> - for f in bgpd babeld isisd ospfd ripngd zebra ripd ospf6d; do touch ${sysconfdir}/quagga/$f.conf; done
> >> + for f in bgpd vtysh babeld isisd ospfd ripngd zebra ripd ospf6d; do touch ${sysconfdir}/quagga/$f.conf; done
> >> chown quagga:quaggavty ${sysconfdir}/quagga
> >> chown quagga:quagga ${sysconfdir}/quagga/*.conf
> >> chmod 750 ${sysconfdir}/quagga
> >> - chown 640 ${sysconfdir}/quagga/*.conf
> >> + chmod 640 ${sysconfdir}/quagga/*.conf
> >> if [ -z "$D" ] && [ -e /etc/init.d/populate-volatile.sh ] ; then
> >> ${sysconfdir}/init.d/populate-volatile.sh update
> >> fi
> >> --
> >> 1.7.10.4
> >>
> >> _______________________________________________
> >> Openembedded-devel mailing list
> >> Openembedded-devel at lists.openembedded.org
> >> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> >
> >
> >
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel at lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> >
>
> --
> Best Reagrds,
> Roy | RongQing Li
--
Martin 'JaMa' Jansa jabber: Martin.Jansa at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20130816/cf882ea9/attachment-0002.sig>
More information about the Openembedded-devel
mailing list