From nemysis@gmx.ch  Fri May 31 19:38:33 2013
Return-Path: <nemysis@gmx.ch>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1])
	by hub.freebsd.org (Postfix) with ESMTP id ACABAEBD
	for <freebsd-gnats-submit@freebsd.org>; Fri, 31 May 2013 19:38:33 +0000 (UTC)
	(envelope-from nemysis@gmx.ch)
Received: from mout.gmx.net (mout.gmx.net [212.227.15.18])
	by mx1.freebsd.org (Postfix) with ESMTP id 56282925
	for <freebsd-gnats-submit@freebsd.org>; Fri, 31 May 2013 19:38:33 +0000 (UTC)
Received: from mailout-de.gmx.net ([10.1.76.4]) by mrigmx.server.lan
 (mrigmx001) with ESMTP (Nemesis) id 0M68YO-1UTQJJ3unq-00yAcO for
 <FreeBSD-gnats-submit@freebsd.org>; Fri, 31 May 2013 21:38:32 +0200
Received: (qmail invoked by alias); 31 May 2013 19:38:30 -0000
Received: from 5-80.77-83.cust.bluewin.ch (EHLO something.email.com) [83.77.80.5]
  by mail.gmx.net (mp004) with SMTP; 31 May 2013 21:38:30 +0200
Received: by something.email.com (sSMTP sendmail emulation); Fri, 31 May 2013 21:38:29 +0200
Message-Id: <20130531193833.ACABAEBD@hub.freebsd.org>
Date: Fri, 31 May 2013 21:38:29 +0200
From: nemysis <nemysis@gmx.ch>
To: FreeBSD-gnats-submit@freebsd.org
Cc: jpaetzel@freebsd.org
Subject: [PATCH] multimedia/dv2jpg: Use files/makefile instead of many patches, remove pkg-plist
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         179154
>Category:       ports
>Synopsis:       [PATCH] multimedia/dv2jpg: Use files/makefile instead of many patches, remove pkg-plist
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    nemysis
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Fri May 31 19:40:00 UTC 2013
>Closed-Date:    Wed Mar 26 16:50:07 UTC 2014
>Last-Modified:  Wed Mar 26 17:20:00 UTC 2014
>Originator:     nemysis
>Release:        FreeBSD 9.1-RELEASE-p3 amd64
>Organization:
>Environment:
System: FreeBSD FreeBSD_Ports 9.1-RELEASE-p3 FreeBSD 9.1-RELEASE-p3 #0: Mon Apr 29 18:27:25 UTC 2013
>Description:

- Shorten header
- Bump portrevision
- Comply with OptionsNG
- Use files/makefile instead of many patches
- Use REINPLACE_CMD instead of simple patches
- Remove pkg-plist
- Cleanup port

Added file(s):
- files/makefile

Removed file(s):
- files/patch-avijoin.c
- files/patch-makefile
- pkg-plist

Generated and tested manually, tested with port test and with RedPorts, sent with FreeBSD Port Tools 0.99_6 (mode: change, diff: ports)
>How-To-Repeat:

Build log

https://redports.org/buildarchive/20130531181601-56196/

>Fix:

--- dv2jpg-1.1_11.patch begins here ---
diff -ruN --exclude=CVS /usr/ports/multimedia/dv2jpg/Makefile ./Makefile
--- /usr/ports/multimedia/dv2jpg/Makefile	2013-01-14 17:54:56.000000000 +0100
+++ ./Makefile	2013-03-31 14:09:33.000000000 +0200
@@ -1,50 +1,56 @@
-# New ports collection makefile for:	dv2jpg
-# Date created:				24 February 2003
-# Whom:					Anders Nordby <anders@FreeBSD.org>
-#
+# Created by: Anders Nordby <anders@FreeBSD.org>
 # $FreeBSD: head/multimedia/dv2jpg/Makefile 300896 2012-07-14 13:54:48Z beat $
-#
 
 PORTNAME=	dv2jpg
 PORTVERSION=	1.1
-PORTREVISION=	10
+PORTREVISION=	11
 CATEGORIES=	multimedia
 MASTER_SITES=	SF
 
 MAINTAINER=	ports@FreeBSD.org
 COMMENT=	Convert Type-2 DV codec-encoded AVI streams to mjpeg-encoded AVI
 
-LIB_DEPENDS=	dv.4:${PORTSDIR}/multimedia/libdv
+LIB_DEPENDS=	dv:${PORTSDIR}/multimedia/libdv
 
 USE_GNOME=	gdkpixbuf
 MAKEFILE=	makefile
 
-.if defined(WITH_PAL_VIDEO)
-VID_FORMAT=	PAL
-.else
-VID_FORMAT=	NTSC
-.endif
+PLIST_FILES=	bin/avijoin \
+		bin/dv2jpg
+
+PORTDOCS=	README.avilib README.dv2jpg.html
+
+OPTIONS_SINGLE=	VIDEO-FORMAT
+OPTIONS_SINGLE_VIDEO-FORMAT=	NTSC PAL
+OPTIONS_DEFAULT=	NTSC
+
+VIDEO-FORMAT_DESC=	Use default video format
+NTSC_DESC=	Default video format NTSC
+PAL_DESC=	Default video format PAL
 
-pre-everything::
-	@${ECHO_MSG} "===> The default video format is NTSC"
-	@${ECHO_MSG} "===> You can change this to PAL by defining"
-	@${ECHO_MSG} "===> WITH_PAL_VIDEO"
+.include <bsd.port.options.mk>
 
 post-extract:
-	${RM} ${WRKSRC}/avijoin ${WRKSRC}/dv2jpg ${WRKSRC}/*.o
+	${INSTALL_DATA} ${FILESDIR}/makefile ${WRKSRC}
+	${RM} -f ${WRKSRC}/avijoin ${WRKSRC}/dv2jpg ${WRKSRC}/*.o
 
 post-patch:
-	@${REINPLACE_CMD} -e 's|%%VID_FORMAT%%|${VID_FORMAT}|' \
+	@${REINPLACE_CMD} -e 's|/opt/vidpb/lib/avilib.h|avilib.h|' \
+		${WRKSRC}/avijoin.c
+
+.if ${PORT_OPTIONS:MPAL}
+	@${REINPLACE_CMD} -e 's|VIDEO=NTSC|# VIDEO=NTSC|' \
+		-e 's|# VIDEO=PAL|VIDEO=PAL|' \
 		${WRKSRC}/${MAKEFILE}
+.endif
 
 do-install:
-	${INSTALL_PROGRAM} ${WRKSRC}/avijoin ${PREFIX}/bin/
-	${INSTALL_PROGRAM} ${WRKSRC}/dv2jpg ${PREFIX}/bin/
-.if !defined(NOPORTDOCS)
+	${INSTALL_PROGRAM} ${WRKSRC}/avijoin ${PREFIX}/bin
+	${INSTALL_PROGRAM} ${WRKSRC}/dv2jpg ${PREFIX}/bin
+
+.if ${PORT_OPTIONS:MDOCS}
 	@${MKDIR} ${DOCSDIR}
-.for f in README.avilib	README.dv2jpg.html
-	${INSTALL_DATA} ${WRKSRC}/${f} ${DOCSDIR}/
-.endfor
+	${INSTALL_DATA} ${PORTDOCS:S|^|${WRKSRC}/|} ${DOCSDIR}
 .endif
 
 .include <bsd.port.mk>
diff -ruN --exclude=CVS /usr/ports/multimedia/dv2jpg/files/makefile ./files/makefile
--- /usr/ports/multimedia/dv2jpg/files/makefile	1970-01-01 01:00:00.000000000 +0100
+++ ./files/makefile	2013-03-31 13:08:35.000000000 +0200
@@ -0,0 +1,27 @@
+
+INC=`${GDK_PIXBUF_CONFIG} --cflags` -I${PREFIX}/include/libdv
+LIBS=-ljpeg -lm -ldv `${GDK_PIXBUF_CONFIG} --libs` -L${LOCALBASE}/lib
+
+# what type of video do you want?
+VIDEO=NTSC
+# VIDEO=PAL
+
+all: dv2jpg avijoin
+	
+dv2jpg: dv2jpg.o avilib.o dvavilib.o
+	$(CC) $(CFLAGS) -o dv2jpg dv2jpg.o avilib.o dvavilib.o $(LIBS) $(LDFLAGS)
+
+dv2jpg.o: dv2jpg.c
+	$(CC) $(CFLAGS) $(INC) -D$(VIDEO) -c dv2jpg.c
+
+avilib.o: avilib.c
+	$(CC) $(CFLAGS) -c avilib.c
+
+dvavilib.o: dvavilib.c
+	$(CC) $(CFLAGS) -c dvavilib.c
+
+clean: 
+	/bin/rm -f *.o dv2jpg core avijoin
+
+avijoin: avijoin.c avilib.o
+	$(CC) $(CFLAGS) -o avijoin avijoin.c avilib.o $(LDFLAGS)
diff -ruN --exclude=CVS /usr/ports/multimedia/dv2jpg/files/patch-avijoin.c ./files/patch-avijoin.c
--- /usr/ports/multimedia/dv2jpg/files/patch-avijoin.c	2013-01-14 17:54:56.000000000 +0100
+++ ./files/patch-avijoin.c	1970-01-01 01:00:00.000000000 +0100
@@ -1,9 +0,0 @@
---- avijoin.c.orig	Sun Jun 24 02:04:47 2001
-+++ avijoin.c	Sun Feb 16 01:24:49 2003
-@@ -1,5 +1,5 @@
- #include <stdio.h>
--#include "/opt/vidpb/lib/avilib.h"
-+#include "avilib.h"
- 
- #define MAX_FILES 255
- #define OURBUFSIZ 4096000
diff -ruN --exclude=CVS /usr/ports/multimedia/dv2jpg/files/patch-makefile ./files/patch-makefile
--- /usr/ports/multimedia/dv2jpg/files/patch-makefile	2013-01-14 17:54:56.000000000 +0100
+++ ./files/patch-makefile	1970-01-01 01:00:00.000000000 +0100
@@ -1,38 +0,0 @@
---- makefile.orig	Sat Jun 23 22:04:47 2001
-+++ makefile	Fri Sep 24 12:05:35 2004
-@@ -1,27 +1,27 @@
- 
--INC=-I/opt/gdk-pixbuf/lib -I/usr/include -I/usr/include/libdv 
--LIBS=-ljpeg -lm -lgdk_pixbuf /usr/lib/libdv.a
-+INC=`${GDK_PIXBUF_CONFIG} --cflags` -I${PREFIX}/include/libdv 
-+LIBS=-ljpeg -lm -ldv `${GDK_PIXBUF_CONFIG} --libs` -L${LOCALBASE}/lib
- 
- # what type of video do you want?
--VIDEO=NTSC
-+VIDEO=%%VID_FORMAT%%
- # VIDEO=PAL
- 
- all: dv2jpg avijoin
- 	
- dv2jpg: dv2jpg.o avilib.o dvavilib.o
--	gcc -o dv2jpg dv2jpg.o avilib.o dvavilib.o $(LIBS) `glib-config --libs` `gtk-config --libs`
-+	$(CC) $(CFLAGS) -o dv2jpg dv2jpg.o avilib.o dvavilib.o $(LIBS) $(LDFLAGS)
- 
- dv2jpg.o: dv2jpg.c
--	gcc -g $(INC) `glib-config --cflags` `gtk-config --cflags` -D$(VIDEO) -g -c dv2jpg.c 
-+	$(CC) $(CFLAGS) $(INC) -D$(VIDEO) -c dv2jpg.c 
- 
- avilib.o: avilib.c
--	gcc -g -c avilib.c
-+	$(CC) $(CFLAGS) -c avilib.c
- 
- dvavilib.o: dvavilib.c
--	gcc -g -c dvavilib.c
-+	$(CC) $(CFLAGS) -c dvavilib.c
- 
- clean: 
- 	/bin/rm -f *.o dv2jpg core avijoin
- 
- avijoin: avijoin.c avilib.o
--	gcc -g -o avijoin avijoin.c avilib.o
-+	$(CC) $(CFLAGS) -o avijoin avijoin.c avilib.o $(LDFLAGS)
diff -ruN --exclude=CVS /usr/ports/multimedia/dv2jpg/pkg-plist ./pkg-plist
--- /usr/ports/multimedia/dv2jpg/pkg-plist	2013-01-14 17:54:56.000000000 +0100
+++ ./pkg-plist	1970-01-01 01:00:00.000000000 +0100
@@ -1,5 +0,0 @@
-bin/avijoin
-bin/dv2jpg
-%%PORTDOCS%%%%DOCSDIR%%/README.avilib
-%%PORTDOCS%%%%DOCSDIR%%/README.dv2jpg.html
-%%PORTDOCS%%@dirrm %%DOCSDIR%%
--- dv2jpg-1.1_11.patch ends here ---

>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->beech 
Responsible-Changed-By: edwin 
Responsible-Changed-When: Fri May 31 19:40:08 UTC 2013 
Responsible-Changed-Why:  
beech@ wants this submitter's PRs (via the GNATS Auto Assign Tool) 

http://www.freebsd.org/cgi/query-pr.cgi?pr=179154 
Responsible-Changed-From-To: beech->nemysis 
Responsible-Changed-By: tabthorpe 
Responsible-Changed-When: Fri Jan 10 03:36:36 UTC 2014 
Responsible-Changed-Why:  
Submitter is committer 

http://www.freebsd.org/cgi/query-pr.cgi?pr=179154 
State-Changed-From-To: open->closed 
State-Changed-By: nemysis 
State-Changed-When: Wed Mar 26 16:50:06 UTC 2014 
State-Changed-Why:  
Committed. Thanks! 

http://www.freebsd.org/cgi/query-pr.cgi?pr=179154 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/179154: commit references a PR
Date: Wed, 26 Mar 2014 17:15:34 +0000 (UTC)

 Author: nemysis
 Date: Wed Mar 26 17:15:26 2014
 New Revision: 349268
 URL: http://svnweb.freebsd.org/changeset/ports/349268
 QAT: https://qat.redports.org/buildarchive/r349268/
 
 Log:
   - Bump PORTREVISION
   - Add license (GPLv2)
   - Use the new format for LIB_DEPENDS
   - Comply with OptionsNG
   - Use PLIST_FILES instead of pkg-plist
   - Support STAGEDIR
   - Simplify DOCS installation handling and add Option
   - Add OPTIONS_SINGLE=VIDEO-FORMAT and NTSC, PAL SINGLE Options,
     and REINPLACE
   - Use files/makefile instead of files/patch-makefile and REINPLACE
   - Add REINPLACE instead of simple patch
   
   PR:		ports/179154
   Submitted by:	nemysis (self)
 
 Added:
   head/multimedia/dv2jpg/files/makefile   (contents, props changed)
 Deleted:
   head/multimedia/dv2jpg/files/patch-avijoin.c
   head/multimedia/dv2jpg/files/patch-makefile
   head/multimedia/dv2jpg/pkg-plist
 Modified:
   head/multimedia/dv2jpg/Makefile
 
 Modified: head/multimedia/dv2jpg/Makefile
 ==============================================================================
 --- head/multimedia/dv2jpg/Makefile	Wed Mar 26 16:54:29 2014	(r349267)
 +++ head/multimedia/dv2jpg/Makefile	Wed Mar 26 17:15:26 2014	(r349268)
 @@ -3,45 +3,55 @@
  
  PORTNAME=	dv2jpg
  PORTVERSION=	1.1
 -PORTREVISION=	10
 +PORTREVISION=	11
  CATEGORIES=	multimedia
  MASTER_SITES=	SF
  
  MAINTAINER=	ports@FreeBSD.org
  COMMENT=	Convert Type-2 DV codec-encoded AVI streams to mjpeg-encoded AVI
  
 -LIB_DEPENDS=	dv.4:${PORTSDIR}/multimedia/libdv
 +LICENSE=	GPLv2
 +
 +LIB_DEPENDS=	libdv.so:${PORTSDIR}/multimedia/libdv
  
  USE_GNOME=	gdkpixbuf
  MAKEFILE=	makefile
  
 -.if defined(WITH_PAL_VIDEO)
 -VID_FORMAT=	PAL
 -.else
 -VID_FORMAT=	NTSC
 -.endif
 +PLIST_FILES=	bin/avijoin bin/dv2jpg
 +
 +PORTDOCS=	README.avilib README.dv2jpg.html
 +OPTIONS_DEFINE=	DOCS
 +
 +OPTIONS_SINGLE=	VIDEO-FORMAT
 +OPTIONS_SINGLE_VIDEO-FORMAT=	NTSC PAL
 +OPTIONS_DEFAULT=	NTSC
 +
 +VIDEO-FORMAT_DESC=	Use default video format
 +NTSC_DESC=	Default video format NTSC
 +PAL_DESC=	Default video format PAL
  
 -NO_STAGE=	yes
 -pre-everything::
 -	@${ECHO_MSG} "===> The default video format is NTSC"
 -	@${ECHO_MSG} "===> You can change this to PAL by defining"
 -	@${ECHO_MSG} "===> WITH_PAL_VIDEO"
 +.include <bsd.port.options.mk>
  
  post-extract:
 +	${INSTALL_DATA} ${FILESDIR}/makefile ${WRKSRC}
  	${RM} ${WRKSRC}/avijoin ${WRKSRC}/dv2jpg ${WRKSRC}/*.o
  
  post-patch:
 -	@${REINPLACE_CMD} -e 's|%%VID_FORMAT%%|${VID_FORMAT}|' \
 +	@${REINPLACE_CMD} -e 's|/opt/vidpb/lib/avilib.h|avilib.h|' \
 +		${WRKSRC}/avijoin.c
 +
 +.if ${PORT_OPTIONS:MPAL}
 +	@${REINPLACE_CMD} -e 's|VIDEO=NTSC|# VIDEO=NTSC|; \
 +		s|# VIDEO=PAL|VIDEO=PAL|' \
  		${WRKSRC}/${MAKEFILE}
 +.endif
  
  do-install:
 -	${INSTALL_PROGRAM} ${WRKSRC}/avijoin ${PREFIX}/bin/
 -	${INSTALL_PROGRAM} ${WRKSRC}/dv2jpg ${PREFIX}/bin/
 -.if !defined(NOPORTDOCS)
 -	@${MKDIR} ${DOCSDIR}
 -.for f in README.avilib	README.dv2jpg.html
 -	${INSTALL_DATA} ${WRKSRC}/${f} ${DOCSDIR}/
 +.for f in avijoin dv2jpg
 +	(cd ${WRKSRC} && ${INSTALL_PROGRAM} ${f} ${STAGEDIR}${PREFIX}/bin/)
  .endfor
 -.endif
 +
 +	@${MKDIR} ${STAGEDIR}${DOCSDIR}
 +	${INSTALL_DATA} ${PORTDOCS:S|^|${WRKSRC}/|} ${STAGEDIR}${DOCSDIR}
  
  .include <bsd.port.mk>
 
 Added: head/multimedia/dv2jpg/files/makefile
 ==============================================================================
 --- /dev/null	00:00:00 1970	(empty, because file is newly added)
 +++ head/multimedia/dv2jpg/files/makefile	Wed Mar 26 17:15:26 2014	(r349268)
 @@ -0,0 +1,27 @@
 +
 +INC=`${GDK_PIXBUF_CONFIG} --cflags` -I${PREFIX}/include/libdv
 +LIBS=-ljpeg -lm -ldv `${GDK_PIXBUF_CONFIG} --libs` -L${LOCALBASE}/lib
 +
 +# what type of video do you want?
 +VIDEO=NTSC
 +# VIDEO=PAL
 +
 +all: dv2jpg avijoin
 +	
 +dv2jpg: dv2jpg.o avilib.o dvavilib.o
 +	$(CC) $(CFLAGS) -o dv2jpg dv2jpg.o avilib.o dvavilib.o $(LIBS) $(LDFLAGS)
 +
 +dv2jpg.o: dv2jpg.c
 +	$(CC) $(CFLAGS) $(INC) -D$(VIDEO) -c dv2jpg.c
 +
 +avilib.o: avilib.c
 +	$(CC) $(CFLAGS) -c avilib.c
 +
 +dvavilib.o: dvavilib.c
 +	$(CC) $(CFLAGS) -c dvavilib.c
 +
 +clean: 
 +	/bin/rm -f *.o dv2jpg core avijoin
 +
 +avijoin: avijoin.c avilib.o
 +	$(CC) $(CFLAGS) -o avijoin avijoin.c avilib.o $(LDFLAGS)
 _______________________________________________
 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"
 
>Unformatted:
