Browse Source

travis: reenable check-source (without BOLT text).

We've been slipping, so fix up minor issues too so it compiles.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
ppa-0.6.1
Rusty Russell 7 years ago
committed by Christian Decker
parent
commit
887e9dcc44
  1. 1
      .travis.yml
  2. 2
      common/bolt11.c
  3. 2
      common/bolt11.h
  4. 2
      common/withdraw_tx.c
  5. 2
      gossipd/routing.c
  6. 2
      gossipd/test/run-bench-find_route.c
  7. 2
      lightningd/peer_control.h
  8. 4
      wallet/wallet.c

1
.travis.yml

@ -19,3 +19,4 @@ script:
- docker run --rm=true -e NO_VALGRIND=${NO_VALGRIND:-0} -e TEST_DEBUG=${TEST_DEBUG:-0} -v "${TRAVIS_BUILD_DIR}":/build -t cdecker/lightning-ci:${ARCH}bit make CC=${COMPILER} check DEVELOPER=${DEVELOPER} - docker run --rm=true -e NO_VALGRIND=${NO_VALGRIND:-0} -e TEST_DEBUG=${TEST_DEBUG:-0} -v "${TRAVIS_BUILD_DIR}":/build -t cdecker/lightning-ci:${ARCH}bit make CC=${COMPILER} check DEVELOPER=${DEVELOPER}
# - git clone https://github.com/lightningnetwork/lightning-rfc.git # - git clone https://github.com/lightningnetwork/lightning-rfc.git
# - docker run --rm=true -v "${TRAVIS_BUILD_DIR}":/build -t cdecker/lightning-ci:${ARCH}bit make check-source BOLTDIR=lightning-rfc # - docker run --rm=true -v "${TRAVIS_BUILD_DIR}":/build -t cdecker/lightning-ci:${ARCH}bit make check-source BOLTDIR=lightning-rfc
- docker run --rm=true -v "${TRAVIS_BUILD_DIR}":/build -t cdecker/lightning-ci:${ARCH}bit make check-source BOLTDIR=none

2
common/bolt11.c

@ -401,7 +401,7 @@ static void towire_route_info(u8 **pptr, const struct route_info *route_info)
* *
* * `pubkey` (264 bits) * * `pubkey` (264 bits)
* * `short_channel_id` (64 bits) * * `short_channel_id` (64 bits)
* * `fee_base_msat` (32 bits, big-endian) * * `fee_base_msat` (32 bits, big-endian)
* * `fee_proportional_millionths` (32 bits, big-endian) * * `fee_proportional_millionths` (32 bits, big-endian)
* * `cltv_expiry_delta` (16 bits, big-endian) * * `cltv_expiry_delta` (16 bits, big-endian)
*/ */

2
common/bolt11.h

@ -22,7 +22,7 @@ struct bolt11_field {
/* BOLT #11: /* BOLT #11:
* * `pubkey` (264 bits) * * `pubkey` (264 bits)
* * `short_channel_id` (64 bits) * * `short_channel_id` (64 bits)
* * `fee_base_msat` (32 bits, big-endian) * * `fee_base_msat` (32 bits, big-endian)
* * `fee_proportional_millionths` (32 bits, big-endian) * * `fee_proportional_millionths` (32 bits, big-endian)
* * `cltv_expiry_delta` (16 bits, big-endian) * * `cltv_expiry_delta` (16 bits, big-endian)
*/ */

2
common/withdraw_tx.c

@ -5,8 +5,8 @@
#include <common/key_derive.h> #include <common/key_derive.h>
#include <common/permute_tx.h> #include <common/permute_tx.h>
#include <common/utxo.h> #include <common/utxo.h>
#include <wally_bip32.h>
#include <string.h> #include <string.h>
#include <wally_bip32.h>
struct bitcoin_tx *withdraw_tx(const tal_t *ctx, struct bitcoin_tx *withdraw_tx(const tal_t *ctx,
const struct utxo **utxos, const struct utxo **utxos,

2
gossipd/routing.c

@ -346,7 +346,7 @@ find_route(const tal_t *ctx, struct routing_state *rstate,
msatoshi); msatoshi);
return NULL; return NULL;
} }
/* Reset all the information. */ /* Reset all the information. */
clear_bfg(rstate->nodes); clear_bfg(rstate->nodes);

2
gossipd/test/run-bench-find_route.c

@ -21,7 +21,7 @@ static bool in_bench = 0;
static char *fake_type_to_string_(const tal_t *ctx, const char *typename, static char *fake_type_to_string_(const tal_t *ctx, const char *typename,
union printable_types u) union printable_types u)
{ {
/* We *do* call this at end of route setup. */ /* We *do* call this at end of route setup. */
if (streq(typename, "struct pubkey")) { if (streq(typename, "struct pubkey")) {
size_t n; size_t n;
memcpy(&n, u.pubkey, sizeof(n)); memcpy(&n, u.pubkey, sizeof(n));

2
lightningd/peer_control.h

@ -44,7 +44,7 @@ struct peer {
/* Command which ordered us to open channel, if any. */ /* Command which ordered us to open channel, if any. */
struct command *opening_cmd; struct command *opening_cmd;
/* Is there a single subdaemon responsible for us? */ /* Is there a single subdaemon responsible for us? */
struct subd *owner; struct subd *owner;

4
wallet/wallet.c

@ -171,7 +171,7 @@ static const struct utxo **wallet_select(const tal_t *ctx, struct wallet *w,
w, &available[i]->txid, available[i]->outnum, w, &available[i]->txid, available[i]->outnum,
output_state_available, output_state_reserved)) output_state_available, output_state_reserved))
fatal("Unable to reserve output"); fatal("Unable to reserve output");
/* Input weight: txid + index + sequence */ /* Input weight: txid + index + sequence */
input_weight = (32 + 4 + 4) * 4; input_weight = (32 + 4 + 4) * 4;
@ -217,7 +217,7 @@ const struct utxo **wallet_select_coins(const tal_t *ctx, struct wallet *w,
*changesatoshi = satoshi_in - value - *fee_estimate; *changesatoshi = satoshi_in - value - *fee_estimate;
return utxo; return utxo;
} }
const struct utxo **wallet_select_all(const tal_t *ctx, struct wallet *w, const struct utxo **wallet_select_all(const tal_t *ctx, struct wallet *w,
const u32 feerate_per_kw, const u32 feerate_per_kw,
size_t outscriptlen, size_t outscriptlen,

Loading…
Cancel
Save