Enhance p2 director with "addJREIU" option

[raghucssit]

Support "addJREIU" option for DirectorApplication. Similar to UpdateSitePublisherApplication.
Background for this enhancement is from #450

[iloveeclipse]

@merks: I think we should document this in N&N? Or is there a better place for p2 director options documentation?

[laeubi]

@merks: I think we should document this in N&N? Or is there a better place for p2 director options documentation?

There is https://help.eclipse.org/latest/topic/org.eclipse.platform.doc.isv/guide/p2_director.html

[iloveeclipse]

@merks: I think we should document this in N&N? Or is there a better place for p2 director options documentation?

There is https://help.eclipse.org/latest/topic/org.eclipse.platform.doc.isv/guide/p2_director.html

OK, so we should update that.

[merks]

@iloveeclipse

Note this part about the argument information being generated:

https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/9a3b53da0c6845c7c9a8a5bb78423503d68e12f2/eclipse.platform.common/bundles/org.eclipse.platform.doc.isv/guide/p2_director.html#L148

That code is commented out here:

[raghucssit]

@merks: I think we should document this in N&N? Or is there a better place for p2 director options documentation?

There is https://help.eclipse.org/latest/topic/org.eclipse.platform.doc.isv/guide/p2_director.html

OK, so we should update that.

Documentation is available at https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/master/eclipse.platform.common/bundles/org.eclipse.platform.doc.isv/guide/p2_director.html
I will add this option there.

[iloveeclipse]

With eclipse-platform/eclipse.platform.releng.aggregator#1915 merged this can be closed.
Thanks Raghu for the fix & Ed for review and very helpful hints!

[iloveeclipse]

Doesn't work as expected. Instead of adding Java 21 it adds Java 17. Build is running on java 21.

[laeubi]

Doesn't work as expected. Instead of adding Java 21 it adds Java 17. Build is running on java 21.

It doesn't really matter what the build is running but the director process, just in case... also keep in mind that this will not add anything it will only make it available during the director calls (what might or might not add things to the profile).

[iloveeclipse]

Everythingin the build / install process runs on Java 21

[iloveeclipse]

I guess what happens is that a.jre.javase for Java 17 is already present in the SDK and so the new option simply silently does nothing.

[laeubi]

I'm not sure what is your expectation here or where you seeing what, but the generated IU will just be transient unless you explicitly install it, so if you put a breakpoint there you should see that it is added as an extra unit but ti later will not show up anywhere.

[iloveeclipse]

Sure I'm also trying to install it. Is it possible, doing both actions in same director call depends on processing order of arguments? So if install is called first, and addjre is processed after that, the Java 17 is installed before Java21 is added? I'm currently on vacation, will follow up in a week.

[laeubi]

You usually install things on top of others, so some units are there (inital + extra units), some are requested to be installed (root units) and the units you provide (updatesites).

The all root IUs are selected + their dependencies (if not already part of initial + extra) and added to the profile.