Bug 171478 - [PATCH] multimedia/avidemux2: Fix default options selection
Summary: [PATCH] multimedia/avidemux2: Fix default options selection
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Juergen Lock
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-08 23:20 UTC by Jean-Sébastien Pédron
Modified: 2012-10-02 21:50 UTC (History)
0 users

See Also:


Attachments
file.diff (460 bytes, patch)
2012-09-08 23:20 UTC, Jean-Sébastien Pédron
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jean-Sébastien Pédron freebsd_committer freebsd_triage 2012-09-08 23:20:02 UTC
The following snippet in multimedia/avidemux2/Makefile.common seems incorrect:
  OPTIONS_DEFAULT= GTK2 FREETYPE FONTCONFIG OSS XVIDEO VPX XVID SDL FAAD VORBIS

  .if !defined(PACKAGE_BUILDING)
  OPTIONS_DEFAULT= LAME FAAC AMR
  .endif

The second "OPTIONS_DEFAULT=" overwrites the first one. I attached a patch that simply transforms it to a "OPTIONS_DEFAULT+=".

Fix: Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2012-09-08 23:20:54 UTC
Responsible Changed
From-To: freebsd-ports-bugs->freebsd-multimedia

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Juergen Lock freebsd_committer freebsd_triage 2012-10-02 21:30:29 UTC
Responsible Changed
From-To: freebsd-multimedia->nox

I'll take it.
Comment 3 dfilter service freebsd_committer freebsd_triage 2012-10-02 21:47:57 UTC
Author: nox
Date: Tue Oct  2 20:47:41 2012
New Revision: 305176
URL: http://svn.freebsd.org/changeset/ports/305176

Log:
  - Fix default options when not PACKAGE_BUILDING. [1]
  - Fix CMAKE_ARGS usage when VORBIS is disabled. [2]
  - Strip Makefile headers.
  - Remove a few more shilb major versions from LIB_DEPENDS.
  - Bump PORTREVISIONs.
  
  PR:		ports/171478 [1]
  Submitted by:	dumbbell [1]
  PR:		ports/172036 [2]
  Submitted by:	KATO Tsuguru <tkato432@yahoo.com> [2]

Modified:
  head/multimedia/avidemux2-plugins/Makefile   (contents, props changed)
  head/multimedia/avidemux2/Makefile   (contents, props changed)
  head/multimedia/avidemux2/Makefile.common   (contents, props changed)

Modified: head/multimedia/avidemux2-plugins/Makefile
==============================================================================
--- head/multimedia/avidemux2-plugins/Makefile	Tue Oct  2 20:27:32 2012	(r305175)
+++ head/multimedia/avidemux2-plugins/Makefile	Tue Oct  2 20:47:41 2012	(r305176)
@@ -1,13 +1,9 @@
-# New ports collection makefile for:    avidemux2
-# Date created:         16 April 2003
-# Whom:                 Anish Mistry
-#                       with help from mean
+# Created by: Anish Mistry (with help from mean)
 # $FreeBSD$
-#
 
 PORTNAME=	avidemux2
 PORTVERSION=	${AVIDEMUX2_VERSION}
-PORTREVISION=	5
+PORTREVISION=	6
 CATEGORIES=	multimedia
 PKGNAMESUFFIX=	-plugins
 

Modified: head/multimedia/avidemux2/Makefile
==============================================================================
--- head/multimedia/avidemux2/Makefile	Tue Oct  2 20:27:32 2012	(r305175)
+++ head/multimedia/avidemux2/Makefile	Tue Oct  2 20:47:41 2012	(r305176)
@@ -1,9 +1,5 @@
-# New ports collection makefile for:    avidemux2
-# Date created:         16 April 2003
-# Whom:                 Anish Mistry
-#                       with help from mean
+# Created by: Anish Mistry (with help from mean)
 # $FreeBSD$
-#
 
 PORTNAME=	avidemux2
 PORTVERSION=	${AVIDEMUX2_VERSION}

Modified: head/multimedia/avidemux2/Makefile.common
==============================================================================
--- head/multimedia/avidemux2/Makefile.common	Tue Oct  2 20:27:32 2012	(r305175)
+++ head/multimedia/avidemux2/Makefile.common	Tue Oct  2 20:47:41 2012	(r305176)
@@ -2,7 +2,7 @@
 #
 
 AVIDEMUX2_VERSION=	2.5.6
-PORTREVISION=	4
+PORTREVISION=	5
 MASTER_SITES=	BERLIOS \
 		SF/avidemux/avidemux/${PORTVERSION}
 DISTNAME=	avidemux_${PORTVERSION}
@@ -41,7 +41,7 @@ OPTIONS_DEFINE=	GTK2 QT4 FREETYPE FONTCO
 OPTIONS_DEFAULT=	GTK2 FREETYPE FONTCONFIG OSS XVIDEO VPX XVID SDL FAAD VORBIS
 
 .if !defined(PACKAGE_BUILDING)
-OPTIONS_DEFAULT=	LAME FAAC AMR
+OPTIONS_DEFAULT+=	LAME FAAC AMR
 .endif
 
 .include <bsd.port.pre.mk>
@@ -101,7 +101,7 @@ PLIST_SUB+=	JACK="@comment "
 .endif
 
 .if ${PORT_OPTIONS:MPULSEAUDIO}
-LIB_DEPENDS+=	pulse.0:${PORTSDIR}/audio/pulseaudio
+LIB_DEPENDS+=	pulse:${PORTSDIR}/audio/pulseaudio
 PLIST_SUB+=	PULSE=""
 .else
 CMAKE_ARGS+=	-DPULSEAUDIOSIMPLE:BOOL=OFF
@@ -109,7 +109,7 @@ PLIST_SUB+=	PULSE="@comment "
 .endif
 
 .if ${PORT_OPTIONS:MFREETYPE}
-LIB_DEPENDS+=	freetype.9:${PORTSDIR}/print/freetype2
+LIB_DEPENDS+=	freetype:${PORTSDIR}/print/freetype2
 PLIST_SUB+=	FREETYPE=""
 .else
 CMAKE_ARGS+=	-DFREETYPE2:BOOL=OFF
@@ -117,19 +117,19 @@ PLIST_SUB+=	FREETYPE="@comment "
 .endif
 
 .if ${PORT_OPTIONS:MFONTCONFIG}
-LIB_DEPENDS+=	fontconfig.1:${PORTSDIR}/x11-fonts/fontconfig
+LIB_DEPENDS+=	fontconfig:${PORTSDIR}/x11-fonts/fontconfig
 .else
 CMAKE_ARGS+=	-DFONTCONFIG:BOOL=OFF
 .endif
 
 .if ${PORT_OPTIONS:MXVIDEO}
-LIB_DEPENDS+=	Xv.1:${PORTSDIR}/x11/libXv
+LIB_DEPENDS+=	Xv:${PORTSDIR}/x11/libXv
 .else
 CMAKE_ARGS+=	-DXVIDEO:BOOL=OFF
 .endif
 
 .if ${PORT_OPTIONS:MFAAC}
-LIB_DEPENDS+=	faac.0:${PORTSDIR}/audio/faac
+LIB_DEPENDS+=	faac:${PORTSDIR}/audio/faac
 PLIST_SUB+=	FAAC=""
 .else
 CMAKE_ARGS+=	-DFAAC:BOOL=OFF
@@ -137,7 +137,7 @@ PLIST_SUB+=	FAAC="@comment "
 .endif
 
 .if ${PORT_OPTIONS:MLAME}
-LIB_DEPENDS+=	mp3lame.0:${PORTSDIR}/audio/lame
+LIB_DEPENDS+=	mp3lame:${PORTSDIR}/audio/lame
 PLIST_SUB+=	LAME=""
 .else
 CMAKE_ARGS+=	-DLAME:BOOL=OFF
@@ -145,7 +145,7 @@ PLIST_SUB+=	LAME="@comment "
 .endif
 
 .if ${PORT_OPTIONS:MFAAD}
-LIB_DEPENDS+=	faad.2:${PORTSDIR}/audio/faad
+LIB_DEPENDS+=	faad:${PORTSDIR}/audio/faad
 PLIST_SUB+=	FAAD=""
 .else
 CMAKE_ARGS+=	-DFAAD:BOOL=OFF -DNeAAC:BOOL=OFF
@@ -153,7 +153,7 @@ PLIST_SUB+=	FAAD="@comment "
 .endif
 
 .if ${PORT_OPTIONS:MX264}
-LIB_DEPENDS+=	x264.125:${PORTSDIR}/multimedia/x264
+LIB_DEPENDS+=	x264:${PORTSDIR}/multimedia/x264
 PLIST_SUB+=	X264=""
 .else
 CMAKE_ARGS+=	-DX264:BOOL=OFF
@@ -161,13 +161,13 @@ PLIST_SUB+=	X264="@comment "
 .endif
 
 .if ${PORT_OPTIONS:MVPX}
-LIB_DEPENDS+=	vpx.1:${PORTSDIR}/multimedia/libvpx
+LIB_DEPENDS+=	vpx:${PORTSDIR}/multimedia/libvpx
 .else
 CMAKE_ARGS+=	-DVPXDEC:BOOL=OFF
 .endif
 
 .if ${PORT_OPTIONS:MXVID}
-LIB_DEPENDS+=	xvidcore.4:${PORTSDIR}/multimedia/xvid
+LIB_DEPENDS+=	xvidcore:${PORTSDIR}/multimedia/xvid
 PLIST_SUB+=	XVID=""
 .else
 CMAKE_ARGS+=	-DXVID:BOOL=OFF
@@ -183,15 +183,15 @@ PLIST_SUB+=	SDL="@comment "
 .endif
 
 .if ${PORT_OPTIONS:MVORBIS}
-LIB_DEPENDS+=	vorbis.4:${PORTSDIR}/audio/libvorbis
+LIB_DEPENDS+=	vorbis:${PORTSDIR}/audio/libvorbis
 PLIST_SUB+=	VORBIS=""
 .else
-CMAKE_ARGS+=	-DVORBIS:BOOL=OFF
+CMAKE_ARGS+=	-DLIBVORBIS:BOOL=OFF -DVORBIS:BOOL=OFF
 PLIST_SUB+=	VORBIS="@comment "
 .endif
 
 .if ${PORT_OPTIONS:MAMR}
-LIB_DEPENDS+=	opencore-amrnb.0:${PORTSDIR}/audio/opencore-amr
+LIB_DEPENDS+=	opencore-amrnb:${PORTSDIR}/audio/opencore-amr
 PLIST_SUB+=	AMRNB=""
 PLIST_SUB+=	AMRWB=""
 .else
_______________________________________________
svn-ports-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-ports-all
To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"
Comment 4 Juergen Lock freebsd_committer freebsd_triage 2012-10-02 21:49:13 UTC
State Changed
From-To: open->closed

Committed. Thanks!