svn commit: r216119 - head
Ulrich Spoerlein
uqs at FreeBSD.org
Thu Dec 2 20:43:51 UTC 2010
On Thu, 02.12.2010 at 15:45:22 +0100, Alexander Leidinger wrote:
> Quoting Ulrich Spoerlein <uqs at FreeBSD.org> (from Thu, 2 Dec 2010
> 10:46:05 +0000 (UTC)):
>
> > Author: uqs
> > Date: Thu Dec 2 10:46:05 2010
> > New Revision: 216119
> > URL: http://svn.freebsd.org/changeset/base/216119
> >
> > Log:
> > Remove more duplicates.
> >
> > Just so that I can lookup the command for the next time:
>
> Can you please add it to OldFiles.inc instead, with a comment what it does?
>
> > for t in `make -V TARGETS universe`; do
> > __MAKE_CONF=/dev/null make -f Makefile.inc1 -m $PWD/share/mk TARGET=$t \
> > -V OLD_FILES -V OLD_LIBS -V OLD_DIRS delete-old | \
> > xargs -n1|sort|uniq -c|egrep -v "^ *1 ";
> > done
>
> While it does not matter for "-V OLD*", the spirit of the command is
> more like the check-old target (which is including shared libs) than
> the delete-old target (which does not delete shared libs but only
> static libs). Using check-old would also remove the (wrong, but not
> far fetched) fear of actually deleting something with this command.
Something that bugs me more, is all the TARGET_ARCH conditionals in that
file, that serve practically no purpose. If a file only existed on i386,
and it has been removed on i386, there's no point in wrapping this under
TARGET_ARCH=i386. The wrapping of all lib32 files for amd64 is a great
example :/
The wrapping is only ever required, if some file has been removed from a
set of arches, but must be retained on others.
Anyway, please see attached.
Uli
-------------- next part --------------
A non-text attachment was scrubbed...
Name: obsolete.diff
Type: text/x-diff
Size: 2397 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/svn-src-head/attachments/20101202/dc6eb5c7/obsolete.bin
More information about the svn-src-head
mailing list