Skip to content
This repository has been archived by the owner on Oct 18, 2018. It is now read-only.

Include symbols in sharedfx #841

Closed
wants to merge 2 commits into from
Closed

Conversation

JunTaoLuo
Copy link
Contributor

Addresses part of #131. The aspnetcore shared framework now includes .ni.pdbs produced by crossgen. Note that on linux this will produce .map and on osx there are no debug symbols available after crossgen'ing

@JunTaoLuo
Copy link
Contributor Author

image
Contents of the shared framework after this change.

@natemcmaster
Copy link
Contributor

What is the debugger experience in VS like with these symbols present?

</ItemGroup>

<!-- Create symbols archive -->
<ZipArchive
File="$(SharedFxOutputPath)$(SharedFxInstallerName)-symbols-$(PackageVersion)-$(SharedFxRID).zip"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also update publish settings since we won't be producing this anymore?

<Target Name="GetFilesToPublish" DependsOnTargets="GetArtifactInfo;GeneratePublishFiles">
<PropertyGroup>
<BlobBasePath>Runtime/$(PackageVersion)/</BlobBasePath>
<AliasBlobBasePath>Runtime/$(SharedFxCliBlobChannel)/</AliasBlobBasePath>
<PackageArchiveFileName>nuGetPackagesArchive-$(PackageVersion).lzma</PackageArchiveFileName>
<InstallerBaseFileName>aspnetcore-runtime-$(PackageVersion)</InstallerBaseFileName>
<InstallerAliasBaseFileName>aspnetcore-runtime-latest</InstallerAliasBaseFileName>
<SymbolsArchiveBaseFileName>aspnetcore-runtime-symbols-$(PackageVersion)</SymbolsArchiveBaseFileName>
<IntermediateInstallerBaseFileName>aspnetcore-runtime-internal-$(PackageVersion)</IntermediateInstallerBaseFileName>
</PropertyGroup>
<ItemGroup>
<!-- symbols -->
<FilesToPublish Include="$(DependencyAssetsDir)$(SymbolsArchiveBaseFileName)-%(RuntimeSymbolsArchive.Identity)%(RuntimeSymbolsArchive.FileExt)" Condition=" '%(RuntimeSymbolsArchive.Identity)' != '' ">
<RelativeBlobPath>$(BlobBasePath)$(SymbolsArchiveBaseFileName)-%(RuntimeSymbolsArchive.Identity)%(RuntimeSymbolsArchive.FileExt)</RelativeBlobPath>
<ManifestArtifactData>ShipInstaller=dotnetcli</ManifestArtifactData>
</FilesToPublish>

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This too:

<RuntimeSymbolsArchive Include="win-x86" FileExt=".zip" />
<RuntimeSymbolsArchive Include="win-x64" FileExt=".zip" />
<RuntimeSymbolsArchive Include="linux-x64" FileExt=".tar.gz" />

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated. I'm updating my VS to test out the debugging experience.

@JunTaoLuo
Copy link
Contributor Author

No longer needed

@JunTaoLuo JunTaoLuo closed this Feb 5, 2018
@JunTaoLuo JunTaoLuo deleted the johluo/symbols-in-sharedfx branch February 5, 2018 19:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants