Inline o.e.jdt.launching.macosx into jdt.launching#855
Conversation
|
Before this is submitted, the plugin should also be removed from the JDT feature: I can provide a PR for that as well. |
|
@iloveeclipse what do you think about this effort respectively can you tell who else would be interested? |
99fedc6 to
fd29413
Compare
|
@HannesWell : technically the change should work. I just have some weird feeling hard coding previously separately contributed extension & bundle, but I also don't see how it would be possible today to avoid that this extension is being used on Mac, so it is "implicitly" hard coded to be always contributed, except one manipulates extension registry via code. Looking at the bundle history, I see it was originally using native scripts and binaries to launch something on Mac. That was probably the reason why it was provided as separated bundle. See for example c8ace81 or 9b792ae. I'm not sure if we would need to have some special native bits again on Mac in the future. |
fd29413 to
1226433
Compare
|
This pull request changes some projects for the first time in this development cycle. An additional commit containing all the necessary changes was pushed to the top of this PR's branch. To obtain these changes (for example if you want to push more changes) either fetch from your fork or apply the git patch. Git patchFurther information are available in Common Build Issues - Missing version increments. |
8ccb7e5 to
3f80e6b
Compare
Agree. But I don't have a better idea to restore the exact state as before. I'm not aware that Extensions can be contributed conditionally in a
I cannot say that, but I hope not. @iloveeclipse if you agree with this and don't have requests for code changes, I can prepare the changes necessary in other repositories as outlined in |
dab87da to
20eaece
Compare
|
@iloveeclipse how to proceed here? |
20eaece to
cec9d0e
Compare
cec9d0e to
8b908e9
Compare
|
@iloveeclipse any update on this? |
8b908e9 to
3f00245
Compare
|
@iloveeclipse can we proceed with this? I think now would be a good time. |
|
@SougandhS : I believe you are using Mac? Can you test this patch? |
ef2ff8b to
71ddbad
Compare
|
Hi @HannesWell
EDIT: |
This simplifies the assembly of Eclipse runtimes on MacOS. Fixes eclipse-jdt#849
0e31378 to
98514b7
Compare
Thank you for testing this! I have now added a guard to prevent adding the macOS VM type twice. I think that's still useful in case some installations don't have the fragment properly removed. |
|
I tested again with and without eclipse-jdt/eclipse.jdt#160 and it is now not showing twice in both cases. |
Thanks! OK, let us merge this. |
|
Great! Thanks to all of you. |
It's been inlined into o.e.jdt.launching (see eclipse-jdt/eclipse.jdt.debug#855) Fixes the build Signed-off-by: David Thompson <davthomp@redhat.com>

What it does
This simplifies the assembly of Eclipse runtimes on MacOS.
Fixes #849
Author checklist