bug#26749: closed (mask files from shebang patcher (git, libtool and stagit))

[GNU bug Tracking System]

Your message dated Mon, 16 Nov 2020 17:17:05 +0100
with message-id <871rgtxon2.fsf@gmail.com>
and subject line Re: bug#25304: Libtool’s ltmain.sh still contains a /gnu/store 
shebang
has caused the debbugs.gnu.org bug report #25304,
regarding mask files from shebang patcher (git, libtool and stagit)
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
25304: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=25304
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> stagit: do not patch example.sh shebang
>
>
>
>
> Date: 
>
> Tue, 2 May 2017 16:15:52 +0000
>
>
>
>
> Hi,
> I just noticed that stagit's example.sh has a patched shebang.
> While I would assume that everyone notices when this fails,
> it would probably be good to not make too optimistic assumptions
> and keep #!/bin/sh in the file.
> -- 
> https://pragmatique.xyz
> PGP: https://people.pragmatique.xyz/ng0/
>
>
>
> --- End Message ---

--- Begin Message --- Subject: Re: bug#25304: Libtool’s ltmain.sh still contains a /gnu/store shebang Date: Mon, 16 Nov 2020 17:17:05 +0100 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux)

Hi!

Ludovic Courtès <ludo@gnu.org> writes:
> Hi!
>
> Miguel Ángel Arruga Vivas <rosen644835@gmail.com> skribis:
>
>> I was looking through the lists because I have a patch that does exactly
>> what you describe here.  I guess this goes to core updates, so this
>> version is on top of it.  WDYT?
>
> Yes, looks like a change for ‘core-updates’.
>
>> From 145273418d3131bcf3b73d416d19f641645cf3f8 Mon Sep 17 00:00:00 2001
>> From: =?UTF-8?q?Miguel=20=C3=81ngel=20Arruga=20Vivas?=
>>  <rosen644835@gmail.com>
>> Date: Fri, 13 Nov 2020 15:24:46 +0100
>> Subject: [PATCH] gnu: libtool: Restore shebangs on all libtoolize files.
>
> You can add “Fixes <https://bugs.gnu.org/25304>.”
>
>> * gnu/packages/autotools.scm (libtool)[restore-build-aux-shebang]: New
>> phase after install.
>> [restore-ltmain-shebang]: Remove phase, it is now performed by the phase
>> restore-build-aux-shebang.
>
> LGTM, thanks for digging through old bugs!
>
> Ludo’.

Pushed as 8d263ff254292c9e13a9848f90b52263da52d570 to core-updates with
that line added.  Thanks for the quick review.  :-)

Happy hacking!
Miguel

signature.asc
Description: PGP signature

--- End Message ---