Skip to content

Conversation

@dmatej
Copy link
Contributor

@dmatej dmatej commented Jan 13, 2026

  • Moved classes and renamed packages to be able to comply JPMS rules
  • The main package is org.glassfish.shoal now.
  • Using Grizzly 5 for build now
  • Fixed XSD links in poms
  • Added module-info files
  • Fixed Felix plugin configuration, so we don't need to write manifest by hand.

Tested with eclipse-ee4j/glassfish#25859 (with Grizzly 5.0.0-SNAPSHOT)

dmatej added 4 commits January 8, 2026 11:24
Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
…grade

- It had to be done at once as all these things are related
- org.shoal.adapter.store package was spread over two jar files,
  had to be renamed
- com.sun.enterprise caused some reflection issues then, so I decided to align
  all shoal modules under org.glassfish.shoal
- impl packages are not automatically added to OSGI bundle, but were used from
  the outside, also Felix complained, so I renamed them too.
- Added JPMS module-info files, Felix works with them quite well

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
…er versions

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
@dmatej dmatej marked this pull request as ready for review January 13, 2026 20:58
@dmatej dmatej requested a review from a team January 13, 2026 21:00
@dmatej dmatej merged commit 04dd253 into eclipse-ee4j:master Jan 13, 2026
2 checks passed
@dmatej dmatej deleted the grizzly5 branch January 13, 2026 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Code Cleaning enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants