Open eric-914 opened 6 years ago
I've just discovered that these two options are incompatible with each other:
%msbuild% /t:Publish /p:DeployOnBuild=true ==> There is a circular dependency in the target dependency graph involving target "Publish"
%msbuild% /t:Publish /p:DeployOnBuild=true
This works: %msbuild% /t:Publish
%msbuild% /t:Publish
And this works: %msbuild% /p:DeployOnBuild=true /p:PublishProfile=MyProfile.pubxml
%msbuild% /p:DeployOnBuild=true /p:PublishProfile=MyProfile.pubxml
Notice that the /target option isn't part of this. But msbuild-command-builder.js (line 11) requires a targets option, and will error if undefined.
I have found this as a workaround for the time being: %msbuild% /t:Build /p:DeployOnBuild=true /p:PublishProfile=MyProfile.pubxml
%msbuild% /t:Build /p:DeployOnBuild=true /p:PublishProfile=MyProfile.pubxml
The fix, I believe, is to change line 11 to: if (options.targets) { args.push('/target:' + options.targets.join(';')); }
if (options.targets) { args.push('/target:' + options.targets.join(';')); }
I've just discovered that these two options are incompatible with each other:
%msbuild% /t:Publish /p:DeployOnBuild=true==> There is a circular dependency in the target dependency graph involving target "Publish"This works:
%msbuild% /t:PublishAnd this works:
%msbuild% /p:DeployOnBuild=true /p:PublishProfile=MyProfile.pubxmlNotice that the /target option isn't part of this. But msbuild-command-builder.js (line 11) requires a targets option, and will error if undefined.
I have found this as a workaround for the time being:
%msbuild% /t:Build /p:DeployOnBuild=true /p:PublishProfile=MyProfile.pubxmlThe fix, I believe, is to change line 11 to:
if (options.targets) { args.push('/target:' + options.targets.join(';')); }