Skip to content

Conversation

@Diapolo
Copy link

@Diapolo Diapolo commented Jun 23, 2013

  • move SelectParamsFromCommandLine() from init.cpp to bitcoin.cpp to allow
    to use TestNet() for Bitcoin-Qt instead of GetBoolArg("-testnet", false)

Does this still work for bitcoind, whose check resides in bitcoind.cpp AFAIK?

- move SelectParamsFromCommandLine() from init.cpp to bitcoin.cpp to allow
  to use TestNet() for Bitcoin-Qt instead of GetBoolArg("-testnet", false)
Diapolo referenced this pull request Jun 23, 2013
Refactor chain-specific tweaks into a CChainParams class and introduce a regtest mode
@sipa
Copy link
Member

sipa commented Jun 23, 2013

This won't work. Bitcoind wouldn't ever call SelectParamsFromCommandLine() anymore?

@Diapolo
Copy link
Author

Diapolo commented Jun 23, 2013

@sipa
Copy link
Member

sipa commented Jun 23, 2013

@Diapolo Ah, that's for the RPC client, not for bitcoind's normal functions.

@Diapolo
Copy link
Author

Diapolo commented Jun 23, 2013

@sipa Could we then place that call (SelectParamsFromCommandLine()) here for bitcoind to work in all vairants?
https://github.com/bitcoin/bitcoin/blob/master/src/bitcoind.cpp#L39

@sipa
Copy link
Member

sipa commented Jun 23, 2013

No, not before ReadConfigFile.

@Diapolo
Copy link
Author

Diapolo commented Jun 24, 2013

Doesn't yet do, what it should ;) -> closed.

@Diapolo Diapolo closed this Jun 24, 2013
@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.

2 participants