update 2012 copyrights to 2013 #2251

pull colindean wants to merge 1 commits into bitcoin:master from colindean:doc/copyright-update-2013 changing 111 files +171 −171
  1. colindean commented at 3:07 AM on January 31, 2013: contributor
    • add 2013 where there was only 2012

    Notable omissions from updates:

    • contrib/bitcoind.bash-completion -- no copyright by The Bitcoin developers
    • contrib/testgen/gen_base58_test_vectors.py -- no copyright by The Bitcoin developers
    • src/addrman.cpp -- no copyright by The Bitcoin developers
    • src/addrman.h -- no copyright by The Bitcoin developers
    • src/leveldb/* -- I figure this can be updated by pulling in upstream code

    Fixes #2217.

  2. update 2012 copyrights to 2013
    * add 2013 where there was only 2012
    
    Notable omissions from updates:
    
    * contrib/bitcoind.bash-completion -- no copyright by The Bitcoin developers
    * contrib/testgen/gen_base58_test_vectors.py -- no copyright by The Bitcoin developers
    * src/addrman.cpp -- no copyright by The Bitcoin developers
    * src/addrman.h -- no copyright by The Bitcoin developers
    * src/leveldb/* -- I figure this can be updated by pulling in upstream code
    
    Fixes #2217.
    2d36e02773
  3. colindean commented at 3:08 AM on January 31, 2013: contributor

    For the omissions, can obviously update them to increment the years or add The Bitcoin developers where necessary. Your wish is my command, haha.

  4. Diapolo commented at 6:48 AM on January 31, 2013: none

    Sorry, but this does not fix #2217 completely, as I intended to add copyright info to all Qt files also and harmonize the used format to that of the core.

    I'm also not sure about changing translations here, as they need to go to Transifex then and overwrite the current state there. Please leave that out IMHO.

  5. in src/qt/forms/aboutdialog.ui:None in 2d36e02773
      90 | @@ -91,7 +91,7 @@
      91 |          <cursorShape>IBeamCursor</cursorShape>
      92 |         </property>
      93 |         <property name="text">
      94 | -        <string>Copyright © 2009-2012 The Bitcoin developers</string>
      95 | +        <string>Copyright © 2009-2013 The Bitcoin developers</string>
    


    Diapolo commented at 6:52 AM on January 31, 2013:

    I think it makes sense to split this string with another pull, so that no translator ever has to touch this again. Perhaps Copyright © 2009-2012 (untranslatable and so easily updatable every year) and The Bitcoin developers (translatable by our translators). @laanwj What do you think?


    luke-jr commented at 7:26 AM on January 31, 2013:

    Move it into code as _("Copyright © 2009-%d The Bitcoin developers") or similar?


    Diapolo commented at 7:32 AM on January 31, 2013:

    I'm sure we should avoid %d and such things in translatable strings. A translator could break it entirely by making a fault there.


    laanwj commented at 1:00 PM on January 31, 2013:

    I think Luke's solution is fine; there are more instances of string interpolation chars in translated strings already. Do use %1 instead of %d (and qt's fmt function)

    Agree with Diapolo that non english translation files should not be changed in this pull, they are always sourced from transifex.

  6. colindean commented at 10:43 PM on January 31, 2013: contributor

    Shall I

    1. Back out the changes to the translation files other than English?
    2. Find where @luke-jr's suggestion should be inserted into the code and change the English translation to match it?
    3. Add the copyright notice to all of the Qt files?

    I unfortunately am going out of the country for the next week and won't likely be near an Internet connection until the evening of Thursday, 2/7. If whatever action is recommended is timely (I don't recall when 0.8 is due, or if this is even fodder for it), someone else is welcome to take my work and build on it.

  7. Diapolo commented at 6:10 AM on February 1, 2013: none

    I suggest to remove all changes to translation files (and the .ui file) and if you want add our copyright header to all Qt files. I'm going to implement Lukes suggestion.

  8. gavinandresen commented at 3:20 PM on February 7, 2013: contributor

    Relevant: http://stackoverflow.com/questions/3487007/when-to-update-the-year-in-open-source-copyright-notice

    Updating every single copyright is not the right thing to do.

    And there is a good reason NOT to: it messes up the change history.

  9. colindean commented at 1:25 AM on February 10, 2013: contributor

    That makes a whole lot more sense.

    I guess then I/we need to determine what files have changed thus far in 2013 and simply ensure that those have had the copyright updated.

    Do I correctly assess this?

  10. gavinandresen commented at 3:08 PM on February 11, 2013: contributor

    See: http://www.contentious.com/2007/01/07/copyright-notice-is-the-year-really-necessary/

    In particular:

    "If the work is a derivative work or a compilation incorporating previously published material, the year date of first publication of the derivative work or compilation is sufficient."

    I say we treat edits to source code files as creating derivative works, and leave the copyright year alone.

    If you create a new source file then put the correct copyright date on it.

    Simply changing the year from (say) 2010 to 2013 is definitely the WRONG thing to do. Doing nothing most of the time seems to me to be the right thing to do, and has the HUGE advantage of also being the easiest thing to do.

  11. colindean commented at 8:04 PM on February 11, 2013: contributor

    So the resolution is not to do anything, and simply have some kind of a policy to update the copyright information when a file is updated?

  12. laanwj commented at 8:55 PM on February 11, 2013: member

    Even better, he's proposing to only update the year on creating a new file.

  13. colindean commented at 1:04 AM on February 12, 2013: contributor

    Ah, OK. Upon re-reading, I understand that sentiment better.

    If it's so resolved, well, I guess this can be closed!

  14. colindean closed this on Feb 12, 2013

  15. colindean deleted the branch on Feb 16, 2013
  16. DrahtBot locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-15 15:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me