Skip to content

Conversation

sandakersmann
Copy link
Contributor

Cryptocurrency was added to Oxford Dictionaries Online in May this year. I see no reason why the name of Satoshi’s revolutionary invention should be divided into two words. I propose that we adopt the word blockchain and merge this pull request.

Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
Block chain corrected to blockchain
@sandakersmann
Copy link
Contributor Author

If this get merged I will make a pull for bitcoin.org as well.

@gmaxwell
Copy link
Contributor

gmaxwell commented Nov 8, 2014

Nak. I'd prefer not to do this.

@sandakersmann
Copy link
Contributor Author

Ok. Any specific reason why you prefer not to do it?

@kanzure
Copy link
Contributor

kanzure commented Nov 8, 2014

Independent of whether or not this pull request is accepted, I would suggest squashing these commits into a single commit only. Normally I am a strong supporter of individual, numerous and atomic commits, but when the commit message between each never differs, there's really no strong reason to use many individual commits to accomplish what could be done with just one.

@gmaxwell
Copy link
Contributor

gmaxwell commented Nov 8, 2014

Yes, I also agree that it should be squashed... right now it's hard to review and would harm bisect.. Just didn't mention it because I don't think this should go in at all.

The source code isn't the right place to advocate a new word. This change doesn't improve clarity, but it touches a lot of files. Pedantically, I don't know how much "blockchain" makes much sense as a free standing concept; After all, you could apply some minor serialization and commitment transformations and no longer have a blockchain at all, but still have bitcoin identically functional.

@jonasschnelli
Copy link
Contributor

NACK on source-code, source-docs level.
It complicates searches.
Does not really improve the bitcoin project.

I would also say, the "blockchain" (bitcoin) is a "block chain". Block chain is more a type of trusted storage.

@sandakersmann
Copy link
Contributor Author

Blockchain is not really a new word. It is used along with block chain and block-chain all over the repo. This is a move to establish a standard.

@gmaxwell
Copy link
Contributor

gmaxwell commented Nov 8, 2014

The code is pretty consistent currently; it's hyphenated in four places. The Bitcoin core source isn't the place to establish word standards (esp not by making dozens of commits.)

@sandakersmann
Copy link
Contributor Author

Ok. I see your point about squashing these commits into one. Just need to do some research on how to do that.

@jgarzik
Copy link
Contributor

jgarzik commented Nov 8, 2014

This is known colloquially as urinating into the wind.

The same nerds got twisted about when "e-mail" became "email".

"blockchain" is just fine, and appears to be coming into common usage.

@gmaxwell
Copy link
Contributor

gmaxwell commented Nov 8, 2014

I don't think anyone is complaining about the word, just the changes. :)

@sandakersmann
Copy link
Contributor Author

Squashing these commits into one in #5248

@jgarzik
Copy link
Contributor

jgarzik commented Nov 8, 2014

For future reference, please do not create a new pull request for the same change. That just makes it more difficult to track previous comments.

Typically, after squashing, you push the new branch to the github branch associated with the pull request using "git push --force"

Meta: We should put this in a doc somewhere...

@sandakersmann
Copy link
Contributor Author

Ok, I will research that. I have just been using Firefox until today. Steep learning curve :)

@fanquake fanquake mentioned this pull request Sep 12, 2015
@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.

5 participants