Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 15 Aug 2013 10:32:23 +0000 (UTC)
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r324756 - head/x11/xmascot
Message-ID:  <201308151032.r7FAWNi8072786@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: danfe
Date: Thu Aug 15 10:32:22 2013
New Revision: 324756
URL: http://svnweb.freebsd.org/changeset/ports/324756

Log:
  In modern world of parallel builds and our new bmake(1) imake's popular
  MakeSubdirs() macro becomes very unsafe, as it hooks the subdirectories'
  build commands to the "all" target, along with main program executable.
  If MakeSubdirs() was used to build a dependency of the main program, it
  is highly likely that parallel build (-jX) will fail; and that's what's
  happening to this port here.
  
  To remedy the problem, use NamedTargetSubdirs() instead; also make sure
  that correct make(1) implementation would be used for sub-makes (ignore
  bogus "MAKE = make" set by imake in generated Makefiles).
  
  While here, improve wording and formatting in the port description.
  
  Reported by:	marino

Modified:
  head/x11/xmascot/Makefile
  head/x11/xmascot/pkg-descr

Modified: head/x11/xmascot/Makefile
==============================================================================
--- head/x11/xmascot/Makefile	Thu Aug 15 09:16:32 2013	(r324755)
+++ head/x11/xmascot/Makefile	Thu Aug 15 10:32:22 2013	(r324756)
@@ -13,10 +13,18 @@ COMMENT=	Moving mascot on your X11 scree
 
 USES=		imake
 USE_XORG=	ice sm x11 xaw xext xmu xpm xt
+MAKE_ARGS=	MAKE="${MAKE_CMD}"
+
 MAN1=		xmascot.1
 
+# Unbreak parallel (-jX) builds: do not use unsafe MakeSubdirs() macro
+post-patch:
+	@${REINPLACE_CMD} -e 's/MakeSubdirs(/Named&$$(IMAGELIB),/' \
+		${WRKSRC}/Imakefile
+
 post-install:
-	-@${MKDIR} ${PREFIX}/lib/X11/ja_JP.EUC/app-defaults
-	${CP} ${PREFIX}/lib/X11/ja_JP.eucJP/app-defaults/XMascot ${PREFIX}/lib/X11/ja_JP.EUC/app-defaults/
+	@${MKDIR} ${PREFIX}/lib/X11/ja_JP.EUC/app-defaults
+	${INSTALL_DATA} ${PREFIX}/lib/X11/ja_JP.eucJP/app-defaults/XMascot \
+		${PREFIX}/lib/X11/ja_JP.EUC/app-defaults
 
 .include <bsd.port.mk>

Modified: head/x11/xmascot/pkg-descr
==============================================================================
--- head/x11/xmascot/pkg-descr	Thu Aug 15 09:16:32 2013	(r324755)
+++ head/x11/xmascot/pkg-descr	Thu Aug 15 10:32:22 2013	(r324756)
@@ -1,22 +1,23 @@
-XMascot displays a moving mascot on your X11 screen.  XMascot has
-the following options,
+XMascot displays a moving mascot on your X11 screen.  XMascot has the
+following options:
 
- -Moving	pretty mascot moving
- -Strectch	stretch as you like
- -Talking	mascot talks with extract command and data
- -Alarm		mascot may make some actions at time you define
- -BIFF		mascot may let you know arriving a mail.
+  - Moving		pretty mascot moving
+  - Stretch		stretch it as you like
+  - Talking		mascot talks with extract command and data
+  - Alarm		mascot may make some actions at defined time
+  - BIFF		mascot may let you know arriving a mail
 
 XMascot supports these image formats:
 
-MAG	*.mag	16 colors and 256 colors
-TIFF	*.tif	16 colors and 256 colors, in raw or lzw
-PPM	*.ppm	256 level color	, in raw
-PGM	*.pgm	256 level gray scale, in raw
-PBM	*.pbm	2 level monocrome, in raw
-PNM	*.pnm	PPM, PGM, or PBM
+  - MAG (*.mag)		16 colors and 256 colors
+  - TIFF (*.tif)	16 colors and 256 colors, in raw or lzw
+  - PPM (*.ppm)		256 level color, in raw
+  - PGM (*.pgm)		256 level gray scale, in raw
+  - PBM (*.pbm)		2 level monochrome, in raw
+  - PNM (*.pnm)		PPM, PGM, or PBM
 
-XMascot distinguishes images from their suffix and can load other image formats
-when {suffix}topnm, *topgm, or *topbm commands are found in your system.
+XMascot distinguishes images from their suffix and can load other image
+formats when corresponding *topnm, *topgm, or *topbm commands are found
+in your system.
 
 WWW: http://cclub-flying.dsl.gr.jp/products/xmascot/



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201308151032.r7FAWNi8072786>