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

Heat should escape dollar signs ($$) in file names #4562

Closed
wixbot opened this issue Oct 20, 2014 · 6 comments
Closed

Heat should escape dollar signs ($$) in file names #4562

wixbot opened this issue Oct 20, 2014 · 6 comments
Milestone

Comments

@wixbot
Copy link

wixbot commented Oct 20, 2014

When harvesting the file AuthController$$TestAuthParams.html, heat generates this component:

<File Id="fil50E06941A55A672424B7E311B8A289FC" KeyPath="yes" Source="$(var.WebAppSource)\share\doc\api\controllers\AuthController$$TestAuthParams.html" />

Light interprets the filename as AuthController$TestAuthParams.html, resulting in a LGHT0103 error (file not found).

Heat should instead escape $$ to $$$$ according to the docs:

The preprocessor treats the $ character in a special way if it is followed by a $ or (. If you want to use a literal $$, use $$$$ instead. Every two $ characters will be replaced with one. For example, $$$$$ will be replaced with $$$.

Originally opened by dwickern

@wixbot
Copy link
Author

wixbot commented Oct 23, 2014

Release changed from v3.8 to v3.x

@wixbot
Copy link
Author

wixbot commented Nov 3, 2014

Why is this a bug. According to the Quote you used for Reference: Every two $ characters will be replaced with one. The pre-processor is behaving as expected, with respect to the documented behavior. Would it dot be easier to have heat ignore Filenames with special characters. I'll try and tackle this one but implementimg a count of special characters will be tough.

Originally posted by eyoung100

@wixbot
Copy link
Author

wixbot commented Nov 4, 2014

The preprocessor behaves as expected, the problem is heat. The original file name had two $ characters and heat generated a component with only one $ according do the preprocessor escaping rules.

I opened a PR wixtoolset/wix3#163

Originally posted by dwickern

@wixbot
Copy link
Author

wixbot commented Nov 4, 2014

Since you submitted a pull request for the $, you should test the other special characters, if there are any.

Originally posted by eyoung100

@wixbot
Copy link
Author

wixbot commented Dec 5, 2014

wixtoolset/wix3#163

Originally posted by barnson
AssignedTo set to dwickern
Resolution set to fixed
Status changed from Open to Resolved

@wixbot
Copy link
Author

wixbot commented Dec 5, 2014

Originally changed by barnson
Release changed from v3.x to v3.10

@wixbot wixbot added this to the v3.10 milestone Dec 20, 2015
@wixbot wixbot closed this as completed Dec 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant