Bug 168464 - [patch] graphics/gegl: prevent automake-1.11 rerun
Summary: [patch] graphics/gegl: prevent automake-1.11 rerun
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: freebsd-gnome (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-30 14:00 UTC by Jan Beich
Modified: 2013-01-25 21:20 UTC (History)
0 users

See Also:


Attachments
check.diff (850 bytes, patch)
2012-05-30 14:00 UTC, Jan Beich
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Beich freebsd_committer freebsd_triage 2012-05-30 14:00:18 UTC
Modifying .am files without proper version[1] of automake or defining
USE_AUTOTOOLS may not always work. And so .am changes are not propagated
to .in files leading to confusion.

The origin of that reinplace line is a bit unclear to me. It was added
during gegl-0.1.2 supposedly[2] to help --disable-docs. But from my testing
--disable-docs comments out `docs' from SUBDIRS just fine in 0.1.2.

So, instead of skipping tests do something useful with them, like
actually running them before install.

[1] gegl-0.1.8 wants automake-1.11 while devel/automake provides automake-1.12
[2] nowadays, it tries to disable examples, tests, tools and can be written as

	${REINPLACE_CMD} -Ee '/^SUBDIRS/,/^$$/s/tools|examples|tests//g' \
		${WRKSRC}/Makefile.am ${WRKSRC}/Makefile.in

How-To-Repeat: $ make
===>  Building for gegl-0.1.8_3
 cd . && /bin/sh /usr/ports/graphics/gegl/work/gegl-0.1.8/missing --run automake-1.11 --gnu Makefile
/usr/ports/graphics/gegl/work/gegl-0.1.8/missing: automake-1.11: not found
WARNING: `automake-1.11' is missing on your system.  You should only need it if
         you modified `Makefile.am', `acinclude.m4' or `configure.ac'.
         You might want to install the `Automake' and `Perl' packages.
         Grab them from any GNU archive site.
 cd . && /bin/sh ./config.status Makefile
config.status: creating Makefile
[...]

$ ls $(make -V WRKSRC)/tools/.libs
img_cmp             introspect          operation_reference
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2012-05-31 12:00:28 UTC
Responsible Changed
From-To: freebsd-ports-bugs->gnome

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Jeremy Messenger freebsd_committer freebsd_triage 2012-12-29 18:34:25 UTC
State Changed
From-To: open->feedback

I can't reproduce with your #1 issue about automake rerun here. As for the #2, 
I don't see there is anything that need to be changed when it works fine as 
now.
Comment 3 Jan Beich freebsd_committer freebsd_triage 2013-01-12 10:58:49 UTC
mezz@FreeBSD.org writes:

> I can't reproduce with your #1 issue about automake rerun here.
> As for the #2, I don't see there is anything that need to be changed
> when it works fine as now.

Those aren't separate issues. #1 is due to bitrot[1] of #2. And the
issue *probably* only happens on 10.0[2].

[1] lines change regularly but maintainer may notice not notice
[2] http://pastebin.com/xuyMGrhh # gegl-0.1.8_4.log
Comment 4 Koop Mast freebsd_committer freebsd_triage 2013-01-25 21:10:28 UTC
State Changed
From-To: feedback->closed

I managed to reproduce this while doing some other work. I fixed this in another 
way, by patching Makefile.in instead of the Makefile.am file. 
Thanks for reporting!
Comment 5 dfilter service freebsd_committer freebsd_triage 2013-01-25 21:10:46 UTC
Author: kwm
Date: Fri Jan 25 21:10:02 2013
New Revision: 310987
URL: http://svnweb.freebsd.org/changeset/ports/310987

Log:
  Patch Makefile.in instead of Makefile.am this prevents autofoo from being run [1]
  
  Use USE_PKGCONFIG instead of USE_GNOME=pkgconfig [2]
  Switch to OptionsNG. This renames the RSVG->LIBRSVG2 and GDK->PIXBUF options[2]
  Remove shlib version in lib_DEPENDS lines [2]
  Trim Makefile Header while here
  Bump portrevision since default depends changed.
  
  PR:		ports/168464 (reported by) [1]
  Submitted by:	Jan Beich <jbeich@tormail.org> [1]
  Obtained from:	GNOME staging area [2]

Modified:
  head/graphics/gegl/Makefile

Modified: head/graphics/gegl/Makefile
==============================================================================
--- head/graphics/gegl/Makefile	Fri Jan 25 20:58:07 2013	(r310986)
+++ head/graphics/gegl/Makefile	Fri Jan 25 21:10:02 2013	(r310987)
@@ -3,7 +3,7 @@
 
 PORTNAME=	gegl
 PORTVERSION=	0.1.8
-PORTREVISION=	5
+PORTREVISION=	6
 CATEGORIES=	graphics
 MASTER_SITES=	ftp://ftp.gimp.org/pub/gegl/0.1/
 
@@ -13,28 +13,17 @@ COMMENT=	A graph based image processing 
 BUILD_DEPENDS=	g-ir-scanner:${PORTSDIR}/devel/gobject-introspection
 LIB_DEPENDS=	babl-0.1:${PORTSDIR}/x11/babl
 
-OPTIONS=	CAIRO		"Enable Cairo" on \
-		ENSCRIPT	"Enable Enscript" on \
-		FFMPEG		"Enable videos through ffmpeg (BROKEN)" off \
-		GDK		"Enable gdk-pixbuf2 formats support" on \
-		GRAPHVIZ	"Enable Graphviz" on \
-		JP2		"Enable JPEG2000 format support" on \
-		JPEG		"Enable JPEG format support" on \
-		LUA		"Enable LUA" on \
-		OPENEXR		"Enable High Dynamic Range (HDR) format support" on \
-		OPENRAW		"Enable RAW camera format support" on \
-		OPTIMIZE	"Enable optimizations autodetection (MMX, SSE)" off \
-		PANGO		"Enable rendering through Pango " on \
-		PNG		"Enable PNG format support" on \
-		RSVG		"Enable RSVG format support" on \
-		SDL		"Enable SDL" on \
-		SPIRO		"Enable Spiro" on \
-		V4L		"Enable Video4Linux" on
+OPTIONS_DEFINE=	CAIRO DOCS ENSCRIPT EXIV2 FFMPEG PIXBUF GRAPHVIZ JASPER JPEG \
+		LUA OPENEXR OPENRAW SIMD PANGO PNG LIBRSVG2 SDL SPIRO V4L
+OPTIONS_DEFAULT=CAIRO DOCS ENSCRIPT EXIV2 PIXBUF GRAPHVIZ JASPER JPEG LUA \
+		OPENEXR OPENRAW PANGO PNG LIBRSVG2 SDL SPIRO V4L
+
+ENSCRIPT_DESC=	Enscript support
+SPIRO_DESC=	Spiro support
+#UMFPACK_DESC=	UMFpack support
 
-# Disable docs until I fix the problem with graphviz's dot
-NOPORTDOCS=	yes
-
-USE_GNOME=	glib20 pkgconfig gnomehack
+USE_GNOME=	glib20 gnomehack
+USE_PKGCONFIG=	build
 USE_BZIP2=	yes
 USE_GMAKE=	yes
 USE_GETTEXT=	yes
@@ -54,7 +43,7 @@ EXAMPLES=	2geglbuffer csugar float-looku
 
 .include <bsd.port.options.mk>
 
-.if !defined(WITHOUT_GDK)
+.if ${PORT_OPTIONS:MPIXBUF}
 USE_GNOME+=	gdkpixbuf2
 PLIST_SUB+=	GDK=""
 .else
@@ -62,15 +51,15 @@ CONFIGURE_ARGS+=--without-gdk-pixbuf
 PLIST_SUB+=	GDK="@comment "
 .endif
 
-.if !defined(WITHOUT_CAIRO)
-LIB_DEPENDS+=	cairo.2:${PORTSDIR}/graphics/cairo
+.if ${PORT_OPTIONS:MCAIRO}
+LIB_DEPENDS+=	cairo:${PORTSDIR}/graphics/cairo
 PLIST_SUB+=	CAIRO=""
 .else
 CONFIGURE_ARGS+=--without-cairo
 PLIST_SUB+=	CAIRO="@comment "
 .endif
 
-.if !defined(WITHOUT_PANGO)
+.if ${PORT_OPTIONS:MPANGO}
 USE_GNOME+=	pango
 PLIST_SUB+=	PANGO=""
 .else
@@ -78,55 +67,54 @@ CONFIGURE_ARGS+=--without-pango
 PLIST_SUB+=	PANGO="@comment "
 .endif
 
-.if !defined(WITHOUT_PANGO) && !defined(WITHOUT_CAIRO)
+.if ${PORT_OPTIONS:MPANGO} && ${PORT_OPTIONS:MCAIRO}
 PLIST_SUB+=	PANGOCAIRO=""
 .else
 CONFIGURE_ARGS+=--without-pangocairo
 PLIST_SUB+=	PANGOCAIRO="@comment "
 .endif
 
-.if !defined(WITHOUT_ENSCRIPT)
+.if ${PORT_OPTIONS:MENSCRIPT}
 BUILD_DEPENDS+=	enscript:${PORTSDIR}/print/enscript-a4
 .endif
 
-.if !defined(WITHOUT_FFMPEG)
-#BROKEN=	ffmpeg support is currently broken
-LIB_DEPENDS+=	avformat.1:${PORTSDIR}/multimedia/ffmpeg
+.if ${PORT_OPTIONS:MFFMPEG}
+LIB_DEPENDS+=	avformat:${PORTSDIR}/multimedia/ffmpeg
 PLIST_SUB+=	FFMPEG=""
 .else
 CONFIGURE_ARGS+=--without-libavformat
 PLIST_SUB+=	FFMPEG="@comment "
 .endif
 
-.if !defined(WITHOUT_GRAPHVIZ)
+.if ${PORT_OPTIONS:MGRAPHVIZ}
 LIB_DEPENDS+=	graph:${PORTSDIR}/graphics/graphviz
 .else
 CONFIGURE_ARGS+=--without-graphviz
 .endif
 
-.if !defined(WITHOUT_JP2)
-LIB_DEPENDS+=	jasper.4:${PORTSDIR}/graphics/jasper
+.if ${PORT_OPTIONS:MJASPER}
+LIB_DEPENDS+=	jasper:${PORTSDIR}/graphics/jasper
 PLIST_SUB+=	JP2=""
 .else
 CONFIGURE_ARGS+=--without-jasper
 PLIST_SUB+=	JP2="@comment "
 .endif
 
-.if !defined(WITHOUT_JPEG)
-LIB_DEPENDS+=	jpeg.11:${PORTSDIR}/graphics/jpeg
+.if ${PORT_OPTIONS:MJPEG}
+LIB_DEPENDS+=	jpeg:${PORTSDIR}/graphics/jpeg
 PLIST_SUB+=	JPEG=""
 .else
 CONFIGURE_ARGS+=--without-libjpeg
 PLIST_SUB+=	JPEG="@comment "
 .endif
 
-.if !defined(WITHOUT_LUA)
+.if ${PORT_OPTIONS:MLUA}
 USE_LUA=	5.1
 .else
 CONFIGURE_ARGS+=--without-lua
 .endif
 
-.if !defined(WITHOUT_OPENEXR)
+.if ${PORT_OPTIONS:MOPENEXR}
 LIB_DEPENDS+=	IlmImf:${PORTSDIR}/graphics/OpenEXR
 PLIST_SUB+=	OPENEXR=""
 .else
@@ -134,20 +122,20 @@ CONFIGURE_ARGS+=--without-openexr
 PLIST_SUB+=	OPENEXR="@comment "
 .endif
 
-.if !defined(WITHOUT_OPENRAW)
-LIB_DEPENDS+=	openraw.6:${PORTSDIR}/graphics/libopenraw
+.if ${PORT_OPTIONS:MOPENRAW}
+LIB_DEPENDS+=	openraw:${PORTSDIR}/graphics/libopenraw
 PLIST_SUB+=	OPENRAW=""
 .else
 CONFIGURE_ARGS+=--without-libopenraw
 PLIST_SUB+=	OPENRAW="@comment "
 .endif
 
-.if defined(WITHOUT_OPTIMIZE)
+.if ${PORT_OPTIONS:MSIMD}
 CONFIGURE_ARGS+=--disable-mmx \
 		--disable-sse
 .endif
 
-.if !defined(WITHOUT_PNG)
+.if ${PORT_OPTIONS:MPNG}
 LIB_DEPENDS+=	png15:${PORTSDIR}/graphics/png
 PLIST_SUB+=	PNG=""
 .else
@@ -155,15 +143,15 @@ CONFIGURE_ARGS+=--without-libpng
 PLIST_SUB+=	PNG="@comment "
 .endif
 
-.if !defined(WITHOUT_RSVG)
-LIB_DEPENDS+=	rsvg-2.2:${PORTSDIR}/graphics/librsvg2
+.if ${PORT_OPTIONS:MLIBRSVG2}
+LIB_DEPENDS+=	rsvg-2:${PORTSDIR}/graphics/librsvg2
 PLIST_SUB+=	RSVG=""
 .else
 CONFIGURE_ARGS+=--without-librsvg
 PLIST_SUB+=	RSVG="@comment "
 .endif
 
-.if !defined(WITHOUT_SDL)
+.if ${PORT_OPTIONS:MSDL}
 USE_SDL=	sdl
 PLIST_SUB+=	SDL=""
 .else
@@ -171,13 +159,13 @@ CONFIGURE_ARGS+=--without-sdl
 PLIST_SUB+=	SDL="@comment "
 .endif
 
-.if !defined(WITHOUT_SPIRO)
-LIB_DEPENDS+=	spiro.0:${PORTSDIR}/graphics/libspiro
+.if ${PORT_OPTIONS:MSPIRO}
+LIB_DEPENDS+=	spiro:${PORTSDIR}/graphics/libspiro
 .else
 CONFIGURE_ARGS+=--without-libspiro
 .endif
 
-.if !defined(WITHOUT_V4L)
+.if ${PORT_OPTIONS:MV4L}
 BUILD_DEPENDS+=	${LOCALBASE}/include/linux/videodev.h:${PORTSDIR}/multimedia/v4l_compat
 PLIST_SUB+=	V4L=""
 .else
@@ -185,16 +173,33 @@ CONFIGURE_ARGS+=--without-libv4l
 PLIST_SUB+=	V4L="@comment "
 .endif
 
-.if !defined(NOPORTDOCS)
+# umfpack library is broken
+#.if ${PORT_OPTIONS:MUMFPACK}
+#LIB_DEPENDS+=	umfpack:${PORTSDIR}/math/suitesparse
+#PLIST_SUB+=	UMFPACK=""
+#.else
+CONFIGURE_ARGS+=--without-umfpack
+PLIST_SUB+=	UMFPACK="@comment "
+#.endif
+
+.if ${PORT_OPTIONS:MEXIV2}
+LIB_DEPENDS+=	exiv2:${PORTSDIR}/graphics/exiv2
+PLIST_SUB+=	EXIV2=""
+.else
+CONFIGURE_ARGS+=--without-exiv2
+PLIST_SUB+=	EXIV2="@comment "
+.endif
+
+.if ${PORT_OPTIONS:MDOCS}
 BUILD_DEPENDS+=	a2x:${PORTSDIR}/textproc/asciidoc
 .else
-CONFIGURE_ARGS+=	--disable-docs
+CONFIGURE_ARGS+=--disable-docs
 .endif
 
 post-patch:
-.if defined(NOPORTDOCS)
-	${REINPLACE_CMD} -e '10s|\\||; 11,13d' \
-		${WRKSRC}/Makefile.am
+.if empty(PORT_OPTIONS:MDOCS)
+	@${REINPLACE_CMD} -e '/^SUBDIRS/s|tools examples tests||' \
+		${WRKSRC}/Makefile.in
 .endif
 	${REINPLACE_CMD} -e 's|-T||' \
 		${WRKSRC}/docs/Makefile.am ${WRKSRC}/docs/Makefile.in
@@ -204,12 +209,12 @@ post-patch:
 		${WRKSRC}/tools/gobj2dot.rb
 
 post-build:
-.if !defined(NOPORTDOCS)
+.if ${PORT_OPTIONS:MDOCS}
 	cd ${WRKSRC}/docs && ${GMAKE}
 .endif
 
 post-install:
-.if !defined(NOPORTDOCS)
+.if ${PORT_OPTIONS:MDOCS}
 	${MKDIR} ${DOCSDIR}/gallery
 	(cd ${WRKSRC}/docs && \
 	    ${COPYTREE_SHARE} \*.html ${DOCSDIR} && \
@@ -218,7 +223,7 @@ post-install:
 	    ${INSTALL_DATA} gallery/clones* ${DOCSDIR}/gallery && \
 	    ${INSTALL_DATA} gallery/index.html ${DOCSDIR}/gallery )
 .endif
-.if !defined(NOPORTEXAMPLES)
+.if ${PORT_OPTIONS:MEXAMPLES}
 	${MKDIR} ${EXAMPLESDIR}/data
 	${INSTALL_DATA} ${WRKSRC}/examples/data/surfer.png ${EXAMPLESDIR}/data
 .for ex in ${EXAMPLES}
_______________________________________________
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"