1 |
>>>>> On Wed, 05 Jun 2019, Michał Górny wrote: |
2 |
|
3 |
> +# @FUNCTION: acct-group_pkg_pretend |
4 |
> +# @DESCRIPTION: |
5 |
> +# Performs sanity checks for correct eclass usage, and early-checks |
6 |
> +# whether requested GID can be enforced. |
7 |
> +acct-group_pkg_pretend() { |
8 |
> + debug-print-function ${FUNCNAME} "${@}" |
9 |
> + |
10 |
> + # verify ACCT_GROUP_ID |
11 |
> + [[ -n ${ACCT_GROUP_ID} ]] || die "Ebuild error: ACCT_GROUP_ID must be set!" |
12 |
> + [[ ${ACCT_GROUP_ID} -ge 0 ]] || die "Ebuild errors: ACCT_GROUP_ID=${ACCT_GROUP_ID} invalid!" |
13 |
|
14 |
Add a sanity check for [[ ${CATEGORY} == acct-group ]] too? |
15 |
|
16 |
> + |
17 |
> + # check for ACCT_GROUP_ID collisions early |
18 |
> + if [[ -n ${ACCT_GROUP_ENFORCE_ID} ]]; then |
19 |
> + local grp=$(egetent group "${ACCT_GROUP_ID}") |
20 |
> + if [[ -n ${grp} ]]; then |
21 |
> + eerror "The required GID is already taken by another group." |
22 |
> + eerror " GID: ${ACCT_GROUP_ID} (needed for ${PN})" |
23 |
> + eerror " current group: ${grp}" |
24 |
> + die "GID ${ACCT_GROUP_ID} taken already" |
25 |
> + fi |
26 |
> + fi |
27 |
> +} |
28 |
|
29 |
> [...] |
30 |
|
31 |
> +# @FUNCTION: acct-user_pkg_pretend |
32 |
|
33 |
And a similar check for acct-user here. |
34 |
|
35 |
Ulrich |