Remove ALL remaining references to X11 licence #5470

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:x11_removal changing 189 files +189 −189
  1. fanquake commented at 7:03 AM on December 13, 2014: member

    This just leaves us the rest of the X11 license mentions in /src, which will continue to be removed as code is changed.

  2. laanwj commented at 9:08 AM on December 15, 2014: member

    I'd say let's just remove all of them from the tree at once.

  3. laanwj added the label Improvement on Dec 15, 2014
  4. fanquake commented at 9:12 AM on December 15, 2014: member

    I had considered that, but didn't think we'd want to touch a few hundred files at once. Will update this pull.

    On Monday, December 15, 2014, Wladimir J. van der Laan < notifications@github.com> wrote:

    I'd say let's just remove all of them from the tree at once.

    Reply to this email directly or view it on GitHub #5470 (comment).

  5. fanquake force-pushed on Dec 15, 2014
  6. fanquake force-pushed on Dec 15, 2014
  7. laanwj commented at 7:28 AM on December 16, 2014: member

    It is a big change but also easy to validate. e.g

    checkout fanquake/x11_removal~1
    git ls-files -z | xargs -0 sed -i 's/MIT\/X11/MIT/g'
    git diff fanquake/x11_removal
    

    Looks like you forgot the following files:

     src/bitcoin-cli-res.rc
     src/bitcoind-res.rc
     src/qt/res/bitcoin-qt-res.rc
     src/test/bctest.py
     src/test/bitcoin-util-test.py
     src/univalue/gen.cpp
     src/univalue/univalue.cpp
     src/univalue/univalue.h
     src/univalue/univalue_read.cpp
     src/univalue/univalue_write.cpp
    
  8. fanquake commented at 7:32 AM on December 16, 2014: member

    I left the remaining .py and .cpp files because they have the BitPay copyright header, was going to ask about just changing that?

    As for the .rc files, wasn't sure wether they were generated somehow, hadn't had a chance to check yet.

  9. fanquake renamed this:
    Remove remaining references to X11 licence from contrib/doc/qa
    Remove remaining references to X11 licence
    on Dec 16, 2014
  10. laanwj commented at 7:52 AM on December 16, 2014: member

    Just change it all, really all. The .rc files are not generated, so they too.

  11. Remove references to X11 licence 78253fcbad
  12. fanquake force-pushed on Dec 16, 2014
  13. fanquake commented at 7:57 AM on December 16, 2014: member

    Thanks. Done.

  14. laanwj commented at 8:12 AM on December 16, 2014: member

    ACK commithash 78253fcbad69b28f1c0c3cfc7f8dcaecbb68c390 pull 5470 (verified using the above script, no output) (signature)

  15. laanwj renamed this:
    Remove remaining references to X11 licence
    Remove ALL remaining references to X11 licence
    on Dec 16, 2014
  16. laanwj merged this on Dec 19, 2014
  17. laanwj closed this on Dec 19, 2014

  18. laanwj referenced this in commit 111a7d45f1 on Dec 19, 2014
  19. fanquake deleted the branch on May 12, 2016
  20. 0xartem referenced this in commit 4fa0836e61 on Jun 1, 2018
  21. 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-13 21:15 UTC

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