-
Notifications
You must be signed in to change notification settings - Fork 742
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
Make Uno.[Win]UI.<Lottie|DevServer> packages to work also on windows #14196
Comments
Any updates on this issue? My Windows project is not built since updated to latest version of Uno. |
Hello? |
@LITTOMA Can you provide more information about your issue? It's always best to provide logs, samples, etc... |
@jeromelaban It's exactly the same as what @dr1rrb mention in the post. The error output for me is like:
|
This is unfortunately not enough to troubleshoot your issue. If you have either Uno.WinUI.Lottie or Uno.WinUI.DevServer included in your |
Hello @LITTOMA ; were you able to troubleshoot this further? We've love to action this but would love to get more information please. |
Hi @sasakrsmanovic . I tried to remove Uno.WinUI.Lottie package from the windows project. However another error occurs on build: All packages I've installed: <ItemGroup>
<PackageReference Include="Uno.Extensions.Http.WinUI" />
<PackageReference Include="Uno.WinUI" />
<PackageReference Include="Microsoft.WindowsAppSDK" />
<PackageReference Include="Microsoft.Windows.SDK.BuildTools" />
<PackageReference Include="CommunityToolkit.Mvvm" />
<PackageReference Include="Uno.Extensions.Configuration" />
<PackageReference Include="Uno.Extensions.Http" />
<PackageReference Include="Uno.Extensions.Http.Refit" />
<PackageReference Include="Uno.Extensions.Logging.WinUI" />
<PackageReference Include="Uno.Extensions.Logging.Serilog" />
<PackageReference Include="Uno.Extensions.Serialization.Http" />
<PackageReference Include="Uno.Extensions.Serialization.Refit" />
<PackageReference Include="Uno.Toolkit.WinUI" />
<PackageReference Include="Uno.Extensions.Hosting.WinUI" />
<PackageReference Include="Uno.Extensions.Localization.WinUI" />
<PackageReference Include="Uno.Extensions.Navigation.Toolkit.WinUI" />
<PackageReference Include="Uno.Extensions.Navigation.WinUI" />
<PackageReference Include="Microsoft.Extensions.Logging.Console" />
<PackageReference Include="Uno.Core.Extensions.Logging.Singleton" />
<PackageReference Include="Uno.UI.Adapter.Microsoft.Extensions.Logging" />
<PackageReference Include="System.IO.Ports" />
</ItemGroup> |
Thanks. Can you provide a binlog of your build? It will help troubleshoot further. |
I suppose it has already been fixed with |
Yes, this has been fixed by the uno.sdk. Thanks! |
What would you like to be added
We should add a
_._
file and remove all dependencies on windows target.Why is this needed
Adding a reference to this packages in the core "library project" (from our app template) will break the windows build. Reference has to be conditional to NOT windows.
For which platform
No response
Anything else we need to know?
Even if our app template does include those reference conditionally to NOT windows, any dev that would add those references later in their app development (or while updating an exciting project) will encounter some weird build error:
Lottie
The windows head project build will fail with
DevServer
When adding
MainWindow.EnableHotReload();
The text was updated successfully, but these errors were encountered: