Re: WITHOUT_JAIL and make delete-old{,-libs}

On Sun Mar 20 11, David Demelier wrote:
> Hello,
> 
> I was surprised to see there is no ${MK_JAIL} conditional to remove old 
> files on 8.2-RELEASE so I started to write it without watching if 
> -CURRENT already make it in 
> /usr/src/tools/build/mk/OptionalObsoleteFiles.inc.

i think it's best to post any OptionalObsoleteFiles.inc related additions as
problem reports.
i've also added quite a lot of files for not yet implemented WITHOUT_* knobs
and sent them to freebsd-{current,hackers}, but nobody was willing to commit
my patches. my experience so far has been:

- for patches that fix bugs, it's better to send these to the mailinglists
- for patches that implement any new functionality the problem report database
  is better suited and chances are much higher your patch will get picked up by
  somebody with commit rights

cheers.
alex

> 
> .if ${MK_JAIL} == no
> OLD_FILES+=usr/sbin/jail
> OLD_FILES+=usr/sbin/jexec
> OLD_FILES+=usr/sbin/jls
> OLD_FILES+=usr/share/man/man8/jail.8.gz
> OLD_FILES+=usr/share/man/man8/jexec.8.gz
> OLD_FILES+=usr/share/man/man8/jls.8.gz
> .endif
> 
> I personnaly added more files :
> 
> OLD_LIBS+=lib/libjail.so.1
> OLD_LIBS+=usr/lib/libjail.a
> OLD_LIBS+=usr/lib/libjail_p.a
> OLD_FILES+=usr/lib/libjail.so
> OLD_FILES+=etc/rc.d/jail
> 
> (/usr/lib/libjail.so is a symbolic link)
> 
> I think they should be removed too, thus can you merge it to -STABLE if 
> it's not already done? (sorry I'm not used to the cvs web interface and 
> I don't have -STABLE right now)
> 
> Cheers,
> 
> -- 
> David Demelier

-- 
a13x