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

OF-2668 / OF-2723: Allow plugins to be compiled against Openfire 4.8 #2330

Merged

Conversation

guusdk
Copy link
Member

@guusdk guusdk commented Nov 11, 2023

In this commit, a second artifact is produced from the xmppserver module. Apart from the prexisting 'jar' artifact, there now is a 'taglib' artifact.

This taglib artifact is a jar file (that needs to go into our maven repository), that holds the Admin Console taglibs, in a spec-compliant directory structure.

The plugin's parent pom-file is modified to be able to make use of this new artifact.

With this, compilation of a plugin that defins the plugin parent pom as its parent succeeds (although an unhealty amount of errors is logged). The resulting admin console pages appear functional, at least in Openfire 4.8.

In this commit, a second artifact is produced from the xmppserver module. Apart from the prexisting 'jar' artifact, there now is a 'taglib' artifact.

This taglib artifact is a jar file (that needs to go into our maven repository), that holds the Admin Console taglibs, in a spec-compliant directory structure.

The plugin's parent pom-file is modified to be able to make use of this new artifact.

With this, compilation of a plugin that defins the plugin parent pom as its parent succeeds (although an unhealty amount of errors is logged). The resulting admin console pages appear functional, at least in Openfire 4.8.
@guusdk guusdk requested a review from Fishbowler November 11, 2023 18:13
…Openfire 4.8.0

Disabling the 'scanManifest' option for the Jspc compiler should prevent illogical lookups by the compiler of JSP pages.
Copy link
Member

@Fishbowler Fishbowler left a comment

Choose a reason for hiding this comment

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

Change looks sane

Testing a broken plugin:

  • Fails to build when referencing 4.8.0 snapshot due to web compilation
  • Builds after mvn installing this branch
  • Web assets appear correct when installing the plugin

@Fishbowler Fishbowler merged commit 7a53b23 into igniterealtime:main Nov 13, 2023
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.

2 participants