From rv@arabica.esil.univ-mrs.fr  Tue May 23 18:18:57 2006
Return-Path: <rv@arabica.esil.univ-mrs.fr>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 2E50916A49E
	for <FreeBSD-gnats-submit@freebsd.org>; Tue, 23 May 2006 18:18:57 +0000 (UTC)
	(envelope-from rv@arabica.esil.univ-mrs.fr)
Received: from arabica.esil.univ-mrs.fr (arabica.esil.univ-mrs.fr [139.124.41.108])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 5843043D76
	for <FreeBSD-gnats-submit@freebsd.org>; Tue, 23 May 2006 18:18:56 +0000 (GMT)
	(envelope-from rv@arabica.esil.univ-mrs.fr)
Received: from arabica.esil.univ-mrs.fr (localhost [127.0.0.1])
	by arabica.esil.univ-mrs.fr (8.13.6/8.13.6) with ESMTP id k4NIItQo036049;
	Tue, 23 May 2006 20:18:55 +0200 (CEST)
	(envelope-from rv@arabica.esil.univ-mrs.fr)
Received: (from rv@localhost)
	by arabica.esil.univ-mrs.fr (8.13.6/8.13.6/Submit) id k4NIIsxG035877;
	Tue, 23 May 2006 20:18:54 +0200 (CEST)
	(envelope-from rv)
Message-Id: <200605231818.k4NIIsxG035877@arabica.esil.univ-mrs.fr>
Date: Tue, 23 May 2006 20:18:54 +0200 (CEST)
From: Herve Quiroz <hq@freebsd.org>
To: FreeBSD-gnats-submit@freebsd.org
Cc: daniel@roe.ch
Subject: [PATCH] games/spicetrade: Update to bsd.java.mk 2.0 and other improvements
X-Send-Pr-Version: 3.113
X-GNATS-Notify: daniel@roe.ch

>Number:         97700
>Category:       ports
>Synopsis:       [PATCH] games/spicetrade: Update to bsd.java.mk 2.0 and other improvements
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    hq
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Tue May 23 18:20:24 GMT 2006
>Closed-Date:    Wed May 24 13:53:13 GMT 2006
>Last-Modified:  Wed May 24 13:53:13 GMT 2006
>Originator:     Herve Quiroz
>Release:        FreeBSD 6.1-RELEASE i386
>Organization:
>Environment:
System: FreeBSD arabica.esil.univ-mrs.fr 6.1-RELEASE FreeBSD 6.1-RELEASE #0: Tue May  9 17:19:22 CEST
>Description:

- Update to bsd.java.mk 2.0
- Set JAVA_VERSION in launcher shell script
- Use absolute path for javavmwrapper in launcher shell script
- Add $FreeBSD$ tag
- Unquote NO_PACKAGE [1]

Reported by:	portlint [1]

NOTE: is there a reason why the port uses USE_X_PREFIX? The Porter's Handbook
encourages the use of DATADIR=${JAVASHAREDIR}/${PORTNAME} for applications
which only install machine-independent java-related files:

http://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/using-java.html#JAVA-BEST-PRACTICES

Port maintainer (daniel@roe.ch) is cc'd.

Generated with FreeBSD Port Tools 0.77
>How-To-Repeat:
>Fix:

--- spicetrade-1.2.patch begins here ---
Index: Makefile
===================================================================
RCS file: /home/ncvs/ports/games/spicetrade/Makefile,v
retrieving revision 1.2
diff -u -u -r1.2 Makefile
--- Makefile	11 Dec 2005 22:46:28 -0000	1.2
+++ Makefile	23 May 2006 18:15:20 -0000
@@ -17,9 +17,10 @@
 MAINTAINER=	daniel@roe.ch
 COMMENT=	Colourful 14th century Arab spice trading game
 
-NO_PACKAGE=	"Package will be 186MB, set FORCE_PACKAGE if you really want to build it"
+NO_PACKAGE=	Package will be 186MB, set FORCE_PACKAGE if you really want to build it
 
-USE_JAVA=	1.4+
+USE_JAVA=	yes
+JAVA_VERSION=	1.4+
 USE_X_PREFIX=	yes
 NO_BUILD=	yes
 
Index: files/spicetrade.in
===================================================================
RCS file: /home/ncvs/ports/games/spicetrade/files/spicetrade.in,v
retrieving revision 1.1
diff -u -u -r1.1 spicetrade.in
--- files/spicetrade.in	11 Sep 2005 02:24:26 -0000	1.1
+++ files/spicetrade.in	23 May 2006 18:15:20 -0000
@@ -1,2 +1,6 @@
 #!/bin/sh
-exec java -jar %%JARFILE%% "$@"
+#
+# $FreeBSD$
+
+JAVA_VERSION="%%JAVA_VERSION%%" \
+	exec "%%LOCALBASE%%/bin/java" -jar "%%JARFILE%%" "$@"
--- spicetrade-1.2.patch ends here ---

>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->hq 
Responsible-Changed-By: edwin 
Responsible-Changed-When: Tue May 23 18:24:14 UTC 2006 
Responsible-Changed-Why:  
Submitter is a commiter 

http://www.freebsd.org/cgi/query-pr.cgi?pr=97700 
State-Changed-From-To: open->feedback 
State-Changed-By: edwin 
State-Changed-When: Tue May 23 18:24:27 UTC 2006 
State-Changed-Why:  
Awaiting maintainers feedback 

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

From: Daniel Roethlisberger <daniel@roe.ch>
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/97700: [PATCH] games/spicetrade: Update to bsd.java.mk 2.0 and other improvements
Date: Wed, 24 May 2006 10:41:03 +0200

 I approve, thanks for the patch.
 
 I used USE_X_PREFIX since from a users perspective, I'd expect a game
 requiring X11 to be installed under the X prefix.  But since the
 porter's handbook strongly recommends otherwise, feel free to change the
 port accordingly when committing the patch.
 
 -- 
 Daniel Roethlisberger
State-Changed-From-To: feedback->closed 
State-Changed-By: hq 
State-Changed-When: Wed May 24 13:53:02 UTC 2006 
State-Changed-Why:  
Patch approved and commited. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=97700 
>Unformatted:
