Update DemaConsulting.Rendering.Skia to 0.2.0 and add SkiaSharp assets - #59
Merged
Merged
Conversation
…ve assets Rendering.Skia 0.2.0 no longer depends on SkiaSharp.NativeAssets.* (managed SkiaSharp core only - see demaconsulting/Rendering PR that removed the library-baked, OS-conditioned native asset references, which were unsafe for a packed library and could collide with other SkiaSharp native asset package IDs in a consumer's dependency graph). Bump DemaConsulting.Rendering.Svg/Skia to 0.2.0 in both the tool and its test project, and add explicit SkiaSharp.NativeAssets.* PackageReferences at these two consumption points, which are the correct application/publish boundaries for that choice: - DemaConsulting.SysML2Tools.Tool.csproj: unconditioned Win32/macOS and Linux.NoDependencies references (the tool is packed once as a single cross-platform dotnet tool, with TrimPublishedNativeAssets curating the final RID set - unchanged behavior, verified below). - DemaConsulting.SysML2Tools.Tests.csproj: OS-conditioned references (this project builds and runs on a single host per CI job, never packed for cross-platform distribution). Verified on native Ubuntu (WSL): - Build succeeds, 0 warnings/errors. - All non-skipped tests pass (net8.0/net10.0; net9.0 runtime is simply not installed in this WSL image). - Published tool contains exactly the curated native asset RIDs, no stray .pdb files, and the Linux binary is confirmed (by hash) to be the SkiaSharp.NativeAssets.Linux.NoDependencies flavor as intended. - The published tool runs successfully on Linux (--version loads libSkiaSharp.so without error). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This was referenced Aug 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates both the tool and test projects to use newer versions of the rendering packages and to explicitly manage native SkiaSharp assets, ensuring proper cross-platform support for both the application and its tests.
Dependency Updates:
DemaConsulting.Rendering.SvgandDemaConsulting.Rendering.Skiapackage references from version0.1.0to0.2.0in bothDemaConsulting.SysML2Tools.Tool.csprojandDemaConsulting.SysML2Tools.Tests.csproj. [1] [2]Native Asset Management:
DemaConsulting.SysML2Tools.Tool.csproj, added unconditional package references forSkiaSharp.NativeAssets.Win32,SkiaSharp.NativeAssets.Linux.NoDependencies, andSkiaSharp.NativeAssets.macOSto ensure all native assets are included for cross-platform packaging.DemaConsulting.SysML2Tools.Tests.csproj, added OS-conditioned package references for the appropriateSkiaSharp.NativeAssetspackage based on the host OS, ensuring the test project provides the correct native asset at runtime.