Would you be comfortable for letting this be fixed next release Justin? I raised this on the dev list when the vote came up - http://mail-archives.apache.org/mod_mbox/incubator-freemarker-dev/201603.mbox/%3C601980662.20160322084059%40freemail.hu%3E
John On Fri, Mar 25, 2016 at 5:54 PM Justin Mclean <jus...@classsoftware.com> wrote: > Hi, > > -1 (binding) due to category B license in source. As these file are > mention in NOTICE there’s probably a reason for this but it’s unclear to me. > > I checked: > - incubating in name > - signatures good but form a non apache email > - LICENSE is good > - NOTICE has unneeded information in it > - DISCLAIMER exists > - No unexpected binary files, but no sure why 2 files are mentioned in > NOTICE. IF these field are covered by an Apache license there should be no > need to put them in NOTICE. > - Can compile from source > > Can you provide some details on these files. This files have Apache > headers on them but as far as I can tell the originals are CDDL/GPU which > is category B and not allowed to be included in a source release. > src/main/resources/freemarker/ext/jsp/web-app_2_2.dtd > src/main/resources/freemarker/ext/jsp/web-app_2_3.dtd > src/main/resources/freemarker/ext/jsp/web-jsptaglibrary_1_1.dtd > src/main/resources/freemarker/ext/jsp/web-jsptaglibrary_1_2.dtd > > For original license see [1][2][3]. I’m not sure how the first file is > licensed. > > Thanks, > Justin > > 1. http://java.sun.com/dtd/web-app_2_3.dtd > 2. http://java.sun.com/j2ee/dtds/web-jsptaglibrary_1_1.dtd > 3.http://java.sun.com/j2ee/dtds/web-jsptaglibrary_1_2.dtd > --------------------------------------------------------------------- > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > For additional commands, e-mail: general-h...@incubator.apache.org > >
