Skip to content

Commit

Permalink
Enable or disable shipping different packages.
Browse files Browse the repository at this point in the history
* By default always ship WCF packages.
* By default do not ship dotnet-svcutil.xmlserializer packages.
* Enable or disable either one by setting their respective properties to true or false, can be done from the build definition that produces final release packages.
* Fixes dotnet#3975
  • Loading branch information
StephenBonikowsky committed Oct 25, 2019
1 parent 5802354 commit 9ea864a
Show file tree
Hide file tree
Showing 8 changed files with 9 additions and 1 deletion.
2 changes: 2 additions & 0 deletions eng/Versioning.props
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
<Project>
<PropertyGroup>
<GenerateAssemblyInfo Condition="'$(GenerateAssemblyInfo)'==''">true</GenerateAssemblyInfo>
<Ship_SvcUtilXmlSerPackages Condition="'$(Ship_SvcUtilXmlSerPackages)'==''">false</Ship_SvcUtilXmlSerPackages>
<Ship_WcfPackages Condition="'$(Ship_WcfPackages)'==''">true</Ship_WcfPackages>
</PropertyGroup>

<PropertyGroup Condition="'$(GenerateAssemblyInfo)'=='true'">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
<CLSCompliant>true</CLSCompliant>
<IsImplementationAssembly>true</IsImplementationAssembly>
<IsPackable>true</IsPackable>
<IsShipping>$(Ship_WcfPackages)</IsShipping>
</PropertyGroup>

<!-- [todo:arcade] Added this because our released S.P.SM package includes the "_BlockReflectionAttribute" but it is included not only in the UAP* assembly but also the other shipped assembly, should this only be in the binary built for uap? -->
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
<TargetFrameworks>$(BuildFrameworks)$(HarvestFrameworks)</TargetFrameworks>
<IsBuilding Condition="$(BuildFrameworks.Contains('$(TargetFramework);'))">true</IsBuilding>
<IsHarvesting Condition="$(HarvestFrameworks.Contains('$(TargetFramework);'))">true</IsHarvesting>
<IsShipping>$(Ship_WcfPackages)</IsShipping>
</PropertyGroup>

<ItemGroup Condition="'$(IsBuilding)' == 'true'">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
<TargetFrameworks>$(BuildFrameworks)$(HarvestFrameworks)</TargetFrameworks>
<IsBuilding Condition="$(BuildFrameworks.Contains('$(TargetFramework);'))">true</IsBuilding>
<IsHarvesting Condition="$(HarvestFrameworks.Contains('$(TargetFramework);'))">true</IsHarvesting>
<IsShipping>$(Ship_WcfPackages)</IsShipping>
</PropertyGroup>

<ItemGroup Condition="'$(IsBuilding)' == 'true'">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
<TargetFrameworks>$(BuildFrameworks)$(HarvestFrameworks)</TargetFrameworks>
<IsBuilding Condition="$(BuildFrameworks.Contains('$(TargetFramework);'))">true</IsBuilding>
<IsHarvesting Condition="$(HarvestFrameworks.Contains('$(TargetFramework);'))">true</IsHarvesting>
<IsShipping>$(Ship_WcfPackages)</IsShipping>
</PropertyGroup>

<ItemGroup Condition="'$(IsBuilding)' == 'true'">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
<TargetFrameworks>$(BuildFrameworks)$(HarvestFrameworks)</TargetFrameworks>
<IsBuilding Condition="$(BuildFrameworks.Contains('$(TargetFramework);'))">true</IsBuilding>
<IsHarvesting Condition="$(HarvestFrameworks.Contains('$(TargetFramework);'))">true</IsHarvesting>
<IsShipping>$(Ship_WcfPackages)</IsShipping>
</PropertyGroup>

<ItemGroup Condition="'$(IsBuilding)' == 'true' And '$(TargetFramework)' != 'net461'">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
<TargetFrameworks>$(BuildFrameworks)$(HarvestFrameworks)</TargetFrameworks>
<IsBuilding Condition="$(BuildFrameworks.Contains('$(TargetFramework);'))">true</IsBuilding>
<IsHarvesting Condition="$(HarvestFrameworks.Contains('$(TargetFramework);'))">true</IsHarvesting>
<IsShipping>$(Ship_WcfPackages)</IsShipping>
</PropertyGroup>

<ItemGroup Condition="'$(IsBuilding)' == 'true'">
Expand Down
2 changes: 1 addition & 1 deletion src/svcutilcore/src/dotnet-svcutil.xmlserializer.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
<DebugSymbols>true</DebugSymbols>
<PlatformTarget>AnyCPU</PlatformTarget>
<ProjectGuid>{EAB2959D-0A16-4F60-A453-765491E1C069}</ProjectGuid>
<IsShipping>false</IsShipping>
<IsShipping>$(Ship_SvcUtilXmlSerPackages)</IsShipping>
</PropertyGroup>

<!--Package Properties-->
Expand Down

0 comments on commit 9ea864a

Please sign in to comment.