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 fails with error HEAT5305, when interpreting a path in an imported .targets file #4850

Open
wixbot opened this issue Aug 4, 2015 · 5 comments
Assignees
Milestone

Comments

@wixbot
Copy link

wixbot commented Aug 4, 2015

I have a .csproj file that imports a .targets file that itself imports another .targets file with this task:

<Import Project="$(MSBuildThisFileDirectory)\Hasp.targets"/>

The project including the original .targets file compiles correctly with Visual Studio but fails when used as a Heat reference in an Installer project. Worked fine with Wix 3.9.

Originally opened by melkor

@wixbot
Copy link
Author

wixbot commented Aug 4, 2015

We need more information to diagnose this: Please provide a repro and logs so we can see the problem.

@wixbot
Copy link
Author

wixbot commented Aug 5, 2015

As a consequence of us moving partially to VS2015 we've changed the root of our projects to be:

<Project ToolsVersion="$(VisualStudioVersion)" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003"/>

With VisualStudioVersion being an environment variable. If I run heat on the problematic project I get this:

Found ToolsVersion $(VisualStudioVersion) inside project file.
Loading project using MSBuild version $(VisualStudioVersion).
heat.exe : error HEAT5305 : Failed to load project D:\Users\Melkor\Projects\Insight\src\Insight.Router\Router.csproj: The imported project "D:\Hasp.targets" was not found. Confirm that the path in the <Import> declaration is correct, and that the file exists on disk.  D:\Users\Melkor\Projects\Insight\ext\Licensing\bin\Support.targets

If I change the ToolsVersion to 14.0 the error disappears.

Originally posted by melkor

@wixbot
Copy link
Author

wixbot commented Aug 11, 2015

Heat could be enhanced to support this variable use.

Release changed from v3.10 to v3.x

@wixbot
Copy link
Author

wixbot commented Aug 12, 2015

Originally changed by jmcooper8654
AssignedTo set to jocooper

@wixbot
Copy link
Author

wixbot commented Aug 12, 2015

I have been approved by my management to investigate this bug and come up with a proposed fix.

Originally posted by jmcooper8654

@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
Projects
None yet
Development

No branches or pull requests

2 participants