uima-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eddie Epstein" <eaepst...@gmail.com>
Subject Re: [VOTE] Release uimacpp-2.2.2-02 as uimacpp-2.2.2-incubating
Date Wed, 13 Aug 2008 13:27:53 GMT
Thilo,

Thanks for the pointers. Comments and question below...

On Tue, Aug 12, 2008 at 11:16 AM, Thilo Goetz <twgoetz@gmx.de> wrote:
> 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?

Good point.

>
> - 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.

Will do.

>
> - 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.

I can't find instructions to that effect. Can you please give me a pointer?

>
> - 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?

Nope, will fix.

>
> The bit about the ICU notice should be fixed before we take
> this to the incubator, I think.
>
> --Thilo

molte grazie,
Eddie

Mime
View raw message