From: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
To: Richard Purdie <richard.purdie@linuxfoundation.org>,
Otavio Salvador <otavio.salvador@ossystems.com.br>,
"Maxin B. John" <maxin.john@intel.com>
Cc: "OE Core \(openembedded-core@lists.openembedded.org\)"
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCHv2 1/1] Revert "useradd.bbclass: remove user/group created by the package in clean* task"
Date: Thu, 14 Apr 2016 11:46:29 +0000 [thread overview]
Message-ID: <5d0ce4d1b23e4a10adbb3552ff1525fb@XBOX02.axis.com> (raw)
In-Reply-To: <1460630455.9308.168.camel@linuxfoundation.org>
> -----Original Message-----
> From: Richard Purdie [mailto:richard.purdie@linuxfoundation.org]
> Sent: den 14 april 2016 12:41
> To: Peter Kjellerstedt; Otavio Salvador
> Cc: OE Core (openembedded-core@lists.openembedded.org)
> Subject: Re: [OE-core] [PATCHv2 1/1] Revert "useradd.bbclass: remove
> user/group created by the package in clean* task"
>
> On Wed, 2016-04-13 at 17:29 +0100, Richard Purdie wrote:
> > There is a comparatively neat way we could use pkgdata to track the
> > provider of a given user, specifically:
> >
> > http://git.yoctoproject.org/cgit.cgi/poky-contrib/commit/?h=rpurdie/wip&id=5cd646ea185eaafaa341f26310f2eddc75766175
> >
> > The above is a quick patch I've just put together which illustrates
> > what could be done so that the user gets better warnings about
> > conflicting users. It needs cleaning up but thought it worth sharing
> > now as if might give some ideas.
> >
> > This is in keeping with how bitbake detects multiple providers of the
> > same thing.
>
> http://git.yoctoproject.org/cgit.cgi/poky-contrib/commit/?h=rpurdie/wip&id=a362e6da0eec1f7a7d4db19d47f16b1c729562ba
>
> is an improved version.
>
> > > Here I must show my lack of knowledge. How and where should I go
> > > about
> > > adding a regression test that verifies the support for that
> > > multiple
> > > recipes can add the same user/group? Since this does not test a
> > > specific recipe, but rather a part of the build framework, I do not
> > > know if, e.g., ptest is applicable (of which I have no experience
> > > either).
> >
> > oe-selftest would be the place to add something like this, see the
> > /meta/lib/oeqa/selftest directory. We could add some test useradd
> > recipes to meta-selftest.
>
> http://git.yoctoproject.org/cgit.cgi/poky-contrib/commit/?h=rpurdie/wip&id=23134a7d837bc7985ba8e7069d405913c2055a04
>
> is a really simplistic new selftest for useradd. It does however test
> the bug the original patch fixes. There are clearly a much wider range
> of things it could test, I just wanted to show this isn't hard.
>
> Cheers,
>
> Richard
Ok, this looks like a good start for me to continue working on.
What is the easiest way to execute the useradd selftest?
//Peter
next prev parent reply other threads:[~2016-04-14 11:46 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-12 13:18 [PATCHv2 0/1] Revert cleaning of users/groups Peter Kjellerstedt
2016-04-12 13:18 ` [PATCHv2 1/1] Revert "useradd.bbclass: remove user/group created by the package in clean* task" Peter Kjellerstedt
2016-04-12 13:34 ` Otavio Salvador
2016-04-12 14:54 ` Richard Purdie
2016-04-12 16:35 ` Otavio Salvador
2016-04-12 17:28 ` Peter Kjellerstedt
2016-04-13 11:05 ` Richard Purdie
2016-04-13 15:14 ` Peter Kjellerstedt
2016-04-13 16:04 ` Maxin B. John
2016-04-13 16:29 ` Richard Purdie
2016-04-14 10:40 ` Richard Purdie
2016-04-14 11:46 ` Peter Kjellerstedt [this message]
2016-04-14 11:50 ` Richard Purdie
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5d0ce4d1b23e4a10adbb3552ff1525fb@XBOX02.axis.com \
--to=peter.kjellerstedt@axis.com \
--cc=maxin.john@intel.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=otavio.salvador@ossystems.com.br \
--cc=richard.purdie@linuxfoundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.