jasperreports icon indicating copy to clipboard operation
jasperreports copied to clipboard

Fix issue-17, iText 2.1.7 to OpenPDF 1.3.11

Open srbala opened this issue 6 years ago • 13 comments

Resolves issue #17, replaces outdated and partial PR #29.

srbala avatar Mar 11 '19 03:03 srbala

This also related to https://github.com/LibrePDF/OpenPDF/issues/145, solves JasperReport support

srbala avatar Mar 11 '19 16:03 srbala

Can you provide a binary distribution of JasperReports library with OpenPDF? I still can't compile JasperReports with the changes you've provided.

ozawa-hi avatar Apr 08 '19 02:04 ozawa-hi

@ozawa-hi Please share the compilation error that you get.

@srbala Can you please update this PR to the latest version of OpenPDF, which is 1.2.15 now?

andreasrosdal avatar Apr 08 '19 06:04 andreasrosdal

Sorry my bad. Got it to build. Thanks.

ozawa-hi avatar Apr 10 '19 22:04 ozawa-hi

I think TIBCO Jaspersoft is sticking with itext2.1.7 because of reasons written in the following page. https://community.jaspersoft.com/wiki/issue-concerning-itext-licensing-tibco-jaspersoft

ozawa-hi avatar Apr 10 '19 22:04 ozawa-hi

Hi ozawa-hi,

Were you able to run JasperReports using OpenPDF. How did you compile?

shamimOnGitHub avatar Jun 04 '19 12:06 shamimOnGitHub

Any news on this? The latest version of OpenPDF is now 1.3.11. Maybe this pull request could be updated?

@ozawa-hi All the issues mentioned on that page are now addressed by OpenPDF. We have done a code review and resolved all questions related to licensing. More info here: https://github.com/LibrePDF/OpenPDF/wiki/OpenPDF---License-MPL---LGPL-background-info

andreasrosdal avatar Oct 02 '19 15:10 andreasrosdal

Hi @teodord,

you have closed https://github.com/TIBCOSoftware/jasperreports/issues/17

When the situation will change in the future, we'll reconsider.

but you reference this PR #53 in https://github.com/TIBCOSoftware/jasperreports/pull/29

Does it mean that there is a reconsideration ? If not, why not close all PR ( and not just duplicate) about openpdf dependencies as wont-fix ?

douph1 avatar Dec 19 '19 14:12 douph1

We'll probably keep only one of these items open for the foreseeable future, as we cannot completely exclude that one day will make the switch.

teodord avatar Dec 19 '19 15:12 teodord

@andreasrosdal updated OpenPDF to 1.3.11 @teodord resolved merge conflicts, this pull is ready for merge

srbala avatar Dec 25 '19 17:12 srbala

@teodord Do we have plan to merge this anytime soon? We are using JasperReport and we want to migrate to OpenPDF. This is what we exactly need.

paksboda avatar Jul 07 '20 10:07 paksboda

My understanding is that you can put OpenPDF in your application classpath instead of iText and it should work just fine. If you look at the proposed changes, none are about Java source code modifications and are only about JAR references. So you can do what you need to do without JasperReports project actually making any change itself.

teodord avatar Jul 07 '20 13:07 teodord

Thanks for response. I agree with you and if not merged then I will do the same thing but if merge is happening anytime soon then it will be more cleaner implementation so I thought to confirm once.

paksboda avatar Jul 07 '20 16:07 paksboda

Looks like this PR is outdated

OleksandrShkurat avatar Mar 03 '23 18:03 OleksandrShkurat

A fork of OpenPDF 1.3.30 is going to be used in the next JasperReports Library release.

teodord avatar Mar 06 '23 12:03 teodord

A fork of OpenPDF 1.3.30 is going to be used in the next JasperReports Library release.

That's why I said this PR is outdated. I'd suggest closing it

OleksandrShkurat avatar Mar 06 '23 13:03 OleksandrShkurat

Closing, iText has been replaces with a OpenPDF 1.3.30 patched dependency.

dadza avatar Mar 10 '23 11:03 dadza

@teodord Is there any information available on the differences of the forked OpenPDF vs. mainline OpenPDF? (Are there any compatibility concerns when replacing the forked one with the mainline one locally?)

soc avatar Mar 24 '23 19:03 soc

The only difference is that our forked version includes a fix for https://github.com/LibrePDF/OpenPDF/issues/676

You can use the official OpenPDF 1.3.30 release if you don't mind the effects of that bug.

dadza avatar Mar 24 '23 19:03 dadza

@dadza Thank you!

soc avatar Mar 24 '23 19:03 soc