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

Unicode hyphen is not accepted as the start of an option on Wix 3.10 heat.exe #5183

Open
earthengine opened this issue Jan 13, 2016 · 5 comments

Comments

@earthengine
Copy link

I download and installed Wix 3.10 yesterday and found that this version no longer regards my "-srd" flag, and so always creates "Debug" or "Release" sub-folders when running Heat.exe.

I fixed this by adding the line to the filter xslt file:

  <xsl:template match="wix:Directory[@Name='Debug' or @Name='Release']">
      <xsl:apply-templates/>
   </xsl:template>

But ideally we should be able to use "-srd" option.

@barnson
Copy link
Member

barnson commented Jan 13, 2016

Works for me. What does your Heat.exe command line look like?

@earthengine
Copy link
Author

Emmm... I tried that again, it seems to work again. As I didn't touch anything rather than the project settings to remove and add back the "-srd" option, it confused me. But this was reported by our team member so it is not just me to have this problem. Let us keep an eye on this.

@earthengine
Copy link
Author

After further careful check on our git history, it seems that the old Wix (3.9 or earlier) heat.exe recognize the Unicode hyphen (U+2212) as the start of an option. But the 3.10 version does not.

Our original command line, was copied from some online blog and only made minimum changes for our project. However, nobody noticed there is a different hyphen (and it was the only one of all 10 hyphens in the command line).

Can you find the difference of the hyphens?

  –srd -o

:)

@earthengine earthengine changed the title "-srd" option does not work on Wix 3.10 heat.exe Unicode hyphen does not accepted as the start of an option on Wix 3.10 heat.exe Jan 14, 2016
@earthengine earthengine reopened this Jan 14, 2016
@earthengine earthengine changed the title Unicode hyphen does not accepted as the start of an option on Wix 3.10 heat.exe Unicode hyphen is not accepted as the start of an option on Wix 3.10 heat.exe Jan 14, 2016
@barnson barnson added this to the v3.x milestone Jan 15, 2016
@barnson
Copy link
Member

barnson commented Jan 15, 2016

Nobody during triage was excited about the feature but we'd consider such a change if someone did the work and it wasn't too disruptive to the command-line parsing.

@bsktcase
Copy link

OMG! This is exactly the problem I was having. I'll bet my predecessor copied from the same blog. Whether it ever gets fixed or not, bless you for posting the question - I don't think I would have spotted that errant hyphen in a million years. :)

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

3 participants