Re: GitHub license display confused by LICENSE-header.txt
Turns out that it is sufficient to rename additional top-level files with the string 'license' in them. (and update any references, of course) However GH does not re-evaluate the license unless the file itself is touched. All the components now show up as AL2.0 apart from math and weaver. Not sure what to do about those yet. N.B. to do a quick check of the GH license attribution: curl -sq https://api.github.com/repos/apache/commons-exec/license | grep -v '"content":' On Tue, 9 Mar 2021 at 18:22, Gary Gregory wrote: > > Awesome, thank you Sebb! > > Gary > > On Tue, Mar 9, 2021, 12:11 sebb wrote: > > > Good idea. > > > > I have done that for commons-codec, and GH has now agreed to use AL2.0 > > > > However it did seem to need a dummy change to the LICENSE file to > > re-evaluate the license. > > > > I'll see about fixing the others > > > > On Tue, 9 Mar 2021 at 01:15, Melloware Inc wrote: > > > > > > In commons beanutils we recommend using /src/conf for these type of > > files. > > > > > > Sent from my iPhone > > > > > > > On Mar 8, 2021, at 8:13 PM, sebb wrote: > > > > > > > > On Tue, 9 Mar 2021 at 01:08, Bernd Eckenfels > > wrote: > > > >> > > > >> Checkstyle-header.txt sounds good and maybe also moving it to a > > subdir like src/build/ or src/test/? > > > > > > > > There are other checkstyle files at the top-level; best to keep them > > together. > > > > > > > >> Gruss > > > >> Bernd > > > >> > > > >> > > > >> -- > > > >> http://bernd.eckenfels.net > > > >> > > > >> Von: sebb > > > >> Gesendet: Tuesday, March 9, 2021 1:41:02 AM > > > >> An: CommonsDev > > > >> Betreff: GitHub license display confused by LICENSE-header.txt > > > >> > > > >> Most of the Commons projects show up in GitHub as having the Apache > > 2.0 License > > > >> > > > >> However a few show up as 'other': > > > >> > > > >> commons-codec > > > >> commons-csv > > > >> commons-dbutils > > > >> commons-exec > > > >> commons-jelly > > > >> commons-logging > > > >> commons-math > > > >> commons-rdf > > > >> commons-text > > > >> commons-weaver > > > >> > > > >> AFAICT this is because there is more than 1 LICENSE file at the top > > level: > > > >> I forked codec and deleted the LICENSE-header.txt file, and the > > > >> license then showed up as AL 2.0. > > > >> > > > >> I think it's not only GitHub that might be confused by such files, so > > > >> I suggest they are moved or renamed to avoid the confusion. They are > > > >> only needed for Checkstyle, so perhaps could be named accordingly, > > > >> e.g. checkstyle-header.txt > > > >> > > > >> [Obviously there would need to be corresponding changes in the pom and > > > >> assembly files] > > > >> > > > >> WDYT? > > > >> > > > >> Sebb. > > > >> > > > >> - > > > >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > > >> For additional commands, e-mail: dev-h...@commons.apache.org > > > >> > > > > > > > > - > > > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > > > > > > > - > > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > > > > - > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: GitHub license display confused by LICENSE-header.txt
Awesome, thank you Sebb! Gary On Tue, Mar 9, 2021, 12:11 sebb wrote: > Good idea. > > I have done that for commons-codec, and GH has now agreed to use AL2.0 > > However it did seem to need a dummy change to the LICENSE file to > re-evaluate the license. > > I'll see about fixing the others > > On Tue, 9 Mar 2021 at 01:15, Melloware Inc wrote: > > > > In commons beanutils we recommend using /src/conf for these type of > files. > > > > Sent from my iPhone > > > > > On Mar 8, 2021, at 8:13 PM, sebb wrote: > > > > > > On Tue, 9 Mar 2021 at 01:08, Bernd Eckenfels > wrote: > > >> > > >> Checkstyle-header.txt sounds good and maybe also moving it to a > subdir like src/build/ or src/test/? > > > > > > There are other checkstyle files at the top-level; best to keep them > together. > > > > > >> Gruss > > >> Bernd > > >> > > >> > > >> -- > > >> http://bernd.eckenfels.net > > >> > > >> Von: sebb > > >> Gesendet: Tuesday, March 9, 2021 1:41:02 AM > > >> An: CommonsDev > > >> Betreff: GitHub license display confused by LICENSE-header.txt > > >> > > >> Most of the Commons projects show up in GitHub as having the Apache > 2.0 License > > >> > > >> However a few show up as 'other': > > >> > > >> commons-codec > > >> commons-csv > > >> commons-dbutils > > >> commons-exec > > >> commons-jelly > > >> commons-logging > > >> commons-math > > >> commons-rdf > > >> commons-text > > >> commons-weaver > > >> > > >> AFAICT this is because there is more than 1 LICENSE file at the top > level: > > >> I forked codec and deleted the LICENSE-header.txt file, and the > > >> license then showed up as AL 2.0. > > >> > > >> I think it's not only GitHub that might be confused by such files, so > > >> I suggest they are moved or renamed to avoid the confusion. They are > > >> only needed for Checkstyle, so perhaps could be named accordingly, > > >> e.g. checkstyle-header.txt > > >> > > >> [Obviously there would need to be corresponding changes in the pom and > > >> assembly files] > > >> > > >> WDYT? > > >> > > >> Sebb. > > >> > > >> - > > >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > >> For additional commands, e-mail: dev-h...@commons.apache.org > > >> > > > > > > - > > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > > > > - > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > - > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > >
Re: GitHub license display confused by LICENSE-header.txt
Good idea. I have done that for commons-codec, and GH has now agreed to use AL2.0 However it did seem to need a dummy change to the LICENSE file to re-evaluate the license. I'll see about fixing the others On Tue, 9 Mar 2021 at 01:15, Melloware Inc wrote: > > In commons beanutils we recommend using /src/conf for these type of files. > > Sent from my iPhone > > > On Mar 8, 2021, at 8:13 PM, sebb wrote: > > > > On Tue, 9 Mar 2021 at 01:08, Bernd Eckenfels > > wrote: > >> > >> Checkstyle-header.txt sounds good and maybe also moving it to a subdir > >> like src/build/ or src/test/? > > > > There are other checkstyle files at the top-level; best to keep them > > together. > > > >> Gruss > >> Bernd > >> > >> > >> -- > >> http://bernd.eckenfels.net > >> ________ > >> Von: sebb > >> Gesendet: Tuesday, March 9, 2021 1:41:02 AM > >> An: CommonsDev > >> Betreff: GitHub license display confused by LICENSE-header.txt > >> > >> Most of the Commons projects show up in GitHub as having the Apache 2.0 > >> License > >> > >> However a few show up as 'other': > >> > >> commons-codec > >> commons-csv > >> commons-dbutils > >> commons-exec > >> commons-jelly > >> commons-logging > >> commons-math > >> commons-rdf > >> commons-text > >> commons-weaver > >> > >> AFAICT this is because there is more than 1 LICENSE file at the top level: > >> I forked codec and deleted the LICENSE-header.txt file, and the > >> license then showed up as AL 2.0. > >> > >> I think it's not only GitHub that might be confused by such files, so > >> I suggest they are moved or renamed to avoid the confusion. They are > >> only needed for Checkstyle, so perhaps could be named accordingly, > >> e.g. checkstyle-header.txt > >> > >> [Obviously there would need to be corresponding changes in the pom and > >> assembly files] > >> > >> WDYT? > >> > >> Sebb. > >> > >> - > >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > >> For additional commands, e-mail: dev-h...@commons.apache.org > >> > > > > - > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > - > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: GitHub license display confused by LICENSE-header.txt
On Tue, 9 Mar 2021 at 12:51, Gilles Sadowski wrote: > > Le mar. 9 mars 2021 à 11:58, sebb a écrit : > > > > On Tue, 9 Mar 2021 at 01:39, Gilles Sadowski wrote: > > > > > > Le mar. 9 mars 2021 à 01:41, sebb a écrit : > > > > > > > > Most of the Commons projects show up in GitHub as having the Apache 2.0 > > > > License > > > > > > > > However a few show up as 'other': > > > > > > > > commons-codec > > > > commons-csv > > > > commons-dbutils > > > > commons-exec > > > > commons-jelly > > > > commons-logging > > > > commons-math > > > > commons-rdf > > > > commons-text > > > > commons-weaver > > > > > > > > AFAICT this is because there is more than 1 LICENSE file at the top > > > > level: > > > > I forked codec and deleted the LICENSE-header.txt file, and the > > > > license then showed up as AL 2.0. > > > > > > For commons-math, there is a single license file; so the reason > > > must be different. Perhaps the contents (several sections were > > > added below the license text)? > > > Should those be moved to the "NOTICE" file? > > > > I don't thnk that is allowed. > > NOTICE is for required attributions only. > > So, after all, do you mean that > 1. GitHub is just wrong in not detecting that CM is AL2.0, and Probably. > 2. there is nothing to change for CM No idea. Feel free to experiment in a spare branch. > ? > > Gilles > > - > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: GitHub license display confused by LICENSE-header.txt
Note that GitHub CI files are never in the root. Gary On Tue, Mar 9, 2021, 09:34 Emmanuel Bourg wrote: > +1 for moving these files elsewhere. The project root should just > contain the pom.xml, license, notice, readme and CI files. > > Emmanuel Bourg > > Le 09/03/2021 à 02:15, Melloware Inc a écrit : > > In commons beanutils we recommend using /src/conf for these type of > files. > > > > Sent from my iPhone > > > >> On Mar 8, 2021, at 8:13 PM, sebb wrote: > >> > >> On Tue, 9 Mar 2021 at 01:08, Bernd Eckenfels > wrote: > >>> > >>> Checkstyle-header.txt sounds good and maybe also moving it to a subdir > like src/build/ or src/test/? > >> > >> There are other checkstyle files at the top-level; best to keep them > together. > >> > >>> Gruss > >>> Bernd > > > - > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > >
Re: GitHub license display confused by LICENSE-header.txt
+1 for moving these files elsewhere. The project root should just contain the pom.xml, license, notice, readme and CI files. Emmanuel Bourg Le 09/03/2021 à 02:15, Melloware Inc a écrit : > In commons beanutils we recommend using /src/conf for these type of files. > > Sent from my iPhone > >> On Mar 8, 2021, at 8:13 PM, sebb wrote: >> >> On Tue, 9 Mar 2021 at 01:08, Bernd Eckenfels wrote: >>> >>> Checkstyle-header.txt sounds good and maybe also moving it to a subdir like >>> src/build/ or src/test/? >> >> There are other checkstyle files at the top-level; best to keep them >> together. >> >>> Gruss >>> Bernd - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: GitHub license display confused by LICENSE-header.txt
I think we should try and get rid of these -header.txt files, there must be a way since not all components use them. Gary On Mon, Mar 8, 2021, 19:41 sebb wrote: > Most of the Commons projects show up in GitHub as having the Apache 2.0 > License > > However a few show up as 'other': > > commons-codec > commons-csv > commons-dbutils > commons-exec > commons-jelly > commons-logging > commons-math > commons-rdf > commons-text > commons-weaver > > AFAICT this is because there is more than 1 LICENSE file at the top level: > I forked codec and deleted the LICENSE-header.txt file, and the > license then showed up as AL 2.0. > > I think it's not only GitHub that might be confused by such files, so > I suggest they are moved or renamed to avoid the confusion. They are > only needed for Checkstyle, so perhaps could be named accordingly, > e.g. checkstyle-header.txt > > [Obviously there would need to be corresponding changes in the pom and > assembly files] > > WDYT? > > Sebb. > > - > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > >
Re: GitHub license display confused by LICENSE-header.txt
Le mar. 9 mars 2021 à 11:58, sebb a écrit : > > On Tue, 9 Mar 2021 at 01:39, Gilles Sadowski wrote: > > > > Le mar. 9 mars 2021 à 01:41, sebb a écrit : > > > > > > Most of the Commons projects show up in GitHub as having the Apache 2.0 > > > License > > > > > > However a few show up as 'other': > > > > > > commons-codec > > > commons-csv > > > commons-dbutils > > > commons-exec > > > commons-jelly > > > commons-logging > > > commons-math > > > commons-rdf > > > commons-text > > > commons-weaver > > > > > > AFAICT this is because there is more than 1 LICENSE file at the top level: > > > I forked codec and deleted the LICENSE-header.txt file, and the > > > license then showed up as AL 2.0. > > > > For commons-math, there is a single license file; so the reason > > must be different. Perhaps the contents (several sections were > > added below the license text)? > > Should those be moved to the "NOTICE" file? > > I don't thnk that is allowed. > NOTICE is for required attributions only. So, after all, do you mean that 1. GitHub is just wrong in not detecting that CM is AL2.0, and 2. there is nothing to change for CM ? Gilles - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: GitHub license display confused by LICENSE-header.txt
On Tue, 9 Mar 2021 at 01:39, Gilles Sadowski wrote: > > Le mar. 9 mars 2021 à 01:41, sebb a écrit : > > > > Most of the Commons projects show up in GitHub as having the Apache 2.0 > > License > > > > However a few show up as 'other': > > > > commons-codec > > commons-csv > > commons-dbutils > > commons-exec > > commons-jelly > > commons-logging > > commons-math > > commons-rdf > > commons-text > > commons-weaver > > > > AFAICT this is because there is more than 1 LICENSE file at the top level: > > I forked codec and deleted the LICENSE-header.txt file, and the > > license then showed up as AL 2.0. > > For commons-math, there is a single license file; so the reason > must be different. Perhaps the contents (several sections were > added below the license text)? > Should those be moved to the "NOTICE" file? I don't thnk that is allowed. NOTICE is for required attributions only. > Gilles > > > [...] > > - > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: GitHub license display confused by LICENSE-header.txt
Le mar. 9 mars 2021 à 01:41, sebb a écrit : > > Most of the Commons projects show up in GitHub as having the Apache 2.0 > License > > However a few show up as 'other': > > commons-codec > commons-csv > commons-dbutils > commons-exec > commons-jelly > commons-logging > commons-math > commons-rdf > commons-text > commons-weaver > > AFAICT this is because there is more than 1 LICENSE file at the top level: > I forked codec and deleted the LICENSE-header.txt file, and the > license then showed up as AL 2.0. For commons-math, there is a single license file; so the reason must be different. Perhaps the contents (several sections were added below the license text)? Should those be moved to the "NOTICE" file? Gilles > [...] - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: GitHub license display confused by LICENSE-header.txt
In commons beanutils we recommend using /src/conf for these type of files. Sent from my iPhone > On Mar 8, 2021, at 8:13 PM, sebb wrote: > > On Tue, 9 Mar 2021 at 01:08, Bernd Eckenfels wrote: >> >> Checkstyle-header.txt sounds good and maybe also moving it to a subdir like >> src/build/ or src/test/? > > There are other checkstyle files at the top-level; best to keep them together. > >> Gruss >> Bernd >> >> >> -- >> http://bernd.eckenfels.net >> >> Von: sebb >> Gesendet: Tuesday, March 9, 2021 1:41:02 AM >> An: CommonsDev >> Betreff: GitHub license display confused by LICENSE-header.txt >> >> Most of the Commons projects show up in GitHub as having the Apache 2.0 >> License >> >> However a few show up as 'other': >> >> commons-codec >> commons-csv >> commons-dbutils >> commons-exec >> commons-jelly >> commons-logging >> commons-math >> commons-rdf >> commons-text >> commons-weaver >> >> AFAICT this is because there is more than 1 LICENSE file at the top level: >> I forked codec and deleted the LICENSE-header.txt file, and the >> license then showed up as AL 2.0. >> >> I think it's not only GitHub that might be confused by such files, so >> I suggest they are moved or renamed to avoid the confusion. They are >> only needed for Checkstyle, so perhaps could be named accordingly, >> e.g. checkstyle-header.txt >> >> [Obviously there would need to be corresponding changes in the pom and >> assembly files] >> >> WDYT? >> >> Sebb. >> >> - >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org >> For additional commands, e-mail: dev-h...@commons.apache.org >> > > - > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: GitHub license display confused by LICENSE-header.txt
On Tue, 9 Mar 2021 at 01:08, Bernd Eckenfels wrote: > > Checkstyle-header.txt sounds good and maybe also moving it to a subdir like > src/build/ or src/test/? There are other checkstyle files at the top-level; best to keep them together. > Gruss > Bernd > > > -- > http://bernd.eckenfels.net > > Von: sebb > Gesendet: Tuesday, March 9, 2021 1:41:02 AM > An: CommonsDev > Betreff: GitHub license display confused by LICENSE-header.txt > > Most of the Commons projects show up in GitHub as having the Apache 2.0 > License > > However a few show up as 'other': > > commons-codec > commons-csv > commons-dbutils > commons-exec > commons-jelly > commons-logging > commons-math > commons-rdf > commons-text > commons-weaver > > AFAICT this is because there is more than 1 LICENSE file at the top level: > I forked codec and deleted the LICENSE-header.txt file, and the > license then showed up as AL 2.0. > > I think it's not only GitHub that might be confused by such files, so > I suggest they are moved or renamed to avoid the confusion. They are > only needed for Checkstyle, so perhaps could be named accordingly, > e.g. checkstyle-header.txt > > [Obviously there would need to be corresponding changes in the pom and > assembly files] > > WDYT? > > Sebb. > > - > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
Re: GitHub license display confused by LICENSE-header.txt
Checkstyle-header.txt sounds good and maybe also moving it to a subdir like src/build/ or src/test/? Gruss Bernd -- http://bernd.eckenfels.net Von: sebb Gesendet: Tuesday, March 9, 2021 1:41:02 AM An: CommonsDev Betreff: GitHub license display confused by LICENSE-header.txt Most of the Commons projects show up in GitHub as having the Apache 2.0 License However a few show up as 'other': commons-codec commons-csv commons-dbutils commons-exec commons-jelly commons-logging commons-math commons-rdf commons-text commons-weaver AFAICT this is because there is more than 1 LICENSE file at the top level: I forked codec and deleted the LICENSE-header.txt file, and the license then showed up as AL 2.0. I think it's not only GitHub that might be confused by such files, so I suggest they are moved or renamed to avoid the confusion. They are only needed for Checkstyle, so perhaps could be named accordingly, e.g. checkstyle-header.txt [Obviously there would need to be corresponding changes in the pom and assembly files] WDYT? Sebb. - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
GitHub license display confused by LICENSE-header.txt
Most of the Commons projects show up in GitHub as having the Apache 2.0 License However a few show up as 'other': commons-codec commons-csv commons-dbutils commons-exec commons-jelly commons-logging commons-math commons-rdf commons-text commons-weaver AFAICT this is because there is more than 1 LICENSE file at the top level: I forked codec and deleted the LICENSE-header.txt file, and the license then showed up as AL 2.0. I think it's not only GitHub that might be confused by such files, so I suggest they are moved or renamed to avoid the confusion. They are only needed for Checkstyle, so perhaps could be named accordingly, e.g. checkstyle-header.txt [Obviously there would need to be corresponding changes in the pom and assembly files] WDYT? Sebb. - To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org