Wow, so reading these threads tells me that Scott had concerns which hit the nail right on the head with respect to licensing but the files were committed anyway. Unfortunate!
+1 for removing them. On Mar 25, 2017 10:03 AM, "Jacques Le Roux" <[email protected]> wrote: > Good point Scott, we discussed it indeed https://lists.apache.org/list. > [email protected]:gte=7y:add%20BIRT%20branch%20to%20trunk. > > I had not the guts to deep into all details, but yes you and others > expressed concerns then: https://s.apache.org/GyKu > > The most important point here is more that AFAIK these files are not even > needed! > > Jacques > > > Le 24/03/2017 à 21:04, Scott Gray a écrit : > >> I'm curious as to how it was ever added, when the birt component was first >> introduced I'm pretty sure we agreed to not include this part of it >> because >> of licensing concerns. >> >> On 25 March 2017 at 02:34, Jacopo Cappellato < >> [email protected]> wrote: >> >> On Thu, Mar 23, 2017 at 10:01 AM, Jacques Le Roux < >>> [email protected]> wrote: >>> >>> Hi, >>>> >>>> plugins/birt/webapp/birt/webcontent is a part of the birt component but >>>> AFAIK is not used by OFBiz. >>>> >>>> Also Birt is licensed under EPL, it's category B >>>> http://www.apache.org/legal/resolved.html#category-b < >>>> http://www.apache.org/legal/resolved.html#category-b>, so files should >>>> >>> be >>> >>>> only provided in object/binary form >>>> >>>> So I think we need to remove this code. I was able to generate a PDF and >>>> >>> a >>> >>>> XLS report after removing this part, so I think it's OK w/o it. >>>> >>>> I also tried the new Birt feature (Flexible Reports) and it works with >>>> >>> any >>> >>>> kind of format provided. >>>> I begin the documentation of this new part and will let you know when >>>> >>> it's >>> >>>> done... >>>> >>>> Jacques >>>> >>>> If the code is not used then I cast a great +1 for its removal. >>> >>> Jacopo >>> >>> >
