incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Crossley <>
Subject Re: [VOTE] Release Apache ManifoldCF 0.1
Date Sat, 08 Jan 2011 01:18:22 GMT
Karl Wright wrote:
> The RAT report after these changes looks good except for two files,
> which come from the skins in the site:
> [rat:report] Unapproved licenses:
> [rat:report]
> [rat:report]   C:/wip/mcf-release/release-0.1-branch/site/src/documentation/skins/common/xslt/html/split.xsl
> [rat:report]   C:/wip/mcf-release/release-0.1-branch/site/src/documentation/skins/lucene/note.txt
> [rat:report]
> [rat:report] *******************************
> One of these looks like it comes from Forrest itself (the first), and
> has this header:
> <!--
>  This stylesheet was taken from the XSLT FAQ
>  Comments and adaption to be used without normalize-space()
>   by
> -->
> No idea what to do about that one.

At Forrest we exclude it from our license header checking tools.

> The other is merely a "todo" list in the Lucene skin, which I believe
> could simply be removed.

Or just exclude it too from your RAT processing.

> Thoughts?

I don't understand why ManifoldCF needs this special "skin" processing
that then needs to live in your svn.
At Forrest, we advise not to create their own skin unless absolutely
necessary. We prefer to address any needs in the default skin.

With a quick flick through the ManifoldCF site i do not see anything
that needs a special skin, just a couple of colour specifications.

Of course this would be better discussed at the Forrest mail lists.
At the dev@ list, seeing that you are using the development version.

However, i reckon that you should not hold up your release
because of this.


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message