From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from p3plsmtpa07-01.prod.phx3.secureserver.net (p3plsmtpa07-01.prod.phx3.secureserver.net [173.201.192.230]) by mail.openembedded.org (Postfix) with ESMTP id 6339F65CED for ; Wed, 12 Nov 2014 11:15:13 +0000 (UTC) Received: from [192.168.65.10] ([75.72.225.8]) by p3plsmtpa07-01.prod.phx3.secureserver.net with id EbF71p00L0BVjqb01bF8JQ; Wed, 12 Nov 2014 04:15:12 -0700 Message-ID: <5463413B.3050902@pabigot.com> Date: Wed, 12 Nov 2014 05:15:07 -0600 From: "Peter A. Bigot" Organization: Peter Bigot Consulting, LLC User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: zhangxiao References: <1414861479-5493-1-git-send-email-pab@pabigot.com> <1414861479-5493-3-git-send-email-pab@pabigot.com> <54630E02.3070500@windriver.com> In-Reply-To: <54630E02.3070500@windriver.com> Cc: openembedded-core@lists.openembedded.org Subject: Re: [PATCH 2/3] pseudo: provide fallback passwd and group files X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 12 Nov 2014 11:15:17 -0000 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit On 11/12/2014 01:36 AM, zhangxiao wrote: > On 2014年11月02日 01:04, Peter A. Bigot wrote: >> Normally pseudo is built with --without-passwd-fallback, which requires >> that somebody provide target passwd and group files. Those come from >> base-passwd in OE, but base-passwd cannot be built without first >> invoking operations under pseudo that require getpw*/getgr*. >> >> Provide the absolute minimum stub files, matching in content what will >> eventually be on the target, that can be used in the cases where the >> target files are not yet available. The requirements for minimum stub >> are the usernames and groups identified in meta/files/fs-perms.txt. >> >> Signed-off-by: Peter A. Bigot >> --- >> meta/recipes-devtools/pseudo/files/fallback-group | 2 ++ >> meta/recipes-devtools/pseudo/files/fallback-passwd | 1 + >> meta/recipes-devtools/pseudo/pseudo_1.6.2.bb | 8 ++++++++ >> 3 files changed, 11 insertions(+) >> create mode 100644 meta/recipes-devtools/pseudo/files/fallback-group >> create mode 100644 meta/recipes-devtools/pseudo/files/fallback-passwd >> >> diff --git a/meta/recipes-devtools/pseudo/files/fallback-group >> b/meta/recipes-devtools/pseudo/files/fallback-group >> new file mode 100644 >> index 0000000..81bf732 >> --- /dev/null >> +++ b/meta/recipes-devtools/pseudo/files/fallback-group >> @@ -0,0 +1,2 @@ >> +root:*:0: > > How about adding those two lines below? > > daemon:*:1: > bin:*:2: What's the rationale for adding these? My intent is for pseudo to provide the absolute minimum required for the default fs-perms.txt, not to attempt to provide everything that's in base-passwd. If you override fs-perms.txt somehow and need those extra groups in early build stages, I think the right approach is to also provide your own fallback-group and fallback-passwd at the same time. If too much gets added to these fallback specifications, we're back in the same position of getting target files that don't match the configuration used during build/packaging. In most cases, adding DEPENDS += "base-passwd" is going to be the right approach to things not being in the fallback files. Peter > > Thanks > Xiao > >> +mail:*:8: >> diff --git a/meta/recipes-devtools/pseudo/files/fallback-passwd >> b/meta/recipes-devtools/pseudo/files/fallback-passwd >> new file mode 100644 >> index 0000000..c1458dc >> --- /dev/null >> +++ b/meta/recipes-devtools/pseudo/files/fallback-passwd >> @@ -0,0 +1 @@ >> +root::0:0:root:/home/root:/bin/sh >> diff --git a/meta/recipes-devtools/pseudo/pseudo_1.6.2.bb >> b/meta/recipes-devtools/pseudo/pseudo_1.6.2.bb >> index fb85f89..95fd2b8 100644 >> --- a/meta/recipes-devtools/pseudo/pseudo_1.6.2.bb >> +++ b/meta/recipes-devtools/pseudo/pseudo_1.6.2.bb >> @@ -5,9 +5,17 @@ SRC_URI = " \ >> file://0001-pseudo_client.c-protect-pwd_lck-against-magic.patch \ >> file://0002-pseudo_util-modify-interface-to-pseudo_etc_file.patch \ >> file://0003-pseudo_client.c-support-multiple-directories-in-PSEU.patch \ >> + file://fallback-passwd \ >> + file://fallback-group \ >> " >> >> SRC_URI[md5sum] = "4d7b4f9d1b4aafa680ce94a5a9a52f1f" >> SRC_URI[sha256sum] = >> "c72be92689511ced7c419149c6aaa1b1a9e4dfc6409d1f16ab72cc35bc1e376a" >> >> PSEUDO_EXTRA_OPTS ?= "--enable-force-async --without-passwd-fallback" >> + >> +do_install_append_class-native () { >> + install -d ${D}${sysconfdir} >> + install -m 644 ${WORKDIR}/fallback-passwd ${D}${sysconfdir}/passwd >> + install -m 644 ${WORKDIR}/fallback-group ${D}${sysconfdir}/group >> +} >>