On 10/8/2019 4:27 PM, Alexey Semenyuk wrote:

Anyway - the code to get the wix tools version only once is not really a part of this fix, just triggered by your desire to print out all of the usage text from candle and light when getting their version with --verbose option.

So I can remove that from this fix.
Just making the fields non static would be good, I guess.

- Alexey


I can't do that, the two variables are used from a static method.

I might look at just avoiding calling validateWixTools() multiple times, but that still leaves 2 calls to findToolVersion(getLightPath())


/Andy

Reply via email to