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

$(MSDeployPath) seems missing #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ir2000
Copy link

@ir2000 ir2000 commented Nov 6, 2014

at least on my machine I needed to add $(MSDeployPath) manually

at least on my machine I needed to add it manually
@sayedihashimi
Copy link
Owner

That shouldn't be required. Have you tried from another machine?

@ir2000
Copy link
Author

ir2000 commented Nov 10, 2014

Verified on another machine (Win7 / VS2012, same setup as mine) <Message Text="MSDeployPath: $(MSDeployPath)" Importance="high"/> shows "7>MSDeployPath: " in the output window.
Another thing I found was that using a saved password doesn't work either, the $(Password) would be empty. I needed to add the password in plain text either in the wpp.targets or the .pubxml <Password>1234</Password>

@sayedihashimi
Copy link
Owner

Is there anything funny about your machine setup? Did you install MSDeploy normally or are you trying to use an xcopy version of msdeploy.exe? If its a vanilla install then this shouldn't be required.

@ir2000
Copy link
Author

ir2000 commented Nov 11, 2014

Nothing funny, installers from microsoft.com installed with next, next, next... Only difference to your system might be german language.

@jlee1201
Copy link

jlee1201 commented Jul 2, 2015

I can also verify that MSDeployPath is empty on mine, and also that Password is empty (I also have to manually type password in plain text).

Using Win 8 / VS2013 Update 4

@jlee1201
Copy link

jlee1201 commented Jul 2, 2015

verified that ir2000's patch fixed MSDeployPath issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants