Here's what I propose to use as a header in all files. Any comments / suggestions?
Bruno
Afficher les réponses par date
On 2011-04-07, at 1:31 PM, Bruno Dufour wrote:
Here's what I propose to use as a header in all files. Any comments / suggestions?
Bruno
Nice! Here are my comments.
1) The text of the license is fine, except a reference to <organization> which you should change to Universite de Montreal, and "This file is part of the Tachyon project" -> "This file is part of the Tachyon JavaScript project" (to be specific).
2) Remove the accents from Universite de Montreal (it will trip some editors that don't do Unicode). Our sources should be ASCII everywhere.
3) The box is too wide... Currently it is 80 characters wide and that will cause line breaks on most terminals. We don't need to close the box on the right side. That way we can stay under 75 characters wide, so that a "diff" will be also less than 80 characters wide (diff often adds a 2 character prefix).
4) We need a version for other languages too, in particular shell scripts.
Marc
On 2011-04-07, at 13:54 , Marc Feeley wrote:
On 2011-04-07, at 1:31 PM, Bruno Dufour wrote:
Here's what I propose to use as a header in all files. Any comments / suggestions?
Bruno
Nice! Here are my comments.
The text of the license is fine, except a reference to <organization> which you should change to Universite de Montreal, and "This file is part of the Tachyon project" -> "This file is part of the Tachyon JavaScript project" (to be specific).
Remove the accents from Universite de Montreal (it will trip some editors that don't do Unicode). Our sources should be ASCII everywhere.
The box is too wide... Currently it is 80 characters wide and that will cause line breaks on most terminals. We don't need to close the box on the right side. That way we can stay under 75 characters wide, so that a "diff" will be also less than 80 characters wide (diff often adds a 2 character prefix).
We need a version for other languages too, in particular shell scripts.
Here's the new set of licenses with all points addressed. The maximum width in now 76 chars.
Bruno
On 2011-04-07, at 15:04 , Marc Feeley wrote:
On 2011-04-07, at 2:10 PM, Bruno Dufour wrote:
Here's the new set of licenses with all points addressed. The maximum width in now 76 chars.
Bruno
There remain references to <COPYRIGHT HOLDER> which should be changed to Universite de Montreal.
Sorry, not sure how I keep missing these. Hopefully this set is fine.
Bruno
On 2011-04-07, at 3:04 PM, Marc Feeley wrote:
On 2011-04-07, at 2:10 PM, Bruno Dufour wrote:
Here's the new set of licenses with all points addressed. The maximum width in now 76 chars.
Bruno
There remain references to <COPYRIGHT HOLDER> which should be changed to Universite de Montreal.
Marc
It occurred to me that, to make things clear, we should prefix the license with a brief summary "This software is licensed under the following license (Modified BSD License):".
Marc
The latest version of the header has been added to all source files on github (only the json files were excluded). Everything seems to work for me, but let me know if you encounter any issue.
Bruno
On 2011-04-07, at 15:18 , Bruno Dufour wrote:
On 2011-04-07, at 15:15 , Marc Feeley wrote:
It occurred to me that, to make things clear, we should prefix the license with a brief summary "This software is licensed under the following license (Modified BSD License):".
Like this?
Bruno
<license.js><license.sh><license.scm>_______________________________________________ Tachyon-list mailing list Tachyon-list@iro.umontreal.ca https://webmail.iro.umontreal.ca/mailman/listinfo/tachyon-list
-
Bruno Dufour -
Marc Feeley