[OE-core] [PATCH 1/1] useradd_base.bbclass: fix simultaneous with flock
Kang Kai
Kai.Kang at windriver.com
Wed Feb 17 05:40:27 UTC 2016
On 2016年02月16日 20:09, Burton, Ross wrote:
>
> On 16 February 2016 at 07:14, <kai.kang at windriver.com
> <mailto:kai.kang at windriver.com>> wrote:
>
> +FLOCK_FILE=$SYSROOT/etc
>
>
> useradd_base is also used by extrausers.bbclass, so it would be best
> if useradd_base decided where to use as a lock ($rootdir/${sysconfdir}
> sounds sensible) instead of expecting that the caller sets FLOCK_FILE
> and mysteriously failing if it wasn't set.
OK. Sounds good.
>
> Also if we're locking and waiting properly instead of using timeouts
> and loops, can we remove the "does it exist?" check before the
> operation (eg groupadd -f will exit 0 if the group exists) and after
> (with proper locking that shouldn't be happening).
I prefer to keep such check. There is more than one case that causes
operation error such as pass wrong parameters. The messages are helpful
for developers.
Thanks,
Kai
>
> Ross
--
Regards,
Neil | Kai Kang
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20160217/2ad94e44/attachment-0002.html>
More information about the Openembedded-core
mailing list