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 throws ArgumentNullException #2246

Open
wixbot opened this issue Jun 27, 2009 · 6 comments
Open

heat throws ArgumentNullException #2246

wixbot opened this issue Jun 27, 2009 · 6 comments
Labels
Milestone

Comments

@wixbot
Copy link

wixbot commented Jun 27, 2009

Obviously a minor issue, but if bad command line arguments (I'd forgotten to specify "file" before the filename) are passed to heat (RC2/3.0.5217.0), then it throws System.ArgumentNullException

E:\trunk\Software\bin\Release.Win32>"c:\Program Files\Windows Installer XML v3\bin\heat.exe" CADEV060.OCX -out z.z
Microsoft (R) Windows Installer Xml Toolset Harvester version 3.0.5217.0
Copyright (C) Microsoft Corporation. All rights reserved.

heat.exe : error HEAT0001 : Value cannot be null.
Parameter name: argument

Exception Type: System.ArgumentNullException

Stack Trace:
at Microsoft.Tools.WindowsInstallerXml.Harvester.Harvest(String argument)
at Microsoft.Tools.WindowsInstallerXml.Tools.Heat.Run(String[] args)

Originally opened by john-aldridge from http://sourceforge.net/p/wix/bugs/1745/

@wixbot
Copy link
Author

wixbot commented Sep 21, 2012

Indeed. We moved this to 4.0 as there will be a lot of work around command line and overall functionality. I put the example in place for others 'glancing' at the bug for support.

The command line handling is poor and something that needs to be revamped. It would definitely be a breaking change to v3.5.

@wixbot
Copy link
Author

wixbot commented Sep 21, 2012

Yes indeed... that's what I said. This bug report is just noting that crashing is a rather ungraceful way to report invalid command line arguments!

@wixbot
Copy link
Author

wixbot commented Sep 21, 2012

heat.exe CADEV060.OCX -out z.z

try

heat.exe file CADEV060.OCX -out z.z

@wixbot
Copy link
Author

wixbot commented Dec 12, 2013

verify

Originally posted by barnson
Area changed from heat to
Releasechanged fromfuturetov3.x

@wixbot
Copy link
Author

wixbot commented Jan 22, 2014

Originally changed by bmurri
AssignedTo set to wixsupport

@wixbot
Copy link
Author

wixbot commented Jul 19, 2015

In 3.10 and 4.0, heat issues an erroneous error message ("heat.exe : error HEAT0319 : The '-out' or '-o' parameter must specify a file path.") even with z.z replaced with a valid .wxs filename or even a complete path.

At least it no longer crashes, but it still doesn't present a correct error message.

Originally posted by bmurri
Status changed from Open to Untriaged

@wixbot wixbot added the bug label Dec 20, 2015
@wixbot wixbot added this to the v3.x milestone Dec 20, 2015
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

1 participant