As there is a String field (SubVersionNum) I think it would be better not to increment the protocol version along with the client version number. It makes creating alternative clients easier.
Make the version number the protocol version and not the client version #5
issue cdecker opened this issue on December 19, 2010-
cdecker commented at 8:06 PM on December 19, 2010: contributor
-
tcatm commented at 1:25 AM on February 6, 2011: none
Anyone thoughts on this? It's now one month old. I'll close it in a few days if no one comments on this.
-
cdecker commented at 3:03 PM on February 6, 2011: contributor
Well we do have 4 upvotes, so it is something that has some interest. Generally we should distinguish better between the Protocol and Satoshis client.
-
tcatm commented at 6:16 PM on February 6, 2011: none
Can you make a patch?
-
cdecker commented at 6:19 PM on February 6, 2011: contributor
It's not really something you can "patch", it's all about not increasing the version number in the version message if the update does not break backward compatibility. instead of using the version number use the version string to identify the client and client version.
-
tcatm commented at 6:24 PM on February 6, 2011: none
Well, a patch would at least add a constant PROTOCOL_VERSION = xyz, wouldn't it?
-
cdecker commented at 6:46 PM on February 6, 2011: contributor
I'll look into it ^^ How should I submit it?
-
tcatm commented at 6:53 PM on February 6, 2011: none
pull request :)
-
cdecker commented at 6:21 PM on February 10, 2011: contributor
Ok, submitted the pull request. Please review :D It's not much but this is more of a design change than a code change ^^
-
jgarzik commented at 7:02 AM on February 13, 2011: contributor
This seems to be opposite of satoshi's intent.
-
cdecker commented at 9:22 AM on February 13, 2011: contributor
Comments moved to pull request.
- laanwj referenced this in commit 679592720a on Sep 18, 2011
- devurandom cross-referenced this on Dec 1, 2011 from issue bitcoin-qt crashes in db on startup by devurandom
- dooglus cross-referenced this on Dec 3, 2011 from issue RPC 'sendfrom' with invalid address crashes bitcoind by dooglus
- luke-jr cross-referenced this on Mar 25, 2012 from issue Startup blocks on mutex at "Loading Addresses" by luke-jr
- gavinandresen cross-referenced this on Apr 8, 2012 from issue addrMan crash on CentOS by gavinandresen
- luke-jr cross-referenced this on Apr 28, 2012 from issue Addrman crash by luke-jr
- rebroad cross-referenced this on May 2, 2012 from issue bitcoin-qt usually SEGVs when exited using unix signals by rebroad
- wizeman cross-referenced this on May 14, 2012 from issue NULL ptr deref when press Ctrl+C right after starting bitcoind by wizeman
- grarpamp cross-referenced this on Jun 16, 2012 from issue detachdb - coredump backtrace by grarpamp
- grarpamp cross-referenced this on Jun 22, 2012 from issue Boost exceptions in ReadConfigFile() not caught by grarpamp
- rebroad cross-referenced this on Mar 21, 2013 from issue ReserveKeyFromKeyPool(): read failed by rebroad
- dvdkhlng cross-referenced this on Mar 28, 2013 from issue Bitcoind aborts on startup: ./db/skiplist.h:152 Assertion `n >= 0' failed. by dvdkhlng
- wtogami cross-referenced this on May 27, 2013 from issue bitcoin-qt hangs during shutdown by wtogami
- sipa referenced this in commit 21b74c7e5f on Jun 2, 2013
- nelisky referenced this in commit a9c69f2a26 on Aug 8, 2013
- sje397 cross-referenced this on Aug 27, 2013 from issue [QT] Add network traffic graph to debug window by sje397
- kac- referenced this in commit e3944f5779 on Mar 25, 2014
- justmoon referenced this in commit 2e9eab1867 on Apr 7, 2014
- kac- referenced this in commit 0dc2b71ed0 on Apr 22, 2014
- gmaxwell cross-referenced this on Jun 9, 2014 from issue CBasicKeyStore::AddCScript() : redeemScripts > 520 bytes are invalid by gmaxwell
- pstratem cross-referenced this on Jun 15, 2014 from issue libsecp256k1 integration by theuni
- rebroad cross-referenced this on Jun 25, 2014 from issue bitcoin-qt core dumping by rebroad
- ahmedbodi referenced this in commit 71bdd18af3 on Aug 6, 2014
- rebroad cross-referenced this on Sep 11, 2014 from issue . by rebroad
- imharrywu cross-referenced this on Sep 28, 2014 from issue null pointer of pindexPrev crash. by imharrywu
- kac- referenced this in commit 35f0a4e2e9 on Oct 8, 2014
- rdponticelli referenced this in commit 50713b0e64 on Nov 26, 2014
- eduffield222 cross-referenced this on Jan 23, 2015 from issue Code vulnerable to segfault after a network fork by eduffield222
- lecbee cross-referenced this on Feb 16, 2015 from issue core dumped when closing bitcoin-qt by lecbee
- rustyrussell cross-referenced this on Mar 26, 2015 from issue Add autoprune functionality by sdaftuar
- laanwj referenced this in commit e688d9edb0 on Mar 26, 2015
- dexX7 referenced this in commit abac370c4b on May 20, 2015
- primezeta cross-referenced this on Jun 24, 2015 from issue Segmentation fault on bitcoind or bitcoin-qt startup (64bit Linux) by primezeta
- seigneur cross-referenced this on Jul 7, 2015 from issue After upgrading to 0.10.2 bitcoind hangs at Opened LevelDB successfully by seigneur
- TheBlueMatt cross-referenced this on Jul 9, 2015 from issue connection limit calculation for select() is overly ambitious with -txindex by TheBlueMatt
- destenson referenced this in commit 280d6c0583 on Jun 26, 2016
- sdaftuar cross-referenced this on Aug 4, 2016 from issue Mempool DoS risk in segwit due to malleated transactions by petertodd
- MarcoFalke referenced this in commit e504cff291 on Dec 9, 2016
- MarcoFalke referenced this in commit 8501bedd75 on Dec 9, 2016
- Celean cross-referenced this on Mar 8, 2017 from issue v0.14.0 shutdown hangs by Celean
- gmaxwell cross-referenced this on Mar 15, 2017 from issue Running out of disk space during GUI init results in hang and non-drawing windows by gmaxwell
- ptschip referenced this in commit f5b8a11780 on May 1, 2017
- gmaxwell cross-referenced this on May 22, 2017 from issue Switch chainstate db and cache to per-txout model by sipa
- kleetus referenced this in commit 1ab2dda4b2 on Jun 13, 2017
- AkioNak referenced this in commit a593ffa0c9 on Jul 24, 2017
- sipa cross-referenced this on Aug 1, 2017 from issue Update to latest Bitcoin patches for LevelDB by sipa
- laanwj referenced this in commit b13a68e129 on Aug 2, 2017
- laanwj referenced this in commit 659c096134 on Aug 2, 2017
- fametrano referenced this in commit ecbd242045 on Aug 4, 2017
- JeremyRubin referenced this in commit 7f378349b2 on Aug 9, 2017
- dooglus cross-referenced this on Aug 25, 2017 from issue gui, wallet: random abort (segmentation fault) running master/HEAD by dooglus
- destenson referenced this in commit 99cfe5237f on Aug 31, 2017
- MarcoFalke referenced this in commit a243b3d2e6 on Sep 29, 2017
- achow101 referenced this in commit 26bef49795 on Nov 17, 2017
- MarcoFalke referenced this in commit 4b11505e71 on Jan 15, 2018
- MarcoFalke referenced this in commit 03e5f0a312 on Feb 7, 2018
- MarcoFalke referenced this in commit a570098021 on Feb 10, 2018
- hkjn referenced this in commit 62d55d023a on Feb 12, 2018
- esotericnonsense referenced this in commit 1e9e390eda on Feb 19, 2018
- Willtech referenced this in commit d5f6f4cc0d on Feb 23, 2018
- HashUnlimited referenced this in commit fb8585691c on Mar 9, 2018
- WorkShop-Office cross-referenced this on Apr 11, 2018 from issue Segfault on startup at a04440 by hkjn
- effectsToCause referenced this in commit df7c60fa84 on Jun 22, 2018
- MarcoFalke referenced this in commit 4d550ffab6 on Jul 30, 2018
- girishvg cross-referenced this on Aug 11, 2018 from issue tinyformat.h porting problem? by girishvg
- funasty1985 cross-referenced this on Aug 13, 2018 from issue bitcoind Segmentation fault (core dumped) by funasty1985
- hashtobewild referenced this in commit b0311244c3 on Aug 26, 2018
- ashishchandr70 cross-referenced this on Sep 13, 2018 from issue bitcoind aborts with boost exception in WSL guest by ashishchandr70
- Rspigler cross-referenced this on Oct 13, 2018 from issue Bus error (core dumped) crash on Fedora (txindex migration?) by Rspigler
- dooglus referenced this in commit c2f5dec2ce on Oct 20, 2018
- lionello referenced this in commit 99f0128041 on Nov 7, 2018
- cryptapus referenced this in commit b74dfba83e on Jan 12, 2019
- farrilis cross-referenced this on Mar 29, 2019 from issue v0.18.0 testing by Sjors
- kayront cross-referenced this on Apr 18, 2019 from issue bitcoin core does not compile on Solaris by kayront
- MarcoFalke referenced this in commit 6899ef3f0e on Jun 11, 2019
- MarcoFalke referenced this in commit 98958c81f5 on Jun 17, 2019
- jonatack cross-referenced this on Sep 7, 2019 from issue Follow ups from #15450 by fanquake
- stefanwouldgo cross-referenced this on Sep 23, 2019 from issue strange behavior on termux by stefanwouldgo
- sipsorcery referenced this in commit e9c85bb139 on Nov 7, 2019
- laanwj referenced this in commit 89e93135ae on Nov 10, 2019
- MarcoFalke referenced this in commit cf43f3f0a8 on Dec 5, 2019
- MarcoFalke referenced this in commit bb03765e2d on Dec 8, 2019
- Warchant referenced this in commit 7958da26e8 on Dec 31, 2019
- laanwj referenced this in commit 66480821b3 on Feb 10, 2020
- Bushstar referenced this in commit 707f3ec860 on Apr 8, 2020
- laanwj referenced this in commit 31bdd86631 on Jul 15, 2020
- KolbyML referenced this in commit 99aafab856 on Aug 1, 2020
- laanwj referenced this in commit 924a4ff7eb on Oct 29, 2020
- jonasschnelli referenced this in commit c33662a0ea on Dec 2, 2020
- KolbyML referenced this in commit 3d9f0283ff on Dec 5, 2020
- laanwj referenced this in commit e520e091db on Jan 6, 2021
- rajarshimaitra referenced this in commit 3f5d261950 on Mar 23, 2021
- rajarshimaitra referenced this in commit db9c5dfbc1 on Mar 23, 2021
- MarcoFalke referenced this in commit 590e49ccf2 on Apr 4, 2021
- rednil cross-referenced this on Apr 5, 2021 from issue wallet: Segmentation fault during sync by rednil
- MarcoFalke referenced this in commit bce09da122 on Apr 28, 2021
- fanquake referenced this in commit fa00bb2c5c on Apr 29, 2021
- MarcoFalke referenced this in commit eb9a1fe037 on May 7, 2021
- laanwj referenced this in commit ee9befe8b4 on May 12, 2021
- MarcoFalke referenced this in commit c857148636 on May 15, 2021
- fanquake cross-referenced this on Jun 9, 2021 from issue build: split depends Qt into native and target builds by fanquake
- satindergrewal referenced this in commit 23c8b025b7 on Jun 22, 2021
- satindergrewal referenced this in commit 9e9ab1cb30 on Jun 22, 2021
- satindergrewal referenced this in commit c7c6c0c551 on Jun 22, 2021
- rajarshimaitra referenced this in commit 37e068de4a on Aug 5, 2021
- mjdietzx referenced this in commit a01b804085 on Sep 2, 2021
- mjdietzx referenced this in commit 84c88ec984 on Sep 2, 2021
- cdecker cross-referenced this on Sep 8, 2021 from issue Differentiate Protocol version from client version by cdecker
- rrybarczyk referenced this in commit 4f72c30655 on Sep 21, 2021
- mjdietzx referenced this in commit 85a69f40dd on Sep 26, 2021
- mjdietzx referenced this in commit e6c88a358a on Oct 28, 2021
- mjdietzx referenced this in commit a9a9d3f047 on Oct 29, 2021
- mjdietzx referenced this in commit 12509a080e on Nov 21, 2021
- mjdietzx referenced this in commit 3fc1f8b612 on Nov 21, 2021
- mjdietzx referenced this in commit 54782ae605 on Nov 28, 2021
- mjdietzx referenced this in commit 02ea5b482f on Nov 28, 2021
- mjdietzx referenced this in commit 88a9a46056 on Nov 28, 2021
- mjdietzx referenced this in commit e1d9773000 on Dec 1, 2021
- mjdietzx referenced this in commit 64c20402b2 on Dec 1, 2021
- Idan-Neeman referenced this in commit 9a7b9f03db on Dec 16, 2021
- bitcoin locked this on Dec 16, 2021