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 http://www.dpawson.co.uk/xsl/ > > Comments and adaption to be used without normalize-space() > by forrest-...@xml.apache.org > --> > > 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. -David --------------------------------------------------------------------- To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org
