difftreelog
build upgrade clap to v3.1
in: master
8 files changed
cmds/jrsonnet/Cargo.tomldiffbeforeafterboth--- a/cmds/jrsonnet/Cargo.toml
+++ b/cmds/jrsonnet/Cargo.toml
@@ -18,11 +18,5 @@
mimallocator = { version = "0.1.3", optional = true }
thiserror = "1.0"
gcmodule = { git = "https://github.com/CertainLach/gcmodule", branch = "jrsonnet" }
-
-[dependencies.clap]
-git = "https://github.com/clap-rs/clap"
-rev = "f0c5ea5e1503de5c8e74d8c047a799cf51498e83"
-
-[dependencies.clap_generate]
-git = "https://github.com/clap-rs/clap"
-rev = "f0c5ea5e1503de5c8e74d8c047a799cf51498e83"
+clap = { version = "3.1", features = ["derive"] }
+clap_complete = { version = "3.1" }
cmds/jrsonnet/src/main.rsdiffbeforeafterboth--- a/cmds/jrsonnet/src/main.rs
+++ b/cmds/jrsonnet/src/main.rs
@@ -1,4 +1,5 @@
-use clap::{AppSettings, Clap, IntoApp};
+use clap::{AppSettings, IntoApp, Parser};
+use clap_complete::Shell;
use jrsonnet_cli::{ConfigureState, GcOpts, GeneralOpts, InputOpts, ManifestOpts, OutputOpts};
use jrsonnet_evaluator::{error::LocError, EvaluationState};
use std::{
@@ -6,51 +7,39 @@
io::Read,
io::Write,
path::PathBuf,
- str::FromStr,
};
#[cfg(feature = "mimalloc")]
#[global_allocator]
static GLOBAL: mimallocator::Mimalloc = mimallocator::Mimalloc;
-#[derive(Clap)]
-#[clap(help_heading = "DEBUG")]
+#[derive(Parser)]
+enum SubOpts {
+ /// Generate completions for specified shell
+ Generate {
+ /// Target shell name
+ shell: Shell,
+ },
+}
+
+#[derive(Parser)]
+#[clap(next_help_heading = "DEBUG")]
struct DebugOpts {
/// Required OS stack size.
/// This shouldn't be changed unless jrsonnet is failing with stack overflow error.
#[clap(long, name = "size")]
pub os_stack: Option<usize>,
- /// Generate completions script
- #[clap(long)]
- generate: Option<GenerateTarget>,
}
-enum GenerateTarget {
- Bash,
- Zsh,
- Fish,
- PowerShell,
-}
-impl FromStr for GenerateTarget {
- type Err = &'static str;
-
- fn from_str(s: &str) -> Result<Self, Self::Err> {
- match s {
- "bash" => Ok(Self::Bash),
- "zsh" => Ok(Self::Zsh),
- "fish" => Ok(Self::Fish),
- "powershell" => Ok(Self::PowerShell),
- _ => Err("unknown target"),
- }
- }
-}
-
-#[derive(Clap)]
+#[derive(Parser)]
#[clap(
- global_setting = AppSettings::ColoredHelp,
global_setting = AppSettings::DeriveDisplayOrder,
+ // args_conflicts_with_subcommands = true,
)]
struct Opts {
+ #[clap(subcommand)]
+ sub: Option<SubOpts>,
+
#[clap(flatten)]
input: InputOpts,
#[clap(flatten)]
@@ -68,20 +57,17 @@
fn main() {
let opts: Opts = Opts::parse();
- if let Some(target) = opts.debug.generate {
- use clap_generate::{generate, generators};
- use GenerateTarget::*;
- let app = &mut Opts::into_app();
- let buf = &mut std::io::stdout();
- let bin = "jrsonnet";
- match target {
- Bash => generate::<generators::Bash, _>(app, bin, buf),
- Zsh => generate::<generators::Zsh, _>(app, bin, buf),
- Fish => generate::<generators::Fish, _>(app, bin, buf),
- PowerShell => generate::<generators::PowerShell, _>(app, bin, buf),
+ if let Some(sub) = opts.sub {
+ match sub {
+ SubOpts::Generate { shell } => {
+ use clap_complete::generate;
+ let app = &mut Opts::command();
+ let buf = &mut std::io::stdout();
+ generate(shell, app, "jrsonnet", buf);
+ std::process::exit(0)
+ }
}
- std::process::exit(0);
- };
+ }
let success = if let Some(size) = opts.debug.os_stack {
std::thread::Builder::new()
crates/jrsonnet-cli/Cargo.tomldiffbeforeafterboth--- a/crates/jrsonnet-cli/Cargo.toml
+++ b/crates/jrsonnet-cli/Cargo.toml
@@ -14,6 +14,4 @@
jrsonnet-parser = { path = "../../crates/jrsonnet-parser", version = "0.4.2" }
gcmodule = { git = "https://github.com/CertainLach/gcmodule", branch = "jrsonnet" }
-[dependencies.clap]
-git = "https://github.com/clap-rs/clap"
-rev = "f0c5ea5e1503de5c8e74d8c047a799cf51498e83"
+clap = { version = "3.1", features = ["derive"] }
crates/jrsonnet-cli/src/ext.rsdiffbeforeafterboth--- a/crates/jrsonnet-cli/src/ext.rs
+++ b/crates/jrsonnet-cli/src/ext.rs
@@ -1,5 +1,5 @@
use crate::ConfigureState;
-use clap::Clap;
+use clap::Parser;
use jrsonnet_evaluator::{error::Result, EvaluationState};
use std::{fs::read_to_string, str::FromStr};
@@ -53,8 +53,8 @@
}
}
-#[derive(Clap)]
-#[clap(help_heading = "EXTERNAL VARIABLES")]
+#[derive(Parser)]
+#[clap(next_help_heading = "EXTERNAL VARIABLES")]
pub struct ExtVarOpts {
/// Add string external variable.
/// External variables are globally available so it is preferred
crates/jrsonnet-cli/src/lib.rsdiffbeforeafterboth--- a/crates/jrsonnet-cli/src/lib.rs
+++ b/crates/jrsonnet-cli/src/lib.rs
@@ -8,7 +8,7 @@
pub use tla::*;
pub use trace::*;
-use clap::Clap;
+use clap::Parser;
use jrsonnet_evaluator::{error::Result, EvaluationState, FileImportResolver};
use std::{env, path::PathBuf};
@@ -16,24 +16,19 @@
fn configure(&self, state: &EvaluationState) -> Result<()>;
}
-#[derive(Clap)]
-#[clap(help_heading = "INPUT")]
+#[derive(Parser)]
+#[clap(next_help_heading = "INPUT")]
pub struct InputOpts {
- #[clap(
- long,
- short = 'e',
- about = "Treat input as code, evaluate them instead of reading file"
- )]
+ /// Treat input as code, evaluate them instead of reading file
+ #[clap(long, short = 'e')]
pub exec: bool,
- #[clap(
- about = "Path to the file to be compiled if `--evaluate` is unset, otherwise code itself"
- )]
+ /// Path to the file to be compiled if `--evaluate` is unset, otherwise code itself
pub input: String,
}
-#[derive(Clap)]
-#[clap(help_heading = "OPTIONS")]
+#[derive(Parser)]
+#[clap(next_help_heading = "OPTIONS")]
pub struct MiscOpts {
/// Disable standard library.
/// By default standard library will be available via global `std` variable.
@@ -74,7 +69,7 @@
}
/// General configuration of jsonnet
-#[derive(Clap)]
+#[derive(Parser)]
#[clap(name = "jrsonnet", version, author)]
pub struct GeneralOpts {
#[clap(flatten)]
@@ -100,8 +95,8 @@
}
}
-#[derive(Clap)]
-#[clap(help_heading = "GARBAGE COLLECTION")]
+#[derive(Parser)]
+#[clap(next_help_heading = "GARBAGE COLLECTION")]
pub struct GcOpts {
/// Do not skip gc on exit
#[clap(long)]
crates/jrsonnet-cli/src/manifest.rsdiffbeforeafterboth1use crate::ConfigureState;2use clap::Clap;3use jrsonnet_evaluator::{error::Result, EvaluationState, ManifestFormat};4use std::{path::PathBuf, str::FromStr};56pub enum ManifestFormatName {7 /// Expect string as output, and write them directly8 String,9 Json,10 Yaml,11}1213impl FromStr for ManifestFormatName {14 type Err = &'static str;15 fn from_str(s: &str) -> std::result::Result<Self, Self::Err> {16 Ok(match s {17 "string" => ManifestFormatName::String,18 "json" => ManifestFormatName::Json,19 "yaml" => ManifestFormatName::Yaml,20 _ => return Err("no such format"),21 })22 }23}2425#[derive(Clap)]26#[clap(help_heading = "MANIFESTIFICATION OUTPUT")]27pub struct ManifestOpts {28 /// Output format, wraps resulting value to corresponding std.manifest call.29 /// If set to `string` then plain string value is expected to be returned,30 /// otherwise output will be serialized to the specified format.31 #[clap(long, short = 'f', default_value = "json", possible_values = &["string", "json", "yaml"])]32 format: ManifestFormatName,33 /// Expect plain string as output.34 /// Shortcut for `--format=string` thus this option is mutually exclusive with `format` option.35 #[clap(long, short = 'S')]36 string: bool,37 /// Write output as YAML stream, can be used with --format json/yaml38 #[clap(long, short = 'y')]39 yaml_stream: bool,40 /// Number of spaces to pad output manifest with.41 /// `0` for hard tabs, `-1` for single line output [default: 3 for json, 2 for yaml]42 #[clap(long)]43 line_padding: Option<usize>,44}45impl ConfigureState for ManifestOpts {46 fn configure(&self, state: &EvaluationState) -> Result<()> {47 if self.string {48 state.set_manifest_format(ManifestFormat::String);49 } else {50 match self.format {51 ManifestFormatName::String => state.set_manifest_format(ManifestFormat::String),52 ManifestFormatName::Json => {53 state.set_manifest_format(ManifestFormat::Json(self.line_padding.unwrap_or(3)))54 }55 ManifestFormatName::Yaml => {56 state.set_manifest_format(ManifestFormat::Yaml(self.line_padding.unwrap_or(2)))57 }58 }59 }60 if self.yaml_stream {61 state.set_manifest_format(ManifestFormat::YamlStream(Box::new(62 state.manifest_format(),63 )))64 }65 Ok(())66 }67}6869#[derive(Clap)]70pub struct OutputOpts {71 /// Write to the output file rather than stdout72 #[clap(long, short = 'o')]73 pub output_file: Option<PathBuf>,74 /// Automatically creates all parent directories for files75 #[clap(long, short = 'c')]76 pub create_output_dirs: bool,77 /// Write multiple files to the directory, list files on stdout78 #[clap(long, short = 'm')]79 pub multi: Option<PathBuf>,80}crates/jrsonnet-cli/src/tla.rsdiffbeforeafterboth--- a/crates/jrsonnet-cli/src/tla.rs
+++ b/crates/jrsonnet-cli/src/tla.rs
@@ -1,9 +1,9 @@
use crate::{ConfigureState, ExtFile, ExtStr};
-use clap::Clap;
+use clap::Parser;
use jrsonnet_evaluator::{error::Result, EvaluationState};
-#[derive(Clap)]
-#[clap(help_heading = "TOP LEVEL ARGUMENTS")]
+#[derive(Parser)]
+#[clap(next_help_heading = "TOP LEVEL ARGUMENTS")]
pub struct TLAOpts {
/// Add top level string argument.
/// Top level arguments will be passed to function before manifestification stage.
crates/jrsonnet-cli/src/trace.rsdiffbeforeafterboth--- a/crates/jrsonnet-cli/src/trace.rs
+++ b/crates/jrsonnet-cli/src/trace.rs
@@ -1,5 +1,5 @@
use crate::ConfigureState;
-use clap::Clap;
+use clap::Parser;
use jrsonnet_evaluator::{
error::Result,
trace::{CompactFormat, ExplainingFormat, PathResolver},
@@ -24,8 +24,8 @@
}
}
-#[derive(Clap)]
-#[clap(help_heading = "STACK TRACE VISUAL")]
+#[derive(Parser)]
+#[clap(next_help_heading = "STACK TRACE VISUAL")]
pub struct TraceOpts {
/// Format of stack traces' display in console.
/// `compact` format only shows `filename:line:column`s