From owner-freebsd-eclipse@FreeBSD.ORG Mon Oct 10 15:10:17 2005 Return-Path: X-Original-To: freebsd-eclipse@hub.freebsd.org Delivered-To: freebsd-eclipse@hub.freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 8C89216A41F for ; Mon, 10 Oct 2005 15:10:17 +0000 (GMT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id 4402843D5E for ; Mon, 10 Oct 2005 15:10:14 +0000 (GMT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.13.3/8.13.3) with ESMTP id j9AFAEkW094174 for ; Mon, 10 Oct 2005 15:10:14 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.13.3/8.13.1/Submit) id j9AFAEfl094173; Mon, 10 Oct 2005 15:10:14 GMT (envelope-from gnats) Date: Mon, 10 Oct 2005 15:10:14 GMT Message-Id: <200510101510.j9AFAEfl094173@freefall.freebsd.org> To: freebsd-eclipse@FreeBSD.org From: Herve Quiroz Cc: Subject: Re: ports/86860: [patch] Align java/eclipse invocation with javavmwrapper X-BeenThere: freebsd-eclipse@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Herve Quiroz List-Id: "FreeBSD users of eclipse EDI, tools, rich client apps & ports." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 10 Oct 2005 15:10:17 -0000 The following reply was made to PR ports/86860; it has been noted by GNATS. From: Herve Quiroz To: bug-followup@FreeBSD.org, past@ebs.gr Cc: Subject: Re: ports/86860: [patch] Align java/eclipse invocation with javavmwrapper Date: Mon, 10 Oct 2005 17:00:27 +0200 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" $@