On Wed, Sep 15, 2004 at 09:56:27AM +0100, Jonathan Perkin wrote:
> * On 2004-09-15 at 09:51 BST, Adam Ciarcinski wrote:
> 
> > Added Files:
> > 	pkgsrc/net/samba/patches: patch-ab
> > 
> > Log Message:
> > Strip binaries on install
> 
> Please don't blindly strip packages, as some systems don't support it
> all that well (read "Solaris").  I'd suggest either making it dependant
> on INSTALL_UNSTRIPPED, or simplify that entire section by using the
> INSTALL macro.

We could create a ${TOOLSDIR}/bin/strip that is either a symlink to
the real strip command, or does one that does nothing, and set
STRIP=strip in the {CONFIGURE,MAKE}_ENV.  That should allow us to
prevent stripping without having to patch a whole bunch of packages
or having to check _INSTALL_STRIPFLAG, etc.  Anyone see any flaws in
this idea?

	Cheers,

	-- Johnny Lam <jlam@NetBSD.org>