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

New argument parsing for bsputil broken #435

Open
dsvensson opened this issue Aug 3, 2024 · 1 comment
Open

New argument parsing for bsputil broken #435

dsvensson opened this issue Aug 3, 2024 · 1 comment

Comments

@dsvensson
Copy link
Contributor

dsvensson commented Aug 3, 2024

The new argument parsing to bsputil expects operations such as --extract-textures to take an argument.

That is, this fails:

bsputil --extract-textures foo.bsp

...while...

bsputil --extract-textures dummy foo.bsp

...works.

Given the new argument parsing I also suspect that #224 is invalid.

@ericwa ericwa closed this as completed in 0b2395d Aug 4, 2024
@ericwa
Copy link
Owner

ericwa commented Aug 4, 2024

Thanks for catching.

I fixed --extract-textures and --extract-entities. I'll need to test the other bsputil commands and ideally add automated tests for all commands.

@ericwa ericwa reopened this Aug 4, 2024
KurtLoeffler pushed a commit to KurtLoeffler/ericw-tools that referenced this issue Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants