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

cargo wix clean on 0.3.5 broken? #218

Closed
bryanlarsen opened this issue Sep 5, 2023 · 13 comments · Fixed by #221
Closed

cargo wix clean on 0.3.5 broken? #218

bryanlarsen opened this issue Sep 5, 2023 · 13 comments · Fixed by #221
Assignees

Comments

@bryanlarsen
Copy link

thread 'main' panicked at 'Mismatch between definition and access of `verbose`. Unknown argument or group id.  Make sure you are using the argument id and not the short or long flags
', C:\Users\bryan\.cargo\registry\src\index.crates.io-6f17d22bba15001f\cargo-wix-0.3.5\src\main.rs:1794:6
@bryanlarsen
Copy link
Author

Given how new cargo-wix 3.5 is I thought a quick report would be useful. I'll keep digging and add details.

My command line:

 cargo wix -p bowtie_node -o "..\..\target\wix\bowtie.msi" -C "-dConfExists=no"  -C "-dConfName=" -C "-dGitVersion=${git_version}"  -C "-dBundleVariant=yes" --no-build --nocapture --install-version $install_version

@Gankra
Copy link
Contributor

Gankra commented Sep 5, 2023

oof, the update to clap 4 subtly broke a bunch of flags at runtime only, I missed this one 😿

@volks73
Copy link
Owner

volks73 commented Sep 5, 2023

@Gankra Can you please take a look at this? I have not had a chance to look at clap v4 in any depth.

@bryanlarsen
Copy link
Author

Yes, it smelled like something very general where tons of detail on my particular environment wasn't particularly helpful. If that's not the case let me know and I'll send more info.

@Gankra
Copy link
Contributor

Gankra commented Sep 5, 2023

Yep, i'll look into this asap (not sure what's breaking here yet...)

@Gankra
Copy link
Contributor

Gankra commented Sep 5, 2023

@bryanlarsen can you confirm that the CLI invocation you shared does not in fact set --verbose (or -v)? (Very confused why verbose is being mentioned at all)

@Gankra
Copy link
Contributor

Gankra commented Sep 5, 2023

Also could you provide the values of git_version and install_version? I'm wondering if one of those is expanding weird.

@Gankra
Copy link
Contributor

Gankra commented Sep 5, 2023

This pair of commands seems to work fine on cargo wix 0.3.5, running in the root of cargo-wix itself:

cargo wix init -p cargo-wix

cargo wix -p cargo-wix -o "target\wix\bowtie.msi" -C "-dConfExists=no"  -C "-dConfName=" -C "-dGitVersion=1.2.3"  -C "-dBundleVariant=yes" --nocapture --install-version 1.2.3 

Adding --verbose or -v or -vvv seems to also work fine.

@volks73
Copy link
Owner

volks73 commented Sep 5, 2023

Also could you provide the values of git_version and install_version? I'm wondering if one of those is expanding weird.

I am guessing, $git_version or $install_version has a v in it, like v0.3.5 and a - is being prepended to make it appear a -v flag or something.

@Gankra
Copy link
Contributor

Gankra commented Sep 5, 2023

Just tossing out more things worth debugging with: what does cargo -Vv print for you? (We just bumped our msrv by a lot)

@bryanlarsen
Copy link
Author

I'm sorry, the line that was crashing for me was cargo wix clean.

cargo 1.70.0 (ec8a8a0ca 2023-04-25)
release: 1.70.0
commit-hash: ec8a8a0cabb0e0cadef58902470f6c7ee7868bdc
commit-date: 2023-04-25
host: x86_64-pc-windows-msvc
libgit2: 1.6.3 (sys:0.17.0 vendored)
libcurl: 8.0.1-DEV (sys:0.4.61+curl-8.0.1 vendored ssl:Schannel)
os: Windows 10.0.22621 (Windows 10 Pro) [64-bit]

@bryanlarsen
Copy link
Author

Removing cargo wix clean from my build script enabled the script to succeed.

@bryanlarsen bryanlarsen changed the title cargo-wix 3.5 broken? cargo wix clean on 0.3.5 broken? Sep 6, 2023
@Gankra
Copy link
Contributor

Gankra commented Sep 6, 2023

Aha, perfect I can definitely reproduce that one!

Gankra added a commit to Gankra/cargo-wix that referenced this issue Sep 6, 2023
volks73 pushed a commit that referenced this issue Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants