Installing 'update-copyright' on PQM

Vincent Ladeuil v.ladeuil+lp at free.fr
Thu Jun 24 07:38:50 BST 2010


>>>>> Martin Pool <mbp at canonical.com> writes:

    > On 24 June 2010 15:44, Martin Pool <mbp at canonical.com> wrote:
    >> As it happens an example of this just came up in Vincent's cleanup
    >> branch <https://code.edge.launchpad.net/~vila/bzr/cleanup/+merge/28306>:
    >> 
    >> On 24 June 2010 04:37, Vincent Ladeuil <v.ladeuil+lp at free.fr> wrote:
    >>> === modified file 'bzrlib/annotate.py'
    >>> --- bzrlib/annotate.py  2010-06-04 03:09:35 +0000
    >>> +++ bzrlib/annotate.py  2010-06-23 18:34:40 +0000
    >>> @@ -1,4 +1,4 @@
    >>> -# Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Canonical Ltd
    >>> +# Copyright (C) 2005-2010 Canonical Ltd
    >>>  #
    >>>  # This program is free software; you can redistribute it and/or modify
    >>>  # it under the terms of the GNU General Public License as published by
    >> 
    >> I had written some small amount of Python code for Bazaar in 2004, and
    >> other people had written things that merged into Bazaar.  I don't know
    >> off hand if any of that code still exists in annotate.py, but you
    >> probably don't know that either.  It is a bit icky that
    >> update_copyright (I presume) is changing this header even when Vincent
    >> is making no other overall changes to this file.

    > I should say that in practice, there probably isn't enough annotate
    > code from 2004 to be worth worrying about, and while I would prefer
    > Vincent merge without those changes, I wouldn't veto the patch.  But
    > it is a mark against using this code automatically.

Definitely something I should take into account. This strongly suggests
that update_copyright usage should be split:

- add *current* year only if a file is modified,

- have a mode where all files are checked and the copyright changes
  *proposed* so we can check each file individually.

I'm afraid there are already too much corner cases to hope for an
automated process there :-/

          Vincent



More information about the bazaar mailing list