On Wed, May 12, 2010 at 09:26:39AM -0700, David O'Brien wrote: > On Tue, May 11, 2010 at 11:30:41AM -0700, Doug Barton wrote: > > On 05/11/10 00:47, Emanuel Haupt wrote: > > > > > > Doug, the OPTIONS patch as you posted it on ports@ does not play so > > > well with shells/bash-static. I've created shells/bash-static in order > > > to have packages created by the build cluster. > > > > Yeah, I forgot about that port, sorry. I can rework the patch easily > > enough if there is interest. > > An OPTIONS patch would be find - as long as it isn't the default.
The OPTIONS framework should be default, the below patch is just clutter that diverges this port even further from best common practice. Which of the individual options are turned on by default can of course be discussed. -erwin > > -- > -- David ([email protected]) > > Index: Makefile > =================================================================== > RCS file: /home/pcvs/ports/shells/bash/Makefile,v > retrieving revision 1.120 > diff -u -p -r1.120 Makefile > --- Makefile 11 May 2010 09:34:44 -0000 1.120 > +++ Makefile 12 May 2010 16:23:55 -0000 > @@ -31,6 +31,12 @@ COMMENT= The GNU Project's Bourne Again > > IGNOREFILES= FAQ > > +.if defined(WITH_OPTIONS) || defined(WITH_BASH_OPTIONS) > +.include "${.CURDIR}/../bash/options" > +.endif > + > +.include <bsd.port.pre.mk> > + > .if !defined(WITHOUT_IMPLICITCD) > EXTRA_PATCHES+= ${PATCHDIR}/xpatch-implicitcd > .endif > @@ -95,8 +101,6 @@ CONFIGURE_ENV= CPPFLAGS="${CPPFLAGS}" \ > LDFLAGS="${LDFLAGS}" \ > YACC="bison -y" > > -.include <bsd.port.pre.mk> > - > post-patch: > @${REINPLACE_CMD} -e "s|%%PREFIX%%|${PREFIX}|g" ${WRKSRC}/doc/bash.1 > > Index: options > =================================================================== > RCS file: options > diff -N options > --- /dev/null 1 Jan 1970 00:00:00 -0000 > +++ options 12 May 2010 16:23:55 -0000 > @@ -0,0 +1,6 @@ > +OPTIONS= IMPLICITCD "Use directory name alone to cd into it" on \ > + COLONBREAKSWORDS "Colons break words" on \ > + HELP "Install the help files" on \ > + INST_READL "Use readline from base, otherwise bash's" on \ > + NLS "National Language Support" on \ > + STATIC_BASH "Compile without shared libs" off > -- Erwin Lansing http://droso.org Prediction is very difficult especially about the future [email protected]
pgpn5aLomJKWH.pgp
Description: PGP signature
