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

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 <
[email protected]> 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).

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

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.

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

Thanks. Done.

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