chore: remove dependency on YAML

Cargo.toml

@@ -6,7 +6,7 @@ authors = ["Nervos Core Dev <[email protected]>"]
 build = "build.rs"
 
 [dependencies]
-clap = { version = "2", features = ["yaml"] }
+clap = { version = "2" }
 serde = "1.0"
 serde_derive = "1.0"
 log = "0.4"

src/cli/args.rs

@@ -0,0 +1,106 @@
+// use build_info::Version;
+use clap::{App, AppSettings, Arg, ArgMatches, SubCommand};
+
+use build_info::Version;
+
+pub fn get_matches() -> ArgMatches<'static> {
+    let version = get_version!();
+
+    App::new("ckb")
+        .author("Nervos Core Dev <[email protected]>")
+        .about("Nervos CKB - The Common Knowledge Base")
+        .version(version.short().as_str())
+        .long_version(version.long().as_str())
+        .setting(AppSettings::ArgRequiredElseHelp)
+        .arg(
+            Arg::with_name("config")
+            .short("c")
+            .long("config")
+            .value_name("CONFIG")
+            .takes_value(true)
+            .help("Specify the configuration file PATH. Tries ckb.json, nodes/default.json in working directory when omitted.")
+        )
+        .subcommand(run())
+        .subcommand(export())
+        .subcommand(import())
+        .subcommand(cli())
+        .get_matches()
+}
+
+fn run() -> App<'static, 'static> {
+    SubCommand::with_name("run").about("Running ckb node")
+}
+
+fn arg_format() -> Arg<'static, 'static> {
+    Arg::with_name("format")
+        .short("f")
+        .long("format")
+        .value_name("FORMAT")
+        .required(true)
+        .takes_value(true)
+        .help("Specify the format.")
+}
+
+fn export() -> App<'static, 'static> {
+    SubCommand::with_name("export")
+        .about("Export ckb data")
+        .arg(arg_format())
+        .arg(
+            Arg::with_name("target")
+                .short("t")
+                .long("target")
+                .value_name("PATH")
+                .required(true)
+                .index(1)
+                .help("Specify the export target path."),
+        )
+}
+
+fn import() -> App<'static, 'static> {
+    SubCommand::with_name("import")
+        .about("Import ckb data")
+        .arg(arg_format())
+        .arg(
+            Arg::with_name("source")
+                .short("s")
+                .long("source")
+                .value_name("PATH")
+                .required(true)
+                .index(1)
+                .help("Specify the exported data path."),
+        )
+}
+
+fn arg_private_key() -> Arg<'static, 'static> {
+    Arg::with_name("private-key")
+        .short("p")
+        .long("private-key")
+        .value_name("H256")
+        .help("Specify the private key")
+        .takes_value(true)
+        .required(true)
+}
+
+fn cli() -> App<'static, 'static> {
+    SubCommand::with_name("cli")
+        .about("Running ckb cli")
+        .setting(AppSettings::ArgRequiredElseHelp)
+        .subcommand(
+            SubCommand::with_name("sign")
+                .about("Sign transaction using sha3-secp256k1 defined in system cell")
+                .arg(arg_private_key())
+                .arg(
+                    Arg::with_name("unsigned-transaction")
+                        .short("u")
+                        .long("unsigned-transaction")
+                        .value_name("JSON")
+                        .help("Specify the unsigned transaction json string")
+                        .takes_value(true)
+                        .required(true),
+                ),
+        ).subcommand(
+            SubCommand::with_name("type_hash")
+                .about("Generate script type hash using sha3-secp256k1 defined in system cell")
+                .arg(arg_private_key()),
+        ).subcommand(SubCommand::with_name("keygen").about("Generate new key"))
+}

src/cli/mod.rs

@@ -1,7 +1,9 @@
+mod args;
 mod export;
 mod import;
 mod run_impl;
 
+pub use self::args::get_matches;
 pub use self::export::export;
 pub use self::import::import;
 pub use self::run_impl::{keygen, run, sign, type_hash};

src/main.rs

@@ -1,33 +1,34 @@
 #[macro_use]
+extern crate build_info;
+#[macro_use]
 extern crate clap;
-extern crate ctrlc;
-extern crate dir;
 #[macro_use]
 extern crate log;
+#[macro_use]
+extern crate serde_derive;
+
 extern crate bigint;
 extern crate ckb_chain;
 extern crate ckb_chain_spec;
 extern crate ckb_core;
 extern crate ckb_db;
+extern crate ckb_instrument;
 extern crate ckb_miner;
 extern crate ckb_network;
 extern crate ckb_notify;
 extern crate ckb_pool;
+extern crate ckb_pow;
 extern crate ckb_rpc;
 extern crate ckb_shared;
 extern crate ckb_sync;
 extern crate ckb_util;
-extern crate hash;
-extern crate logger;
-#[macro_use]
-extern crate serde_derive;
-#[macro_use]
-extern crate build_info;
-extern crate ckb_instrument;
-extern crate ckb_pow;
 extern crate config as config_tool;
 extern crate crypto;
+extern crate ctrlc;
+extern crate dir;
 extern crate faster_hex;
+extern crate hash;
+extern crate logger;
 extern crate serde_json;
 #[cfg(test)]
 extern crate tempfile;
@@ -36,20 +37,13 @@ mod cli;
 mod helper;
 mod setup;
 
-use build_info::Version;
 use setup::{get_config_path, Setup};
 
 fn main() {
     // Always print backtrace on panic.
     ::std::env::set_var("RUST_BACKTRACE", "full");
 
-    let yaml = load_yaml!("cli/app.yml");
-    let version = get_version!();
-    let matches = clap::App::from_yaml(yaml)
-        .version(version.short().as_str())
-        .long_version(version.long().as_str())
-        .get_matches();
-
+    let matches = cli::get_matches();
     let config_path = get_config_path(&matches);
     let setup = match Setup::setup(&config_path) {
         Ok(setup) => {