Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include PDB for all TfmRuntimeSpecificPackageFile #111879

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

ericstj
Copy link
Member

@ericstj ericstj commented Jan 27, 2025

Previously this would only include the PDB for the primary output which missed any other additions to TfmRuntimeSpecificPackageFile - such as those from references or packages.

Currently the only place this is done is

<ItemGroup Condition="'$(TargetPlatformIdentifier)' == 'windows'">
<ProjectReference Include="Messages\System.Diagnostics.EventLog.Messages.csproj"
ReferenceOutputAssembly="false"
OutputItemType="TfmRuntimeSpecificPackageFile"
PrivateAssets="true" />
</ItemGroup>

Previously this would only include the PDB for the primary output which
missed any other additions to TfmRuntimeSpecificPackageFile - such as
those from references or packages.
@ericstj ericstj requested review from ViktorHofer and a team January 27, 2025 23:42
@ericstj ericstj marked this pull request as ready for review January 27, 2025 23:43
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

@ericstj
Copy link
Member Author

ericstj commented Feb 3, 2025

/ba-g failing jobs were timeouts
Relevant job (all-configurations build) passed.

@ericstj ericstj merged commit fd9c728 into dotnet:main Feb 3, 2025
142 of 150 checks passed
@ericstj
Copy link
Member Author

ericstj commented Feb 4, 2025

/backport to release/9.0-staging

@ericstj
Copy link
Member Author

ericstj commented Feb 4, 2025

/backport to release/8.0-staging

Copy link
Contributor

github-actions bot commented Feb 4, 2025

Started backporting to release/9.0-staging: https://github.com/dotnet/runtime/actions/runs/13142353069

Copy link
Contributor

github-actions bot commented Feb 4, 2025

Started backporting to release/8.0-staging: https://github.com/dotnet/runtime/actions/runs/13142356380

grendello added a commit to grendello/runtime that referenced this pull request Feb 4, 2025
* main: (30 commits)
  JIT: Optimize bit-wise AND with a constant mask in combination with a left shift in a compare (dotnet#111979)
  Change how we build the cross-OS DAC to support building in the VMR (dotnet#111927)
  Add Windows Server 2025 to test configurations (dotnet#111938)
  [PERF] Move performance testing YAML from dotnet/runtime to dotnet/performance (dotnet#111454)
  arm64: Add support for Bitwise OR NOT & XOR NOT (dotnet#111893)
  JIT: Fix cross crossgen comparison failures (dotnet#112078)
  Bump `StyleCop.Analyzers` to `1.2.0-beta.556` (dotnet#111278)
  Remove `RequiresProcessIsolation` on InterfaceFolding tests (dotnet#112098)
  Use hardlinks in helixpublishwitharcade (dotnet#112091)
  Update breaking change rules regarding byref/objref fields. (dotnet#112087)
  [daccess] Do not use USE_DAC_TABLE_RVA on Apple platforms (dotnet#112076)
  use collection syntax in illink (dotnet#108458)
  Include PDB for all TfmRuntimeSpecificPackageFile (dotnet#111879)
  [main] Update dependencies from dotnet/emsdk (dotnet#111690)
  Enable Mono tests (dotnet#111981)
  Let the debugger knows DATAS is on (dotnet#107115)
  Tests ran counter (dotnet#111145)
  Some System.Decimal performance improvements (dotnet#99212)
  [mono][mini] Remove support for the Xamarin.iOS and Xamarin.Mac assemblies in the AOT compiler. (dotnet#108886)
  Remove one usage of `Unsafe.AsPointer`. (dotnet#112079)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants