[bisq-network/bisq] [WIP] Add API test harness for Linux (#4366)
Stan
notifications at github.com
Fri Jul 10 18:15:27 UTC 2020
@ghubstan commented on this pull request.
> +// bitcoin-cli -regtest getbalances
+// bitcoin-cli -regtest getrpcinfo
+
+ at Slf4j
+public class BitcoinDaemon extends AbstractLinuxProcess implements LinuxProcess {
+
+ public BitcoinDaemon(ApiTestConfig config) {
+ super("bitcoind", config);
+ }
+
+ @Override
+ public void start() throws InterruptedException, IOException {
+ String bitcoindCmd = "export LD_LIBRARY_PATH=" + config.berkeleyDbLibPath + ";"
+ + " " + config.bitcoinPath + "/bitcoind"
+ + " -datadir=" + config.bitcoinDatadir
+ + " -daemon";
`export LD_LIBRARY_PATH` is necessary for my bitcoin-core installation. This issue is something we'll need to sort out for an unknown variety of setups.
I think `config.bitcoinPath` should point to the directory containing bitcoin-core binaries, not the `bitcoind` and `bitcoin-cli` binaries themselves -- requiring two `apitest` config options instead of one. On the other hand, my `bitcoin-core` binaries are isolated in my `bitcoin-core` src directory (I never ran `make install`), not a default system `bin` directory where many unrelated executables are installed. Let's keep discussing...
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4366#discussion_r452999082
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200710/fd2cd358/attachment.html>
More information about the bisq-github
mailing list