summaryrefslogtreecommitdiff
path: root/meta/classes/useradd.bbclass
diff options
context:
space:
mode:
authorScott Garman <scott.a.garman@intel.com>2011-11-08 18:23:48 -0800
committerRichard Purdie <richard.purdie@linuxfoundation.org>2011-11-10 11:35:30 +0000
commit2f73466eb5018040a123ccb0e2af8c519525f958 (patch)
tree0f5241c879bf3655405c240e587b217e120ee2a3 /meta/classes/useradd.bbclass
parent11506f702a83a77d06d259ce05c89808d53a9661 (diff)
downloadopenembedded-core-2f73466eb5018040a123ccb0e2af8c519525f958.tar.gz
openembedded-core-2f73466eb5018040a123ccb0e2af8c519525f958.tar.bz2
openembedded-core-2f73466eb5018040a123ccb0e2af8c519525f958.tar.xz
openembedded-core-2f73466eb5018040a123ccb0e2af8c519525f958.zip
useradd.bbclass: only modify packages in USERADD_PACKAGES
Previously we injected the user/group preinstall script into all output packages. This fixes that so that only packages listed in USERADD_PACKAGES get modified. It also removes the USERADDPN variable, which is no longer needed. Signed-off-by: Scott Garman <scott.a.garman@intel.com>
Diffstat (limited to 'meta/classes/useradd.bbclass')
-rw-r--r--meta/classes/useradd.bbclass42
1 files changed, 16 insertions, 26 deletions
diff --git a/meta/classes/useradd.bbclass b/meta/classes/useradd.bbclass
index a8a1c14f4..3b2d1dbfe 100644
--- a/meta/classes/useradd.bbclass
+++ b/meta/classes/useradd.bbclass
@@ -1,5 +1,3 @@
-USERADDPN ?= "${PN}"
-
# base-passwd-cross provides the default passwd and group files in the
# target sysroot, and shadow -native and -sysroot provide the utilities
# and support files needed to add and modify user and group accounts
@@ -106,30 +104,29 @@ SYSROOTPOSTFUNC_virtclass-nativesdk = ""
# Recipe parse-time sanity checks
def update_useradd_after_parse(d):
- if not d.getVar('USERADD_PACKAGES', False):
- if not d.getVar('USERADD_PARAM', False) and not d.getVar('GROUPADD_PARAM', False):
- raise bb.build.FuncFailed, "%s inherits useradd but doesn't set USERADD_PARAM or GROUPADD_PARAM" % bb.data.getVar('FILE', d)
+ useradd_packages = d.getVar('USERADD_PACKAGES', True)
+
+ if not useradd_packages:
+ raise bb.build.FuncFailed, "%s inherits useradd but doesn't set USERADD_PACKAGES" % bb.data.getVar('FILE', d)
+
+ for pkg in useradd_packages.split():
+ if not d.getVar('USERADD_PARAM_%s' % pkg, True) and not d.getVar('GROUPADD_PARAM_%s' % pkg, True):
+ raise bb.build.FuncFailed, "%s inherits useradd but doesn't set USERADD_PARAM or GROUPADD_PARAM for package %s" % (bb.data.getVar('FILE', d), pkg)
python __anonymous() {
update_useradd_after_parse(d)
}
# Return a single [GROUP|USER]ADD_PARAM formatted string which includes the
-# [group|user]add parameters for all packages in this recipe
+# [group|user]add parameters for all USERADD_PACKAGES in this recipe
def get_all_cmd_params(d, cmd_type):
import string
param_type = cmd_type.upper() + "ADD_PARAM_%s"
params = []
- pkgs = d.getVar('USERADD_PACKAGES', True)
- if not pkgs:
- pkgs = d.getVar('USERADDPN', True)
- packages = (d.getVar('PACKAGES', True) or "").split()
- if packages and pkgs not in packages:
- pkgs = packages[0]
-
- for pkg in pkgs.split():
+ useradd_packages = d.getVar('USERADD_PACKAGES', True) or ""
+ for pkg in useradd_packages.split():
param = d.getVar(param_type % pkg, True)
if param:
params.append(param)
@@ -156,17 +153,10 @@ fakeroot python populate_packages_prepend () {
rdepends = d.getVar("RDEPENDS_%s" % pkg, True) or ""
rdepends += " base-passwd shadow"
bb.data.setVar("RDEPENDS_%s" % pkg, rdepends, d)
-
- # We add the user/group calls to all packages to allow any package
- # to contain files owned by the users/groups defined in the recipe.
- # The user/group addition code is careful not to create duplicate
- # entries, so this is safe.
- pkgs = d.getVar('USERADD_PACKAGES', True)
- if not pkgs:
- pkgs = d.getVar('USERADDPN', True)
- packages = (d.getVar('PACKAGES', True) or "").split()
- if packages and pkgs not in packages:
- pkgs = packages[0]
- for pkg in pkgs.split():
+
+ # Add the user/group preinstall scripts and RDEPENDS requirements
+ # to packages specified by USERADD_PACKAGES
+ useradd_packages = d.getVar('USERADD_PACKAGES', True) or ""
+ for pkg in useradd_packages.split():
update_useradd_package(pkg)
}