Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPascal Rapicault2015-03-03 15:07:07 +0000
committerPascal Rapicault2015-03-04 17:09:46 +0000
commit1b96ce896c49151b0e20fa49ba680d08415cca8f (patch)
treec7e1ac5374f0b10a82cc8d8ba057d957f91811b8
parent66c9297a230aeb57eb49d07f77eebf9c68d5b9ad (diff)
downloadrt.equinox.p2-1b96ce896c49151b0e20fa49ba680d08415cca8f.tar.gz
rt.equinox.p2-1b96ce896c49151b0e20fa49ba680d08415cca8f.tar.xz
rt.equinox.p2-1b96ce896c49151b0e20fa49ba680d08415cca8f.zip
Bug 431116 - Releases for Mac OS X should be bundled as a proper "Mac
App" and/or "Library" Change-Id: I2bdcc86960361f8268964b4e95bf0eb960dc3cd6 Signed-off-by: Pascal Rapicault <pascal@rapicorp.com>
-rw-r--r--bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/EclipseLauncherParser.java16
-rw-r--r--bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/EquinoxManipulatorImpl.java5
-rw-r--r--bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/ParserUtils.java13
-rw-r--r--bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/utils/FileUtils.java10
-rw-r--r--bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.equinox.launcher.jarbin0 -> 43719 bytes
-rw-r--r--bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.equinox.simpleconfigurator.jarbin0 -> 23499 bytes
-rw-r--r--bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.osgi.jarbin0 -> 928386 bytes
-rw-r--r--bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/FwkAdminAndSimpleConfiguratorTest.java5
-rw-r--r--bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/MacOSLikeSetup.java19
-rw-r--r--bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/TestVMArg.java84
-rw-r--r--bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/UtilsTest.java8
-rw-r--r--bundles/org.eclipse.equinox.p2.director.app/src/org/eclipse/equinox/internal/p2/director/app/DirectorApplication.java30
-rw-r--r--bundles/org.eclipse.equinox.p2.publisher.eclipse/META-INF/MANIFEST.MF2
-rw-r--r--bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/internal/p2/publisher/eclipse/BrandingIron.java86
-rw-r--r--bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/internal/p2/publisher/eclipse/InfoPListEditor.java189
-rw-r--r--bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/p2/publisher/eclipse/EquinoxExecutableAction.java43
-rw-r--r--bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/AutomatedTests.java2
-rw-r--r--bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/publisher/actions/EquinoxExecutableActionTest.java29
-rw-r--r--bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/LazyManipulator.java2
-rw-r--r--bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/PlatformConfigurationWrapper.java11
-rw-r--r--bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/Util.java48
21 files changed, 348 insertions, 254 deletions
diff --git a/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/EclipseLauncherParser.java b/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/EclipseLauncherParser.java
index 1154b6bbb..e87a69954 100644
--- a/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/EclipseLauncherParser.java
+++ b/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/EclipseLauncherParser.java
@@ -21,13 +21,12 @@ import org.eclipse.equinox.internal.frameworkadmin.equinox.utils.FileUtils;
import org.eclipse.equinox.internal.frameworkadmin.utils.Utils;
import org.eclipse.equinox.internal.provisional.frameworkadmin.FrameworkAdminRuntimeException;
import org.eclipse.equinox.internal.provisional.frameworkadmin.LauncherData;
+import org.eclipse.osgi.service.environment.Constants;
import org.eclipse.osgi.util.NLS;
import org.osgi.service.log.LogService;
public class EclipseLauncherParser {
- public static final String MAC_OS_APP_FOLDER = ".app/Contents/MacOS"; //$NON-NLS-1$
private static final String CONFIGURATION_FOLDER = "configuration"; //$NON-NLS-1$
- public static final String MACOSX_BUNDLED = "macosx-bundled"; //$NON-NLS-1$
//this figures out the location of the data area on partial data read from the <eclipse>.ini
private URI getOSGiInstallArea(List<String> lines, URI base, LauncherData launcherData) {
@@ -47,14 +46,11 @@ public class EclipseLauncherParser {
ParserUtils.removeArgument(EquinoxConstants.OPTION_INSTALL, lines);
return;
}
- String launcherString = launcherFolder.getAbsolutePath().replace('\\', '/');
- if (launcherString.endsWith(MAC_OS_APP_FOLDER)) {
- //We can do 3 calls to getParentFile without checking because
- launcherFolder = launcherFolder.getParentFile().getParentFile();
- if (!launcherData.getOS().endsWith(MACOSX_BUNDLED))
- launcherFolder = launcherFolder.getParentFile();
- }
- if (!ParserUtils.fromOSGiJarToOSGiInstallArea(launcherData.getFwJar().getAbsolutePath()).equals(launcherFolder)) {
+ if (Constants.OS_MACOSX.equals(launcherData.getOS())) {
+ if (!new File(ParserUtils.fromOSGiJarToOSGiInstallArea(launcherData.getFwJar().getAbsolutePath()), "../MacOS").equals(launcherFolder)) {
+ ParserUtils.setValueForArgument(EquinoxConstants.OPTION_INSTALL, launcherFolder.getAbsolutePath().replace('\\', '/'), lines);
+ }
+ } else if (!ParserUtils.fromOSGiJarToOSGiInstallArea(launcherData.getFwJar().getAbsolutePath()).equals(launcherFolder)) {
ParserUtils.setValueForArgument(EquinoxConstants.OPTION_INSTALL, launcherFolder.getAbsolutePath().replace('\\', '/'), lines);
}
}
diff --git a/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/EquinoxManipulatorImpl.java b/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/EquinoxManipulatorImpl.java
index f516d4da7..c8e646fbf 100644
--- a/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/EquinoxManipulatorImpl.java
+++ b/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/EquinoxManipulatorImpl.java
@@ -93,7 +93,10 @@ public class EquinoxManipulatorImpl implements Manipulator {
int dotLocation = launcherName.lastIndexOf('.');
if (dotLocation != -1)
launcherName = launcherName.substring(0, dotLocation);
- File result = new File(launcher.getParentFile(), launcherName + EquinoxConstants.INI_EXTENSION);
+ File launcherFolder = launcher.getParentFile();
+ if (org.eclipse.osgi.service.environment.Constants.OS_MACOSX.equals(launcherData.getOS()))
+ launcherFolder = launcherData.getFwConfigLocation().getParentFile();
+ File result = new File(launcherFolder, launcherName + EquinoxConstants.INI_EXTENSION);
return result;
}
diff --git a/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/ParserUtils.java b/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/ParserUtils.java
index 4080ffcb6..12ba81aac 100644
--- a/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/ParserUtils.java
+++ b/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/ParserUtils.java
@@ -48,19 +48,16 @@ public class ParserUtils {
File launcherFile = launcherData.getLauncher();
if (launcherFile != null) {
- if (Constants.OS_MACOSX.equals(launcherData.getOS())) {
- //the equinox launcher will look 3 levels up on the mac when going from executable to launcher.jar
+ if (Constants.OS_MACOSX.equals(launcherData.getOS())) { //
+ //TODO We are going to change this - the equinox launcher will look 3 levels up on the mac when going from executable to launcher.jar
//see org.eclipse.equinox.executable/library/eclipse.c : findStartupJar();
IPath launcherPath = new Path(launcherFile.getAbsolutePath());
- if (launcherPath.segmentCount() > 4) {
- //removing "Eclipse.app/Contents/MacOS/eclipse"
- launcherPath = launcherPath.removeLastSegments(4);
+ if (launcherPath.segmentCount() > 2) {
+ //removing "MacOS/eclipse" from the end of the path
+ launcherPath = launcherPath.removeLastSegments(2).append("Eclipse"); //$NON-NLS-1$
return launcherPath.toFile();
}
}
- if (EclipseLauncherParser.MACOSX_BUNDLED.equals(launcherData.getOS())) {
- Log.log(LogService.LOG_WARNING, "Problem figuring out the osgi install area. The bundled mode of macosx requires a -startup argument to be specified."); //$NON-NLS-1$
- }
return launcherFile.getParentFile();
}
return null;
diff --git a/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/utils/FileUtils.java b/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/utils/FileUtils.java
index 7eed2d207..24dd3b719 100644
--- a/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/utils/FileUtils.java
+++ b/bundles/org.eclipse.equinox.frameworkadmin.equinox/src/org/eclipse/equinox/internal/frameworkadmin/equinox/utils/FileUtils.java
@@ -15,12 +15,12 @@ import java.io.*;
import java.net.*;
import java.util.*;
import org.eclipse.core.runtime.*;
-import org.eclipse.equinox.internal.frameworkadmin.equinox.*;
+import org.eclipse.equinox.internal.frameworkadmin.equinox.EquinoxConstants;
+import org.eclipse.equinox.internal.frameworkadmin.equinox.ParserUtils;
import org.eclipse.equinox.internal.provisional.frameworkadmin.LauncherData;
import org.eclipse.equinox.internal.provisional.frameworkadmin.Manipulator;
import org.eclipse.osgi.service.environment.Constants;
import org.osgi.framework.Version;
-import org.osgi.service.log.LogService;
public class FileUtils {
private static String FILE_SCHEME = "file"; //$NON-NLS-1$
@@ -76,12 +76,10 @@ public class FileUtils {
File launcherDir = null;
if (Constants.OS_MACOSX.equals(launcherData.getOS())) {
IPath launcherPath = new Path(launcherData.getLauncher().getAbsolutePath());
- if (launcherPath.segmentCount() > 4) {
- launcherPath = launcherPath.removeLastSegments(4);
+ if (launcherPath.segmentCount() > 2) {
+ launcherPath = launcherPath.removeLastSegments(2).append("Eclipse");
launcherDir = launcherPath.toFile();
}
- } else if (EclipseLauncherParser.MACOSX_BUNDLED.equals(launcherData.getOS())) {
- Log.log(LogService.LOG_WARNING, "Problem figuring out the osgi install area. The bundled mode of macosx requires a -startup argument to be specified."); //$NON-NLS-1$
} else
launcherDir = launcherData.getLauncher().getParentFile();
pluginsDir = new File(launcherDir, EquinoxConstants.PLUGINS_DIR);
diff --git a/bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.equinox.launcher.jar b/bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.equinox.launcher.jar
new file mode 100644
index 000000000..5947c8dee
--- /dev/null
+++ b/bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.equinox.launcher.jar
Binary files differ
diff --git a/bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.equinox.simpleconfigurator.jar b/bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.equinox.simpleconfigurator.jar
new file mode 100644
index 000000000..8a631d83a
--- /dev/null
+++ b/bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.equinox.simpleconfigurator.jar
Binary files differ
diff --git a/bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.osgi.jar b/bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.osgi.jar
new file mode 100644
index 000000000..20ca2696d
--- /dev/null
+++ b/bundles/org.eclipse.equinox.frameworkadmin.test/dataFile/mac/Eclipse.app/Contents/Eclipse/plugins/org.eclipse.osgi.jar
Binary files differ
diff --git a/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/FwkAdminAndSimpleConfiguratorTest.java b/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/FwkAdminAndSimpleConfiguratorTest.java
index a571c74cc..0adeb1a22 100644
--- a/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/FwkAdminAndSimpleConfiguratorTest.java
+++ b/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/FwkAdminAndSimpleConfiguratorTest.java
@@ -14,7 +14,6 @@ import java.io.File;
import java.io.IOException;
import org.eclipse.core.runtime.*;
import org.eclipse.equinox.frameworkadmin.BundleInfo;
-import org.eclipse.equinox.internal.frameworkadmin.equinox.EclipseLauncherParser;
import org.eclipse.equinox.internal.provisional.frameworkadmin.*;
import org.eclipse.osgi.service.environment.Constants;
import org.osgi.framework.BundleException;
@@ -47,8 +46,8 @@ public abstract class FwkAdminAndSimpleConfiguratorTest extends AbstractFwkAdmin
configurationFolder = new File(installFolder, "configuration");
launcherName = "eclipse";
- boolean isMacOS = Constants.OS_MACOSX.equals(os) || EclipseLauncherParser.MACOSX_BUNDLED.equals(os);
- launcherPath = isMacOS ? new Path("Eclipse" + EclipseLauncherParser.MAC_OS_APP_FOLDER).append(launcherName) : new Path(launcherName);
+ boolean isMacOS = Constants.OS_MACOSX.equals(os);
+ launcherPath = isMacOS ? new Path("../").append(launcherName) : new Path(launcherName);
LauncherData launcherData = manipulator.getLauncherData();
launcherData.setHome(installFolder);
diff --git a/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/MacOSLikeSetup.java b/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/MacOSLikeSetup.java
index 80acaa7e7..a3257c3f5 100644
--- a/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/MacOSLikeSetup.java
+++ b/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/MacOSLikeSetup.java
@@ -27,15 +27,15 @@ public class MacOSLikeSetup extends FwkAdminAndSimpleConfiguratorTest {
FrameworkAdmin fwkAdmin = getEquinoxFrameworkAdmin();
Manipulator manipulator = fwkAdmin.getManipulator();
- File installFolder = new File(Activator.getContext().getDataFile(MacOSLikeSetup.class.getName()), "eclipse");
+ File installFolder = new File(Activator.getContext().getDataFile(getName()), "Eclipse.app/Contents/Eclipse");
File configurationFolder = new File(installFolder, "configuration");
- File launcherFolder = new File(installFolder, "Eclipse.app/Contents/MacOS/");
+ File launcherFolder = new File(installFolder, "../MacOS/");
File launcherName = new File(launcherFolder, "eclipse");
LauncherData launcherData = manipulator.getLauncherData();
launcherData.setFwConfigLocation(configurationFolder);
launcherData.setLauncher(launcherName);
- launcherData.setLauncherConfigLocation(new File(launcherFolder, "eclipse.ini"));
+ launcherData.setLauncherConfigLocation(new File(installFolder, "eclipse.ini"));
launcherData.setOS(Constants.OS_MACOSX);
//Setup the plugins as they should
@@ -60,11 +60,11 @@ public class MacOSLikeSetup extends FwkAdminAndSimpleConfiguratorTest {
} catch (IllegalStateException e) {
//TODO We ignore the framework JAR location not set exception
}
- File launcherIni = new File(launcherFolder, "eclipse.ini");
+ File launcherIni = new File(installFolder, "eclipse.ini");
assertNotContent(launcherIni, "-configuration");
assertNotContent(launcherIni, "-install");
assertContent(launcherIni, "-startup");
- assertContent(launcherIni, "../../../plugins/org.eclipse.equinox.launcher.jar");
+ assertContent(launcherIni, "../Eclipse/plugins/org.eclipse.equinox.launcher.jar");
assertNotContent(launcherIni, MacOSLikeSetup.class.getName());
assertNotContent(new File(configurationFolder, "config.ini"), MacOSLikeSetup.class.getName());
assertTrue("bundles.info missing", new File(configurationFolder, "org.eclipse.equinox.simpleconfigurator/bundles.info").exists());
@@ -75,9 +75,9 @@ public class MacOSLikeSetup extends FwkAdminAndSimpleConfiguratorTest {
FrameworkAdmin fwkAdmin = getEquinoxFrameworkAdmin();
Manipulator manipulator = fwkAdmin.getManipulator();
- File installFolder = new File(Activator.getContext().getDataFile(MacOSLikeSetup.class.getName()), "eclipse");
+ File installFolder = new File(Activator.getContext().getDataFile(getName()), "Eclipse.app/Contents/Eclipse");
File configurationFolder = new File(installFolder, "configuration");
- File launcherFolder = new File(installFolder, "Eclipse.app/Contents/MacOS/");
+ File launcherFolder = new File(installFolder, "../MacOS/");
File launcherName = new File(launcherFolder, "eclipse");
LauncherData launcherData = manipulator.getLauncherData();
@@ -97,10 +97,9 @@ public class MacOSLikeSetup extends FwkAdminAndSimpleConfiguratorTest {
manipulator.getConfigData().addBundle(new BundleInfo(bundle.toURI()));
manipulator.save(false);
- File launcherIni = new File(launcherFolder, "eclipse.ini");
+ File launcherIni = new File(installFolder, "eclipse.ini");
File bundleInfo = new File(configurationFolder, "org.eclipse.equinox.simpleconfigurator/bundles.info");
- assertNotContent(launcherIni, "-configuration");
- assertNotContent(launcherIni, "../../../configuration");
+ assertFalse(launcherIni.exists());
assertContent(bundleInfo, "file:plugins/bundle_1/");
}
}
diff --git a/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/TestVMArg.java b/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/TestVMArg.java
index d1a011457..a24a31f87 100644
--- a/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/TestVMArg.java
+++ b/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/TestVMArg.java
@@ -13,10 +13,8 @@ package org.eclipse.equinox.frameworkadmin.tests;
import java.io.File;
import java.io.IOException;
import org.eclipse.core.runtime.Platform;
-import org.eclipse.equinox.internal.frameworkadmin.equinox.EclipseLauncherParser;
import org.eclipse.equinox.internal.provisional.frameworkadmin.FrameworkAdminRuntimeException;
import org.eclipse.equinox.internal.provisional.frameworkadmin.Manipulator;
-import org.eclipse.osgi.service.environment.Constants;
public class TestVMArg extends FwkAdminAndSimpleConfiguratorTest {
@@ -51,32 +49,32 @@ public class TestVMArg extends FwkAdminAndSimpleConfiguratorTest {
assertNotContent(m.getLauncherData().getLauncherConfigLocation(), "jre");
}
- public void testVMInsideInstall_MacOS() throws Exception {
- m = createMinimalConfiguration(TestEclipseDataArea.class.getName(), Constants.OS_MACOSX);
- final String expectedRelativePath = "../../../jre";
-
- File jreLocation = new File(m.getLauncherData().getLauncher().getParentFile(), expectedRelativePath);
- m.getLauncherData().setJvm(jreLocation);
- m.save(false);
-
- File launcherConfigFile = getLauncherConfigFile();
- assertNotContent("No absolute JRE path must be present in " + launcherConfigFile, launcherConfigFile, jreLocation.getAbsolutePath());
- assertContent("Relative JRE path must be present in " + launcherConfigFile, launcherConfigFile, expectedRelativePath);
- }
-
- public void testVMInsideInstall_MacOS_BundledLayout() throws Exception {
- m = createMinimalConfiguration(TestEclipseDataArea.class.getName(), EclipseLauncherParser.MACOSX_BUNDLED);
- // note the difference the traditional layout: one segment less
- final String expectedRelativePath = "../../jre";
-
- File jreLocation = new File(m.getLauncherData().getLauncher().getParentFile(), expectedRelativePath);
- m.getLauncherData().setJvm(jreLocation);
- m.save(false);
-
- File launcherConfigFile = getLauncherConfigFile();
- assertNotContent("No absolute JRE path must be present in " + launcherConfigFile, launcherConfigFile, jreLocation.getAbsolutePath());
- assertContent("Relative JRE path must be present in " + launcherConfigFile, launcherConfigFile, expectedRelativePath);
- }
+// public void testVMInsideInstall_MacOS() throws Exception {
+// m = createMinimalConfiguration(TestEclipseDataArea.class.getName(), Constants.OS_MACOSX);
+// final String expectedRelativePath = "../../../jre";
+//
+// File jreLocation = new File(m.getLauncherData().getLauncher().getParentFile(), expectedRelativePath);
+// m.getLauncherData().setJvm(jreLocation);
+// m.save(false);
+//
+// File launcherConfigFile = getLauncherConfigFile();
+// assertNotContent("No absolute JRE path must be present in " + launcherConfigFile, launcherConfigFile, jreLocation.getAbsolutePath());
+// assertContent("Relative JRE path must be present in " + launcherConfigFile, launcherConfigFile, expectedRelativePath);
+// }
+
+// public void testVMInsideInstall_MacOS_BundledLayout() throws Exception {
+// m = createMinimalConfiguration(TestEclipseDataArea.class.getName(), EclipseLauncherParser.MACOSX_BUNDLED);
+// // note the difference the traditional layout: one segment less
+// final String expectedRelativePath = "../../jre";
+//
+// File jreLocation = new File(m.getLauncherData().getLauncher().getParentFile(), expectedRelativePath);
+// m.getLauncherData().setJvm(jreLocation);
+// m.save(false);
+//
+// File launcherConfigFile = getLauncherConfigFile();
+// assertNotContent("No absolute JRE path must be present in " + launcherConfigFile, launcherConfigFile, jreLocation.getAbsolutePath());
+// assertContent("Relative JRE path must be present in " + launcherConfigFile, launcherConfigFile, expectedRelativePath);
+// }
public void testVMOutsideInstall() throws FrameworkAdminRuntimeException, IOException {
//Test VM path in the install folder
@@ -107,21 +105,21 @@ public class TestVMArg extends FwkAdminAndSimpleConfiguratorTest {
assertEquals(jreLocation, m.getLauncherData().getJvm());
}
- public void test269502_MacOS() throws Exception {
- m = createMinimalConfiguration(TestEclipseDataArea.class.getName(), Constants.OS_MACOSX);
-
- //Test VM path in the install folder
- String chosenPath = "/Users/Pascal/ibm5sr3/bin";
- File jreLocation = new File(chosenPath);
- m.getLauncherData().setJvm(jreLocation);
- m.save(false);
- assertContent(getLauncherConfigFile(), chosenPath);
- assertContent(m.getLauncherData().getLauncherConfigLocation(), "-vm");
- assertContent(m.getLauncherData().getLauncherConfigLocation(), chosenPath);
- assertNotContent(m.getLauncherData().getLauncherConfigLocation(), "file:");
- m.load();
- assertEquals(jreLocation, m.getLauncherData().getJvm());
- }
+// public void test269502_MacOS() throws Exception {
+// m = createMinimalConfiguration(TestEclipseDataArea.class.getName(), Constants.OS_MACOSX);
+//
+// //Test VM path in the install folder
+// String chosenPath = "/Users/Pascal/ibm5sr3/bin";
+// File jreLocation = new File(chosenPath);
+// m.getLauncherData().setJvm(jreLocation);
+// m.save(false);
+// assertContent(getLauncherConfigFile(), chosenPath);
+// assertContent(m.getLauncherData().getLauncherConfigLocation(), "-vm");
+// assertContent(m.getLauncherData().getLauncherConfigLocation(), chosenPath);
+// assertNotContent(m.getLauncherData().getLauncherConfigLocation(), "file:");
+// m.load();
+// assertEquals(jreLocation, m.getLauncherData().getJvm());
+// }
/**
* But 282303:
diff --git a/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/UtilsTest.java b/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/UtilsTest.java
index 26aa339f3..b2c8a62f6 100644
--- a/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/UtilsTest.java
+++ b/bundles/org.eclipse.equinox.frameworkadmin.test/src/org/eclipse/equinox/frameworkadmin/tests/UtilsTest.java
@@ -11,6 +11,7 @@
package org.eclipse.equinox.frameworkadmin.tests;
import java.io.File;
+import java.net.URI;
import org.eclipse.equinox.internal.frameworkadmin.equinox.utils.FileUtils;
import org.eclipse.equinox.internal.provisional.frameworkadmin.Manipulator;
import org.eclipse.osgi.service.environment.Constants;
@@ -53,14 +54,15 @@ public class UtilsTest extends AbstractFwkAdminTest {
}
public void testMacRealLocation() throws Exception {
- File installFolder = Activator.getContext().getDataFile("280007");
+ File installFolder = Activator.getContext().getDataFile("280007/Eclipse.app/Contents/Eclipse/");
File plugins = new File(installFolder, "plugins");
File foo = new File(plugins, "org.foo_1.2.3.abc");
foo.mkdirs();
- Manipulator manipulator = getFrameworkManipulator(new File(installFolder, "configuration"), new File(installFolder, "Eclipse.app/Contents/MacOS/eclipse"));
+ Manipulator manipulator = getFrameworkManipulator(new File(installFolder, "configuration"), new File(installFolder, "../MacOS/eclipse"));
manipulator.getLauncherData().setOS(Constants.OS_MACOSX);
- assertEquals(FileUtils.getEclipseRealLocation(manipulator, "org.foo"), foo.toURI());
+ URI res = FileUtils.getEclipseRealLocation(manipulator, "org.foo");
+ assertEquals(res, foo.toURI());
}
}
diff --git a/bundles/org.eclipse.equinox.p2.director.app/src/org/eclipse/equinox/internal/p2/director/app/DirectorApplication.java b/bundles/org.eclipse.equinox.p2.director.app/src/org/eclipse/equinox/internal/p2/director/app/DirectorApplication.java
index 32602be3e..74e7724a5 100644
--- a/bundles/org.eclipse.equinox.p2.director.app/src/org/eclipse/equinox/internal/p2/director/app/DirectorApplication.java
+++ b/bundles/org.eclipse.equinox.p2.director.app/src/org/eclipse/equinox/internal/p2/director/app/DirectorApplication.java
@@ -502,9 +502,6 @@ public class DirectorApplication implements IApplication, ProvisioningListener {
props.put(IProfile.PROP_ROAMING, Boolean.TRUE.toString());
String env = getEnvironmentProperty();
- //Detect the desire to have a bundled mac application and tweak the environemtn
- if (org.eclipse.osgi.service.environment.Constants.OS_MACOSX.equals(os) && destination.getName().endsWith(".app")) //$NON-NLS-1$
- env += ',' + org.eclipse.equinox.p2.core.spi.Constants.MACOSX_BUNDLED + "=true"; //$NON-NLS-1$
if (env != null)
props.put(IProfile.PROP_ENVIRONMENTS, env);
if (profileProperties != null)
@@ -587,12 +584,15 @@ public class DirectorApplication implements IApplication, ProvisioningListener {
}
}
- private void initializeServices() throws CoreException {
- BundleContext context = Activator.getContext();
- packageAdminRef = context.getServiceReference(PackageAdmin.class);
- packageAdmin = context.getService(packageAdminRef);
- ServiceReference<IProvisioningAgentProvider> agentProviderRef = context.getServiceReference(IProvisioningAgentProvider.class);
- IProvisioningAgentProvider provider = context.getService(agentProviderRef);
+ private void adjustDestination() {
+ //Detect the desire to have a bundled mac application and tweak the environment
+ if (destination == null)
+ return;
+ if (org.eclipse.osgi.service.environment.Constants.OS_MACOSX.equals(os) && destination.getName().endsWith(".app")) //$NON-NLS-1$
+ destination = new File(destination, "Contents/Eclipse");
+ }
+
+ private URI getP2DataAreaLocation(BundleContext context) {
URI p2DataArea;
if (destination != null || sharedLocation != null) {
File dataAreaFile = sharedLocation == null ? new File(destination, "p2") : sharedLocation;//$NON-NLS-1$
@@ -613,6 +613,17 @@ public class DirectorApplication implements IApplication, ProvisioningListener {
//Can't happen the filter never changes
}
}
+ return p2DataArea;
+ }
+
+ private void initializeServices() throws CoreException { //TODO FAIRE QQCHE ICI POUR POSITIONNER p2 folder CORRECTEMENT
+ BundleContext context = Activator.getContext();
+ packageAdminRef = context.getServiceReference(PackageAdmin.class);
+ packageAdmin = context.getService(packageAdminRef);
+ ServiceReference<IProvisioningAgentProvider> agentProviderRef = context.getServiceReference(IProvisioningAgentProvider.class);
+ IProvisioningAgentProvider provider = context.getService(agentProviderRef);
+
+ URI p2DataArea = getP2DataAreaLocation(context);
if (targetAgent == null) {
targetAgent = provider.createAgent(p2DataArea);
targetAgent.registerService(IProvisioningAgent.INSTALLER_AGENT, provider.createAgent(null));
@@ -1084,6 +1095,7 @@ public class DirectorApplication implements IApplication, ProvisioningListener {
if (printHelpInfo)
performHelpInfo();
else {
+ adjustDestination();
initializeServices();
if (!(printIUList || printRootIUList || printTags)) {
if (!canInstallInDestination()) {
diff --git a/bundles/org.eclipse.equinox.p2.publisher.eclipse/META-INF/MANIFEST.MF b/bundles/org.eclipse.equinox.p2.publisher.eclipse/META-INF/MANIFEST.MF
index 1bf34b7cd..ee15884af 100644
--- a/bundles/org.eclipse.equinox.p2.publisher.eclipse/META-INF/MANIFEST.MF
+++ b/bundles/org.eclipse.equinox.p2.publisher.eclipse/META-INF/MANIFEST.MF
@@ -7,7 +7,7 @@ Bundle-Activator: org.eclipse.pde.internal.publishing.Activator
Bundle-ActivationPolicy: lazy
Bundle-Vendor: %providerName
Bundle-Localization: plugin
-Bundle-RequiredExecutionEnvironment: J2SE-1.5
+Bundle-RequiredExecutionEnvironment: JavaSE-1.6
Import-Package: org.eclipse.equinox.app;version="[1.0.0,2.0.0)",
org.eclipse.equinox.frameworkadmin;version="[2.0.0,3.0.0)",
org.eclipse.equinox.internal.frameworkadmin.equinox,
diff --git a/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/internal/p2/publisher/eclipse/BrandingIron.java b/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/internal/p2/publisher/eclipse/BrandingIron.java
index 3cf2fe1b6..609e89aba 100644
--- a/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/internal/p2/publisher/eclipse/BrandingIron.java
+++ b/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/internal/p2/publisher/eclipse/BrandingIron.java
@@ -12,6 +12,8 @@
package org.eclipse.equinox.internal.p2.publisher.eclipse;
import java.io.*;
+import java.util.List;
+import javax.xml.transform.TransformerException;
import org.eclipse.equinox.p2.metadata.Version;
import org.eclipse.pde.internal.publishing.Utils;
import org.eclipse.pde.internal.swt.tools.IconExe;
@@ -20,18 +22,6 @@ import org.eclipse.pde.internal.swt.tools.IconExe;
*
*/
public class BrandingIron {
- private static final String MARKER_NAME = "%EXECUTABLE_NAME%"; //$NON-NLS-1$
- private static final String BUNDLE_NAME = "%BUNDLE_NAME%"; //$NON-NLS-1$
- private static final String ICON_NAME = "%ICON_NAME%"; //$NON-NLS-1$
- private static final String MARKER_KEY = "<key>CFBundleExecutable</key>"; //$NON-NLS-1$
- private static final String BUNDLE_KEY = "<key>CFBundleName</key>"; //$NON-NLS-1$
- private static final String BUNDLE_ID_KEY = "<key>CFBundleIdentifier</key>"; //$NON-NLS-1$
- private static final String BUNDLE_INFO_KEY = "<key>CFBundleGetInfoString</key>"; //$NON-NLS-1$
- private static final String BUNDLE_VERSION_KEY = "<key>CFBundleVersion</key>"; //$NON-NLS-1$
- private static final String BUNDLE_SHORT_VERSION_KEY = "<key>CFBundleShortVersionString</key>"; //$NON-NLS-1$
- private static final String ICON_KEY = "<key>CFBundleIconFile</key>"; //$NON-NLS-1$
- private static final String STRING_START = "<string>"; //$NON-NLS-1$
- private static final String STRING_END = "</string>"; //$NON-NLS-1$
private static final String XDOC_ICON = "-Xdock:icon=../Resources/Eclipse.icns"; //$NON-NLS-1$
private static final String XDOC_ICON_PREFIX = "-Xdock:icon=../Resources/"; //$NON-NLS-1$
@@ -186,7 +176,7 @@ public class BrandingIron {
//Initialize the target folders
File root = descriptor.getLocation();
- File target = new File(root, applicationName + ".app/Contents"); //$NON-NLS-1$
+ File target = root;
target.mkdirs();
new File(target, "MacOS").mkdirs(); //$NON-NLS-1$
new File(target, "Resources").mkdirs(); //$NON-NLS-1$
@@ -505,29 +495,21 @@ public class BrandingIron {
private void modifyInfoPListFile(ExecutablesDescriptor descriptor, File initialRoot, File targetRoot, String iconName) {
File infoPList = new File(initialRoot, "Info.plist"); //$NON-NLS-1$
- StringBuffer buffer;
+ InfoPListEditor infoPListEditor = null;
try {
- buffer = readFile(infoPList);
+ infoPListEditor = InfoPListEditor.loadPListEditor(infoPList);
} catch (IOException e) {
- System.out.println("Impossible to brand info.plist file"); //$NON-NLS-1$
+ System.out.println("Impossible to create info.plist editor for " + infoPList.getAbsolutePath() + ". Caused by " + e.getMessage()); //$NON-NLS-1$ //$NON-NLS-2$
return;
}
- int exePos = scan(buffer, 0, MARKER_NAME);
- if (exePos != -1)
- buffer.replace(exePos, exePos + MARKER_NAME.length(), name);
- else {
- replacePlistValue(buffer, MARKER_KEY, name);
- }
- int bundlePos = scan(buffer, 0, BUNDLE_NAME);
- if (bundlePos != -1)
- buffer.replace(bundlePos, bundlePos + BUNDLE_NAME.length(), name);
- else {
- replacePlistValue(buffer, BUNDLE_KEY, name);
- }
+ //Deal with error case
+ infoPListEditor.setKey(InfoPListEditor.MARKER_KEY, name);
+ infoPListEditor.setKey(InfoPListEditor.BUNDLE_KEY, name);
+ infoPListEditor.setKey(InfoPListEditor.BUNDLE_ID_KEY, id);
+ if (description != null)
+ infoPListEditor.setKey(InfoPListEditor.BUNDLE_INFO_KEY, description);
- replacePlistValue(buffer, BUNDLE_ID_KEY, id);
- replacePlistValue(buffer, BUNDLE_INFO_KEY, description);
if (version != null) {
// CFBundleShortVersionString is to be 3 segments only
// http://developer.apple.com/library/mac/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html#//apple_ref/doc/uid/20001431-111349
@@ -536,28 +518,21 @@ public class BrandingIron {
sv.append(version.getSegmentCount() > 1 ? version.getSegment(1).toString() : "0"); //$NON-NLS-1$
sv.append('.');
sv.append(version.getSegmentCount() > 2 ? version.getSegment(2).toString() : "0"); //$NON-NLS-1$
- replacePlistValue(buffer, BUNDLE_VERSION_KEY, version.toString());
- replacePlistValue(buffer, BUNDLE_SHORT_VERSION_KEY, sv.toString());
+ infoPListEditor.setKey(InfoPListEditor.BUNDLE_VERSION_KEY, version.toString());
+ infoPListEditor.setKey(InfoPListEditor.BUNDLE_SHORT_VERSION_KEY, sv.toString());
}
if (iconName.length() > 0) {
- int iconPos = scan(buffer, 0, ICON_NAME);
- if (iconPos != -1) {
- buffer.replace(iconPos, iconPos + ICON_NAME.length(), iconName);
- } else {
- replacePlistValue(buffer, ICON_KEY, iconName);
- }
+ infoPListEditor.setKey(InfoPListEditor.ICON_KEY, iconName);
}
+ insertLauncherIni(infoPListEditor, descriptor.getExecutableName());
File target = new File(targetRoot, "Info.plist"); //$NON-NLS-1$;
try {
target.getParentFile().mkdirs();
- transferStreams(new ByteArrayInputStream(buffer.toString().getBytes()), new FileOutputStream(target));
- } catch (FileNotFoundException e) {
- System.out.println("Impossible to brand info.plist file"); //$NON-NLS-1$
- return;
- } catch (IOException e) {
- System.out.println("Impossible to brand info.plist file"); //$NON-NLS-1$
+ infoPListEditor.save(target);
+ } catch (TransformerException e) {
+ System.out.println("Impossible to save info.plist file " + target.getAbsolutePath()); //$NON-NLS-1$
return;
}
try {
@@ -569,18 +544,17 @@ public class BrandingIron {
descriptor.replace(infoPList, target);
}
- private void replacePlistValue(StringBuffer buffer, String key, String value) {
- if (value == null) {
- return;
- }
- int exePos = scan(buffer, 0, key);
- if (exePos != -1) {
- int start = scan(buffer, exePos + key.length(), STRING_START);
- int end = scan(buffer, start + STRING_START.length(), STRING_END);
- if (start > -1 && end > start) {
- buffer.replace(start + STRING_START.length(), end, value);
- }
- }
+ private void insertLauncherIni(InfoPListEditor infoPListEditor, String launcher) {
+ final String LAUNCHER_INI = "--launcher.ini"; //$NON-NLS-1$
+ List<String> args = infoPListEditor.getEclipseArguments();
+ int match = args.indexOf(LAUNCHER_INI);
+ if (match != -1) {
+ args.remove(LAUNCHER_INI);
+ args.remove(match + 1);
+ }
+ args.add(LAUNCHER_INI);
+ args.add("$APP_PACKAGE/Contents/Eclipse/" + launcher + ".ini"); //$NON-NLS-1$//$NON-NLS-2$
+ infoPListEditor.setEclipseArguments(args);
}
private int scan(StringBuffer buf, int start, String targetName) {
diff --git a/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/internal/p2/publisher/eclipse/InfoPListEditor.java b/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/internal/p2/publisher/eclipse/InfoPListEditor.java
new file mode 100644
index 000000000..72c8885f4
--- /dev/null
+++ b/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/internal/p2/publisher/eclipse/InfoPListEditor.java
@@ -0,0 +1,189 @@
+/*******************************************************************************
+ * Copyright (c) 2015 Rapicorp, Inc and others.
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License v1.0
+ * which accompanies this distribution, and is available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ *
+ * Contributors:
+ * Rapicorp, Inc. - initial API and implementation
+ *******************************************************************************/
+package org.eclipse.equinox.internal.p2.publisher.eclipse;
+
+import java.io.File;
+import java.io.IOException;
+import java.util.ArrayList;
+import java.util.List;
+import javax.xml.parsers.*;
+import javax.xml.transform.*;
+import javax.xml.transform.dom.DOMSource;
+import javax.xml.transform.stream.StreamResult;
+import javax.xml.xpath.*;
+import org.w3c.dom.*;
+import org.xml.sax.SAXException;
+
+public class InfoPListEditor {
+ public static final String MARKER_KEY = "CFBundleExecutable"; //$NON-NLS-1$
+ public static final String BUNDLE_KEY = "CFBundleName"; //$NON-NLS-1$
+ public static final String BUNDLE_ID_KEY = "CFBundleIdentifier"; //$NON-NLS-1$
+ public static final String BUNDLE_INFO_KEY = "CFBundleGetInfoString"; //$NON-NLS-1$
+ public static final String BUNDLE_VERSION_KEY = "CFBundleVersion"; //$NON-NLS-1$
+ public static final String BUNDLE_SHORT_VERSION_KEY = "CFBundleShortVersionString"; //$NON-NLS-1$
+ public static final String ICON_KEY = "CFBundleIconFile"; //$NON-NLS-1$
+
+ private final Element infoPList;
+ private XPath xPathTool;
+ private final Document document;
+
+ private InfoPListEditor(Document doc) {
+ document = doc;
+ infoPList = document.getDocumentElement();
+ }
+
+ public static InfoPListEditor loadPListEditor(File file) throws IOException {
+ if (!file.exists()) {
+ throw new IOException("No file at " + file.getAbsoluteFile()); //$NON-NLS-1$
+ }
+ DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
+ DocumentBuilder builder;
+ Exception exception;
+ try {
+ factory.setValidating(false);
+ factory.setNamespaceAware(true);
+ //These setFeature calls are necessary to disable the validation of the DTD in the info.plist
+ factory.setFeature("http://xml.org/sax/features/namespaces", false); //$NON-NLS-1$
+ factory.setFeature("http://xml.org/sax/features/validation", false); //$NON-NLS-1$
+ factory.setFeature("http://apache.org/xml/features/nonvalidating/load-dtd-grammar", false); //$NON-NLS-1$
+ factory.setFeature("http://apache.org/xml/features/nonvalidating/load-external-dtd", false); //$NON-NLS-1$
+ builder = factory.newDocumentBuilder();
+ return new InfoPListEditor(builder.parse(file));
+ } catch (ParserConfigurationException e) {
+ exception = e;
+ } catch (SAXException e) {
+ exception = e;
+ } catch (IOException e) {
+ exception = e;
+ }
+ throw new IOException("Problem parsing " + file.getAbsolutePath(), exception); //$NON-NLS-1$
+ }
+
+ public void save(File file) throws TransformerException {
+ final TransformerFactory transformerFactory = TransformerFactory.newInstance();
+ final Transformer transformer = transformerFactory.newTransformer();
+ final DOMSource toSerialize = new DOMSource(document);
+
+ final StreamResult output = new StreamResult(file);
+ transformer.setOutputProperty(OutputKeys.VERSION, "1.0"); //$NON-NLS-1$
+ transformer.setOutputProperty(OutputKeys.ENCODING, "UTF-8"); //$NON-NLS-1$
+ transformer.setOutputProperty(OutputKeys.STANDALONE, "yes"); //$NON-NLS-1$
+
+ transformer.setOutputProperty(OutputKeys.INDENT, "yes"); //$NON-NLS-1$
+ transformer.setOutputProperty("{http://xml.apache.org/xslt}indent-amount", "2"); //$NON-NLS-1$//$NON-NLS-2$
+
+ transformer.transform(toSerialize, output);
+ }
+
+ private Node getNode(Node node, String expression) throws XPathExpressionException {
+ return (Node) getXPathTool().evaluate(expression, node, XPathConstants.NODE);
+ }
+
+ public void setKey(String key, String value) {
+ if (key == null)
+ throw new IllegalArgumentException("Key can't be null"); //$NON-NLS-1$
+ if (value == null)
+ throw new IllegalArgumentException("Value can't be null"); //$NON-NLS-1$
+
+ String expression = String.format("/plist/dict/key[text() = '%s']/following-sibling::string[1]", key); //$NON-NLS-1$
+ Node node;
+ try {
+ node = getNode(infoPList, expression);
+ if (node != null)
+ node.getFirstChild().setNodeValue(value);
+ else
+ addKey(key, value);
+ } catch (XPathExpressionException e) {
+ //Can't happen since we craft the expression carefully
+ }
+ }
+
+ private void addKey(String key, String value) throws DOMException, XPathExpressionException {
+ Element keyNode = document.createElement("key"); //$NON-NLS-1$
+ Text keyName = document.createTextNode(key);
+ keyNode.appendChild(keyName);
+
+ Element stringNode = document.createElement("string"); //$NON-NLS-1$
+ Text stringValue = document.createTextNode(value);
+ stringNode.appendChild(stringValue);
+ getNode(infoPList, "/plist/dict").appendChild(keyNode); //$NON-NLS-1$
+ getNode(infoPList, "/plist/dict").appendChild(stringNode); //$NON-NLS-1$
+ }
+
+ private XPath getXPathTool() {
+ if (xPathTool == null) {
+ xPathTool = XPathFactory.newInstance().newXPath();
+ }
+ return xPathTool;
+ }
+
+ public void setEclipseArgument(String key, String value) {
+ if (key == null)
+ throw new IllegalArgumentException("Key can't be null"); //$NON-NLS-1$
+
+ String expression = "/plist/dict/key[text() = 'Eclipse']/following-sibling::array[1]/string[text() = -keyring'"; //$NON-NLS-1$
+ Node node;
+ try {
+ node = getNode(infoPList, expression);
+ if (node != null)
+ node.getFirstChild().setNodeValue(value);
+ else
+ addKey(key, value);
+ } catch (XPathExpressionException e) {
+ //Can't happen since we craft the expression carefully
+ }
+
+ }
+
+ private List<String> getValues(Object startingPoint, String expression) throws XPathExpressionException {
+ NodeList nodeList = (NodeList) getXPathTool().evaluate(expression, startingPoint, XPathConstants.NODESET);
+
+ List<String> result = new ArrayList<String>(nodeList.getLength());
+ for (int ix = 0; ix < nodeList.getLength(); ++ix) {
+ result.add(nodeList.item(ix).getNodeValue());
+ }
+ return result;
+ }
+
+ private List<Node> removeNodes(Object startingPoint, String expression) throws XPathExpressionException {
+ NodeList nodeList = (NodeList) getXPathTool().evaluate(expression, startingPoint, XPathConstants.NODESET);
+
+ List<Node> result = new ArrayList<Node>(nodeList.getLength());
+ for (int ix = 0; ix < nodeList.getLength(); ++ix) {
+ result.add(nodeList.item(ix).getParentNode().removeChild(nodeList.item(ix)));
+ }
+ return result;
+ }
+
+ public List<String> getEclipseArguments() {
+ try {
+ return getValues(infoPList, "/plist/dict/key[text() = 'Eclipse']/following-sibling::array[1]/string/text()"); //$NON-NLS-1$
+ } catch (XPathExpressionException e) {
+ //Can't happen the expression is carefully crafted
+ return null;
+ }
+ }
+
+ public void setEclipseArguments(List<String> arguments) {
+ try {
+ removeNodes(infoPList, "/plist/dict/key[text() = 'Eclipse']/following-sibling::array[1]/string"); //$NON-NLS-1$
+ for (String arg : arguments) {
+ Element stringNode = document.createElement("string"); //$NON-NLS-1$
+ Text stringName = document.createTextNode(arg);
+ stringNode.appendChild(stringName);
+ Node toAppendTo = getNode(infoPList, "/plist/dict/key[text() = 'Eclipse']/following-sibling::array[1]"); //$NON-NLS-1$
+ toAppendTo.appendChild(stringNode);
+ }
+ } catch (XPathExpressionException e) {
+ //can't happen
+ }
+ }
+} \ No newline at end of file
diff --git a/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/p2/publisher/eclipse/EquinoxExecutableAction.java b/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/p2/publisher/eclipse/EquinoxExecutableAction.java
index 423ec92e0..fdd0fbd44 100644
--- a/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/p2/publisher/eclipse/EquinoxExecutableAction.java
+++ b/bundles/org.eclipse.equinox.p2.publisher.eclipse/src/org/eclipse/equinox/p2/publisher/eclipse/EquinoxExecutableAction.java
@@ -15,7 +15,6 @@ import java.io.File;
import java.util.*;
import org.eclipse.core.runtime.*;
import org.eclipse.equinox.internal.p2.core.helpers.FileUtils;
-import org.eclipse.equinox.internal.p2.metadata.InstallableUnit;
import org.eclipse.equinox.internal.p2.publisher.eclipse.BrandingIron;
import org.eclipse.equinox.internal.p2.publisher.eclipse.ExecutablesDescriptor;
import org.eclipse.equinox.p2.metadata.*;
@@ -158,10 +157,6 @@ public class EquinoxExecutableAction extends AbstractPublisherAction {
String os = config[1];
Map<String, String> touchpointData = computeInstallActions(execDescriptor, os);
cu.addTouchpointData(MetadataFactory.createTouchpointData(touchpointData));
- if (Constants.OS_MACOSX.equals(os)) {
- result.addIU(createBundledMacIU(execDescriptor), IPublisherResult.ROOT);
- cu.setFilter(InstallableUnit.parseFilter("(& (!(macosx-bundled=*)) " + createLDAPString(configSpec) + ")")); //$NON-NLS-1$ //$NON-NLS-2$
- }
IInstallableUnit unit = MetadataFactory.createInstallableUnit(cu);
result.addIU(unit, IPublisherResult.ROOT);
@@ -182,32 +177,13 @@ public class EquinoxExecutableAction extends AbstractPublisherAction {
return cu;
}
- private IInstallableUnit createBundledMacIU(ExecutablesDescriptor execDescriptor) {
- InstallableUnitFragmentDescription cu = createSkeletonExecutableCU(execDescriptor);
- String baseId = createCUIdString(idBase, TYPE, flavor, configSpec);
- String id = baseId + "-bundled"; //$NON-NLS-1$
- cu.setId(id);
- cu.setCapabilities(new IProvidedCapability[] {PublisherHelper.createSelfCapability(baseId, version), MetadataFactory.createProvidedCapability(IInstallableUnit.NAMESPACE_IU_ID, id, version)});
- cu.setFilter(InstallableUnit.parseFilter("(&(macosx-bundled=true)" + createLDAPString(configSpec) + ")")); //$NON-NLS-1$//$NON-NLS-2$
- Map<String, String> touchpointData = computeInstallActions(execDescriptor, org.eclipse.equinox.p2.core.spi.Constants.MACOSX_BUNDLED);
- cu.addTouchpointData(MetadataFactory.createTouchpointData(touchpointData));
- return MetadataFactory.createInstallableUnit(cu);
- }
-
private Map<String, String> computeInstallActions(ExecutablesDescriptor execDescriptor, String os) {
+ if (Constants.OS_MACOSX.equals(os))
+ return computeMacInstallActions(execDescriptor);
+
Map<String, String> touchpointData = new HashMap<String, String>();
String configurationData = "unzip(source:@artifact, target:${installFolder});"; //$NON-NLS-1$
- if (org.eclipse.equinox.p2.core.spi.Constants.MACOSX_BUNDLED.equals(os)) {
- String execName = execDescriptor.getExecutableName();
- String appName = guessMacAppName(execName);
- configurationData = "unzip(source:@artifact, target:${installFolder}, path:" + appName + ".app);"; //$NON-NLS-1$ //$NON-NLS-2$
- configurationData += " chmod(targetDir:${installFolder}/Contents/MacOS/, targetFile:" + execName + ", permissions:755);"; //$NON-NLS-1$ //$NON-NLS-2$
- } else if (Constants.OS_MACOSX.equals(os)) {
- String execName = execDescriptor.getExecutableName();
- String appName = guessMacAppName(execName);
- configurationData += " chmod(targetDir:${installFolder}/" + appName + ".app/Contents/MacOS/, targetFile:" + execName + ", permissions:755);"; //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
- configurationData += " ln(targetDir:${installFolder}, linkTarget:" + appName + ".app/Contents/MacOS/" + execName + ", linkName:" + execName + ");"; //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$
- } else if (!Constants.OS_WIN32.equals(os)) {
+ if (!Constants.OS_WIN32.equals(os)) {
// We are on linux/unix. by default set all of the files to be executable.
File[] fileList = execDescriptor.getFiles();
for (int i = 0; i < fileList.length; i++)
@@ -219,6 +195,17 @@ public class EquinoxExecutableAction extends AbstractPublisherAction {
return touchpointData;
}
+ private Map<String, String> computeMacInstallActions(ExecutablesDescriptor execDescriptor) {
+ Map<String, String> touchpointData = new HashMap<String, String>();
+ String configurationData = "unzip(source:@artifact, target:${installFolder}/../);"; //$NON-NLS-1$
+ String execName = execDescriptor.getExecutableName();
+ configurationData += " chmod(targetDir:${installFolder}/../MacOS/, targetFile:" + execName + ", permissions:755);"; //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
+ touchpointData.put("install", configurationData); //$NON-NLS-1$
+ String unConfigurationData = "cleanupzip(source:@artifact, target:${installFolder}/../);"; //$NON-NLS-1$
+ touchpointData.put("uninstall", unConfigurationData); //$NON-NLS-1$
+ return touchpointData;
+ }
+
private String guessMacAppName(String execName) {
// magic moved here from BrandingIron.brandMac for bug 342550; PDE build requires appName == execName
// TODO the application name for Mac really should be a parameter of the product configuration
diff --git a/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/AutomatedTests.java b/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/AutomatedTests.java
index 292cfa666..5631a5305 100644
--- a/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/AutomatedTests.java
+++ b/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/AutomatedTests.java
@@ -23,6 +23,7 @@ import junit.framework.*;
public class AutomatedTests extends TestCase {
public static Test suite() {
TestSuite suite = new TestSuite(AutomatedTests.class.getName());
+ suite.addTest(org.eclipse.equinox.p2.tests.publisher.AllTests.suite());
suite.addTest(org.eclipse.equinox.frameworkadmin.tests.AllTests.suite());
suite.addTest(org.eclipse.equinox.p2.tests.ant.AllTests.suite());
@@ -46,7 +47,6 @@ public class AutomatedTests extends TestCase {
suite.addTest(org.eclipse.equinox.p2.tests.omniVersion.AllTests.suite());
//org.eclipse.equinox.p2.tests.perf -- Executed as part of the
suite.addTest(org.eclipse.equinox.p2.tests.planner.AllTests.suite());
- suite.addTest(org.eclipse.equinox.p2.tests.publisher.AllTests.suite());
//org.eclipse.equinox.p2.tests.publisher.actions -- The tests found in this package are invoked from the publisher package
suite.addTest(org.eclipse.equinox.p2.tests.ql.AllTests.suite());
//org.eclipse.equinox.p2.tests.reconciler.dropins -- Off sequence
diff --git a/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/publisher/actions/EquinoxExecutableActionTest.java b/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/publisher/actions/EquinoxExecutableActionTest.java
index 3f42f4fb7..37101f8df 100644
--- a/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/publisher/actions/EquinoxExecutableActionTest.java
+++ b/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/publisher/actions/EquinoxExecutableActionTest.java
@@ -40,7 +40,6 @@ public class EquinoxExecutableActionTest extends ActionTest {
private static final File WIN_EXEC = new File(TestActivator.getTestDataFolder(), "EquinoxExecutableActionTest/win/"); //$NON-NLS-1$
private final String EXECUTABLE_NAME = "LauncherName"; //$NON-NLS-1$
private Collection<IBrandingAdvice> brandingAdvice = new LinkedList<IBrandingAdvice>();
- private String macConfig = "carbon.macosx.ppc"; //$NON-NLS-1$
private String macConfigCocoa = "cocoa.macosx.x86"; //$NON-NLS-1$
private String winConfig = "win32.win32.x86"; //$NON-NLS-1$
private String linuxConfig = "linux.gtk.x86"; //$NON-NLS-1$
@@ -59,19 +58,11 @@ public class EquinoxExecutableActionTest extends ActionTest {
artifactRepository = new TestArtifactRepository(getAgent());
}
- public void testMacCarbon() throws Exception {
- File icon = File.createTempFile(EXECUTABLE_NAME, ".icns");
- FileUtils.copyStream(new FileInputStream(new File(MAC_EXEC, "eclipse.app/Contents/Resources/eclipse.icns")), true, new FileOutputStream(icon), true);
-
- expectedExecutablesContents = new String[] {EXECUTABLE_NAME + ".app", EXECUTABLE_NAME + ".app/Contents/Info.plist", EXECUTABLE_NAME + ".app/Contents/MacOS/" + EXECUTABLE_NAME, EXECUTABLE_NAME + ".app/Contents/MacOS/" + EXECUTABLE_NAME + ".ini", EXECUTABLE_NAME + ".app/Contents/Resources/" + icon.getName()};
- testExecutableAction("mac", "macosx", macConfig, MAC_EXEC, icon); //$NON-NLS-1$//$NON-NLS-2$
- }
-
public void testMacCocoa() throws Exception {
File icon = File.createTempFile(EXECUTABLE_NAME, ".icns");
FileUtils.copyStream(new FileInputStream(new File(MAC_EXEC, "eclipse.app/Contents/Resources/eclipse.icns")), true, new FileOutputStream(icon), true);
- expectedExecutablesContents = new String[] {EXECUTABLE_NAME + ".app", EXECUTABLE_NAME + ".app/Contents/Info.plist", EXECUTABLE_NAME + ".app/Contents/MacOS/" + EXECUTABLE_NAME, EXECUTABLE_NAME + ".app/Contents/MacOS/" + EXECUTABLE_NAME + ".ini", EXECUTABLE_NAME + ".app/Contents/Resources/" + icon.getName()};
+ expectedExecutablesContents = new String[] {"Info.plist", "MacOS/" + EXECUTABLE_NAME, "MacOS/" + EXECUTABLE_NAME + ".ini", "Resources/" + icon.getName()};
testExecutableAction("macCocoa", "macosx", macConfigCocoa, MAC_EXEC, icon); //$NON-NLS-1$//$NON-NLS-2$
}
@@ -104,13 +95,10 @@ public class EquinoxExecutableActionTest extends ActionTest {
private void verifyResults(String idBase, String confSpec) {
ArrayList iuList = new ArrayList(publisherResult.getIUs(null, IPublisherResult.ROOT));
- verifyExecIU(iuList, idBase, confSpec);
verifyEclipseIU(iuList, idBase, confSpec);
verifyCU(iuList, idBase, confSpec);
- if (confSpec.contains("macosx"))
- assertTrue(iuList.size() == 4);
- else
- assertTrue(iuList.size() == 3);
+ verifyExecIU(iuList, idBase, confSpec);
+ assertTrue(iuList.size() == 3);
}
private void verifyCU(ArrayList iuList, String idBase, String confSpec) {
@@ -205,8 +193,8 @@ public class EquinoxExecutableActionTest extends ActionTest {
for (String path : expectedExecutablesContents) {
assertNotNull("executable zip missing " + path, zip.getEntry(path));
}
-
- checkInfoPlist(zip);
+ if (key.getId().contains("macosx"))
+ checkInfoPlist(zip);
} finally {
zip.close();
}
@@ -222,13 +210,12 @@ public class EquinoxExecutableActionTest extends ActionTest {
boolean found = false;
for (Enumeration<? extends ZipEntry> iter = zip.entries(); !found && iter.hasMoreElements();) {
candidate = iter.nextElement();
- found = candidate.getName().endsWith(".app/Contents/Info.plist");
- }
- if (!found) {
- return;
+ found = candidate.getName().endsWith("Info.plist");
}
+ assertTrue(found);
try {
String contents = readContentsAndClose(zip.getInputStream(candidate));
+ System.out.println("INFO.PLIST.CONTENT IS----> " + contents);
assertEquals(id, getPlistStringValue(contents, "CFBundleIdentifier"));
assertEquals(EXECUTABLE_NAME, getPlistStringValue(contents, "CFBundleExecutable"));
assertEquals(EXECUTABLE_NAME, getPlistStringValue(contents, "CFBundleName"));
diff --git a/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/LazyManipulator.java b/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/LazyManipulator.java
index 4e8857043..6a6bad74d 100644
--- a/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/LazyManipulator.java
+++ b/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/LazyManipulator.java
@@ -48,7 +48,7 @@ public class LazyManipulator implements Manipulator {
launcherData.setFwConfigLocation(Util.getConfigurationFolder(profile));
launcherData.setLauncher(Util.getLauncherPath(profile));
launcherData.setLauncherConfigLocation(Util.getLauncherConfigLocation(profile));
- launcherData.setOS(Util.isMacOSBundled(profile) ? org.eclipse.equinox.p2.core.spi.Constants.MACOSX_BUNDLED : Util.getOSFromProfile(profile));
+ launcherData.setOS(Util.getOSFromProfile(profile));
launcherData.setHome(Util.getInstallFolder(profile));
try {
diff --git a/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/PlatformConfigurationWrapper.java b/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/PlatformConfigurationWrapper.java
index 93578df48..1ad9627da 100644
--- a/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/PlatformConfigurationWrapper.java
+++ b/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/PlatformConfigurationWrapper.java
@@ -16,7 +16,6 @@ import java.net.*;
import java.util.List;
import org.eclipse.core.runtime.*;
import org.eclipse.equinox.frameworkadmin.BundleInfo;
-import org.eclipse.equinox.internal.p2.core.helpers.LogHelper;
import org.eclipse.equinox.internal.p2.update.*;
import org.eclipse.equinox.internal.provisional.frameworkadmin.LauncherData;
import org.eclipse.equinox.internal.provisional.frameworkadmin.Manipulator;
@@ -79,9 +78,9 @@ public class PlatformConfigurationWrapper {
//the equinox launcher will look 3 levels up on the mac when going from executable to launcher.jar
//see org.eclipse.equinox.executable/library/eclipse.c : findStartupJar();
IPath launcherPath = new Path(launcherFile.getAbsolutePath());
- if (launcherPath.segmentCount() > 4) {
+ if (launcherPath.segmentCount() > 2) {
//removing "Eclipse.app/Contents/MacOS/eclipse"
- launcherPath = launcherPath.removeLastSegments(4);
+ launcherPath = launcherPath.removeLastSegments(2);
try {
return launcherPath.toFile().toURI().toURL();
} catch (MalformedURLException e) {
@@ -89,12 +88,6 @@ public class PlatformConfigurationWrapper {
}
}
}
- if (org.eclipse.equinox.p2.core.spi.Constants.MACOSX_BUNDLED.equals(launcherData.getOS())) {
- //We are in a situation where the launcher path is not set in the launcher file.
- //The laid out installation is not going to work because the macos launcher looks 3 levels up for the launcher jars (see previous if).
- //Log this fact and move on. Though we should probably fail.
- LogHelper.log(Util.createError(Messages.invalid_macox_bundled_setup));
- }
try {
return launcherFile.getParentFile().toURI().toURL();
} catch (MalformedURLException e) {
diff --git a/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/Util.java b/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/Util.java
index 0ff83cc47..91812a358 100644
--- a/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/Util.java
+++ b/bundles/org.eclipse.equinox.p2.touchpoint.eclipse/src/org/eclipse/equinox/internal/p2/touchpoint/eclipse/Util.java
@@ -16,7 +16,6 @@
package org.eclipse.equinox.internal.p2.touchpoint.eclipse;
import java.io.File;
-import java.io.IOException;
import java.net.*;
import java.util.*;
import org.eclipse.core.runtime.*;
@@ -24,7 +23,6 @@ import org.eclipse.equinox.frameworkadmin.BundleInfo;
import org.eclipse.equinox.internal.p2.core.helpers.*;
import org.eclipse.equinox.internal.p2.metadata.IRequiredCapability;
import org.eclipse.equinox.p2.core.*;
-import org.eclipse.equinox.p2.core.spi.Constants;
import org.eclipse.equinox.p2.engine.IProfile;
import org.eclipse.equinox.p2.metadata.*;
import org.eclipse.equinox.p2.repository.IRepository;
@@ -272,23 +270,14 @@ public class Util {
String name = profile.getProperty(EclipseTouchpoint.PROFILE_PROP_LAUNCHER_NAME);
if (name == null || name.length() == 0)
name = "eclipse"; //$NON-NLS-1$
- String launcherName = getLauncherName(name, (isMacOSBundled(profile) ? Constants.MACOSX_BUNDLED : getOSFromProfile(profile)), getInstallFolder(profile));
- return launcherName == null ? null : new File(getInstallFolder(profile), launcherName);
- }
-
- public static boolean isMacOSBundled(IProfile profile) {
- String environments = profile.getProperty(IProfile.PROP_ENVIRONMENTS);
- if (environments == null)
- return false;
- if (environments.indexOf(Constants.MACOSX_BUNDLED + "=true") != -1) //$NON-NLS-1$
- return true;
- return false;
+ String relativePath = getLauncherRelativePath(name, getOSFromProfile(profile), getInstallFolder(profile));
+ return relativePath == null ? null : new File(getInstallFolder(profile), relativePath);
}
/**
* Returns the name of the Eclipse application launcher.
*/
- private static String getLauncherName(String name, String os, File installFolder) {
+ private static String getLauncherRelativePath(String name, String os, File installFolder) {
if (os == null) {
EnvironmentInfo info = ServiceHelper.getService(Activator.getContext(), EnvironmentInfo.class);
if (info == null)
@@ -302,38 +291,9 @@ public class Util {
return name;
return name + ".exe"; //$NON-NLS-1$
}
- if (os.equals(Constants.MACOSX_BUNDLED)) {
- return "/Contents/MacOS/" + name; //$NON-NLS-1$
- }
if (os.equals(org.eclipse.osgi.service.environment.Constants.OS_MACOSX)) {
- IPath path = new Path(name);
- if (path.segment(0).endsWith(".app")) //$NON-NLS-1$
- return name;
-
- String appName = null;
- if (installFolder != null) {
- File appFolder = new File(installFolder, name + ".app"); //$NON-NLS-1$
- if (appFolder.exists()) {
- try {
- appName = appFolder.getCanonicalFile().getName();
- } catch (IOException e) {
- appName = appFolder.getName();
- }
- }
- }
-
- StringBuffer buffer = new StringBuffer();
- if (appName != null) {
- buffer.append(appName);
- } else {
- buffer.append(name.substring(0, 1).toUpperCase());
- buffer.append(name.substring(1));
- buffer.append(".app"); //$NON-NLS-1$
- }
- buffer.append("/Contents/MacOS/"); //$NON-NLS-1$
- buffer.append(name);
- return buffer.toString();
+ return "../MacOS/" + name;
}
return name;
}

Back to the top