There are 18 emails so far in this thread of which 13 are yours. - You mentioned stuff from wikipedia - then you mentioned stuff about licensing - then you switched to birt - then you talked about the author - then you go back to questioning how to render PDFs in BIRT - then you talk about your test logs for PDF rendering in BIRT (or something like that) - then you talk about the gradle cache - then you talk about digital signatures - then you talk about discussing things with apache legal - then you ask people for their opinion - then you go back to BIRT
So to answer your question, YES, it's very hard to read you :) Many of your emails are long with lots of URLs and jump around multiple topics. I personally cannot keep up, and that's why when you present a question to the community, I have to ask you to pin down exactly what you want. Now back to this thread: I'm always in favor of completely removing libraries where possible. This means refactoring CompDocServices.java and PdfSurveyServices.java. I'm not sure how much work would that be, but if it is a lot of work, then the work that I proposed might be a quick fix for now (exclusion in gradle). On Mon, Jun 11, 2018 at 8:25 PM, Jacques Le Roux <jacques.le.r...@les7arts.com> wrote: > No, I'm suggesting to drop itext as a whole, not only itextpdf. > > Is it so difficult to read me :-o ? > > I 1st spoke about "itext/4.2.0" (not itextpdf at all). Then I suggested to > remove "it". > > <<Also from few tests I did, it seems we don't need it to render PDF with > Birt. Please confirm...>> > > I believe (it's no clear from Birt side) itext is something we drag from the > 1st contribution of Birt in OFBiz. And Birt is now able to render PDF w/o > itext. > In some edge cases (at least: digital signature[1], 4 bytes UTF-8[2]) users > would still need to use itext. See my previous last message for other > details: > <<Since it works for me w/ "compile 'com.lowagie:itext" commented out after > clearing the Gradle cache from all itext files>> > > Jacques > > [1] https://s.apache.org/b2sQ > > [2] https://s.apache.org/Ib78 > > > > Le 11/06/2018 à 16:37, Taher Alkhateeb a écrit : >> >> I'm a bit lost. What are you _exactly_ proposing to do here? Are you >> suggesting my exclusion syntax above (BTW better remove the version), >> or are you suggesting something else? >> >> On Mon, Jun 11, 2018 at 3:10 PM, Jacques Le Roux >> <jacques.le.r...@les7arts.com> wrote: >>> >>> Le 08/06/2018 à 16:29, Jacques Le Roux a écrit : >>>> >>>> Are we sure there are no legal issues doing so? >>>> >>>> It seems OK at >>>> https://mvnrepository.com/artifact/com.lowagie/itext/4.2.0 >>>> (MPL) >>>> >>>> But reading >>>> https://developers.itextpdf.com/question/versions-older-than-5 >>>> which applies also to 4.2.0 (see bottom "Some people claim that they use >>>> iText 4.2.0, but that version has never been officially released") itext >>>> seems a legal issue globally (not only itextpdf) >>>> >>>> Maybe we should ask legal? >>>> >>>> Also from few tests I did, it seems we don't need it to render PDF with >>>> Birt. Please confirm... >>> >>> >>> Did someone else tests? >>> Since it works for me w/ "compile 'com.lowagie:itext" commented out >>> after >>> clearing the Gradle cache from all itext files I believe it should work >>> for >>> everyone else. Please confirm, should I open a Jira now? >>> >>> Now if users are of need of itext for other reasons (I found a couple of >>> them Googling) they should take their responsibility. What are other >>> opinions here? >>> >>> Jacques >>> >>> >>>> Jacques >>>> >>>> Le 08/06/2018 à 16:03, Scott Gray a écrit : >>>>> >>>>> Thanks Taher! Perfect simple solution. >>>>> >>>>> Regards >>>>> Scott >>>>> >>>>> On Fri, 8 Jun 2018, 23:19 Taher Alkhateeb, <slidingfilame...@gmail.com> >>>>> wrote: >>>>> >>>>>> So we exclude the transitive dependency in build.gradle and if >>>>>> everything >>>>>> works then we're fine. >>>>>> >>>>>> Syntax: >>>>>> >>>>>> compile('com.lowagie:itext:4.2.0') { >>>>>> exclude 'com.itextpdf:itextpdf:5.5.6' >>>>>> } >>>>>> >>>>>> On Fri, Jun 8, 2018, 11:40 AM Scott Gray >>>>>> <scott.g...@hotwaxsystems.com> >>>>>> wrote: >>>>>> >>>>>>> Hey Jacques, >>>>>>> >>>>>>> Maybe I wasn't clear, OFBiz is downloading 5.5.6 as a dependency of >>>>>> >>>>>> 4.2.0, >>>>>>> >>>>>>> does it make sense? >>>>>>> >>>>>>> Regards >>>>>>> Scott >>>>>>> >>>>>>> >>>>>>> On Fri, 8 Jun 2018, 19:30 Jacques Le Roux, >>>>>>> <jacques.le.r...@les7arts.com >>>>>>> >>>>>>> wrote: >>>>>>> >>>>>>>> I suggest this comment, a Jira seems appropriate >>>>>>>> >>>>>>>> - compile 'com.lowagie:itext:4.2.0' >>>>>>>> + compile 'com.lowagie:itext:4.2.0' // don't update to 5+ because >>>>>>>> of >>>>>>>> license change >>>>>>>> >>>>>>>> Jacques >>>>>>>> >>>>>>>> >>>>>>>> Le 08/06/2018 à 09:26, Jacques Le Roux a écrit : >>>>>>>>> >>>>>>>>> Le 08/06/2018 à 09:24, Jacques Le Roux a écrit : >>>>>>>>>> >>>>>>>>>> Hi Scott, >>>>>>>>>> >>>>>>>>>> Reading Wikipedia It's OK as long as we don't update to a version >>>>>>> >>>>>>> = 5 >>>>>>>> >>>>>>>> https://en.wikipedia.org/wiki/IText >>>>>>>>> >>>>>>>>> Here is another source for MPL licensing: >>>>>>>> >>>>>>>> https://www.eclipse.org/forums/index.php/t/175386/ >>>>>>>>>> >>>>>>>>>> <<The source code was initially distributed as open source under >>>>>>>>>> the >>>>>>>> >>>>>>>> Mozilla Public License < >>>>>>>> https://en.wikipedia.org/wiki/Mozilla_Public_License> >>>>>>>>>> >>>>>>>>>> or the GNU Library General Public License < >>>>>>>> >>>>>>>> https://www.gnu.org/licenses/old-licenses/lgpl-2.0.en.html> open >>>>>> >>>>>> source >>>>>>>> >>>>>>>> licenses. However, as of version >>>>>>>>>> >>>>>>>>>> 5.0.0 (released Dec 7, 2009) it is distributed under the Affero >>>>>>> >>>>>>> General >>>>>>>> >>>>>>>> Public License >>>>>>>>>> >>>>>>>>>> <https://en.wikipedia.org/wiki/Affero_General_Public_License> >>>>>> >>>>>> version >>>>>>>> >>>>>>>> 3.>> >>>>>>>>>> >>>>>>>>>> MPL being OK as binary >>>>>>>>>> >>>>>>>>>> Jacques >>>>>>>>>> >>>>>>>>>> Le 08/06/2018 à 03:57, Scott Gray a écrit : >>>>>>>>>>> >>>>>>>>>>> Hi All, >>>>>>>>>>> >>>>>>>>>>> I just noticed that the iText maven bundle is a bit tricksy and >>>>>>>> >>>>>>>> includes >>>>>>>>>>> >>>>>>>>>>> iText 5.6.6 as a dependency, with the latter being GPL licensed. >>>>>> >>>>>> You >>>>>>>> >>>>>>>> can >>>>>>>>>>> >>>>>>>>>>> see it by running "./gradlew -q dependencies": >>>>>>>>>>> +--- com.lowagie:itext:4.2.0 >>>>>>>>>>> | \--- com.itextpdf:itextpdf:5.5.6 >>>>>>>>>>> >>>>>>>>>>> I haven't checked to see if the later version is actually used by >>>>>> >>>>>> our >>>>>>>> >>>>>>>> code >>>>>>>>>>> >>>>>>>>>>> and I'm not sure if merely downloading it causes licensing >>>>>>>>>>> issues, >>>>>>> >>>>>>> but >>>>>>>> >>>>>>>> I >>>>>>>>>>> >>>>>>>>>>> thought I'd bring the question here in case anyone else has >>>>>>>>>>> already >>>>>>>> >>>>>>>> looked >>>>>>>>>>> >>>>>>>>>>> into it. Not sure what the work-around would be if it is an >>>>>>>>>>> issue. >>>>>>>>>>> >>>>>>>>>>> Regards >>>>>>>>>>> Scott >>>>>>>>>>> >>>> >