Re: [PATCH] AS_INIT: don't encode full path

[Luke Mewburn]

On 20-07-04 13:00, Zack Weinberg wrote:
  | On Fri, Jul 3, 2020 at 8:42 PM Luke Mewburn <luke@mewburn.net> wrote:
  | >
  | > Fix AS_INIT to use the basename of __file__ instead of the full path.
  | > Allows for reproducible builds.
  | 
  | I like the idea here, but I think the patch isn't quite ready yet.
  | Most importantly, please write a test for the testsuite to ensure that
  | this doesn't get broken accidentally in the future.  Second, there
  | should be a comment in m4sh.m4 explaining that we are taking the
  | basename of __file__ so that the path to the source directory is not
  | embedded in the output file.  Third, a small tweak for your regular
  | expression: it should be "^.*/\([^/]*\)$" instead of "^.*/\(.*)".  The
  | latter relies too much on the particular greediness behavior of M4's
  | regexp implementation.

Hi Zack,

I've updated the patch to address your comments:
1) Added a testsuite.
2) Added the comment.
3) Improved the regex. (Thanks for the heads up about M4 greedy regex.)

I've attached the revised patch.

Hopefully this still trivial enough to not need a copyright assignment,
since I cribbed most of the new tests from elsewhere in m4sh.at and local.at.
If an assignment is required let me know; happy to provide it.

regards,
Luke.

0001-AS_INIT-basename-__file__.patch
Description: Text document

pgpyVCu8wgHCW.pgp
Description: PGP signature