On Wed, 9 Jan 2013 17:50:10 +0100
Marc Espie <es...@nerim.net> wrote:

> On Wed, Jan 09, 2013 at 04:50:35PM +0100, Christopher Zimmermann
> wrote:
> > Hi,
> > 
> > I'd like to commit this. ok?
> > I don't see how this indirection could make any sense otherwise.
> > 
> > Christopher
> > 
> > 
> > Index: libiconv.port.mk
> > ===================================================================
> > RCS file: /cvs/ports/converters/libiconv/libiconv.port.mk,v
> > retrieving revision 1.6
> > diff -u -p -r1.6 libiconv.port.mk
> > --- libiconv.port.mk    15 Nov 2010 00:22:50 -0000      1.6
> > +++ libiconv.port.mk    9 Jan 2013 15:48:06 -0000
> > @@ -3,10 +3,10 @@
> >  # The RUN_DEPENDS entry is to ensure libiconv is installed. This is
> >  # necessary so that we have charset.alias installed on static
> >  # archs. Typically installed in PREFIX/lib.
> > -MODLIBICONV_LIB_DEPENDS =      converters/libiconv
> > -MODLIBICONV_RUN_DEPENDS =      converters/libiconv
> > +MODLIBICONV_LIB_DEPENDS ?=     converters/libiconv
> > +MODLIBICONV_RUN_DEPENDS ?=     converters/libiconv
> >  
> > -MODLIBICONV_WANTLIB =          iconv>=2
> > +MODLIBICONV_WANTLIB ?=         iconv>=2
> >  
> >  LIB_DEPENDS +=                 ${MODLIBICONV_LIB_DEPENDS}
> >  RUN_DEPENDS +=                 ${MODLIBICONV_RUN_DEPENDS}
> 
> You're lacking imagination.
> This indirection still makes sense, in the context of MULTI_PACKAGES.
> 

Ah, I get it. But I'd still like to commit the diff. OK?

Attachment: pgpLps3U8LO3p.pgp
Description: PGP signature

Reply via email to