Replies: 2 comments 2 replies
-
Re the warnings as errors, I found what I was missing which were the WiX-specific MSBuild properties: <Project Sdk="WixToolset.Sdk/4.0.5">
<PropertyGroup>
<SuppressSpecificWarnings>1148</SuppressSpecificWarnings>
</PropertyGroup>
</Project> ... which then leads to the following validation error:
This can also be suppressed with another WiX-specific MSBuild property: <Project Sdk="WixToolset.Sdk/4.0.5">
<PropertyGroup>
<SuppressIces>ICE24</SuppressIces>
<SuppressSpecificWarnings>1148</SuppressSpecificWarnings>
</PropertyGroup>
</Project> So now I can coerce a build with warnings as errors, with the above suppressions, but the question remains - can this be safely ignored or will it eventually be promoted to an error? |
Beta Was this translation helpful? Give feedback.
-
No, not really. The behavior is undefined. There is a Deployment Dojo episode (maybe S1:E4 - Out with the Old. In with the New. ?) where I explore the ins and outs of it. If it isn't that episode, it's probably in one of the two after it. It's interesting to hypothesize what is going on because they don't document it. If you really want to do what you're doing with version numbers in your MSI file, you'll need to file a feature request with Microsoft. Note: Burn can handle such version numbers (and many more complex ones). |
Beta Was this translation helpful? Give feedback.
-
We're using "deterministic" versioning in builds using the git commit date time formatted in a way captured by the following PowerShell:
Note that leading zeroes, possible in the patch version (
Hmm
), are stripped so any NuGet package artifacts are SemVer 2.0 compliant.Anyway, I've just found out that one of our repos builds an installer using WiX, and since it binds the deterministic file version from a build artifact to the "product" version:
... it throws the following warning during the build:
We can't/won't drop using this deterministic versioning and we'd prefer to consistently use the same version everywhere, including for the MSI product version, so my question is can this be safely ignored or will it eventually be promoted to an error?
Also, there seems to be an issue that when building with "warnings as errors" this cannot be suppressed with:
What am I missing?
Beta Was this translation helpful? Give feedback.
All reactions