Re: [PATCH v2] gold: add --package-metadata

[Luca Boccassi <bluca@debian.org>]

[-- Attachment #1: Type: text/plain, Size: 1288 bytes --]

On Tue, 2022-08-02 at 11:50 +0100, Luca Boccassi wrote:
> On Fri, 29 Jul 2022 at 23:29, <luca.boccassi@gmail.com> wrote:
> > 
> > From: Luca Boccassi <bluca@debian.org>
> > 
> > Following the same format as the implementation in ld:
> > 9e2bb0cb5e74aed4158f08495534922d7108f928
> > 
> > Generate a .note.package FDO package metadata ELF note, following
> > the spec: https://systemd.io/ELF_PACKAGE_METADATA/
> > 
> > If the jansson library is available at build time (and it is explicitly
> > enabled), link ld to it, and use it to validate that the input is
> > correct JSON, to avoid writing garbage to the file. The
> > configure option --enable-jansson has to be used to explicitly enable
> > it (error out when not found). This allows bootstrappers (or others who
> > are not interested) to seamlessly skip it without issues.
> > ---
> > v2: apply fixes from review
> 
> Hello Cary,
> 
> Anything else you'd like me to change here? Thanks!

CC'ing Ian on suggestion from Nick. Would be great if a maintainer
could have a look and let me know if we need more changes.

Meanwhile, Fedora has backported this patch:

https://src.fedoraproject.org/rpms/binutils/c/9ee9ffc894cc632b54d862d6213e2d8fcb8db829?branch=rawhide

-- 
Kind regards,
Luca Boccassi

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]