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

clap assertion failure in cli.yml #2506

Closed
edgex004 opened this issue Aug 21, 2021 · 3 comments
Closed

clap assertion failure in cli.yml #2506

edgex004 opened this issue Aug 21, 2021 · 3 comments

Comments

@edgex004
Copy link
Contributor

Describe the bug

Clap throws an assertion error when running tauri build.

Side note: it seems like load_yaml!("cli.yml"); does not mark main.rs as dirty when cli.yml changes. I need to clean build to get the changes in cli.yml to be utilized.

To Reproduce

Steps to reproduce the behavior:

  1. cd to rs.cli directory.
  2. Clean build rs.cli: cargo clean && cargo build
  3. cd to your project directory. (In this case, project was created with npm x create-tauri-app)
  4. Build project with rs.cli executable: path/to/cargo-tauri tauri build
  5. Note error:
thread 'main' panicked at '
ArgSettings::TakesValue is required when ArgSettings::MultipleValues is set.

Expected behavior

Project is built successfully.

Screenshots

Platform and Versions (required):

Operating System - Mac OS, version 11.2.3 X64

Node.js environment
  Node.js - 16.7.0
  @tauri-apps/cli - 1.0.0-beta.9
  @tauri-apps/api - 1.0.0-beta.6

Global packages
  npm - 7.20.3
  yarn - 1.22.11

Rust environment
  rustc - 1.56.0-nightly
  cargo - 1.56.0-nightly

App directory structure
/dist
/target
/node_modules
/src-tauri
/.idea
/src

App
  tauri.rs - 1.0.0-beta.6 (no lockfile)
  build-type - bundle
  CSP - default-src blob: data: filesystem: ws: wss: http: https: tauri: 'unsafe-eval' 'unsafe-inline' 'self' img-src: 'self'
  distDir - ../dist
  devPath - http://localhost:10001/
  bundler - Rollup

Additional context

Built from hash 8fcdc1d47419f8f609b5685d5266d7c5365a7106

Stack Trace

@edgex004
Copy link
Contributor Author

I have updated cli.yml to fix on my system. Will submit a PR.

@edgex004
Copy link
Contributor Author

Would it be a good idea to add a test for cli.yml?

lucasfernog pushed a commit that referenced this issue Aug 22, 2021
…n. (#2506) (#2507)

Co-authored-by: David Von Edge <david.vonedge@smiths.com>
@edgex004
Copy link
Contributor Author

Fixed by #2507.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant