Skip to content

[bugfix] Improve content installer. Addresses #6517#6519

Merged
duncdrum merged 2 commits into
eXist-db:developfrom
dizzzz:improve_installer
Jul 6, 2026
Merged

[bugfix] Improve content installer. Addresses #6517#6519
duncdrum merged 2 commits into
eXist-db:developfrom
dizzzz:improve_installer

Conversation

@dizzzz

@dizzzz dizzzz commented Jun 23, 2026

Copy link
Copy Markdown
Member

Improve content installer. Addresses #6517

  • added missing xar files -0 made openAPI xars mandatory install. exist-openapi could be optional but mandatory when eXide is selected?
  • updated xar descriptions
  • order of xar names
  • update final 'startup' text --> not sure about "clickable options" text at finish step
  • @reinhapa what about these 2 errors in picture below?
image

apps panel:

image

@dizzzz dizzzz requested review from a team, duncdrum, line-o and reinhapa June 23, 2026 20:50
@reinhapa

Copy link
Copy Markdown
Member
  • @reinhapa what about these 2 errors in picture below?

There seems no special packsLang.xml available, that defines the basic translations for the Installer Packs... (it is actually missing in exist-installer/src/main/izpack

@duncdrum duncdrum left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the test failure

Error: core] [ERROR] Errors: 
Error: core] [ERROR]   MoveResourceTest.testMove:116 » Execution java.io.IOException: Received RST_STREAM: Stream cancelled
[exist-core] [INFO] 
Error: core] [ERROR] Tests run: 7010, Failures: 0, Errors: 1, Skipped: 102

is unrelated. see #6532

@duncdrum

Copy link
Copy Markdown
Contributor

@reinhapa so I take it we should add a packslang.xml for en right?

@reinhapa

Copy link
Copy Markdown
Member

@reinhapa so I take it we should add a packslang.xml for en right?

Yes, that will eliminate the warning...

@line-o line-o left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@line-o

line-o commented Jun 29, 2026

Copy link
Copy Markdown
Member

@dizzzz would you like to add the packslang.xml to this PR or favor to do it in a follow up ?

@dizzzz

dizzzz commented Jun 30, 2026

Copy link
Copy Markdown
Member Author

I can do it after this week. If it is too late, someone else need to pick it up.

And: what should end up in the file :)

@dizzzz dizzzz force-pushed the improve_installer branch from 42f7841 to e0deb56 Compare July 6, 2026 20:20
@dizzzz

dizzzz commented Jul 6, 2026

Copy link
Copy Markdown
Member Author

all is OK now:

 java -jar target/exist-installer-7.0.0-SNAPSHOT.jar
Using laf com.jgoodies.looks.plastic.Plastic3DLookAndFeel
====================
Installation started
Framework: 5.2.6-ef357 (IzPack)
Platform: mac_osx,version=26.5.2,arch=unknown,symbolicName=null,javaVersion=21.0.9
Installation finished
[ Writing the uninstaller data ... ]

@reinhapa are you ok with this?

@duncdrum duncdrum merged commit 3e22590 into eXist-db:develop Jul 6, 2026
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants