Date: Wed, 26 Aug 2015 16:11:21 +0000 From: bugzilla-noreply@freebsd.org To: freebsd-ports-bugs@FreeBSD.org Subject: [Bug 202404] [MAINTAINER] multimedia/mplayer2: updates to Makefile Message-ID: <bug-202404-13-dSFAkwsAw6@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-202404-13@https.bugs.freebsd.org/bugzilla/> References: <bug-202404-13@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D202404 --- Comment #24 from Jan Beich <jbeich@FreeBSD.org> --- Comment on attachment 160377 --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D160377 v7 As ffmpeg CLI isn't actually used but its libraries are BUILD_DEPENDS=3D ffmpeg:${PORTSDIR}/multimedia/ffmpeg RUN_DEPENDS=3D ffmpeg:${PORTSDIR}/multimedia/ffmpeg can be converted to LIB_DEPENDS=3D libavcodec.so:${PORTSDIR}/multimedia/ffmpeg > +OPTIONS_DEFAULT=3D ASS ENCA LIBAV X11 ENCA and LIBAV are orphaned. LIBAV_* were removed in v7 while ASS_IMPLIES=3DENCA in v6. > post-patch: > -.if ${OSVERSION} >=3D 900010 > - @${REINPLACE_CMD} -e \ > - '/CFLAGS.*-D_LARGEFILE64_SOURCE/ s/-D_LARGEFILE64_SOURCE/-D_FILE_OFFSE= T_BITS=3D64/' \ > - ${CONFIGURE_WRKSRC}/${CONFIGURE_SCRIPT} > -.endif riggs@ added the substitution in ports r253004, so he should probably review its removal. I've only suggested to remove .if guard. > @@ -261,6 +174,13 @@ DEFAULT_KERN_HZ=3D${WITH_KERN_HZ} > DEFAULT_KERN_HZ=3D1024 > .endif > +.include <bsd.port.options.mk> [...] > +BROKEN_sparc64=3D Does not compile on sparc64 [...] > +CONFIGURE_ARGS+=3D --extra-libs=3D"${LIBS}" Why not concatenate value with the top CONFIGURE_ARGS definition? LIBS won't be evaluated before do-configure calls configure script after all options are already processed. > + > +# Depend on RESTRICTED package when built manually > +.if !defined(PACKAGE_BUILDING) > +OPTIONS_DEFAULT_i386+=3D WIN32 > +.endif > + Try to avoid putting anything below .include lines unless they don't work otherwise. For example, all those .if defined() expect variables defined in either environment, make.conf or command line, thus way before the port's Makefile. One exception is underdocumented Makefile.local. (In reply to Carlos J Puga Medina from comment #22) > but the main question here is keep or remove mplayer2 port. A dead project can stay in ports as long as there's a maintainer for it. Unmaintained code in libmpdemux/ and stream/ may cause a security concern but most parsing/decoding is done by ffmpeg libraries. That said, I agree about deprecation/removal as a former user of mplayer2. mpv seems like the continuation of the fast evolution of mplayer code started by Uoti. For a cooling-off period mark with DEPRECATED, otherwise use MOVED file to switch users from mplayer2 to either mpv or mplayer. DEPRECATED and EXPIRATION_DATE are invisible for package users and maybe unnoticed until periodic reminders are restored as the last one was from 2015-05-07. https://www.freebsd.org/doc/en/books/porters-handbook/dads-deprecated.html https://www.freebsd.org/doc/en/books/porters-handbook/moved-and-updating-fi= les.html#moved-and-updating-moved https://lists.freebsd.org/pipermail/freebsd-ports/2015-May/099082.html --=20 You are receiving this mail because: You are the assignee for the bug.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-202404-13-dSFAkwsAw6>