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

[KSP2] command line doesn't support absolute Windows paths #2046

Closed
joffrey-bion opened this issue Aug 22, 2024 · 0 comments · Fixed by #2117
Closed

[KSP2] command line doesn't support absolute Windows paths #2046

joffrey-bion opened this issue Aug 22, 2024 · 0 comments · Fixed by #2117
Assignees
Labels
bug Something isn't working P1 major features or blocking bugs

Comments

@joffrey-bion
Copy link

joffrey-bion commented Aug 22, 2024

The command line doc specifies that lists must be colon-separated, even for lists of paths.

Because of this, Windows paths like C:\MyDir are interpreted as 2 directories:

  • a relative dir C, which is undesired
  • an absolute dir /MyDir, which may end up in the right place if the working dir is in the same drive, but is not correct in general

This is because the generated argument parser simply splits on :, disregarding quotes or system-specific path separators.

It would be great it the CLI accepted the system's path-separator as a separator for paths lists (like Java's -cp argument).

@joffrey-bion joffrey-bion changed the title KSP2 command line doesn't support absolute Windows paths [KSP2] command line doesn't support absolute Windows paths Aug 26, 2024
@ting-yuan ting-yuan self-assigned this Sep 24, 2024
@ting-yuan ting-yuan added bug Something isn't working P1 major features or blocking bugs labels Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working P1 major features or blocking bugs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants