Skip to content

Commit

Permalink
Remove short-named equinox.launcher.cocoa.macosx fragment and its usage
Browse files Browse the repository at this point in the history
The full-named org.eclipse.equinox.launcher.cocoa.macosx.x86_64 fragment
exists for a long time already and users of the old short-named variant
should have had enough time to migrate.

And remove consideration of old development locations of the launcher
fragments.
  • Loading branch information
HannesWell committed May 9, 2024
1 parent 99a8578 commit 740aba9
Show file tree
Hide file tree
Showing 8 changed files with 2 additions and 122 deletions.
22 changes: 0 additions & 22 deletions bundles/org.eclipse.equinox.launcher.cocoa.macosx/.project

This file was deleted.

This file was deleted.

36 changes: 0 additions & 36 deletions bundles/org.eclipse.equinox.launcher.cocoa.macosx/about.html

This file was deleted.

25 changes: 0 additions & 25 deletions bundles/org.eclipse.equinox.launcher.cocoa.macosx/build.properties

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,6 @@
import java.util.Locale;
import java.util.Properties;
import java.util.Set;
import java.util.StringJoiner;
import java.util.StringTokenizer;
import java.util.zip.ZipEntry;
import java.util.zip.ZipFile;
Expand Down Expand Up @@ -447,15 +446,6 @@ private String getArch() {
return name;
}

private String getFragmentString(String fragmentOS, String fragmentWS, String fragmentArch) {
StringJoiner buffer = new StringJoiner("."); //$NON-NLS-1$
buffer.add(PLUGIN_ID).add(fragmentWS).add(fragmentOS);
if (!(fragmentOS.equals(Constants.OS_MACOSX) && !Constants.ARCH_X86_64.equals(fragmentArch))) {
buffer.add(fragmentArch);
}
return buffer.toString();
}

/**
* Sets up the JNI bridge to native calls
*/
Expand All @@ -475,11 +465,8 @@ private void setupJNI(URL[] defaultPath) {
}
if (libPath == null) {
//find our fragment name
String fragmentOS = getOS();
String fragmentWS = getWS();
String fragmentArch = getArch();

libPath = getLibraryPath(getFragmentString(fragmentOS, fragmentWS, fragmentArch), defaultPath);
String fragmentName = String.join(".", PLUGIN_ID, getWS(), getOS(), getArch()); //$NON-NLS-1$
libPath = getLibraryPath(fragmentName, defaultPath);
}
library = libPath;
if (library != null)
Expand Down Expand Up @@ -509,10 +496,6 @@ private String getLibraryPath(String fragmentName, URL[] defaultPath) {
for (int i = urls.length - 1; i >= 0 && libPath == null; i--) {
File entryFile = new File(urls[i].getFile());
String dir = entryFile.getParent();
if (inDevelopmentMode) {
String devDir = dir + "/" + PLUGIN_ID + "/fragments"; //$NON-NLS-1$ //$NON-NLS-2$
fragment = searchFor(fragmentName, devDir);
}
if (fragment == null)
fragment = searchFor(fragmentName, dir);
if (fragment != null)
Expand Down
7 changes: 0 additions & 7 deletions features/org.eclipse.equinox.executable.feature/feature.xml
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,6 @@
arch="aarch64"
version="0.0.0"/>

<plugin
id="org.eclipse.equinox.launcher.cocoa.macosx"
os="macosx"
ws="cocoa"
arch="aarch64"
version="0.0.0"/>

<plugin
id="org.eclipse.equinox.launcher.gtk.linux.ppc64le"
os="linux"
Expand Down
1 change: 0 additions & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,6 @@
<module>bundles/org.eclipse.osgi/supplement</module>

<module>bundles/org.eclipse.equinox.launcher</module>
<module>bundles/org.eclipse.equinox.launcher.cocoa.macosx</module>
<module>bundles/org.eclipse.equinox.launcher.cocoa.macosx.x86_64</module>
<module>bundles/org.eclipse.equinox.launcher.cocoa.macosx.aarch64</module>
<module>bundles/org.eclipse.equinox.launcher.gtk.linux.aarch64</module>
Expand Down

0 comments on commit 740aba9

Please sign in to comment.