svn commit: r416270 - head/net/rsync

[Emanuel Haupt]

Adam Weinberger <adamw at adamw.org> wrote:
> > On 2 Jun, 2016, at 1:35, Emanuel Haupt <ehaupt at FreeBSD.org> wrote:
> > 
> > Author: ehaupt
> > Date: Thu Jun  2 07:35:49 2016
> > New Revision: 416270
> > URL: https://svnweb.freebsd.org/changeset/ports/416270
> > 
> > Log:
> >  Fix the logic of the POPT_PORT option.
> > 
> >  - POPT_CONFIGURE_WITH should actually be POPT_PORT_CONFIGURE_WITH
> >  - The whole ^POPT block logic is reversing the logic
> >  - Bump PORTREVISION since we have a change of dependencie
> > 
> >  PR:		209837 (based on)
> >  Submitted by:	Martin Birgmeier <la5lbtyi at aon.at>
> >  MFH:		2016Q2
> > 
> > -POPT_PORT_LIB_DEPENDS=	libpopt.so:devel/popt
> > -POPT_PORT_CPPFLAGS=	-I${LOCALBASE}/include
> > -POPT_PORT_LDFLAGS=	-L${LOCALBASE}/lib
> > -POPT_CONFIGURE_WITH=	included-popt
> > +POPT_PORT_LIB_DEPENDS_OFF=	libpopt.so:devel/popt
> > +POPT_PORT_CPPFLAGS_OFF=	-I${LOCALBASE}/include
> > +POPT_PORT_LDFLAGS_OFF=	-L${LOCALBASE}/lib
> > +
> > +POPT_PORT_CONFIGURE_WITH=	included-popt
> 
> Wait, now you've made it backwards.
> 
> Now:
> "Use popt from devel/popt"
> ON = use bundled popt
> OFF = use popt from devel/popt
> 
> Those need to be switched!

Of course you're right.

Emanuel