Re: [daisy][PATCH 1/3] layer.conf: Add in useradd dependencies as ABISAFE dependencies

["Peter A. Bigot" <pab@pabigot.com>]

On 09/30/2014 07:55 PM, Denys Dmytriyenko wrote:
> On Thu, Sep 25, 2014 at 02:17:55PM +0200, Steffen Sledz wrote:
>> Currently allarch recipes using useradd rebuild each time MACHINE
>> changes which is not desireable. Adding the useradd dependencies
>> to this list ensures they do not change under these circumstances.
>>
>> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>> Signed-off-by: Steffen Sledz <sledz@dresearch-fe.de>
>> ---
>>   meta/conf/layer.conf | 4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf
>> index 18609dd..a764ee0 100644
>> --- a/meta/conf/layer.conf
>> +++ b/meta/conf/layer.conf
>> @@ -30,4 +30,8 @@ SIGGEN_EXCLUDERECIPES_ABISAFE += " \
>>     udev-extraconf \
>>     packagegroup-x11-xserver \
>>     systemd-serialgetty \
>> +  initscripts \
> This line above ^^^^ has a very long trail of whitespaces and it got commited
> that way, unfortunately :(

If OE developers and patch submitters do:

   cd .git/hooks
   mv pre-commit.sample pre-commit

in each layer, such errors will be detected at commit time. Wouldn't fix 
issues with whitespace in pulled material or git am but it could reduce 
introducing the problem.

Peter