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

[BUG] Virtual argument's virtual path is case-sensitive #130

Open
Coryf88 opened this issue Oct 3, 2020 · 1 comment
Open

[BUG] Virtual argument's virtual path is case-sensitive #130

Coryf88 opened this issue Oct 3, 2020 · 1 comment
Labels
bug Error, flaw, failure or fault that causes to produce an incorrect result or crash help wanted

Comments

@Coryf88
Copy link

Coryf88 commented Oct 3, 2020

Describe the bug
--virtual's virtual path is case-sensitive

To Reproduce

  1. Create file test.cpp, containing #include "\A3\3den\ui\macros.inc"
  2. Execute sqfvm.exe --parse-only --virtual "P:\a3|a3" --input-config test.cpp
  3. See error Failed to include '\A3\3den\ui\macros.inc'
  4. Execute sqfvm.exe --parse-only --virtual "P:\a3|A3" --input-config test.cpp
  5. Notice no error.

Expected behavior
Virtual path to be case-insensitive.

Screenshots
N/A

Additional context
N/A

@Coryf88 Coryf88 added the bug Error, flaw, failure or fault that causes to produce an incorrect result or crash label Oct 3, 2020
@X39
Copy link
Collaborator

X39 commented Oct 18, 2020

Limitation due to compatibility needs with unix.

While arma is indeed, case-insensitive, doing it with SQF-VM would require additional muscle work and double the memory footprint for certain things (eg. config tree, assembly size)

needs to be decided wether to implement this (and potentially fail on unix systems with the same code) or not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Error, flaw, failure or fault that causes to produce an incorrect result or crash help wanted
Projects
None yet
Development

No branches or pull requests

2 participants