<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p><font face="Arial">Thanks!<br>
</font></p>
<p><font face="Arial">Note that checkbashisms 2.17.12-1 failed to
catch this when I debugged this issue.</font></p>
<p><font face="Arial"><a class="moz-txt-link-freetext" href="https://www.archlinux.org/packages/community/any/checkbashisms/">https://www.archlinux.org/packages/community/any/checkbashisms/</a><br>
</font></p>
<p><font face="Arial">Also note that on Arch Linux, I used dash
0.5.9.1-1, whereas Debian only has various older versions</font></p>
<p><font face="Arial"><a class="moz-txt-link-freetext" href="https://packages.debian.org/search?keywords=dash">https://packages.debian.org/search?keywords=dash</a></font></p>
<p>Running shellcheck and checkbashisms regularly is advisable even
for autoconf generated scripts.<br>
</p>
<br>
<div class="moz-cite-prefix">On 22-02-18 12:07, Eike Rathke wrote:<br>
</div>
<blockquote type="cite"
cite="mid:20180222110744.GP12167@remotee.erack.redhat.com">
<pre wrap="">Hi Sander,
On Wednesday, 2018-02-21 14:03:49 +0100, Sander Maijers wrote:
</pre>
<blockquote type="cite">
<pre wrap="">I have set dash as /bin/sh.
</pre>
</blockquote>
<pre wrap="">
"Funny", because Debian has dash there as well and no problem.
Anyway.. this
</pre>
<blockquote type="cite">
<pre wrap="">./configure fails due a syntax error, since `${PRODUCTNAME// /}` not a
portable syntax.
</pre>
</blockquote>
<pre wrap="">
has been changed to
PRODUCTNAME_WITHOUT_SPACES=$(printf %s "$PRODUCTNAME" | sed 's/ //g')
on master.
Eike
</pre>
</blockquote>
<br>
</body>
</html>