svn commit: r229908 - head/usr.bin/brandelf

Alexander Best arundel at freebsd.org
Wed Jan 11 00:47:07 UTC 2012


On Tue Jan 10 12, Dimitry Andric wrote:
> On 2012-01-10 09:21, Pawel Jakub Dawidek wrote:
> >On Tue, Jan 10, 2012 at 02:58:29AM +0000, Eitan Adler wrote:
> >>Author: eadler (ports committer)
> >>Date: Tue Jan 10 02:58:29 2012
> >>New Revision: 229908
> >>URL: http://svn.freebsd.org/changeset/base/229908
> >>
> >>Log:
> >>   Fix warning when compiling with gcc46:
> >>   	 error: variable 'verbose' set but not use
> >
> >Maybe we should remove -v from the manual page as well or document as 
> >no-op?
> >I'd still leave it in the code though, so there is no POLA violation.
> 
> It has been a no-op since the beginning of brandelf, and I have really
> no idea what the original author intended the verbose output to be. :)
> 
> It is probably best to document it as a no-op now, to reduce possible
> confusion.

shouldn't brandelf be replaced with the version from the elftoolchain project?
any news when the first import is going to happen?

cheers.
alex


More information about the svn-src-head mailing list