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

Append and AppendUnique handle quoting differently #4262

Open
mwichmann opened this issue Nov 14, 2022 · 0 comments
Open

Append and AppendUnique handle quoting differently #4262

mwichmann opened this issue Nov 14, 2022 · 0 comments
Assignees
Labels
Append Issues in Append/Prepend + Unique variants

Comments

@mwichmann
Copy link
Collaborator

mwichmann commented Nov 14, 2022

Example on a Windows platform and current

env1.Append(CCFLAGS = '/this /is /a /test')
env2.AppendUnique(CCFLAGS = '/this /is /a /test')

giving in one case three arguments and the other a single quoted argument:

cl /Fotestfile.obj /c testfile.cpp /TP /nologo /this /is /a /test
cl /Fotestfile.obj /c testfile.cpp /TP /nologo "/this /is /a /test"

This was split out from #3876 as it is not specific to CPPDEFINES, and that issue was intended to be devoted to CPPDEFINES issues.

@mwichmann mwichmann added the Append Issues in Append/Prepend + Unique variants label Nov 14, 2022
@mwichmann mwichmann self-assigned this Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Append Issues in Append/Prepend + Unique variants
Projects
None yet
Development

No branches or pull requests

1 participant