-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Enable JDK-EA builds again #13036
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
Closed
Closed
Enable JDK-EA builds again #13036
Changes from 16 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
28be33f
Fix JabRef-post-image.wsf to make it agnostic to the location of the …
alexeysemenyukoracle 407af96
move to jabgui
Siedlerchr 65eb404
fix errors and convert to build dir from layout
Siedlerchr aac904f
Use mise and JDK25-ea
koppor 222c9ae
Fix path
koppor 1a25775
Check if mise sets env vars
koppor f2b7e15
Refine Java installation
koppor 37590f8
Fix path on Windows
koppor 33fc6c3
Use jdpackage from JDK25
koppor bd76b44
Use mise everywhere
koppor 80baef8
Updating the gradle wrapper does not need any JDK
koppor b943996
Update byte-buddy
koppor 09c04a1
More byte buddy
koppor 8f804ea
Try to set JAVA_HOME
koppor 9289d93
Remove main.wxs
koppor dc293db
Try quick hack for .kts
koppor f5859d7
Enable debugging OpenRewrite
koppor 35a5a58
Fix PATH and JAVA_HOME
koppor 3bf0aae
Merge remote-tracking branch 'origin/main' into fix-jpackage-post-ima…
koppor e9e7fbb
Fix brace
koppor 5d0a2b5
Fix typo
koppor 2f4866a
Fix path
koppor bf4328e
Add some debug
koppor 71f7f3a
More debug - and try to fix
koppor 462b953
Merge remote-tracking branch 'origin/main' into fix-jpackage-post-ima…
koppor 4b19e64
Discard changes to .github/workflows/tests.yml
koppor d371a46
Discard changes to .github/workflows/tests-fetchers.yml
koppor 793ed12
Initial -jdk-ea version
koppor 3173d43
Discard changes to .github/workflows/deployment.yml
koppor ae52cce
Discard changes to mise.toml
koppor c2f4f5f
Merge branch 'fix-jpackage-post-image-update-to-JDK25' of github.com:…
koppor a9b85bb
Downgrad regular toolchain
koppor cefcece
Try to update to JDK-ea using mise
koppor File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,40 @@ | ||
<?xml version="1.0" ?> | ||
<package> | ||
<job id="postImage"> | ||
<script language="jscript"> | ||
<![CDATA[ | ||
var fileSystem = new ActiveXObject("Scripting.FileSystemObject"); | ||
|
||
// The script is invoked in the "image" folder and not in "config" for some reason | ||
// DEBUG Output | ||
// var shell = new ActiveXObject("WScript.Shell"); | ||
// shell.Popup(fileSystem.GetFolder(".")); | ||
var jabRefRoot = fileSystem.GetFolder('@jabRefRoot@'); | ||
// jpackage will run a copy of the script from the "config" directory. | ||
var installerConfig = fileSystem.GetFile(WScript.ScriptFullName).ParentFolder; | ||
|
||
WScript.Echo("jabRefRoot: " + jabRefRoot) | ||
WScript.Echo("installerConfig: " + installerConfig) | ||
|
||
// Copy additional installer resources | ||
fileSystem.CopyFile(jabRefRoot + "/buildres/windows/JabRefTopBanner.bmp", installerConfig + "/JabRefTopBanner.bmp"); | ||
|
||
var wxsFilePath = installerConfig + "/main.wxs"; | ||
wxsFile = fileSystem.OpenTextFile(wxsFilePath, 1); | ||
var contents = wxsFile.ReadAll(); | ||
wxsFile.Close(); | ||
|
||
// Add registry values for JabRef Browser Extension | ||
contents = contents.replace("</Product>", "<DirectoryRef Id=\"TARGETDIR\"><Component Id=\"RegistryJabRefBrowserEntries\" Guid=\"b6bc55ad-905c-4258-89b1-8b37abbe559c\" Win64=\"yes\"><RegistryKey Root=\"HKMU\" Key=\"SOFTWARE\\Mozilla\\NativeMessagingHosts\\org.jabref.jabref\" Action=\"createAndRemoveOnUninstall\" ForceCreateOnInstall=\"yes\"><RegistryValue Type=\"string\" Value=\"[INSTALLDIR]jabref-firefox.json\"/></RegistryKey><RegistryKey Root=\"HKMU\" Key=\"Software\\Google\\Chrome\\NativeMessagingHosts\\org.jabref.jabref\" Action=\"createAndRemoveOnUninstall\" ForceCreateOnInstall=\"yes\"><RegistryValue Type=\"string\" Value=\"[INSTALLDIR]jabref-chrome.json\"/></RegistryKey><RegistryKey Root=\"HKMU\" Key=\"Software\\Microsoft\\Edge\\NativeMessagingHosts\\org.jabref.jabref\" Action=\"createAndRemoveOnUninstall\" ForceCreateOnInstall=\"yes\"><RegistryValue Type=\"string\" Value=\"[INSTALLDIR]jabref-chrome.json\"/></RegistryKey><RegistryKey Root=\"HKMU\" Key=\"Software\\Google\\Chrome\\Extensions\\bifehkofibaamoeaopjglfkddgkijdlh\" Action=\"createAndRemoveOnUninstall\" ForceCreateOnInstall=\"yes\"><RegistryValue Type=\"string\" Name=\"update_url\" Value=\"https://clients2.google.com/service/update2/crx\" /></RegistryKey><RegistryKey Root=\"HKMU\" Key=\"Software\\Wow6432Node\\Google\\Chrome\\Extensions\\bifehkofibaamoeaopjglfkddgkijdlh\" Action=\"createAndRemoveOnUninstall\" ForceCreateOnInstall=\"yes\"><RegistryValue Type=\"string\" Name=\"update_url\" Value=\"https://clients2.google.com/service/update2/crx\" /></RegistryKey><RegistryKey Root=\"HKMU\" Key=\"Software\\Microsoft\\Edge\\Extensions\\pgkajmkfgbehiomipedjhoddkejohfna\" Action=\"createAndRemoveOnUninstall\" ForceCreateOnInstall=\"yes\"><RegistryValue Type=\"string\" Name=\"update_url\" Value=\"https://edge.microsoft.com/extensionwebstorebase/v1/crx\" /></RegistryKey><RegistryKey Root=\"HKMU\" Key=\"Software\\Wow6432Node\\Microsoft\\Edge\\Extensions\\pgkajmkfgbehiomipedjhoddkejohfna\" Action=\"createAndRemoveOnUninstall\" ForceCreateOnInstall=\"yes\"><RegistryValue Type=\"string\" Name=\"update_url\" Value=\"https://edge.microsoft.com/extensionwebstorebase/v1/crx\" /></RegistryKey></Component></DirectoryRef><Feature Id=\"BrowserExtension\" Level=\"1\"><ComponentRef Id=\"RegistryJabRefBrowserEntries\" /></Feature></Product>"); | ||
|
||
// Specify banner | ||
contents = contents.replace("</Product>", "<WixVariable Id=\"WixUIBannerBmp\" Value=\"JabRefTopBanner.bmp\" /></Product>"); | ||
//contents = contents.replace("</Product>", "<WixVariable Id=\"WixUIDialogBmp\" Value=\"\" /></Product>"); | ||
|
||
wxsFile = fileSystem.OpenTextFile(wxsFilePath, 2, true); | ||
wxsFile.Write(contents); | ||
wxsFile.Close(); | ||
]]> | ||
</script> | ||
</job> | ||
</package> |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems like the env.JAVA_HOME var is not set correctly
jlink failed with: Error: jlink version 25.0 does not match target java.base version 21.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jdx/mise#4973