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

Library conflict with Microsoft.Bcl.Memory #178

Open
Nness opened this issue Mar 6, 2025 · 5 comments
Open

Library conflict with Microsoft.Bcl.Memory #178

Nness opened this issue Mar 6, 2025 · 5 comments

Comments

@Nness
Copy link

Nness commented Mar 6, 2025

Microsoft has released Microsoft.Bcl.Memory to allow use of index and range in .Net 4.6.2. Where IndexRange package start conflict with other Microsoft owned library due to this.

@Nness
Copy link
Author

Nness commented Mar 6, 2025

Refer to pull request #179

@Nness
Copy link
Author

Nness commented Mar 6, 2025

@saucecontrol when you have time, could you take look this pull request to resolve conflict on index and range in .Net 4.6.1 above projects.

@saucecontrol
Copy link
Owner

Hey, thanks for logging this. I'm aware of the new package, but I chose not to use it for now, because it brings its own set of issues.

The dependencies for the Microsoft.Bcl.Memory package are out of sync with the current version numbers of System.Memory's dependencies, which results in assembly load failures for some legacy project types unless the user manually creates all the necessary binding redirects.

The conflicts with Index and Range types can be resolved through aliasing if you add an explicit reference to the IndexRange package.

I realize this is not ideal, but unfortunately, this is a case where one or the other groups of users will be broken without manual intervention. Since the Microsoft package doesn't offer any benefits beyond reducing potential conflicts, I'm going to wait until they sort out their transitive dependencies before adopting their package.

@Nness
Copy link
Author

Nness commented Mar 9, 2025

That worked, thanks

@Nness Nness closed this as completed Mar 9, 2025
@saucecontrol
Copy link
Owner

That worked, thanks

Glad to hear it!

I'm going to leave this open for visibility and because I do hope to switch over at some point.

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

No branches or pull requests

2 participants