From: Joe MacDonald <Joe.MacDonald@windriver.com>
To: <jackie.huang@windriver.com>
Cc: openembedded-devel@lists.openembedded.org, Zhenfeng.Zhao@windriver.com
Subject: Re: [meta-networking][PATCH] ebtables: move ebtables-save to sbindir to avoid QA issue
Date: Mon, 25 Feb 2013 10:18:20 -0500 [thread overview]
Message-ID: <20130225151820.GE4330@windriver.com> (raw)
In-Reply-To: <1361769851-28342-1-git-send-email-jackie.huang@windriver.com>
[-- Attachment #1: Type: text/plain, Size: 1865 bytes --]
Merged, thanks.
-J.
[[meta-networking][PATCH] ebtables: move ebtables-save to sbindir to avoid QA issue] On 13.02.25 (Mon 13:24) jackie.huang@windriver.com wrote:
> From: Jackie Huang <jackie.huang@windriver.com>
>
> Fix the QA issue:
> WARNING: QA Issue: ebtables: Found a reference to /usr/ in
> /path/to/tmp/work/x86_64-wrs-linux/ebtables-2.0.10-4-r1/packages-split/ebtables/sbin/ebtables-save
> WARNING: QA Issue: Shell scripts in base_bindir and base_sbindir should not reference anything in exec_prefix
>
> Signed-off-by: Jackie Huang <jackie.huang@windriver.com>
> ---
> .../recipes-filter/ebtables/ebtables_2.0.10-4.bb | 7 ++++++-
> 1 files changed, 6 insertions(+), 1 deletions(-)
>
> 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 bca34c0..d6d803a 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
> @@ -2,7 +2,7 @@ SUMMARY = "Utility for basic Ethernet frame filtering on a Linux bridge, MAC NAT
> LICENSE = "GPLv2"
> LIC_FILES_CHKSUM = "file://COPYING;md5=53b4a999993871a28ab1488fdbd2e73e"
> SECTION = "console/network"
> -PR = "r1"
> +PR = "r2"
>
> RDEPENDS_${PN} += "perl"
>
> @@ -44,6 +44,11 @@ do_install () {
> # Fix hardcoded paths in scripts
> sed -i 's!/sbin/!${base_sbindir}/!g' ${D}/${sysconfdir}/init.d/ebtables
> sed -i 's!/etc/!${sysconfdir}/!g' ${D}/${sysconfdir}/init.d/ebtables
> +
> + # The script ebtables-save refernces perl in exec_prefix, so
> + # move it to sbindir to avoid QA issue
> + install -d ${D}/${sbindir}
> + mv ${D}/${base_sbindir}/ebtables-save ${D}/${sbindir}
> }
>
> CONFFILES_${PN} += "${sysconfdir}/default/ebtables"
--
-Joe MacDonald.
:wq
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
prev parent reply other threads:[~2013-02-25 15:34 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-25 5:24 [meta-networking][PATCH] ebtables: move ebtables-save to sbindir to avoid QA issue jackie.huang
2013-02-25 15:18 ` Joe MacDonald [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130225151820.GE4330@windriver.com \
--to=joe.macdonald@windriver.com \
--cc=Zhenfeng.Zhao@windriver.com \
--cc=jackie.huang@windriver.com \
--cc=openembedded-devel@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox