Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Its long since time we removed satoshi from the developers list #506

Merged
merged 1 commit into from Aug 8, 2014

Conversation

Projects
None yet
6 participants
Contributor

TheBlueMatt commented Aug 5, 2014

Satoshi hasn't done anything in a long time. For a list of people who should be considered active developers, he clearly does not belong.

Contributor

saivann commented Aug 5, 2014

FWIW, people often refered to this list for Satoshi's PGP key, and it seems like his email was removed to imply he was inactive implicitly. Do Satoshi still have push access to the repository? As far as I'm concerned, I'm OK both with keeping or removing Satoshi from the list.

Contributor

TheBlueMatt commented Aug 5, 2014

No one should be encrypting to satoshi's PGP key these days anyway, he should be considered gone, IMO.

Contributor

saivann commented Aug 5, 2014

@TheBlueMatt I think it was in order to verify Satoshi's signature in the future rather than encrypting with his PGP keys. But I agree this provides few value here beside historical purpose.

Contributor

gmaxwell commented Aug 5, 2014

I have some mild reason to believe that satoshi doesn't have access to that pgp key anymore. Though if we wanted to move it to a legacy section, that would also be sensible.

Contributor

saivann commented Aug 5, 2014

LGTM. In the absence of critical feedback, this pull request will be merged on August 5th.

jgarzik commented Aug 5, 2014

I think it is quite lame to just remove him like that.

Contributor

saivann commented Aug 5, 2014

@jgarzik Any suggestion regarding how the list should be managed / how to deal with this specific question?

Contributor

luke-jr commented Aug 5, 2014

Note this isn't removing the key, just the list entry. People can still link to the key just fine.

Contributor

TheBlueMatt commented Aug 5, 2014

Yes, sorry, I mentioned this on IRC but not here. This very deliberately does not remove his key file, just the list entry.

Contributor

gmaxwell commented Aug 5, 2014

How about above the list, "The Bitcoin system is the invention and design of Satoshi Nakamoto <email/pgp> who first described it a whitepaper (whitepaper link) and in an original feature complete implementation (link to first release?). Satoshi is no longer active in Bitcoin but the codebase he began is now maintained and developed by a community of volunteers."

Contributor

saivann commented Aug 5, 2014

A simple

tag above the list would look as follow ("Satoshi" could link to the PGP key, and "whitepaper" to the pdf, perhaps)

capture du 2014-08-04 21 31 05

Contributor

instagibbs commented Aug 5, 2014

"The Bitcoin system is the invention and design of Satoshi Nakamoto who first described it in a whitepaper and an original feature-complete implementation. Satoshi is no longer active in Bitcoin but the codebase he began is now maintained and developed by a community of volunteers."

Couple of grammar edits, but I like the gist. Could even be below the list.

Contributor

TheBlueMatt commented Aug 5, 2014

I disagree with linking to satoshi's pgp key anywhere, but in general that LGTM. Maybe an even shorter "The Bitcoin system was originally designed by Satoshi Nakamoto. Their [or his, depending on your choice of pronoun for satoshi] original Bitcoin codebase is now being maintained as Bitcoin Core by the following developers and a community of volunteers." (note that laanjw gets paid for his work on Bitcoin Core, so he's not a volunteer)

Contributor

gmaxwell commented Aug 5, 2014

@TheBlueMatt Contributors who are paid still serve as volunteers (they might be paid to work on Bitcoin stuff, but they are still contributing as individuals), but sure easier to avoid some confusion.

Contributor

luke-jr commented Aug 5, 2014

FWIW, I vote to just delete the list entirely. It's too focused on one codebase and causes more problems than it solves.

Edit: Nevermind, apparently some people see this as useful enough to keep.

jgarzik commented Aug 5, 2014

The URLs to the whitepaper and satoshi PGP key should remain working for all eternity.

Contributor

saivann commented Aug 5, 2014

@jgarzik This pull request isn't removing /satoshinakamoto.asc and /bitcoin.pdf , so no link should be broken. Are you OK with the last suggestion by @TheBlueMatt ?

Here is a live preview and branch:
http://bitcointest1.us.to/en/development
https://github.com/bitcoin/bitcoin.org/commits/satoshi

Contributor

saivann commented Aug 6, 2014

For the record, I am not merging the pull request in order to leave some time to @jgarzik to comment on the last suggestions.

Contributor

saivann commented Aug 7, 2014

In the absence of critical feedback, I will merge changes in the commits and live preview displayed below on August 8th (@TheBlueMatt, feel free to update your branch).

http://bitcointest1.us.to/en/development
https://github.com/bitcoin/bitcoin.org/commits/satoshi

Contributor

TheBlueMatt commented Aug 7, 2014

"being maintained by these project maintainers" sounds very redundant. Maybe "is now maintained as Bitcoin Core by the following individuals as well as a community or volunteers"

Matt

On August 7, 2014 10:46:29 AM PDT, saivann notifications@github.com wrote:

In the absence of critical feedback, I will merge changes in the
commits and live preview displayed below on August 8th (@TheBlueMatt,
feel free to update your branch).

http://bitcointest1.us.to/en/development
https://github.com/bitcoin/bitcoin.org/commits/satoshi


Reply to this email directly or view it on GitHub:
bitcoin#506 (comment)

Contributor

saivann commented Aug 7, 2014

@TheBlueMatt Then let's just use "developers", this last change was suggested by @jgarzik, but both sound good to me.

Contributor

TheBlueMatt commented Aug 7, 2014

Sure, "is now maintained as Bitcoin Core by the following developers as well as a community or volunteers" Sgtm

On August 7, 2014 4:21:17 PM PDT, saivann notifications@github.com wrote:

@TheBlueMatt Then let's just use "developers", this last change was
suggested by @jgarzik, both sound good to me.


Reply to this email directly or view it on GitHub:
bitcoin#506 (comment)

@saivann saivann merged commit c5eda37 into bitcoin-dot-org:master Aug 8, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment