On 01/04/2008, James M Snell <[EMAIL PROTECTED]> wrote: > > > sebb wrote: > > The following do not seem to be test files, nor are they trivially short. > > So they should have AL headers please: > > > > core/src/main/resources/abdera.properties.example > > dependencies/deps.properties > > > > > Done >
Thanks > > The top level NOTICE file should not have the 4 line == header. > > > Removed > Thanks > > Also, SVN includes the file > > > dependencies/i18n/src/main/java/org/apache/abdera/i18n/text/data/CompositionExclusions.txt > > which is not AL licensed; IMO it should be attributed in the NOTICE file. > > > > > I've just removed the file completely as we actually no longer need it > at runtime. In the future, I plan to automate the process of downloading > and processing that file during the build process. > However the product still depends on the file - I don't think the fact that the file is not in SVN makes any dfference. > > - James > > > > Probably something needs to go in the LICENSE file as well > > > > > > > > On 01/04/2008, James M Snell <[EMAIL PROTECTED]> wrote: > >> I've added license headers to the build scripts and to the localization > >> file and log4j.properties file. The rest I intend to leave as is. Is > >> that sufficient? If so, we'll roll a new build. > >> > >> > >> - James > >> > >> sebb wrote: > >> > On 31/03/2008, James M Snell <[EMAIL PROTECTED]> wrote: > >> >> In the FAQ I see this: "A file without any degree of creativity in > >> >> either its literal elements or its structure is not protected by > >> >> copyright law; therefore, such a file does not require a license > header. > >> > > >> > Yes, but: > >> > > >> >> If in doubt about the extent of the file's creativity, add the > license > >> >> header to the file." > >> > > >> >> Here's the breakdown that I see from Rat: > >> >> > >> >> None of the build scripts contain license headers. Given that the > build > >> >> scripts are everyday, normal maven build scripts, I do believe they > >> >> qualify under the "a file without any degree of creativity" clause. > >> > > >> > I disagree. > >> > > >> > Note that all Commons poms include the AL header. > >> > > >> >> ? adapters/pom.xml > >> >> ? client/pom.xml > >> >> ? core/pom.xml > >> >> ? examples/pom.xml > >> >> ? extensions/pom.xml > >> >> ? extensions/gdata/pom.xml > >> >> ? extensions/geo/pom.xml > >> >> ? extensions/html/pom.xml > >> >> ? extensions/json/pom.xml > >> >> ? extensions/main/pom.xml > >> >> ? extensions/media/pom.xml > >> >> ? extensions/oauth/pom.xml > >> >> ? extensions/opensearch/pom.xml > >> >> ? extensions/serializer/pom.xml > >> >> ? extensions/sharing/pom.xml > >> >> ? extensions/wsse/pom.xml > >> >> ? parser/pom.xml > >> >> ? security/pom.xml > >> >> ? server/pom.xml > >> >> ? spring/pom.xml > >> >> ? dependencies/deps.properties > >> >> > >> >> The following file used to provide localization strings may possibly > >> >> require a license header. > >> >> > >> >> ? core/src/main/resources/abderamessages.properties > >> >> > >> >> The following are configuration files used at runtime, the majority > of > >> >> which consist of only a few lines of text and also fall under the > above > >> >> quoted clause. > >> >> > >> >> ? client/src/main/java/log4j.properties > >> >> ? core/src/main/resources/abdera.properties.example > >> >> ? > >> > > >> > These have less content, so the creative aspect is definitely lower. > >> > However, I don't think there's no creativity involved. > >> > > >> >> > core/src/main/resources/META-INF/services/org.apache.abdera.factory.ExtensionFactory.example > >> >> ? > >> >> > core/src/main/resources/META-INF/services/org.apache.abdera.factory.ExtensionFactory > >> >> ? > >> >> > core/src/test/resources/META-INF/services/org.apache.abdera.converter.ConverterProvider > >> >> ? > >> >> > examples/src/main/resources/META-INF/services/org.apache.abdera.factory.ExtensionFactory > >> >> ? > >> >> > extensions/complete/resources/META-INF/services/org.apache.abdera.writer.NamedWriter > >> >> ? > >> >> > extensions/complete/resources/META-INF/services/org.apache.abdera.factory.ExtensionFactory > >> >> ? > >> >> > extensions/complete/resources/META-INF/services/org.apache.abdera.parser.NamedParser > >> >> ? > >> >> > extensions/html/src/main/resources/META-INF/services/org.apache.abdera.parser.NamedParser > >> >> ? > >> >> > extensions/json/src/main/resources/META-INF/services/org.apache.abdera.writer.NamedWriter > >> >> !????? > >> >> > extensions/main/src/main/resources/META-INF/services/org.apache.abdera.factory.ExtensionFactory > >> >> ? > >> >> > extensions/media/src/main/resources/META-INF/services/org.apache.abdera.factory.ExtensionFactory > >> >> ? > >> >> > extensions/opensearch/src/main/resources/META-INF/services/org.apache.abdera.factory.ExtensionFactory > >> >> ? > >> >> > extensions/sharing/src/main/resources/META-INF/services/org.apache.abdera.factory.ExtensionFactory > >> >> ? > >> >> > parser/src/main/resources/META-INF/services/org.apache.abdera.writer.NamedWriter > >> >> ? > >> >> > contrib/rss/src/main/resources/META-INF/services/org.apache.abdera.factory.ExtensionFactory > >> >> > >> > > >> > Agreed. > >> > > >> >> The following simple files are used for example purposes and do not > >> >> contain any creative content whatsoever. > >> >> > >> >> ? examples/src/main/resources/xmlcontent.xml > >> >> ? examples/src/main/resources/log4j.properties > >> >> ? examples/src/main/resources/simple.xml > >> >> ? examples/src/main/resources/test.xslt > >> >> ? examples/src/main/resources/content.xslt > >> >> ? > examples/src/main/resources/org/apache/abdera/examples/appserver/web.xml > >> > > >> > Not much creativity, and small files, so the AL header is not > essential. > >> > > >> >> The following were also (inappropriately) flagged by RAT > >> >> > >> >> ? extensions/complete/resources/META-INF/LICENSE.htmlparser.txt > >> >> ? extensions/complete/resources/META-INF/NOTICE.htmlparser.txt > >> >> ? extensions/complete/resources/META-INF/NOTICE.serializer.txt > >> >> ? extensions/json/src/main/resources/META-INF/LICENSE.htmlparser.txt > >> >> ? extensions/json/src/main/resources/META-INF/NOTICE.htmlparser.txt > >> >> ? extensions/json/src/main/resources/META-INF/NOTICE.serializer.txt > >> >> ? dependencies/legal/servlet-api-LICENSE.txt > >> >> ? dependencies/legal/htmlparser-LICENSE.txt > >> >> > >> >> > >> >> The following resources are test resources that in addition to not > >> >> containing any degree of creative content, they arguably should not > >> >> contain license headers because the presence of the license header > could > >> >> potentially impact the results of the tests. > >> >> > >> > > >> > Probably. > >> > Though if the AL header affects the parsing, then there may well be a > >> > problem with the parsing... > >> > > >> >> ? > adapters/hibernate/src/test/resources/abdera/adapter/hibernate.cfg.xml > >> >> ? > adapters/hibernate/src/test/resources/abdera/adapter/DummyData.hbm.xml > >> >> ? extensions/opensearch/src/test/resources/opensearch.xml > >> >> ? > >> >> > parser/src/test/resources/www.snellspace.com/public/nondefaultnamespace.xml > >> >> ? > >> >> > parser/src/test/resources/www.snellspace.com/public/nondefaultnamespace3.xml > >> >> ? > >> >> > parser/src/test/resources/www.snellspace.com/public/nondefaultnamespace2.xml > >> >> ? parser/src/test/resources/www.snellspace.com/public/xmlbase.xml > >> >> ? parser/src/test/resources/www.snellspace.com/public/ordertest.xml > >> >> ? parser/src/test/resources/www.snellspace.com/public/linktests.xml > >> >> ? > parser/src/test/resources/www.snellspace.com/public/contentsummary.xml > >> >> ? parser/src/test/resources/simpleFeed.xml > >> >> ? parser/src/test/resources/xmlcontent.xml > >> >> ? parser/src/test/resources/feed.xml > >> >> ? parser/src/test/resources/simple.xml > >> >> ? > >> >> > parser/src/test/resources/www.feedparser.org/tests/wellformed/atom10/entry_content_base64_2.xml > >> >> ? > >> >> > parser/src/test/resources/www.feedparser.org/tests/wellformed/atom10/entry_content_base64.xml > >> >> ? > >> >> > parser/src/test/resources/www.feedparser.org/tests/wellformed/atom10/entry_author_email.xml > >> >> ? > >> >> > parser/src/test/resources/www.feedparser.org/tests/wellformed/atom10/entry_author_name.xml > >> >> ? > >> >> > parser/src/test/resources/www.feedparser.org/tests/wellformed/atom10/atom10_namespace.xml > >> >> ? parser/src/test/resources/complete.xml > >> >> ? parser/src/test/resources/simpleService.xml > >> >> ? parser/src/test/resources/simpleEntry.xml > >> >> ? parser/src/test/resources/test.xslt > >> >> ? parser/src/test/resources/entry.xml > >> >> ? parser/src/test/resources/content.xslt > >> >> ? spring/src/test/resources/org/apache/abdera/spring/beans.xml > >> >> ? contrib/rss/src/test/resources/rss1.rdf > >> >> ? > >> >> > adapters/hibernate/src/test/resources/abdera/adapter/hibernate.properties > >> >> ? security/src/test/resources/log4j.properties > >> >> ? server/src/test/resources/abdera/adapter/sample.properties > >> >> > >> >> This one documentation file, which contains only a single one > sentence > >> >> statement, does not contain a license header. > >> >> > >> >> ? docs/knownissues.txt > >> >> > >> >> Which of these files do you think we absolutely have to add license > >> >> headers to? > >> >> > >> >> > >> >> - James > >> >> > >> >> > >> >> sebb wrote: > >> >> > On 31/03/2008, Dan Diephouse <[EMAIL PROTECTED]> wrote: > >> >> >> sebb wrote: > >> >> >> > On 31/03/2008, James M Snell <[EMAIL PROTECTED]> wrote: > >> >> >> > > >> >> >> >> > >> >> >> > > >> >> >> > >> >> >>> -1: There are MD5 and SHA1 digests in the directory, but the > archives > >> >> >> > have no signatures. > >> >> >> > > >> >> >> > > >> >> >> > >> >> >> OK, I will fix this. > >> >> >> > >> >> >>>> Maven Repository: > http://people.apache.org/~dandiep/abdera-take6/ > >> >> >> >> > >> >> >> >> > >> >> >> > > >> >> >> > -0: The abdera-bundle-0.4.0-incubating-sources.jar does not > appear to > >> >> >> > have any content - only the META-INF directory is present. Is > that > >> >> >> > correct? > >> >> >> > > >> >> >> > >> >> >> This is just a by-product of Maven. We can delete it. > >> >> >> > >> >> >>> -1: The NOTICE files in that jar (and others) contains far too > much. > >> >> >> > The NOTICE file is for required attribtions ONLY (e.g. as per > an About box) > >> >> >> > There's really no need to repeat ASF for each project used by > Abdera. > >> >> >> > > >> >> >> > >> >> >> Having too much information in the NOTICE files is not a crime. > The > >> >> >> Maven remote-resources plugin aggregates all this stuff for us > so we > >> >> >> never miss any notice that we need to put in. > >> >> > > >> >> > Unfortunately the plugin generates incorrect information. > >> >> > It *is* a problem having all the redundant information. > >> >> > > >> >> > See for example: > >> >> > > >> >> > http://www.apache.org/legal/src-headers.html > >> >> > also > >> >> > http://wiki.apache.org/legal/3party/notice > >> >> > http://wiki.apache.org/legal/3party/notice/discuss > >> >> > > >> >> >>> -1: The LICENSE files need to either contain copies of the 3rd > party > >> >> >> > licenses, or they need to have a reference to the 3rd party > licences. > >> >> >> > Equally, there is no need for the lib directory to contain > copies of > >> >> >> > the AL for every ASF product. > >> >> >> > > >> >> >> > >> >> >> Why does the LICENSE file need to have a copy of all the other > licenses? > >> >> >> These are contained in the lib/ directory like many other ASF > projects. > >> >> >> > >> >> > > >> >> > See the last paragraph of: > >> >> > > >> >> > http://www.apache.org/dev/apply-license.html#new > >> >> > > >> >> >> Re: the ASL license in lib/ - once again having too much > information is > >> >> >> not a crime. This is a service to uesrs so they know where the > libraries > >> >> >> came from. > >> >> > > >> >> > I agree the source is useful, but the place for this is the > LICENSE file. > >> >> > > >> >> >>> -1: RAT report says: > >> >> >> > > >> >> >> > 99 Unknown Licenses > >> >> >> > > >> >> >> > Some of these are trivial, but most require an AL header. > >> >> >> > > >> >> >> > >> >> >> Not true - there is not consensus that properties/xml files need > to have > >> >> >> headers. All the Java source code files have headers. If there > are > >> >> >> specific files that you feel should have a license that don't > please > >> >> >> list them and explain why. I'm not saying that we didn't miss > something, > >> >> >> but I am saying that the ones that I know about don't necessarily > >> >> >> require a header. > >> >> > > >> >> > Yes, they do, see: > >> >> > > >> >> > http://www.apache.org/legal/src-headers.html#faq-exceptions > >> >> > > >> >> >>> What is the SVN tag that corresponds with the archives? > >> >> >> > > >> >> >> > > >> >> >> > >> >> >> the branch will be tagged once its released. > >> >> >> > >> >> >> Dan > >> >> >> > >> >> >> > >> >> >> -- > >> >> >> Dan Diephouse > >> >> >> MuleSource > >> >> >> http://mulesource.com | http://netzooid.com > >> >> >> > >> >> >> > >> >> >> > >> >> >> > --------------------------------------------------------------------- > >> >> >> To unsubscribe, e-mail: [EMAIL PROTECTED] > >> >> >> For additional commands, e-mail: [EMAIL PROTECTED] > >> >> >> > >> >> >> > >> >> > > >> >> > > --------------------------------------------------------------------- > >> >> > To unsubscribe, e-mail: [EMAIL PROTECTED] > >> >> > For additional commands, e-mail: [EMAIL PROTECTED] > >> >> > > >> >> > > >> >> > >> >> --------------------------------------------------------------------- > >> >> To unsubscribe, e-mail: [EMAIL PROTECTED] > >> >> For additional commands, e-mail: [EMAIL PROTECTED] > >> >> > >> >> > >> > > >> > --------------------------------------------------------------------- > >> > To unsubscribe, e-mail: [EMAIL PROTECTED] > >> > For additional commands, e-mail: [EMAIL PROTECTED] > >> > > >> > > >> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: [EMAIL PROTECTED] > >> For additional commands, e-mail: [EMAIL PROTECTED] > >> > >> > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]