On 04/09/2010 05:31 PM, yuppie wrote: > Hi! > > > Tres Seaver wrote: >> yuppie wrote: >>> Tres Seaver wrote: >>> So depending on a layout difference (line break or not) it produces >>> copyright headers with different content. >>> >>> Doesn't make any sense to me, but the people I already asked don't care. >> >> The "All Rights Reserved" is a little senseless, since the following >> language turns around and says that the files are covered by the ZPL. > > I'm not opposed to removing that phrase. What I don't like is the fact > that it is removed randomly based on a bug in a script that is made for > a different purpose.
Right. I'd consider that behaviour not a bug, but actually point out that all the examples in documentations that I have found considered the copyright to be stand-alone on a single line with the next line following to carry the "All rights reserved" remark. Although the tool screws up, the output of the tool really is intended to be reviewed by the person who does the checkin. > I think a change like that should be based on an official policy change, > including new coding style guidelines: > http://docs.zope.org/zopetoolkit/codingstyle/python-style.html#license-statement-module-docstring I'd move this off from the topic of the tool being broken: if either of you would like to follow up on removing the "all rights reserved" from the header, then please acknowledge that and I'll pick it up (probably that needs to go to the foundation?!? I bet some lawyer wanted that phrase to be in there originally and I guess we have to ask the ZF board whether that part of the policy can be changed.) Christian -- Christian Theune · c...@gocept.com gocept gmbh & co. kg · forsterstraße 29 · 06112 halle (saale) · germany http://gocept.com · tel +49 345 1229889 0 · fax +49 345 1229889 1 Zope and Plone consulting and development _______________________________________________ Zope-Dev maillist - Zope-Dev@zope.org https://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - https://mail.zope.org/mailman/listinfo/zope-announce https://mail.zope.org/mailman/listinfo/zope )