I noticed the following: - The NOTICE files say "Apache UIMACPP". In the sandbox releases, we just say "Apache UIMA". Not sure we should say Apache UIMACPP, since that's not really a project name?
- The src distribution says that it includes the ICU, both in NOTICE and LICENSE. However, the ICU is not part of the src distribution (it is of the binary distribution). We should probably have two different NOTICE/LICENSE files, for the binary and src distributions. - I'm unclear on the use of the APR in UIMACPP. I see APR headers in the UIMACPP binary distribution. The APR NOTICE file contains several acknowledgments that should be in the UIMACPP NOTICE file *if* UIMACPP uses APR. - include/xercesc/util/LogicalPath.c doesn't include a license header, which is hardly our fault :-). Somebody might want to tell the Xerces folks... - minor: examples/data/docforsegmenter.xcas contains a license header, examples/data/docforsegmenter.xmi doesn't. Any reason? The bit about the ICU notice should be fixed before we take this to the incubator, I think. --Thilo Eddie Epstein wrote:
We have finished testing this release candidate and there are no remaining issues that impact reliability or performance. The release artifacts and RAT results are available on p.a.o/~eae/uimacpp-release-candidates/2.2.2/02/ Please cast your vote: [ ] +1 release candidate uimacpp-2.2.2-incubating RC02 is ready and can be released [ ] 0 don't care [ ] -1 don't release yet, I found some more issues Thanks, Eddie
