Unless this fixes an actual problem it should wait until after 5.1 is tagged
On 2012/01/29 21:42, Alexandr Shadchin wrote: > 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 >
