Re: pkg/56497: emacs 27 fails to build on Solaris 10 (autogen.sh shell)

[Claes Nästén <pekdon%gmail.com@localhost>]

The following reply was made to PR pkg/56497; it has been noted by GNATS.

From: =?utf-8?q?Claes_N=C3=A4st=C3=A9n?= <pekdon%gmail.com@localhost>
To: gnats-bugs <gnats-bugs%netbsd.org@localhost>
Cc: 
Subject: Re: pkg/56497: emacs 27 fails to build on Solaris 10 (autogen.sh shell)
Date: Sun, 14 Nov 2021 09:42:27 +0100

 --=-1636879347-193339-10393-9446-3-=
 Content-Type: text/plain; charset=UTF-8
 Content-Disposition: inline
 
 Excerpts from Jonathan Perkin's message of 2021-11-14 08:15:01 +0000:
 > The following reply was made to PR pkg/56497; it has been noted by GNATS.
 > 
 > From: Jonathan Perkin <jperkin%joyent.com@localhost>
 > To: gnats-bugs%netbsd.org@localhost
 > Cc: pkg-manager%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, pkgsrc-bugs%netbsd.org@localhost,
 >     pekdon%gmail.com@localhost
 > Subject: Re: pkg/56497: emacs 27 fails to build on Solaris 10 (autogen.sh
 >  shell)
 > Date: Sun, 14 Nov 2021 08:10:07 +0000
 > 
 >  * On 2021-11-13 at 21:55 GMT, nia wrote:
 >  
 >  > Maybe use bootstrap --full? We probably should replace Solaris 10
 >  > sh with mksh as we do on some other platforms.
 >  
 >  We already do, bootstrap uses bash if available otherwise mksh, so not 
 >  sure what else might be going on here.  May just be /bin/sh hardcoded 
 >  somewhere rather than using CONFIG_SHELL.
 >  
 
 See the attached Makefile.in patch, where autogen.sh is called as is
 (and it has #!/bin/sh hardcoded). Using sh to exec it seems to solve the
 issue but a patch is more work to maintain compared to just replacing
 the shell don't you think?
 
 
 --=-1636879347-193339-10393-9446-3-=
 Content-Disposition: attachment; filename="patch-Makefile.in"
 Content-Type: text/plain; name="patch-Makefile.in"
 Content-Transfer-Encoding: quoted-printable
 
 $NetBSD$
 
 --- Makefile.in.orig	2021-11-14 08:33:10.884831000 +0000
 +++ Makefile.in
 @@ -456,7 +456,7 @@ config.status: ${srcdir}/configure
  	fi
  =
 
  $(srcdir)/configure: $(srcdir)/configure.ac $(srcdir)/m4/*.m4
 -	cd $(srcdir) && ./autogen.sh autoconf
 +	cd $(srcdir) && sh ./autogen.sh autoconf
  =
 
  # =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D Installat=
 ion =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
  =
 
 @@ -1114,7 +1114,7 @@ check-info: info
  #  * Rebuild Makefile, to update the build procedure itself.
  #  * Do the actual build.
  bootstrap: bootstrap-clean
 -	cd $(srcdir) && ./autogen.sh autoconf
 +	cd $(srcdir) && sh ./autogen.sh autoconf
  	$(MAKE) MAKEFILE_NAME=3Dforce-Makefile force-Makefile
  	$(MAKE) all
  =
 
 
 --=-1636879347-193339-10393-9446-3-=--