Control: reopen -1

On Thu, Mar 21, 2019 at 12:54:04AM +0000, Debian Bug Tracking System wrote:
>    * Fix possible infinite loop. (Closes: #924291)

Thank you for the timely fix. Unfortunately, it doesn't work as the
variables are improperly escaped. The build continues to loop.

The patch has:

+       attempts=32; until ./mkkey $(KEYFILE) "Client Of Win" $(DESC) $(MAKER) 
$(COMMENT) "inl,standard2"; do attempts=$(attempts - 1); test $(attempts) -le 0 
&& exit 1; sleep 1; done

It tries to interpolate make variables named "attempts - 1" and
"attempts". Those are empty so the test expression always fails. With
proper escaping it looks like this:

+       attempts=32; until ./mkkey $(KEYFILE) "Client Of Win" $(DESC) $(MAKER) 
$(COMMENT) "inl,standard2"; do attempts=$$((attempts - 1)); test $$(attempts) 
-le 0 && exit 1; sleep 1; done

The double $ is unescaped by make to a single $ and the double braces
are required to perform arithmetic in shell.

Helmut

Reply via email to