From 10f1cf65f1644ce24cba6d4fef324a0acba1f4c8 Mon Sep 17 00:00:00 2001
From: Jose Perez Rodriguez <joperezr@microsoft.com>
Date: Tue, 5 Feb 2019 16:18:14 -0800
Subject: [PATCH] Adding readme to source package and changing the nested file
 path. (dotnet/corefx#35089)

* Adding readme to source package and changing the nested file path.

* Address feedback and add README.md to None

* Adding new class to descriptions.json


Commit migrated from https://github.com/dotnet/corefx/commit/92eda9e19a886cbf7f8c7e97680df42a3ca1a2be
---
 src/libraries/System.Text.Json/Directory.Build.props |  4 ++++
 .../pkg/Microsoft.Bcl.Json.Sources.pkgproj           |  2 ++
 .../System.Text.Json/src/System.Text.Json.csproj     | 12 +++++++++---
 src/libraries/pkg/descriptions.json                  |  3 ++-
 4 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/src/libraries/System.Text.Json/Directory.Build.props b/src/libraries/System.Text.Json/Directory.Build.props
index efdc2a5d029..8c36c9f7347 100644
--- a/src/libraries/System.Text.Json/Directory.Build.props
+++ b/src/libraries/System.Text.Json/Directory.Build.props
@@ -6,5 +6,9 @@
     <StrongNameKeyId>Open</StrongNameKeyId>
     <IsNETCoreApp>true</IsNETCoreApp>
     <IsUAP>true</IsUAP>
+
+    <!-- Properties for source package -->
+    <_sourcePackageName>Microsoft.Bcl.Json.Sources</_sourcePackageName>
+    <_packagePathToSources>contentFiles/cs/netstandard2.0/$(_sourcePackageName)</_packagePathToSources>
   </PropertyGroup>
 </Project>
diff --git a/src/libraries/System.Text.Json/pkg/Microsoft.Bcl.Json.Sources.pkgproj b/src/libraries/System.Text.Json/pkg/Microsoft.Bcl.Json.Sources.pkgproj
index 31a52411784..e86f20ac4ad 100644
--- a/src/libraries/System.Text.Json/pkg/Microsoft.Bcl.Json.Sources.pkgproj
+++ b/src/libraries/System.Text.Json/pkg/Microsoft.Bcl.Json.Sources.pkgproj
@@ -6,6 +6,8 @@
   <PropertyGroup>
     <!-- We need to make sure that all of the surface area on the sources gets rewritten as internal. -->
     <GenerateInternalTypesSource>true</GenerateInternalTypesSource>
+    <!-- We need to disable package file check as the sdk targets will flag README.md file as it is not a known extension. -->
+    <SkipPackageFileCheck>true</SkipPackageFileCheck>
   </PropertyGroup>
 
   <!-- Given that System.Text.Json has Resources, we need to make sure we include them (and the SR.cs from Common) 
diff --git a/src/libraries/System.Text.Json/src/System.Text.Json.csproj b/src/libraries/System.Text.Json/src/System.Text.Json.csproj
index 90667746b77..fa73ef91c07 100644
--- a/src/libraries/System.Text.Json/src/System.Text.Json.csproj
+++ b/src/libraries/System.Text.Json/src/System.Text.Json.csproj
@@ -89,14 +89,19 @@
   <Target Name="GetSourcesToPackage" Returns="@(_sourceToPackage)" DependsOnTargets="Compile">
 
       <PropertyGroup>
+        <_sourcePackageName>Microsoft.Bcl.Json.Sources</_sourcePackageName>
         <_sourcePackageTargetLines><![CDATA[<?xml version="1.0" encoding="utf-8"?>
 <Project>
   <ItemGroup Condition="'%24(ExcludeEmbeddedResourceFromPackage)' != 'true'">@(EmbeddedResource->'
-    <EmbeddedResource Include="%24(MSBuildThisFileDirectory)%(FileName)%(Extension)" Link="System/Text/Json/%(FileName)%(Extension)" ManifestResourceName="%(ManifestResourceName)" />', '')
+    <EmbeddedResource Include="%24(MSBuildThisFileDirectory)%(FileName)%(Extension)" Link="$(_sourcePackageName)/%(FileName)%(Extension)" ManifestResourceName="%(ManifestResourceName)" />', '')
+  </ItemGroup>
+
+  <ItemGroup>
+    <None Include="%24(MSBuildThisFileDirectory)README.md" Link="$(_sourcePackageName)/README.md" Visible="true" />
   </ItemGroup>
 </Project>]]>
         </_sourcePackageTargetLines>
-        <_targetsFileName>Microsoft.Bcl.Json.Sources.targets</_targetsFileName>
+        <_targetsFileName>$(_sourcePackageName).targets</_targetsFileName>
         <_generatedSourcePackageTargetPath>$(IntermediateOutputPath)/$(_targetsFileName)</_generatedSourcePackageTargetPath>
       </PropertyGroup>
 
@@ -104,9 +109,10 @@
 
       <ItemGroup>
         <_sourceToExclude Include="@(Compile)" Condition="$([System.String]::Copy('%(Compile.Identity)').ToLower().EndsWith('assemblyinfo.cs'))" />
-        <_sourceToPackage Include="@(Compile->'%(FullPath)')" PackagePath="contentFiles/cs/netstandard2.0/System/Text/Json/%(FileName)%(Extension)" Exclude="@(_sourceToExclude)"/>
+        <_sourceToPackage Include="@(Compile->'%(FullPath)')" PackagePath="contentFiles/cs/netstandard2.0/$(_sourcePackageName)/%(FileName)%(Extension)" Exclude="@(_sourceToExclude)"/>
         <_sourceToPackage Condition="'%(Extension)' == '.resx'" Include="@(EmbeddedResource->'%(FullPath)')"  PackagePath="build/netstandard2.0/%(FileName)%(Extension)" />
         <_sourceToPackage Include="$(_generatedSourcePackageTargetPath)" PackagePath="build/netstandard2.0/$(_targetsFileName)" />
+        <_sourceToPackage Include="$(MSBuildThisFileDirectory)../source_package/README.md" PackagePath="build/netstandard2.0/README.md" />
       </ItemGroup>
   </Target>
 </Project>
diff --git a/src/libraries/pkg/descriptions.json b/src/libraries/pkg/descriptions.json
index 6e189a7ae0f..0512d12459d 100644
--- a/src/libraries/pkg/descriptions.json
+++ b/src/libraries/pkg/descriptions.json
@@ -1982,7 +1982,8 @@
     "Description": "Provides the source code implementation of System.Text.Json as internal types. This package is only required for projects targeting .NET Framework 4.6.1+ or .NET Core 2.x; and is not for projects that can target .NET Core 3.0 and higher. For guidance on consuming the source pacakge, visit: https://aka.ms/json-source-package-guide.",
     "CommonTypes": [
       "System.Text.Json.Utf8JsonReader",
-      "System.Text.Json.Utf8JsonWriter"
+      "System.Text.Json.Utf8JsonWriter",
+      "System.Text.Json.JsonDocument"
     ]
   },
   {
-- 
2.34.1