[PATCH 01/45] *: Regenerate autoconf and aclocal files

Arsen Arsenović arsen@aarsen.me
Thu Aug 10 09:13:22 GMT 2023


Alan Modra <amodra@gmail.com> writes:

> On Mon, Aug 07, 2023 at 01:07:04PM +0200, Arsen Arsenović via Binutils wrote:
>> bfd/ChangeLog:
>> 
>> 	* aclocal.m4: Regenerate.
>> 
>> gold/ChangeLog:
>> 
>> 	* aclocal.m4: Regenerate.
>> 
>> intl/ChangeLog:
>> 
>> 	* aclocal.m4: Regenerate.
>
> Hmm, these three and the gprofng changes are *not* regeneration,
> because the files don't need regenerating at the start of a patch
> series.

Without this base commit, this diff would be inadvertently merged into
subsequent patches, so I decided to separate it out at the start to get
more self-contained patches.

> What's more, if I apply your patch series then when I regenerate at
> the end I get different generated files..

Yes, I also found this odd.  I couldn't find the cause at the time of
writing.

> Maybe it would be better if changes to generated files are left to a
> final patch that does a bulk regeneration.  It would certainly be
> easier to review.

Hmm, my impression is that it'd be easier to handle these patches if
each commit had accurate generated files.  I can switch to a
regeneration after the fact if so desired.
-- 
Arsen Arsenović
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 381 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20230810/6f828ccd/attachment.sig>


More information about the Binutils mailing list