-
Notifications
You must be signed in to change notification settings - Fork 255
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
Include Wix in Source Control #16
Comments
Hmm, looks like using resolve in the commit message closes an issue as well. Didn't mean that :-) Looks like there are still issues with the WiX Project, so I will re-open this issue for now. Need to test on a machine that doesn't have WiX currently installed, so I will create a Build Configuration on my TeamCity Server and continue working on this issue. |
I'm a bit lost because I cannot build the project any more. Visual studio gives me following error:
(Line 772 in wix2010.targets) I'm not familiar with wix or visual studio workflow. Are there additional steps (besides installing wix) that I need to take? |
Hello, This is completely me fault, and I am going to fix this at lunch time. For now, you should be able to continue by building just the Chocotely-Explorer Project. If you build the entire Solution, then you will see this error. I am trying to get to the solution to a state where Wix doesn't need to be installed on the machine that is doing the build. This is required for the MyGet integration that I am currently working on. Gary |
Okay, thank you for the hint :) |
Not a problem at all. I will let you know here, as soon as I fix what I broke :-) |
Just did a full solution build and it is still looking for the WixTasks dll:
which is probably why you referenced the build and deploy task (i'm moving slow this morning :P) |
Due to the projects gitignore file excluding anything in a bin folder, the necessary files were not checked in, and I missed that. I have now added these files into a folder called tools, and hopefully that will correct the issue.
Hey tarwn, Yip, these two issues are really intertwined :-) I have just checked in a change, and you and @cemrich do a get latest version and try building the solution again? The WiX project should now build the MSI without the need to have WiX installed locally. This means that we don't need to put it onto any build server that we are using. Gary |
Yes, now it does build just fine. |
Good to hear! There is still an issue on the MyGet Server build, but it is a different issue. I am working with the MyGet team to see if this can be resolved. Will keep this issue open up it is resolved. |
All done. The now works on the build server as well :-) |
When using the Build Services from MyGet, you see the following in the log file:
"D:\temp\tmp2F81\Setup_ChocolateyGUI\Setup_ChocolateyGUI.wixproj(43,3): error MSB4019: The imported project "C:\Program Files (x86)\MSBuild\Microsoft\WiX\v3.x\Wix.targets" was not found. Confirm that the path in the declaration is correct, and that the file exists on disk."
This would suggest that Wix is being referenced from the default installation location, rather than in Source Control.
Suggest that this is updated to change the Import Statement to the Source Control location.
Gary
The text was updated successfully, but these errors were encountered: