Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

provide EMBEDDED implementation of ITW_LIBS #196

Closed
judovana opened this issue Apr 23, 2019 · 17 comments
Closed

provide EMBEDDED implementation of ITW_LIBS #196

judovana opened this issue Apr 23, 2019 · 17 comments
Assignees
Labels
enhancement New feature or request
Projects
Milestone

Comments

@judovana
Copy link
Contributor

It will behave same as BUNDLED, but will slightly affect also JDK search path.
For rust launchers, for both 1.8 and head

@judovana judovana self-assigned this Apr 23, 2019
@judovana judovana added the enhancement New feature or request label Apr 23, 2019
@judovana judovana added this to To do in icedtea-web via automation Apr 23, 2019
@karianna karianna added this to the 1.8.x milestone Apr 23, 2019
@karianna
Copy link
Member

Set to 1.8.x milestone but needs to go on HEAD as well.

@karianna
Copy link
Member

This is not a blocker for 1.8.1

@judovana
Copy link
Contributor Author

@akashche @karianna Where do you expect javaws will be placed in jre?

@judovana
Copy link
Contributor Author

RH win installer have javaws in spearate folder JAVA_HOME/webstart...

@karianna
Copy link
Member

@akashche @karianna Where do you expect javaws will be placed in jre?

I'm really not sure.

@judovana
Copy link
Contributor Author

judovana commented May 2, 2019

Ok. We have to settle on this (feel free to bring it to wider adopOJDK - around jdk image distribution - audience)

I vote for {jre,jdk}/webstart and I will be searching for java via ../bin/java. Where symlink from bin/javaws to ../webstart. That will keep us aligned with RH windows images, which are already used and proved themselves. Thoughts?

@karianna
Copy link
Member

karianna commented May 2, 2019

Ok. We have to settle on this (feel free to bring it to wider adopOJDK - around jdk image distribution - audience)

I vote for {jre,jdk}/webstart and I will be searching for java via ../bin/java. Where symlink from bin/javaws to ../webstart. That will keep us aligned with RH windows images, which are already used and proved themselves. Thoughts?

I think it working as it has under RH ITW is what users would expect.

@judovana
Copy link
Contributor Author

judovana commented May 4, 2019

Ok. Second question - how the adopt's jdk will handle the inclusion? It will download some ITW release, and unpack it into its image after the jdk build is done, or it will be building itw and including some snapshot? Will it be building it by itself or by boot jdk?

@douph1
Copy link
Contributor

douph1 commented May 6, 2019

I can say how it works actually in beta/pre release stage :

It will download some ITW release, and unpack it into its image after the jdk build is done

no

it will be building itw and including some snapshot

Actually we checkout a specified branch at HEAD.
Test job are configured to checkout branch "1.8".
We build it and create a zip wich is then integreted into AdoptOpenJdk installer ( unzip into JRE/JKD folder )
Then the AdoptOpenJdk installer is created (not publicy at this time)

Will it be building it by itself or by boot jdk?

It seems we use bootk jdk as I see --with-jdk-home= in build job

@douph1
Copy link
Contributor

douph1 commented May 6, 2019

and we are just waiting for this last issue #184 (tagged as 1.8 ) to be fixed/closed and your GO to build new installer with ITW bundled

@judovana
Copy link
Contributor Author

judovana commented May 6, 2019

I see. Thanx. Then I will go with ../bin/java search and you will have to go with (will be new) EMBEDDED configure switch, I'm currently sick, but will do a pathc once I'm able to focus to programming.

@douph1
Copy link
Contributor

douph1 commented May 7, 2019

Acutally after unpacking itw zip file into installer workspace there seems to be copy task.
Bin from zip files are copied into "bin" folder from JRE/JDK on final install ( as Oracle )

See what is installed in the final computer users:

javaws.exe
itweb-settings.exe
itw-modularjdk.args
policyeditor.exe
are put into C:\Program Files\AdoptOpenJDK\jre-8.0.202.08-hotspot\bin\

So if you search for ../bin/java from javaws.exe it will work for actual build
jre-8.0.202.08-hotspot\bin..\bin\java.exe
and
new one if we change to "webstart" folder
jre-8.0.202.08-hotspot\webstart..\bin\java.exe

See other files locations:
C:\Program Files\AdoptOpenJDK\jre-8.0.202.08-hotspot\icedtea-web-docs
C:\Program Files\AdoptOpenJDK\jre-8.0.202.08-hotspot\share\pixmaps
C:\Program Files\AdoptOpenJDK\jre-8.0.202.08-hotspot\share\icedtea-web\javaws.jar
...
C:\Program Files\AdoptOpenJDK\jre-8.0.202.08-hotspot\share\doc\icedtea-web\netx
..

C:\Program Files\AdoptOpenJDK\jre-8.0.202.08-hotspot\win-deps-runtime\js.jar
C:\Program Files\AdoptOpenJDK\jre-8.0.202.08-hotspot\win-deps-runtime\mslinks.jar
C:\Program Files\AdoptOpenJDK\jre-8.0.202.08-hotspot\win-deps-runtime\tagsoup.jar
C:\Program Files\AdoptOpenJDK\jre-8.0.202.08-hotspot\netx.src.zip
C:\Program Files\AdoptOpenJDK\jre-8.0.202.08-hotspot\plugin.src.zip

@judovana
Copy link
Contributor Author

Hm. So you really wish to put javaws along side with java.exe? hmhmh.
In 2.0 the libraries are no longer share nor itw-modularjdk.args are in bin (you cen set the path to them in configure already in 1.8, and I would advice you to do so, it do not belong to bin), they are all in "libs" (it cans till be changed)

Maybe we can go with 1.8 in this way, but for 2.0 to move itw to separate directory?
In linux, both cna coexist, as you can simply simlink the javaws from bin to its destined folder. On windows..:(

@judovana
Copy link
Contributor Author

please see (and review and test):
1.8: #252
2.0: #253

@judovana
Copy link
Contributor Author

It will work in both jre/bin/javaws and jre/itw-image/bin/javaws

@judovana
Copy link
Contributor Author

This is for native launchers only. I have no intentions to put this behaviour to shell launchers. Any opinion on this intention?

@judovana
Copy link
Contributor Author

pushed.

icedtea-web automation moved this from To do to Done May 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
icedtea-web
  
Done
Development

No branches or pull requests

3 participants