cvs commit: ports/archivers/unrar Makefile

[Sunpoet Po-Chuan Hsieh]

On Sun, Jan 15, 2012 at 12:35:04PM +0600, Yar wrote:
> Hello Poyopoyo,
> 
> Saturday, January 14, 2012, 7:20:45, you wrote:
> 
> > At Sun, 8 Jan 2012 09:30:33 +0000 (UTC),
> > Sunpoet Po-Chuan Hsieh wrote:
> >> @@ -24,6 +24,9 @@ MAKEFILE=     makefile.unix
> >>  PLIST_FILES=   bin/unrar
> >>  PORTDOCS=      license.txt readme.txt
> >> 
> >> +post-patch:
> >> +       @${REINPLACE_CMD} -e '/^CXX/ s|^|#|' ${WRKSRC}/${MAKEFILE}
> >> +
> >>  do-install:
> >>         ${INSTALL_PROGRAM} ${WRKSRC}/unrar ${PREFIX}/bin
> >>  .if !defined(NOPORTDOCS)
> 
> > Hi,
> > Commenting CXX* out after patch phase breaks unrar-iconv since it
> > requires CXXFLAGS to be set in makefile.unix.
> 
> ===>>  Building for unrar-iconv-4.10,5
> > c++  -O2 -pipe -march=native -fno-strict-aliasing
> > -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -DWITH_ICONV -DUNRAR -c rar.cpp
> > In file included from rar.hpp:5,
> >                  from rar.cpp:1:
> > os.hpp:197:19: error: iconv.h: No such file or directory
> > In file included from rar.hpp:76,
> >                  from rar.cpp:1:
> > global.hpp:19: error: 'iconv_t' does not name a type
> > global.hpp:20: error: 'iconv_t' does not name a type
> > gmake: *** [rar.o] Error 1
> > *** Error code 1
> 
> > Here's a candidate:
> > - Comment CXX* out for only unrar not for unrar-iconv.
> I  vote  for it! \if !defined(USE_ICONV) at unrar Makefile\.
> It's more accuracy at my point of view.
> 
> > - in unrar-iconv do the same thing with files/patch-iconv.
> BUT :) I attached tiny patch for unrar-iconv/Makefile
> with dirty workaround.
> 
> 
> 
> -- 
> Best regards,
>  Yar                          mailto:yarodin at gmail.com

Hi,

I have a suggestion for this build failure.
The patch for makefile.unix in unrar-iconv/files/patch-iconv can be replaced
by the following lines in unrar-iconv/Makefile.

CXXFLAGS+= -DWITH_ICONV -I${LOCALBASE}/include
LDFLAGS+=  -L${LOCALBASE}/lib -liconv

Then, we can remove makefile.unix patch from unrar-iconv/files/patch-iconv.

If you agree with this change, I'll commit it ASAP.
Thanks.

Regards,
sunpoet