Skip to content

Conversation

@fanquake
Copy link
Member

Update header licenses in crypter.cpp and crypter.h

@Diapolo
Copy link

Diapolo commented May 17, 2012

Can you plase also fix this in the crypter.cpp and use a more detailed commit-message? If you do so I'm fine with this.

@fanquake
Copy link
Member Author

Done

@Diapolo
Copy link

Diapolo commented May 17, 2012

Good, now if you merge them into 1 commit everything is perfect :).
"git rebase origin -i" helps :).

@laanwj
Copy link
Member

laanwj commented May 17, 2012

I don't want to be a party pooper but uhm... the file is called COPYING:

https://github.com/bitcoin/bitcoin/blob/master/COPYING

I cannot find a license.txt anywhere in the repos.

@fanquake
Copy link
Member Author

My mistake..
So this means all the other headers are wrong? And should instead be changed to COPYING?

@laanwj
Copy link
Member

laanwj commented May 17, 2012

I just noticed that the file is copied as license.txt on win32 for the gitian build.

...confusing... Then again, I suppose the name in git is leading.

@Diapolo
Copy link

Diapolo commented May 17, 2012

Then it's wrong in ALL other headers and cpp files ;).

@fanquake
Copy link
Member Author

Will make a new pull for this, updating the license header in all the necessary files.

@fanquake fanquake closed this May 18, 2012
suprnurd pushed a commit to chaincoin-legacy/chaincoin that referenced this pull request Dec 5, 2017
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Jan 22, 2019
"Formatting" a bunch of python test files
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request May 6, 2020
bf74c6f [Cleanup] Remove stale UNITTEST network (Fuzzbawls)

Pull request description:

  This network is un-necessary as our unit test suite can simply be run
  with MainNet's parameters, which is basically what we already do.

  Minor conflict with bitcoin#1328 due to that PR having already removed one of the unused methods.

ACKs for top commit:
  random-zebra:
    utACK bf74c6f
  furszy:
    utACK bf74c6f and merging..

Tree-SHA512: 1b8c7b4178e2dc8d60fe89b4ce4d4ad476f433ced19d981e1d3e1043fd8468e269600fcbe30ee04f7954250a183f2bc34486ad9b37fd7d78aed828c47d662d77
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request May 6, 2020
…hainparams

47f552d [Cleanup] Remove MineBlocksOnDemand function in chainparams (random-zebra)

Pull request description:

  The only network where fMineBlocksOnDemand is true is regtest (unittest network was removed in bitcoin#1330 )
  Just use IsRegTestNet function.

ACKs for top commit:
  furszy:
    utACK 47f552d

Tree-SHA512: 95bc644532675a55098e1836286b15c462221879b25018c0f89c3c9bcd626556b928d65641996ff41aad34460c2f1940390479575e4845aa43e742409832c1a0
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants