Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 10 Oct 2005 16:10:21 GMT
From:      Panagiotis Astithas <past@ebs.gr>
To:        freebsd-eclipse@FreeBSD.org
Subject:   Re: ports/86860: [patch] Align java/eclipse invocation with javavmwrapper
Message-ID:  <200510101610.j9AGALDD002440@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help
The following reply was made to PR ports/86860; it has been noted by GNATS.

From: Panagiotis Astithas <past@ebs.gr>
To: Herve Quiroz <herve.quiroz@esil.univ-mrs.fr>
Cc: bug-followup@FreeBSD.org
Subject: Re: ports/86860: [patch] Align java/eclipse invocation with javavmwrapper
Date: Mon, 10 Oct 2005 19:02:52 +0300

 Herve Quiroz wrote:
 > Hi,
 > 
 > I am a bit late on this topic (PR already closed) but I just noticed a
 > flaw with this change.
 > 
 > javavmwrapper picks a JDK depending on several variables -- see
 > javavm(1). With this change it is possible that a not suitable JDK gets
 > picked up. Here is a patch that should fix this (I didn't test it
 > though).
 > 
 > 
 > Index: files/eclipse.in
 > ===================================================================
 > RCS file: /home/ncvs/ports/java/eclipse/files/eclipse.in,v
 > retrieving revision 1.5
 > diff -u -r1.5 eclipse.in
 > --- files/eclipse.in	9 Oct 2005 14:54:21 -0000	1.5
 > +++ files/eclipse.in	10 Oct 2005 14:54:13 -0000
 > @@ -46,4 +46,4 @@
 >  	exit 1
 >  fi
 >  
 > -PATH=${JAVA_HOME}/bin:$PATH exec "${ECLIPSE_HOME}/eclipse" $@
 > +JAVA_VERSION="%%JAVA_VERSION%%" JAVA_OS="%%JAVA_OS%%" PATH=${JAVA_HOME}/bin:$PATH exec "${ECLIPSE_HOME}/eclipse" $@
 
 Why do you think this is necessary? What the eclipse executable does is 
 invoke the java executable it finds in the PATH. Since we have made sure 
 the javavmwrapper's symlink will get chosen as the default, it will 
 honour the JAVA_VERSION and JAVA_OS variables, right? Am I missing 
 something here?
 
 Regards,
 
 Panagiotis



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