This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: PATCH: PR ld/11146: Add "readelf --dyn-syms"
- From: Daniel Jacobowitz <dan at codesourcery dot com>
- To: Nick Clifton <nickc at redhat dot com>
- Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, binutils at sourceware dot org
- Date: Mon, 11 Jan 2010 08:51:22 -0500
- Subject: Re: PATCH: PR ld/11146: Add "readelf --dyn-syms"
- References: <20100107192613.GA11400@lucon.org> <4B4B1953.8070007@redhat.com>
On Mon, Jan 11, 2010 at 12:28:03PM +0000, Nick Clifton wrote:
> Hi H.J.
>
> >binutils/
> >2010-01-07 H.J. Lu <hongjiu.lu@intel.com>
> >
> > PR ld/11146
> > * readelf.c (do_dyn_syms): New.
> > (OPTION_DYN_SYMBS): Likewise.
Why syms versus symbs?
It seems to me very confusing to have -Ds (dynamic symbols) and
--dyn-syms. The description in the manual sounds like -Ds and
--dyn-syms should both do what your new patch does. I wouldn't
be able to tell the difference. Is the existing -Ds useful? If both
are useful, the manual should differentiate them. If the other is not
useful, just change -Ds.
--
Daniel Jacobowitz
CodeSourcery