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

added Config.jvm_opts to add arguments to the java call #176

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

massimiliano-della-rovere
Copy link
Contributor

@massimiliano-della-rovere massimiliano-della-rovere commented Aug 21, 2024

I needed to pass the -Djava.jwt.headless=true argument to the java call, to avoid getting an error from java/jasper.

There was no way to do it, so I added:

  • a jvm_opts attribute to the Config class,
  • that is set by the PyReportJasper.config method,
  • and is read by the Report.__init__ method.

Both Config.jvm_opts and PyReportJasper.config(..., jvm_opts) must be a tuple or list, containing zero or more strings; each string must be a valid argument accepted by the java executable.

Example of usage:

prj = PyReportJasper()
prj.config(..., jvm_opts=("-Djava.awt.headless=true",))
prj.process_report()

@massimiliano-della-rovere
Copy link
Contributor Author

There are failed tests involving MacOS, but my modifications are not causing the failures, if I'm not wrong.

What can I do?

@jadsonbr
Copy link
Collaborator

There are failed tests involving MacOS, but my modifications are not causing the failures, if I'm not wrong.

What can I do?

It’s not a problem with your code. It’s a configuration issue with the pipeline that needs attention to update the proper functioning of Java on the Mac; there may have been an update.

@jadsonbr jadsonbr merged commit f2b2c00 into acesseonline:master Aug 21, 2024
48 of 72 checks passed
@jadsonbr jadsonbr linked an issue Aug 21, 2024 that may be closed by this pull request
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.

report.py lines 61 and 63 seem to have a problem
2 participants