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

Move Bundle/@CommandLineVariables into Bal.wixext #6858

Closed
rseanhall opened this issue Aug 7, 2022 · 1 comment
Closed

Move Bundle/@CommandLineVariables into Bal.wixext #6858

rseanhall opened this issue Aug 7, 2022 · 1 comment
Assignees
Labels
Milestone

Comments

@rseanhall
Copy link
Contributor

Feature requests

If this issue is a feature request:

  • Describe the scenario and benefits that the feature supports.

In v4, support was added for hidden variables to be masked in the log even if the name of the variable on the command line was a different case than the one declared in the bundle (#3777). The default is that all public variables should be uppercase, and Bal.wixext will throw an error if a variable is marked as overridable and it's not all uppercase when the bundle specified that all public variables should be uppercase. The built-in wixstdba themes use "InstallFolder" as the variable name in the options dialog, but if the user tries to mark this with bal:Overridable="yes" then that will fail to build.

  • Describe how you're accomplishing the feature today (if possible).

Custom theme or setting Bundle/@CommandLineVariables to caseSensitive.

  • Describe what you'd like the new feature to do.

Change the themes to use INSTALLFOLDER instead of InstallFolder.

@barnson barnson added burn and removed triage labels Aug 11, 2022
@barnson barnson added this to the v4.0 milestone Aug 11, 2022
@rseanhall
Copy link
Contributor Author

We discussed this and we're going to change how the Bundle/@CommandLineVariables work. The engine will always do a case insensitive match, and will hide the value if it finds any hidden variable with the name. It will be moved out of the core toolset and into Bal.wixext. The default setting will be changed to caseSensitive to match v3. The uppercase setting will change to caseInsensitive (this will require changes to how Bal.wixext validates the overridable variables aren't overlapping). This means the InstallFolder name doesn't need to change.

@rseanhall rseanhall changed the title Update wixstdba themes to use INSTALLFOLDER instead of InstallFolder Move Bundle/@CommandLineVariables into Bal.wixext Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants