Skip to content

Commit f9bb17a

Browse files
author
Luke Bakken
committed
Publish does depend on nuget packages being present, and add a few windows checks
1 parent f116eee commit f9bb17a

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

build/build.targets

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,22 +9,24 @@
99
<Import Project="$(BuildPath)\assemblyinfo.targets" />
1010

1111
<Target Name="PublishValidateVersionString">
12+
<Error Condition="'$(OS)' != 'Windows_NT'" Text="Publish targets are Windows-only!" />
1213
<Error Condition="'$(VersionString)' == '' Or '$(VersionString)' == 'unknown'"
1314
Text="Must set VersionString property on command line!" />
1415
</Target>
1516

1617
<Target Name="PublishValidateBranch">
18+
<Error Condition="'$(OS)' != 'Windows_NT'" Text="Publish targets are Windows-only!" />
1719
<GitBranch LocalPath="$(SolutionDir)">
1820
<Output TaskParameter="Branch" PropertyName="GitBranch" />
1921
</GitBranch>
20-
<Message Condition="$(VPreRelease) != '' And '$(GitBranch)' != '$(MasterBranchName)'"
22+
<Message Condition="$(IsDryRun) == 'true' And $(VPreRelease) != '' And '$(GitBranch)' != '$(MasterBranchName)'"
2123
Text="NOTE: You are publishing from a non-master branch for pre-release version $(VPreRelease)" />
22-
<Error Condition="$(VPreRelease) == '' And '$(GitBranch)' != '$(MasterBranchName)'"
24+
<Error Condition="$(IsDryRun) == 'false' And $(VPreRelease) == '' And '$(GitBranch)' != '$(MasterBranchName)'"
2325
Text="Please ensure all features have been merged into $(MasterBranchName) and that you are on the $(MasterBranchName) branch." />
2426
</Target>
2527

2628
<Target Name="PublishValidation" DependsOnTargets="PublishValidateVersionString;PopulateVersionInformation;PublishValidateBranch">
27-
<Error Condition="'$(OS)' != 'Windows_NT'" Text="Publish target is Windows-only!" />
29+
<Error Condition="'$(OS)' != 'Windows_NT'" Text="Publish targets are Windows-only!" />
2830
<!-- Validate parsed VersionString -->
2931
<RegexMatch Input="$(VMajor)" Expression="^[1-9]$">
3032
<Output PropertyName="VMajorValidated" TaskParameter="Output" />
@@ -61,13 +63,15 @@
6163
</Target>
6264

6365
<Target Name="PublishTag" Condition="$(IsDryRun) == 'false'" DependsOnTargets="PublishValidation">
66+
<Error Condition="'$(OS)' != 'Windows_NT'" Text="Publish targets are Windows-only!" />
6467
<Message Text="Tagging with $(GitTagMessage)" />
6568
<GitClient Command="tag" Arguments="--message $(GitTagMessage)" LocalPath="$(SolutionDir)" />
6669
<Message Text="Pushing tag $(GitTagVersion). NOTE: this requires a remote named &quot;basho&quot;" />
6770
<GitClient Command="push" Arguments="basho &quot;$(GitTagVersion)&quot;" LocalPath="$(SolutionDir)" />
6871
</Target>
6972

7073
<Target Name="PublishBuildRelease" DependsOnTargets="GenerateCommonAssemblyInfo">
74+
<Error Condition="'$(OS)' != 'Windows_NT'" Text="Publish targets are Windows-only!" />
7175
<Message Text="MSBuild properties: Configuration=Release;SolutionDir=$(SolutionDir);VersionString=$(VersionString)" />
7276
<MSBuild Projects="$(SolutionFile)" Properties="Configuration=Release;SolutionDir=$(SolutionDir);VersionString=$(VersionString)" />
7377
</Target>
@@ -93,6 +97,7 @@
9397
</Target>
9498

9599
<Target Name="PublishNuGetPackage" Condition="$(IsDryRun) == 'false'" DependsOnTargets="PublishValidation">
100+
<Error Condition="'$(OS)' != 'Windows_NT'" Text="Publish targets are Windows-only!" />
96101
<XmlQuery XmlFileName="$(RiakClientPackagesConfig)" XPath="/packages/package[@id='Newtonsoft.Json']/@version">
97102
<Output TaskParameter="Values" PropertyName="NewtonsoftJsonVersion" />
98103
</XmlQuery>
@@ -121,6 +126,7 @@
121126
</Target>
122127

123128
<Target Name="PublishGitHubRelease" Condition="$(IsDryRun) == 'false'" DependsOnTargets="PublishValidation">
129+
<Error Condition="'$(OS)' != 'Windows_NT'" Text="Publish targets are Windows-only!" />
124130
<PropertyGroup Condition="'$(VPreRelease)' != ''">
125131
<IsPreReleaseArg>-IsPreRelease</IsPreReleaseArg>
126132
</PropertyGroup>
@@ -129,7 +135,7 @@
129135
</Target>
130136

131137
<Target Name="Publish"
132-
DependsOnTargets="PublishValidation;PublishTag;PublishBuildRelease;PublishNuGetPackage;PublishGitHubRelease" />
138+
DependsOnTargets="RestorePackages;PublishValidation;PublishTag;PublishBuildRelease;PublishNuGetPackage;PublishGitHubRelease" />
133139

134140
<Target Name="GitSubmodule" Condition="!Exists('$(ProtoMsgCsvFile)')">
135141
<GitClient Command="submodule" Arguments="update --init" LocalPath="$(SolutionDir)" />

0 commit comments

Comments
 (0)