[PATCH] src/elflint.c: add debug print for GNU_RETAIN and others

Sergei Trofimovich slyfox@gentoo.org
Tue Jan 12 23:28:26 GMT 2021


On Tue, 12 Jan 2021 10:14:55 +0100
Mark Wielaard <mark@klomp.org> wrote:

> Hi Sergei,
> 
> On Thu, 2020-12-31 at 12:11 +0000, Sergei Trofimovich via Elfutils-devel wrote:
> > ---
> >  src/elflint.c | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> > 
> > diff --git a/src/elflint.c b/src/elflint.c
> > index b3cbaade..7b7f7937 100644
> > --- a/src/elflint.c
> > +++ b/src/elflint.c
> > @@ -2789,7 +2789,10 @@ section_flags_string (GElf_Word flags, char *buf, size_t len)
> >        NEWFLAG (OS_NONCONFORMING),
> >        NEWFLAG (GROUP),
> >        NEWFLAG (TLS),
> > -      NEWFLAG (COMPRESSED)
> > +      NEWFLAG (COMPRESSED),
> > +      NEWFLAG (GNU_RETAIN),
> > +      NEWFLAG (ORDERED),
> > +      NEWFLAG (EXCLUDE)
> >      };
> >  #undef NEWFLAG
> >    const size_t nknown_flags = sizeof (known_flags) / sizeof (known_flags[0]);  
> 
> Thanks, this is obviously correct and I pushed it. But normally a patch
> comes with a Signed-off-by line and a ChangeLog entry. I added both.
> But please see:
> https://sourceware.org/git/?p=elfutils.git;a=blob_plain;f=CONTRIBUTING;hb=HEAD

My apologies. Will follow CONTRIBUTING closer to future patches.

Thank you!

-- 

  Sergei


More information about the Elfutils-devel mailing list