[PATCH] Make PE images deterministic by default.

Cory Fields cory@coryfields.com
Thu Nov 21 19:25:00 GMT 2013


Hi Nick

Done, see Patch V2.

I'm not quite sure what the standard procedure is, but Tristan has
approved this series to be backported to 2.24.

As well as committing this change and my other outstanding approved
one (Make windres output deterministic by using constant (zero)
timestamps), would you mind pushing to the 2.24 branch as well? Or
should I ask Tristan for that?

Backports requested are:
5cd84a722e09f301d1e134d075e1cfa421db08d6
[Make windres output deterministic by using constant (zero) timestamps]
[Make PE images deterministic by default.]

Regards,
Cory

On Thu, Nov 21, 2013 at 4:53 AM, nick clifton <nickc@redhat.com> wrote:
> Hi Cory,
>
>
>> binutils/Changelog
>> 2013-11-20  Cory Fields  <cory@coryfields.com>
>>      * bfd/libcoff-in.h: Add insert_timestamp flag to the pe_data struct.
>>      * bfd/libcoff.h: Update after change to bfd/libcoff-in.h.
>>      * bfd/peXXigen.c (_bfd_XXi_only_swap_filehdr_out): Only use a real
>>        timestamp if --insert-timestamp was used.
>>      * ld/pe-dll.c (fill_edata): Likewise.
>>      * ld/emultempl/pe.em: Add the --insert-timestamp option.
>>      * ld/emultempl/pep.em: Likewise for 64bit.
>>      * ld/ld.texinfo:  Document the --insert-timestamp option.
>
>
> Approved, but with one small caveat:
>
>
>> +  // Only use a real timestamp if the option was chosen
>
>
> Could we have a C-style comment here please ?
>
> Cheers
>   Nick
>
>



More information about the Binutils mailing list