From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by yocto-www.yoctoproject.org (Postfix, from userid 118) id 0BC6DE00D05; Thu, 7 Mar 2019 22:00:47 -0800 (PST) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on yocto-www.yoctoproject.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 X-Spam-HAM-Report: * -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% * [score: 0.0000] * -2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at https://www.dnswl.org/, * medium trust * [147.11.1.11 listed in list.dnswl.org] Received: from mail.windriver.com (mail.windriver.com [147.11.1.11]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 6FAABE00D01 for ; Thu, 7 Mar 2019 22:00:46 -0800 (PST) Received: from ALA-HCA.corp.ad.wrs.com ([147.11.189.40]) by mail.windriver.com (8.15.2/8.15.1) with ESMTPS id x2860j3K018112 (version=TLSv1 cipher=AES128-SHA bits=128 verify=FAIL) for ; Thu, 7 Mar 2019 22:00:45 -0800 (PST) Received: from [128.224.162.184] (128.224.162.184) by ALA-HCA.corp.ad.wrs.com (147.11.189.50) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 7 Mar 2019 22:00:45 -0800 To: Mark Hatle , References: <1551927535-18666-1-git-send-email-dengke.du@windriver.com> From: ddu Message-ID: Date: Fri, 8 Mar 2019 14:00:42 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.5.1 MIME-Version: 1.0 In-Reply-To: X-Originating-IP: [128.224.162.184] Subject: Re: [m-c-s][PATCH] libvirt: fix GROUPADD_PARAM override conflict X-BeenThere: meta-virtualization@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: "Discussion of layer enabling hypervisor, virtualization tool stack, and cloud support" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 08 Mar 2019 06:00:47 -0000 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US 在 2019/3/7 下午10:00, Mark Hatle 写道: > On 3/6/19 8:58 PM, Dengke Du wrote: >> The libvirt recipe in meta-virtualization already define the GROUPADD_PARAM, >> so we should append it, not override. >> >> Signed-off-by: Dengke Du >> --- >> meta-openstack/recipes-extended/libvirt/libvirt_4.%.bbappend | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/meta-openstack/recipes-extended/libvirt/libvirt_4.%.bbappend b/meta-openstack/recipes-extended/libvirt/libvirt_4.%.bbappend >> index b55f9a9..0b0f011 100644 >> --- a/meta-openstack/recipes-extended/libvirt/libvirt_4.%.bbappend >> +++ b/meta-openstack/recipes-extended/libvirt/libvirt_4.%.bbappend >> @@ -5,7 +5,7 @@ PACKAGECONFIG ?= "qemu lxc test remote macvtap libvirtd udev yajl \ >> >> inherit useradd >> USERADD_PACKAGES = "${PN}" >> -GROUPADD_PARAM_${PN} = "--system libvirt" >> +GROUPADD_PARAM_${PN}_append = " ;--system libvirt" > The __append should be fine then, but why the ';'? That part looks wrong, since > typically the PARAM is the parameters passed directly to the groupadd program. > I don't believe ';' is a valid parameter. Hi Mark The GROUPADD_PARAM in useradd.bbclass using ';' to create pieces, and pass each piece to groupadd command, https://git.openembedded.org/openembedded-core/tree/meta/classes/useradd.bbclass#n54 such as "-r qemu; -r kvm; --system libvirt" into '-r qemu' '-r kvm' '--system libvirt' to groupadd command. If we don't use the ';'  like this: '-r qemu -r kvm --system libvirt' pass to groupadd command will fail. So we should add ';' > > --Mark > >> do_install_append() { >> sed -e "s:^#unix_sock_group =:unix_sock_group =:g" -i ${D}/etc/libvirt/libvirtd.conf >> >