cvs commit: src Makefile.inc1 ObsoleteFiles.inc src/share/man/man7 build.7

Jeremie Le Hen jeremie at le-hen.org
Sun Jul 31 18:59:06 GMT 2005


Hi Alexander,

> > BTW, I don't really see why {check,delete}-old-* should be restricted
> > to native environments.  DESTDIR might point to e.g. an NFS-exported
> > filesystem from which thin clients of a different architecture boot.
> 
> Because of ObsoleteFiles.inc:
> ---snip---
> .if ${MACHINE_ARCH} != "alpha" && ${MACHINE_ARCH} != "sparc64"
> # 20040130: libkse renamed to libpthread
> OLD_FILES+=usr/lib/libkse.a
> OLD_FILES+=usr/lib/libkse.so
> .endif
> ---snip---
> 
> We could change this to TARGET_ARCH instead, but then the user has to
> set it, else he removes non-obsolete files. Even when we make it a
> documented requirement, I don't have a good feeling regarding this.
> There's too much foot-shooting potential if someone forgets to set it.
> I think it's more user friendly / ergonomic to keep the current
> restriction. With the restriction in place you need to have 2
> files (Makefile.inc1 and ObsoleteFiles.inc) or the NFS exported src
> directory on one client to remove the old files. This is a little bit
> more work, but forces the administrator to think a little bit about what
> he does. It's part of my "as failsafe as possible" design of this
> feature.

This sounds as a hack, but couldn't you use file(1) on ${DESTDIR}/bin/ls
to identify the architecture it's running ?  It would at least avoid
breaking POLA.

Regards,
-- 
Jeremie Le Hen
< jeremie at le-hen dot org >< ttz at chchile dot org >


More information about the cvs-src mailing list