Re: [PATCHv2] strip: keep .ctf section in stripped file

["Guillermo E. Martinez" <guillermo.e.martinez@oracle.com>]

On Wed, Feb 22, 2023 at 05:42:45PM +0100, Mark Wielaard wrote:
> Hi,
> 

Hi Mark,

> On Tue, 2022-12-20 at 22:35 +0100, Mark Wielaard wrote:
> > On Wed, Jun 01, 2022 at 10:55:27AM -0500, Guillermo E. Martinez via Elfutils-devel wrote:
> > > This is the second version patch to avoid remove the CTF section in
> > > stripped files. Changes from v1:
> > > 
> > >   - Add description in tests/run-strip-remove-keep-ctf.sh
> > >     mentioning how to regenerate test input file (testfile-ctf)
> > > 
> > > Please let me know your thoughts.
> > > 
> > > [...]
> > > 
> > > CTF debug format was designed to be present in stripped files, so
> > > this section should not be removed, so a new --remove-ctf option
> > > is added to indicate explicitly that .ctf section will be stripped
> > > out from binary file.
> > 
> > Sorry, I see I never reviewed this v2 variant.  I know we tried to
> > coordinate with binutils so eu-strip and binutils strip would do the
> > same thing. And that Jose had an idea for a new section flag to
> > automatically detect what section should/shouldn't be stripped (into a
> > separate .debug file). What was the conclusion of that?
> 
> Any update on this?
> How should .ctf sections be dealt with by strip like tools?
> 

Sorry for late reply,

The conclusion was basically not use section flags to identify which
section should be stripped out or not, so, it requires other mechanisms
for explicitly specifying which sections should be removed, as eu-strip
does using arguments.

https://sourceware.org/bugzilla/show_bug.cgi?id=29737

Worth it mention here, that my last test shows that .ctf section is not
stripped by ue-strip because it doesn't have a "debug" section name. 

Thanks,
guillermo