Skip to content

Add Humanizer.Core,2.2.0 #553

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

Merged
merged 4 commits into from
Mar 3, 2023
Merged

Conversation

MichaelSimons
Copy link
Member

This is required as fallout from dotnet/source-build-externals#108 where source-build-externals wasn't versioning Humanizer.Core correctly after the update to 2.14.0.

There are existing SBRP's that depend on Humanizer.Core,2.2.0 which is now no longer provided by SBE. This wasn't caught earlier because the darc dependency flow for SBE wasn't working. After this change, there are no more dependencies from SBE, therefore the dependency was removed.

@MichaelSimons MichaelSimons enabled auto-merge (squash) March 3, 2023 20:29
@MichaelSimons MichaelSimons merged commit a3ee227 into dotnet:main Mar 3, 2023
@MichaelSimons MichaelSimons deleted the Humanizer branch March 4, 2023 00:26
@ViktorHofer
Copy link
Member

ViktorHofer commented Apr 14, 2023

I'm very concerned about external packages being part of SBRP. We don't control them and hence can't upgrade TFMs and their underlying targeting packs easily. Allowing externals here would limit SBRP substantially. In addition to that, re-hosting external source code sounds concerning from a legal stand-point as well (even though the repository has an MIT license).

There are existing SBRP's that depend on Humanizer.Core,2.2.0

Do you know which libraries depend on the old Humanizer.Core 2.2.0 version? I assume those aren't SBRP ones but libraries in the stack?

@MichaelSimons
Copy link
Member Author

MichaelSimons commented Apr 14, 2023

Do you know which libraries depend on the old Humanizer.Core 2.2.0 version? I assume those aren't SBRP ones but libraries in the stack?

The references are coming from SBRP packages:
/repos/dotnet/src/source-build-reference-packages/artifacts/source-build/self/src/artifacts/obj/Microsoft.CodeAnalysis.CSharp.Workspaces.4.0.1/project.assets.json
/repos/dotnet/src/source-build-reference-packages/artifacts/source-build/self/src/artifacts/obj/Microsoft.CodeAnalysis.Workspaces.Common.4.0.1/project.assets.json

Our options to remove these are:

  1. Upgrade the product references to eliminate these dependencies It looks like the only two come from runtime -
    • /repos/dotnet/src/source-build-reference-packages/artifacts/source-build/self/src/artifacts/obj/Microsoft.CodeAnalysis.CSharp.Workspaces.4.0.1/project.assets.json
    • /repos/dotnet/src/source-build-reference-packages/artifacts/source-build/self/src/artifacts/obj/Microsoft.CodeAnalysis.Workspaces.Common.4.0.1/project.assets.json
  2. Upgrade the SBRP reference to the Humanizer version we carry in source-build - 2.14.0
  3. Build multiple versions of Humanizer in source-build - Add 2.2.0 to source-build-externals

In addition to that, re-hosting external source code sounds concerning from a legal stand-point as well (even though the repository has an MIT license).

Not justifying it just calling it out, we do this in various text-only packages as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants