[Aptitude-devel] Bug#999383: Bug#999383: aptitude: bashism in configure script

[Andrej Shadura]

Hi again,

On Thu, 11 Nov 2021, at 19:12, Axel Beckert wrote:
> I'm not sure why the following patch fixes the issue, but it does so
> at the expense of some more forks per build:
>
> diff --git a/buildlib/docbook.mk b/buildlib/docbook.mk
> index 9ec1b433..de03f0b4 100644
> --- a/buildlib/docbook.mk
> +++ b/buildlib/docbook.mk
> @@ -129,7 +129,7 @@ docbook-man-stamp: $(DOCBOOK_XML) aptitude-man.xsl 
> aptitude-common.xsl
>  	@if [ -x "$(srcdir)/fixman" ]; then \
>  	  for i in $(DOCBOOK_MANS); do \
>  	    echo "$(srcdir)/fixman $$i"; \
> -	    . $(srcdir)/fixman $$i; \
> +	    $(srcdir)/fixman $$i; \
>            done; \
>          fi
>  	touch docbook-man-stamp

Yes, I see why — "." with arguments is bash-only syntax, in POSIX/dash it accepts only the name of the script and ignores any arguments passed:
https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#dot

-- 
Cheers,
  Andrej