Put back inadvertently removed copyright notices #11318

pull gmaxwell wants to merge 1 commits into bitcoin:master from gmaxwell:fix_copying changing 1 files +1 −0
  1. gmaxwell commented at 7:30 am on September 13, 2017: contributor

    In an abundance of caution this restores “Bitcoin Developers” to the COPYING file in case there were contributors before that point in time that would object to the current label. It’s harmless and more pedantically correct.

    (Change extracted from the Bitcoin-abc github)

  2. Put back inadvertently removed copyright notices
    In an abundance of caution this restores "Bitcoin Developers" to the COPYING file in
    case there were contributors before that point in time that would object to the
    current label.  It's harmless and more pedantically correct.
    
    [Change extracted from the Bitcoin-abc repository, commit message by gmaxwell]
    
    Signed-off-by: Gregory Maxwell <greg@xiph.org>
    d552ed678c
  3. fanquake added the label Docs and Output on Sep 13, 2017
  4. gmaxwell commented at 7:55 am on September 13, 2017: contributor
    FWIW, I contributed under the original listing. I did not like the project rename and I personally have a general preference for the original attribution.
  5. chriswheeler commented at 10:23 am on September 13, 2017: contributor

    The original Bitcoin software released by Satoshi included Satoshi Nakamoto in the license.txt and source code files.

    e.g. https://github.com/trottier/original-bitcoin/blob/master/license.txt

    Should this be restored too?

    Further, it is my understanding that copyright notices should be applied to every file - should all file headers be updated with the correct copyright notices?

  6. awitherow commented at 10:48 am on September 13, 2017: none
    @chriswheeler is there any reason WHY BTC should have changed such fundamentals in the first place?
  7. meshcollider commented at 10:59 am on September 13, 2017: contributor
    @chriswheeler @awitherow I believe Satoshi himself changed the text to Bitcoin Developers while the code was hosted on sourceforge: https://sourceforge.net/p/bitcoin/code/201/
  8. chriswheeler commented at 11:10 am on September 13, 2017: contributor

    I have just read the related discussion here: https://github.com/Bitcoin-ABC/bitcoin-abc/pull/89

    It does appear (thanks @MeshCollider) that Satoshi changed the notice to “Bitcoin Developers”.

    I still believe that this change should be made to all files, not just the COPYING file.

  9. Leviathn commented at 4:20 pm on September 13, 2017: none
    ACK
  10. gmaxwell commented at 8:37 pm on September 13, 2017: contributor

    The original Bitcoin software released by Satoshi included Satoshi Nakamoto in the license.txt and source code files.

    He changed it to the text my change includes himself.

  11. sipa commented at 10:28 pm on September 13, 2017: member
    @chriswheeler Which other files are wrong? This mistake was introduced https://github.com/bitcoin/bitcoin/pull/6135/commits/3b00e7c231648cfe0c36c5f9f0c44953146c1a5b, so it seems like a sole instance.
  12. awitherow commented at 9:45 am on September 14, 2017: none

    Why is it possible to copyright this to the Bitcoin developers in general?

    Is there an actual copyright?

    Why has Satoshi been removed in general because it was his original creation?

  13. meshcollider commented at 9:48 am on September 14, 2017: contributor
    @awitherow as I and others pointed out above, Satoshi changed it himself
  14. awitherow commented at 10:49 am on September 14, 2017: none

    It just was not clear to me due to insufficient grammar. Thank you for the clarification @MeshCollider .

    Still doesn’t answer the question if there is an actual copyright existing.

  15. meshcollider commented at 10:57 am on September 14, 2017: contributor
    @awitherow what do you mean by an “actual copyright”? Yes, the code is licensed under the MIT License and copyright to the bitcoin developers / core developers, as stated in https://github.com/bitcoin/bitcoin/blob/master/COPYING
  16. laanwj commented at 2:46 pm on September 14, 2017: member

    FWIW, I contributed under the original listing. I did not like the project rename and I personally have a general preference for the original attribution.

    We could rename the project back, but I don’t think it’s worth the hassle. ACK anyhow.

  17. laanwj commented at 2:50 pm on September 14, 2017: member

    @chriswheeler Which other files are wrong? This mistake was introduced 3b00e7c, so it seems like a sole instance.

    I think he means the copyright in individual source files, one could re-add the (C) Bitcoin developers there too, for the files that existed before the rename or with code moved from files that existed before the rename (with the respective date range of before the project rename). Would be quite a lot of work tho.

  18. gmaxwell commented at 11:39 pm on September 15, 2017: contributor

    We could rename the project back, but I don’t think it’s worth the hassle.

    I agree. In any case, I agree with the suggestion that the headers get updated everywhere. But I don’t see a problem with making that another PR. I’d prefer to do a change that touches every file later in the development cycle when we’re not backporting everything to 0.15.x still. (unless someone argues otherwise).

    I’d prefer to not delay an improvement everyone agrees with (I think?) while waiting on a change that is going to touch 300 files. This change can also just apply to the 0.15 branch, not so much for a all-files toucher.

    Keep in mind, no copyright holder has ever complained here (beyond, I suppose, my comment above that I also prefer the change… and it’s not like I really matter.) so this is lower urgency.

  19. luke-jr commented at 0:14 am on September 16, 2017: member

    @awitherow Whenever someone writes code, it is automatically copyrighted by them, and remains so until they assign it to someone else. There is no “Bitcoin developers” or “Bitcoin Core developers” legal entity - these are just English phrases that mean each contributor retains copyright to his own work.

    It’s redundant, but harmless, to have both lines present, since they mean the same thing.

  20. gmaxwell commented at 6:29 pm on September 27, 2017: contributor
    ping
  21. jonasschnelli commented at 3:01 am on September 28, 2017: contributor
    ACK. The “Core” suffix may sound nice, but it suddenly created a gap between Bitcoin developers and “Bitcoin Core” developers where some people think the later is an organisation (which is obviously not true).
  22. laanwj commented at 6:00 pm on September 28, 2017: member

    The “Core” suffix may sound nice, but it suddenly created a gap between Bitcoin developers and “Bitcoin Core” developers where some people think the later is an organisation (which is obviously not true).

    Yeah - it was intended to create a distinction between bitcoin the currency, and the node software, as that was confusing people at the time. Of course, it is misinterpreted in the most horrible and creative ways.

  23. laanwj merged this on Sep 28, 2017
  24. laanwj closed this on Sep 28, 2017

  25. laanwj referenced this in commit a72003d794 on Sep 28, 2017
  26. chriswheeler commented at 7:41 pm on September 28, 2017: contributor
    One final thought on this - If it’s reverted, isn’t that violating the rights of the people who contributed under the ‘Bitcoin Core Developers’ notice? I’m not a copyright lawyer, but I would advise getting some advice from someone who is before making changes to copyright notices without the consent of everyone who contributed.
  27. achow101 commented at 7:48 pm on September 28, 2017: member
    @chriswheeler the “Bitcoin Core developers” notice was not removed. A “The Bitcoin Developers” notice was added
  28. chriswheeler commented at 8:12 pm on September 28, 2017: contributor
    Oops. I should learn to read the diff better :) Sorry all.
  29. MarcoFalke referenced this in commit 915dd7c184 on Oct 3, 2017
  30. MarcoFalke referenced this in commit 1d76fe8c16 on Oct 3, 2017
  31. MarcoFalke referenced this in commit 8b61aee690 on Oct 3, 2017
  32. codablock referenced this in commit 3b0df3422e on Sep 25, 2019
  33. barrystyle referenced this in commit 388ce49a69 on Jan 22, 2020
  34. 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-02-17 06:13 UTC

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