Thilo, Thanks for looking this over. Please see some questions below
* LICENSE file needs the variables filled in.
Is this related to the comment about appending other licenses to this file? If not I am at a loss.
* The APR headers still use the old 1.1 Apache license. Is this the latest version of APR? I'm not suggesting you change the license, to the contrary. Just wondering if there is a later version available that's already under the 2.0 license.
Good point. Will move to a more recent version of APR.
* For the release voting, we'll need to say how the documentation is automatically generated with doxygen (because there are no license headers).
The doxygen generated html identifies itself as being generated by doxygen at the bottom of every page. I looked for information on the doxygen site about other requirements to acknowledge doxygen, but could not find any. Can you say more here?
* uimacpp/include/uima/msg.h and uimacpp/include/uima/msgstrtab.h have no license headers.
Fixed.
* The licenses do not go into separate subdirectories. Instead, all licenses are appended to the LICENSE file. In addition, in the NOTICES file, you spell out what software you include that's not Apache licensed, and under which license (which is then in the LICENSE file). You'll need to do that for the ICU, and I guess Xeces and APR, because they are 1.x licensed. We should ask on incubator-general about the ICU license, but I assume it's ok because Xerces is also using it.
The xerces release here is already 2.0 licensed, and APR will be too. Will ask the incubator-general about ICU.
That's it for today, I'll take another look tomorrow, and hopefully also at the source distribution.
Thanks again, Eddie
