Thanks a lot for the detailed review of the distros, Niall !

I will comment on specific points below, but in light of Niall's
feedback I am proposing:

a) We close this vote, and declare it unsuccessful
b) We apply suggested fixes (again, specific comments below) and
recreate the svn tag
c) We vote again on new proposed artifacts with these fixes (and these
fixes only)

This means folks who have reviewed the artifacts will need to do so
again, sorry for the inconvenience.

Any objections to this plan?


On 1/4/07, Niall Pemberton <[EMAIL PROTECTED]> wrote:
I just noticed another thing - theres some JavaScript files which are
being distributed as part of the Cobertura documentation
(sortabletable.js, stringbuilder.js and customsorttypes.js)  which
have two different licenses (GPL, plus others). Looks to me like there
should at least be an attribution in the NOTICE.txt - at a minimum I
think you need to review whether its OK to re-distribute these  since
users will be using that software if they look at the Cobertura
documentation. IMO it would be better if they were excluded from the
distros altogether.

<snip/>

OK, with that information, IMO, we should remove the cobertura plugin
completely for v1.0.4. We can discuss a suitable test coverage plugin
to add later, but as things stand, it seems to be mostly broken
[1],[2] in the current build anyway (we do have some tests for these
modules actually, contrary to what these report says).

[1] http://shale.apache.org/shale-core/cobertura/index.html
[2] http://shale.apache.org/shale-clay/cobertura/index.html

(more comments below ...)


Niall

On 1/4/07, Niall Pemberton <[EMAIL PROTECTED]> wrote:
> A couple of nitpicks
>
> 1) I ran the rat tool on the framework distro, after removing the docs
> directory (which highlighted a load of generated files) there were a
> few missing license headers. Patch available here:
>   https://issues.apache.org/struts/browse/SHALE-384
>
<snap/>

I will fix this (assigned SHALE-384 to me already).


> Also there are two Sun licensed files included in the distro in
> shale-tiger's resources:
>     web-facesconfig_1_0.dtd
>     web-facesconfig_1_1.dtd
>
> Are we authorised to re-distribute these files?
>
<snip/>

Craig is probably the best person here to answer this (though others
may know as well).


> 2) None of the shale jar files contain the usual manifest entries such as:
>     Extension-Name
>     Specification-Title
>     Specification-Vendor
>     Specification-Version
>     Implementation-Title
>     Implementation-Vendor
>     Implementation-Version
>     Implementation-Vendor-Id
>
> I've attached a patch for the pom to include these to the above JIRA ticket
>
<snap/>

I maintained the inertia from previous releases regarding the
manifests, in hindsight, shouldn't have. Thanks again for all the
patches Niall.

-Rahul


> Niall
>
<snip/>

Reply via email to