<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On 19 September 2016 at 12:57, Peter Kjellerstedt <span dir="ltr">&lt;<a href="mailto:peter.kjellerstedt@axis.com" target="_blank">peter.kjellerstedt@axis.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">&gt; ++            mem_exists=&quot;`grep &quot;^$groupname:[!:]*:[!:]*:\([!,<wbr>]*,\)*$username\(,[!,]*\)*&quot;$<wbr>rootdir/etc/group || true`&quot;<br>
&gt;               if test &quot;x$mem_exists&quot; = &quot;x&quot;; then<br>
&gt;                       bbfatal &quot;${PN}: groupmems command did not succeed.&quot;<br>
&gt;               fi<br>
<br>
</span>The above change cannot be correct. Changing the expression &quot;[^:]&quot;<br>
(which means &quot;anything but :&quot;) to &quot;[!:]&quot; (which means &quot;! or :&quot;) is<br>
definitely not the same....</blockquote></div><br>My prediction is that this series is generated by running checkbashism over the fragments, it fires a false-positive here (from memory because it thinks it is processing a bash glob or something, not grep regex).</div><div class="gmail_extra"><br></div><div class="gmail_extra">Note that I just submitted my verify-bashism script which has a whitelist for things - such as use of command - which whilst not in POSIX are in fact in bash/dash/ash so for all intents and purposes will work.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Ross</div></div>