Skip to content

Commit 350a683

Browse files
authored
Consume generator as a package (Azure#15133)
Generator, Autorest shared soure and targets are all packed into a single nuget package now.
1 parent b99eb63 commit 350a683

38 files changed

+46
-2240
lines changed

eng/CodeGeneration.targets

Lines changed: 2 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
<PropertyGroup>
44
<_AutoRestVersion>https://github.com/Azure/autorest/releases/download/autorest-3.0.6236/autorest-3.0.6236.tgz</_AutoRestVersion>
55
<_AutoRestCoreVersion>3.0.6306</_AutoRestCoreVersion>
6-
<_AutoRestCSharpVersion>https://github.com/Azure/autorest.csharp/releases/download/3.0.0-dev.20200911.1/autorest-csharp-v3-3.0.0-dev.20200911.1.tgz</_AutoRestCSharpVersion>
6+
<_AutoRestCSharpVersion>https://github.com/Azure/autorest.csharp/releases/download/3.0.0-dev.20200811.1/autorest-csharp-v3-3.0.0-dev.20200811.1.tgz</_AutoRestCSharpVersion>
77
<_SupportsCodeGeneration Condition="'$(IsClientLibrary)' == 'true'">true</_SupportsCodeGeneration>
88
<_DefaultInputName Condition="Exists('$(MSBuildProjectDirectory)/autorest.md')">$(MSBuildProjectDirectory)/autorest.md</_DefaultInputName>
99
<AutoRestInput Condition="'$(AutoRestInput)' == ''">$(_DefaultInputName)</AutoRestInput>
@@ -12,16 +12,11 @@
1212
use the following command line (remove the space between minus minus): dotnet msbuild /t:GenerateCode /p:AutoRestAdditionalParameters="- -interactive"
1313
-->
1414
<AutoRestAdditionalParameters></AutoRestAdditionalParameters>
15-
<_SharedCodeDirectory>$(MSBuildThisFileDirectory)../sdk/core/Azure.Core/src/Shared/</_SharedCodeDirectory>
16-
<_AutoRestSharedCodeDirectory>$(_SharedCodeDirectory)AutoRest/</_AutoRestSharedCodeDirectory>
1715

1816
<_GenerateCode Condition="'$(_SupportsCodeGeneration)' == 'true' AND '$(AutoRestInput)' != ''">true</_GenerateCode>
1917
</PropertyGroup>
2018

2119
<PropertyGroup>
22-
<_AutoRestVersion Condition="'$(TemporaryUsePreviousGeneratorVersion)' == 'true'">https://github.com/Azure/autorest/releases/download/autorest-3.0.6236/autorest-3.0.6236.tgz</_AutoRestVersion>
23-
<_AutoRestCoreVersion Condition="'$(TemporaryUsePreviousGeneratorVersion)' == 'true'">3.0.6306</_AutoRestCoreVersion>
24-
<_AutoRestCSharpVersion Condition="'$(TemporaryUsePreviousGeneratorVersion)' == 'true'">https://github.com/Azure/autorest.csharp/releases/download/3.0.0-dev.20200811.1/autorest-csharp-v3-3.0.0-dev.20200811.1.tgz</_AutoRestCSharpVersion>
2520
</PropertyGroup>
2621

2722
<Target Name="GenerateCode" Condition="'$(_GenerateCode)' == 'true'" >
@@ -38,17 +33,9 @@
3833
<Error Text="Following GitHub URLs do not contain commit hash: @(GithubUrlsWithoutHash) please use permalinks for code generation inputs (see https://help.github.com/en/github/managing-files-in-a-repository/getting-permanent-links-to-files) " Condition="'@(GithubUrlsWithoutHash)' != ''" />
3934

4035
<RemoveDir Directories="$(MSBuildProjectDirectory)/Generated"/>
41-
<Exec Command="npx autorest@$(_AutoRestVersion) --version=$(_AutoRestCoreVersion) $(AutoRestInput) $(AutoRestAdditionalParameters) --use=$(_AutoRestCSharpVersion) --output-folder=$(MSBuildProjectDirectory) --title=$(RootNamespace) --namespace=$(RootNamespace) --shared-source-folder=$(_SharedCodeDirectory)" />
36+
<Exec Command="npx autorest@$(_AutoRestVersion) --version=$(_AutoRestCoreVersion) $(AutoRestInput) $(AutoRestAdditionalParameters) --use=$(_AutoRestCSharpVersion) --output-folder=$(MSBuildProjectDirectory) --title=$(RootNamespace) --namespace=$(RootNamespace) --shared-source-folder=$(AutoRestSharedCodeDirectory).." />
4237
</Target>
4338

44-
<PropertyGroup Condition="'$(_GenerateCode)' == 'true'">
45-
<NoWarn>$(NoWarn);CA1812</NoWarn>
46-
</PropertyGroup>
47-
48-
<ItemGroup Condition="'$(_GenerateCode)' == 'true'">
49-
<Compile Include="$(_AutoRestSharedCodeDirectory)/**/*.cs" Link="Shared/AutoRest/%(RecursiveDir)/%(Filename)%(Extension)" />
50-
</ItemGroup>
51-
5239
</Project>
5340

5441

eng/Directory.Build.Data.props

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -152,4 +152,12 @@
152152
<!-- Disable doc comments for test projects -->
153153
<DocumentationFile></DocumentationFile>
154154
</PropertyGroup>
155+
156+
<!-- Code generation settings -->
157+
<PropertyGroup>
158+
<!-- Don't use code generator bundled Azure.Core shared source for code generation -->
159+
<AzureCoreSharedCodeDirectory>$(AzureCoreSharedSources)</AzureCoreSharedCodeDirectory>
160+
<!-- Don't use code generator bundled Azure.Core shared source when building, projects in this repo include individual sources when needed -->
161+
<IncludeAzureCoreSharedCode>false</IncludeAzureCoreSharedCode>
162+
</PropertyGroup>
155163
</Project>

eng/Directory.Build.Data.targets

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,10 @@
5050
<PackageReference Include="Microsoft.NETFramework.ReferenceAssemblies" PrivateAssets="all" IsImplicitlyDefined="true" />
5151
</ItemGroup>
5252

53-
<!-- Add Client SDK Analyzers -->
53+
<!-- Add Client SDK Tools -->
5454
<ItemGroup>
55+
<PackageReference Condition="'$(IsClientLibrary)' == 'true'" Include="AutoRest.CSharp.V3" PrivateAssets="All" />
56+
5557
<PackageReference Condition="'$(GenerateAPIListing)' == 'true'" Include="Microsoft.DotNet.GenAPI" PrivateAssets="All" />
5658

5759
<PackageReference Condition="'$(EnableClientSdkAnalyzers)' == 'true'" Include="Azure.ClientSdk.Analyzers" PrivateAssets="All" />
@@ -103,7 +105,7 @@
103105

104106
<Import Project="ApiListing.targets" />
105107

106-
<Import Project="CodeGeneration.targets" />
108+
<Import Project="CodeGeneration.targets" Condition="'$(TemporaryUsePreviousGeneratorVersion)' == 'true'" />
107109

108110
<Import Project="TestFramework.targets" Condition="'$(IsTestProject)' == 'true'"/>
109111

eng/Packages.Data.props

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
<ItemGroup>
1010
<PackageReference Update="ApprovalTests" Version="3.0.22" />
1111
<PackageReference Update="ApprovalUtilities" Version="3.0.22" />
12+
<PackageReference Update="AutoRest.CSharp.V3" Version="1.0.0-alpha.20200914.3" />
1213
<PackageReference Update="Azure.AI.FormRecognizer" Version="3.0.0" />
1314
<PackageReference Update="Azure.AI.TextAnalytics" Version="5.0.0" />
1415
<PackageReference Update="Azure.Data.AppConfiguration" Version="1.0.0" />

eng/scripts/CodeChecks.ps1

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ try {
7272

7373
Write-Host "Re-generating clients"
7474
Invoke-Block {
75-
& dotnet msbuild $PSScriptRoot\..\service.proj /t:GenerateCode /p:ServiceDirectory=$ServiceDirectory
75+
& dotnet msbuild $PSScriptRoot\..\service.proj /restore /t:GenerateCode /p:ServiceDirectory=$ServiceDirectory
7676

7777
# https://github.com/Azure/azure-sdk-for-net/issues/8584
7878
# & $repoRoot\storage\generate.ps1

0 commit comments

Comments
 (0)