Merge pull request #379 from rrybarczyk/use-workspaces

Use workspaces to separate crates

Author: TheBlueMatt

.travis.yml

@@ -11,22 +11,28 @@ before_install:
   - sudo apt-get install -y binutils-dev libunwind8-dev libcurl4-openssl-dev libelf-dev libdw-dev cmake gcc binutils-dev libiberty-dev
 
 script:
-  - RUSTFLAGS="-C link-dead-code" cargo build --verbose
-  - rm -f target/debug/lightning-* # Make sure we drop old test binaries
-  - RUSTFLAGS="-C link-dead-code" cargo test --verbose
-  - if [ "$(rustup show | grep default | grep 1.34.2)" != "" ]; then cd fuzz && cargo test --verbose && ./travis-fuzz.sh; fi
-  - if [ "$(rustup show | grep default | grep stable)" != "" ]; then cd net-tokio && cargo build --verbose && cd ..; fi
-  - if [ "$(rustup show | grep default | grep stable)" != "" ]; then
-      wget https://github.com/SimonKagstrom/kcov/archive/master.tar.gz &&
-      tar xzf master.tar.gz &&
-      cd kcov-master &&
-      mkdir build &&
-      cd build &&
-      cmake .. &&
-      make &&
-      make install DESTDIR=../../kcov-build &&
-      cd ../.. &&
-      rm -rf kcov-master &&
-      for file in target/debug/lightning-*; do [ -x "${file}" ] || continue; mkdir -p "target/cov/$(basename $file)"; ./kcov-build/usr/local/bin/kcov --exclude-pattern=/.cargo,/usr/lib --verify "target/cov/$(basename $file)" "$file"; done &&
-      bash <(curl -s https://codecov.io/bash) &&
-      echo "Uploaded code coverage"; fi
+     # Build workspaces with Rust stable, beta, and 1.34.0
+     - if [ "$(rustup show | grep default | grep 1.22.0)" == "" ]; then RUSTFLAGS="-C link-dead-code" cargo build --verbose; fi
+     # Build lightning workspace with Rust 1.22.0
+     - if [ "$(rustup show | grep default | grep 1.22.0)" != "" ]; then RUSTFLAGS="-C link-dead-code" cargo build --verbose -p lightning; fi
+     - rm -f target/debug/lightning-* # Make sure we drop old test binaries
+     # Test workspaces for Rust stable, beta, and 1.34.2
+     - if [ "$(rustup show | grep default | grep 1.22.0)" == "" ]; then RUSTFLAGS="-C link-dead-code" cargo test --verbose; fi
+     # Test lightning workspace for 1.22.0
+     - if [ "$(rustup show | grep default | grep 1.22.0)" != "" ]; then RUSTFLAGS="-C link-dead-code" cargo test --verbose -p lightning; fi
+     # Run lightning workspace fuzz tests for Rust 1.34.2
+     - if [ "$(rustup show | grep default | grep 1.34.2)" != "" ]; then cd lightning/fuzz && cargo test --verbose && ./travis-fuzz.sh; fi
+     - if [ "$(rustup show | grep default | grep stable)" != "" ]; then
+           wget https://github.com/SimonKagstrom/kcov/archive/master.tar.gz &&
+           tar xzf master.tar.gz &&
+           cd kcov-master &&
+           mkdir build &&
+           cd build &&
+           cmake .. &&
+           make &&
+           make install DESTDIR=../../kcov-build &&
+           cd ../.. &&
+           rm -rf kcov-master &&
+           for file in target/debug/lightning-*; do [ -x "${file}" ] || continue; mkdir -p "target/cov/$(basename $file)"; ./kcov-build/usr/local/bin/kcov --exclude-pattern=/.cargo,/usr/lib --verify "target/cov/$(basename $file)" "$file"; done &&
+           bash <(curl -s https://codecov.io/bash) &&
+           echo "Uploaded code coverage"; fi

Cargo.toml

@@ -1,38 +1,6 @@
-[package]
-name = "lightning"
-version = "0.0.9"
-authors = ["Matt Corallo"]
-license = "Apache-2.0"
-repository = "https://github.com/rust-bitcoin/rust-lightning/"
-description = """
-A Bitcoin Lightning library in Rust.
-Does most of the hard work, without implying a specific runtime, requiring clients implement basic network logic, chain interactions and disk storage.
-Still missing tons of error-handling. See GitHub issues for suggested projects if you want to contribute. Don't have to bother telling you not to use this for anything serious, because you'd have to build a client around it to even try.
-"""
+[workspace]
 
-[features]
-# Supports tracking channels with a non-bitcoin chain hashes. Currently enables all kinds of fun DoS attacks.
-non_bitcoin_chain_hash_routing = []
-fuzztarget = ["secp256k1/fuzztarget", "bitcoin/fuzztarget", "bitcoin_hashes/fuzztarget"]
-# Unlog messages superior at targeted level.
-max_level_off = []
-max_level_error = []
-max_level_warn = []
-max_level_info = []
-max_level_debug = []
-
-[dependencies]
-bitcoin = "0.20"
-bitcoin_hashes = "0.7"
-secp256k1 = "0.15"
-
-[dev-dependencies.bitcoin]
-version = "0.20"
-features = ["bitcoinconsensus"]
-
-[dev-dependencies]
-hex = "0.3"
-rand = "0.4"
-
-[profile.dev]
-opt-level = 1
+members = [
+    "lightning",
+    "lightning-net-tokio",
+]

net-tokio/Cargo.toml renamed to lightning-net-tokio/Cargo.toml

@@ -11,7 +11,7 @@ For Rust-Lightning clients which wish to make direct connections to Lightning P2
 [dependencies]
 bitcoin = "0.20"
 bitcoin_hashes = "0.7"
-lightning = { version = "0.0.9", path = "../" }
+lightning = { version = "0.0.9", path = "../lightning" }
 secp256k1 = "0.15"
 tokio-codec = "0.1"
 futures = "0.1"

net-tokio/src/lib.rs renamed to lightning-net-tokio/src/lib.rs

@@ -0,0 +1,38 @@
+[package]
+name = "lightning"
+version = "0.0.9"
+authors = ["Matt Corallo"]
+license = "Apache-2.0"
+repository = "https://github.com/rust-bitcoin/rust-lightning/"
+description = """
+A Bitcoin Lightning library in Rust.
+Does most of the hard work, without implying a specific runtime, requiring clients implement basic network logic, chain interactions and disk storage.
+Still missing tons of error-handling. See GitHub issues for suggested projects if you want to contribute. Don't have to bother telling you not to use this for anything serious, because you'd have to build a client around it to even try.
+"""
+
+[features]
+# Supports tracking channels with a non-bitcoin chain hashes. Currently enables all kinds of fun DoS attacks.
+non_bitcoin_chain_hash_routing = []
+fuzztarget = ["secp256k1/fuzztarget", "bitcoin/fuzztarget", "bitcoin_hashes/fuzztarget"]
+# Unlog messages superior at targeted level.
+max_level_off = []
+max_level_error = []
+max_level_warn = []
+max_level_info = []
+max_level_debug = []
+
+[dependencies]
+bitcoin = "0.20"
+bitcoin_hashes = "0.7"
+secp256k1 = "0.15"
+
+[dev-dependencies.bitcoin]
+version = "0.20"
+features = ["bitcoinconsensus"]
+
+[dev-dependencies]
+hex = "0.3"
+rand = "0.4"
+
+[profile.dev]
+opt-level = 1