Uno.Core icon indicating copy to clipboard operation
Uno.Core copied to clipboard

Need to remove the dependency to CommonServiceLocator package

Open carldebilly opened this issue 5 years ago • 4 comments

Bug Report

Current behavior

Right now there's a strong dependency to CommonServiceLocator package defined in the csproj:

https://github.com/nventive/Uno.Core/blob/5aecd4909faeacb522fb303cb100c9de14eabb06/src/Uno.Core/Uno.Core.csproj#L34

This dependency can conflict with higher level frameworks implementing the CommonServiceLocator of a more recent version.

Since Uno.Core is forcing a pretty old version, it's expected those conflicts can occurs.

Expected behavior

The code should be refactored to completely remove this dependency.

carldebilly avatar Jun 19 '19 16:06 carldebilly

Just had the issue after referencing the latest version of MVVMLight which uses CommonServiceLocator 2.0.5 while Uno.Core still uses 1.3.0. I downgraded MVVMLight to make it work. Is this the recommended solution until this item is adressed ? @MatFillion

MakanWG avatar Feb 10 '20 02:02 MakanWG

@MakanWG This reference could be removed totally. I think it's used for logging, but we can (and should) use something else for that.

carldebilly avatar Feb 10 '20 16:02 carldebilly

It can be replaced, but it will make for a binary breaking change. We'll be doing a set of breaking changes like those along with the changes in AndroidX (which is going to force a whole set of other breaking changes)

jeromelaban avatar Feb 11 '20 05:02 jeromelaban

Related: #69.

weitzhandler avatar Apr 24 '20 09:04 weitzhandler