Bitcoin-Qt: give testnet a unique IPC message queue name #2147

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:mq_name_testnet changing 3 files +25 −11

Diapolo commented at 9:10 PM on January 3, 2013: none
- this prevents an interference with the IPC message queue (which is used for URI processing) when running a testnet and mainnet instance in parallel
- to check for testnet, I had to raise the ParseParameters() call in main() to the topmost position
BitcoinPullTester commented at 9:51 PM on January 3, 2013: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/a399b4973b5d0f7b00d26d5e059db6479181a36c for binaries and test log.

in src/qt/qtipcserver.cpp:None in a399b4973b outdated

  25 | @@ -26,6 +26,9 @@
  26 |  using namespace boost::interprocess;
  27 |  using namespace boost::posix_time;
  28 |  
  29 | +// holds Bitcoin-Qt message queue name (initialized in bitcoin.cpp)
  30 | +std::string BITCOINURI_QUEUE_NAME;

laanwj commented at 3:48 PM on January 5, 2013:

You could make this lowercase now it is no longer a constant

Diapolo commented at 2:43 AM on January 6, 2013:

Updated to strBitcoinURIQueueName :).

laanwj commented at 3:49 PM on January 5, 2013: member

ACK (apart from my code comment)

Bitcoin-Qt: give testnet a unique IPC message queue name

- this prevents an interference with the IPC message queue (which is used
  for URI processing) when running a testnet and mainnet instance in
  parallel
- to check for testnet, I had to raise the ParseParameters() call in
  main() to the topmost position

8ffbd6c378

BitcoinPullTester commented at 3:01 AM on January 6, 2013: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/8ffbd6c3789843cb024a3ee425ac4cc0482eaeb7 for binaries and test log.
laanwj referenced this in commit 51b05d0dca on Jan 6, 2013
laanwj merged this on Jan 6, 2013
laanwj closed this on Jan 6, 2013
laudney referenced this in commit f470d269b8 on Mar 19, 2014
owlhooter referenced this in commit 7ab5b4a28e on Oct 11, 2018
DrahtBot locked this on Sep 8, 2021

Contributors

Diapolo

BitcoinPullTester

laanwj