Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nydus-image: remove redundant -o short option #550

Merged
merged 1 commit into from
Jul 4, 2022
Merged

nydus-image: remove redundant -o short option #550

merged 1 commit into from
Jul 4, 2022

Commits on Jul 4, 2022

  1. nydus-image: remove redundant -o short option

    Subcommand 'unpack' introduced short option "-p" which is
    redundant with global option log-level ending up with a panic
    
    INFO [utils - 93:run] - /data00/home/gechangwei/git_repo/nydus-rs/target-fusedev/debug/nydus-image create --bootstrap bootstrap_scratched  --blob /data00/home/gechangwei/nydus-test-yard/blobihdgyw05 --log-level info --fs-version 6 --output-json /tmp/tmp46yv4v3_output.json /data00/home/gechangwei/nydus-test-yard/gen_rootfs_scratch
    thread 'main' panicked at 'Argument short must be unique
    
    	-o is already in use', /home/gechangwei/.cargo/registry/src/mirrors.sjtug.sjtu.edu.cn-4f7dbcce21e258a2/clap-2.34.0/src/app/parser.rs:190:13
    stack backtrace:
       0: rust_begin_unwind
                 at /rustc/fe5b13d681f25ee6474be29d748c65adcd91f69e/library/std/src/panicking.rs:584:5
       1: core::panicking::panic_fmt
                 at /rustc/fe5b13d681f25ee6474be29d748c65adcd91f69e/library/core/src/panicking.rs:143:14
       2: clap::app::parser::Parser::debug_asserts
    
    Signed-off-by: Changewei Ge <gechangwei@bytedance.com>
    changweige committed Jul 4, 2022
    Configuration menu
    Copy the full SHA
    51eabe8 View commit details
    Browse the repository at this point in the history