plugin-bcli: changed configs

Changelog-None

Signed-off-by: Nishant Bansal <[email protected]>

Author: NishantBansal2003

cln-rpc/src/codec.rs

@@ -105,6 +105,7 @@ impl Decoder for JsonCodec {
 /// A codec that reads fully formed [crate::messages::JsonRpc]
 /// messages. Internally it uses the [JsonCodec] which itself is built
 /// on the [MultiLineCodec].
+#[allow(dead_code)]
 #[derive(Default)]
 pub(crate) struct JsonRpcCodec {
     inner: JsonCodec,

plugins/bcli.c

@@ -101,7 +101,7 @@ static const char **gather_argsv(const tal_t *ctx, const char *cmd, va_list ap)
 	const char *arg;
 
 	args[0] = bitcoind->cli ? bitcoind->cli : chainparams->cli;
-	add_arg(&args, "-rpcwait");
+
 	if (chainparams->cli_args)
 		add_arg(&args, chainparams->cli_args);
 	if (bitcoind->datadir)
@@ -1038,53 +1038,50 @@ static void wait_and_check_bitcoind(struct plugin *p)
 {
 	int in, from, status, ret;
 	pid_t child;
-	const char **cmd = gather_args(bitcoind, "getnetworkinfo", NULL);
+	const char **cmd =
+	    gather_args(bitcoind, "-rpcwait", "getnetworkinfo", NULL);
 	char *output = NULL;
 
-	for (;;) {
-		tal_free(output);
+	// plugin_log(p, LOG_UNUSUAL, "Waiting for bitcoind to warm up...");
 
-		child = pipecmdarr(&in, &from, &from, cast_const2(char **, cmd));
+	child = pipecmdarr(&in, &from, &from, cast_const2(char **, cmd));
 
-		if (bitcoind->rpcpass)
-			write_all(in, bitcoind->rpcpass, strlen(bitcoind->rpcpass));
+	if (bitcoind->rpcpass)
+		write_all(in, bitcoind->rpcpass, strlen(bitcoind->rpcpass));
 
-		close(in);
+	close(in);
 
-		if (child < 0) {
-			if (errno == ENOENT)
-				bitcoind_failure(p, "bitcoin-cli not found. Is bitcoin-cli "
-						    "(part of Bitcoin Core) available in your PATH?");
-			plugin_err(p, "%s exec failed: %s", cmd[0], strerror(errno));
-		}
+	if (child < 0) {
+		if (errno == ENOENT)
+			bitcoind_failure(
+			    p,
+			    "bitcoin-cli not found. Is bitcoin-cli "
+			    "(part of Bitcoin Core) available in your PATH?");
+		plugin_err(p, "%s exec failed: %s", cmd[0], strerror(errno));
+	}
 
-		output = grab_fd(cmd, from);
-
-		while ((ret = waitpid(child, &status, 0)) < 0 && errno == EINTR);
-		if (ret != child)
-			bitcoind_failure(p, tal_fmt(bitcoind, "Waiting for %s: %s",
-						    cmd[0], strerror(errno)));
-		if (!WIFEXITED(status))
-			bitcoind_failure(p, tal_fmt(bitcoind, "Death of %s: signal %i",
-						   cmd[0], WTERMSIG(status)));
-
-		if (WEXITSTATUS(status) == 0)
-			break;
-
-		/* bitcoin/src/rpc/protocol.h:
-		 *	RPC_IN_WARMUP = -28, //!< Client still warming up
-		 */
-		if (WEXITSTATUS(status) != 28) {
-			if (WEXITSTATUS(status) == 1)
-				bitcoind_failure(p, "Could not connect to bitcoind using"
-						    " bitcoin-cli. Is bitcoind running?");
-			bitcoind_failure(p, tal_fmt(bitcoind, "%s exited with code %i: %s",
-						    cmd[0], WEXITSTATUS(status), output));
-		}
+	output = grab_fd(cmd, from);
 
-		plugin_log(p, LOG_UNUSUAL,
-			   "Waiting for bitcoind to warm up...");
-		sleep(1);
+	while ((ret = waitpid(child, &status, 0)) < 0 && errno == EINTR)
+		;
+	if (ret != child)
+		bitcoind_failure(p, tal_fmt(bitcoind, "Waiting for %s: %s",
+					    cmd[0], strerror(errno)));
+	if (!WIFEXITED(status))
+		bitcoind_failure(p, tal_fmt(bitcoind, "Death of %s: signal %i",
+					    cmd[0], WTERMSIG(status)));
+
+	/* bitcoin/src/rpc/protocol.h:
+	 *	RPC_IN_WARMUP = -28, //!< Client still warming up
+	 */
+	if (WEXITSTATUS(status) != 0 && WEXITSTATUS(status) != 28) {
+		if (WEXITSTATUS(status) == 1)
+			bitcoind_failure(p,
+					 "Could not connect to bitcoind using"
+					 " bitcoin-cli. Is bitcoind running?");
+		bitcoind_failure(p,
+				 tal_fmt(bitcoind, "%s exited with code %i: %s",
+					 cmd[0], WEXITSTATUS(status), output));
 	}
 
 	parse_getnetworkinfo_result(p, output);