On Sat, Jan 28, 2012 at 04:56:33PM -0500, Brad wrote:
> On 28/01/12 4:54 PM, Brad wrote:
> >On 28/01/12 3:20 AM, Alexandr Shadchin wrote:
> >>On Wed, Dec 21, 2011 at 01:00:31AM +0600, Alexandr Shadchin wrote:
> >>>Hi,
> >>>
> >>>Remove audio/faad from LIB_DEPENDS, streamripper not use this library.
> >>>(ripaac.o is empty)
> >>>
> >>>Comments ? OK ?
> >>>
> >>
> >>No objection ?
> >
> >Does this not build Ok without the additional patches?
> 
> and while you're there is USE_GROFF still necessary?
> 

Yes, USE_GROFF is necessary.
Build does ok without the additional patches (remove unneeded patches).

-- 
Alexandr Shadchin

Index: Makefile
===================================================================
RCS file: /cvs/ports/audio/streamripper/Makefile,v
retrieving revision 1.22
diff -u -p -r1.22 Makefile
--- Makefile    1 Dec 2011 18:57:48 -0000       1.22
+++ Makefile    29 Jan 2012 15:43:47 -0000
@@ -3,6 +3,7 @@
 COMMENT=       rip shoutcast streams to local mp3s
 
 DISTNAME=      streamripper-1.64.6
+REVISION=      0
 CATEGORIES=    audio
 
 HOMEPAGE=      http://streamripper.sourceforge.net/
@@ -17,16 +18,16 @@ PERMIT_PACKAGE_FTP= Yes
 PERMIT_DISTFILES_CDROM=        Yes
 PERMIT_DISTFILES_FTP=  Yes
 
-WANTLIB += c faad glib-2.0 iberty m mad>=2 ogg>=5.2 pthread vorbis>=3.0
+WANTLIB += c glib-2.0 iberty m mad>=2 ogg>=5.2 pthread vorbis>=3.0
 
-LIB_DEPENDS=   audio/faad \
-               audio/libmad \
+LIB_DEPENDS=   audio/libmad \
                audio/libogg \
                audio/libvorbis \
                devel/glib2
 
 AUTOCONF_VERSION=2.63
 CONFIGURE_STYLE= autoconf
+CONFIGURE_ARGS += ac_cv_lib_faad_NeAACDecDecode2=no
 CONFIGURE_ENV= CPPLAGS="-I${LOCALBASE}/include" \
                LDFLAGS="-L${LOCALBASE}/lib"
 USE_GROFF =    Yes

Reply via email to