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

For birt-runtime-osgi copy javax.xml.soap jar to the lib folder #1347

Merged
merged 1 commit into from
Jun 24, 2023

Conversation

merks
Copy link
Contributor

@merks merks commented Jun 24, 2023

@merks
Copy link
Contributor Author

merks commented Jun 24, 2023

@speckyspooky

This adds the jar to the example lib folder and in the war's lib folder it's there as well.

I'm going out now, so please feel free to squash and merge this when the builds complete, I assume successfully.

@speckyspooky
Copy link
Contributor

@merks
I will do it when the built is finished.

@speckyspooky speckyspooky merged commit 169bab3 into eclipse-birt:master Jun 24, 2023
@merks merks deleted the issue-1340 branch June 24, 2023 16:12
@speckyspooky
Copy link
Contributor

The change was an partial success. We have a new issue based on the changes.
You comment the plugin-copy of some libraries:

  • org.apache.commons.logging_*.jar
  • org.apache.axis_/lib/.jar
  • org.apache.commons.discovery_/lib/.jar
  • javax.xml.rpc_/lib/.jar

Now we get Error 500 on the example reports (see below). Please remove the comment and leave the copy part to the plugin-folder like before.

issue-after-soap

@speckyspooky
Copy link
Contributor

speckyspooky commented Jun 24, 2023

@merks

With the latest changes we miss 2 files which are required to execute the report-generation:

  • commons-discovery-0.2.jar
  • org.apache.commons.logging_1.2.0.v20180409-1502.jar

I compared the latest version "lib"-folder with the version before.
If I add the both files manuelly at the "lib"-folder then the report can be generated correctly.

@speckyspooky
Copy link
Contributor

PR #1348

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