Add riscv64 build support by kasperk81 · Pull Request #3192 · mono/SkiaSharp

kasperk81 · 2025-03-09T00:09:25Z

Description of Change

Add linux-riscv64 and linux-musl-riscv64 support. Debian 12 with clang 19 and libfontconfig1_2.15.0-2 (http://deb.debian.org/debian/pool/main/f/fontconfig/) is a good baseline to start.

Bugs Fixed

Fixes [FEATURE] riscv64 flavor of libSkiaSharp and libHarfBuzzSharp #3191

API Changes

None.

Behavioral Changes

None.

Required skia PR

None.

PR Checklist

Has tests (if omitted, state reason in description)
Rebased on top of main at time of PR
Merged related skia PRs
Changes adhere to coding standard
Updated documentation

mattleibow · 2025-03-09T12:02:23Z

/azp run

azure-pipelines · 2025-03-09T12:02:31Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mattleibow · 2025-03-10T10:59:19Z

/azp run

azure-pipelines · 2025-03-10T10:59:26Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mattleibow · 2025-03-10T11:04:23Z

While we wait for the build to run again, are you able to confirm that the artifacts work in the first run? https://dev.azure.com/xamarin/public/_build/results?buildId=137319&view=results

mattleibow · 2025-03-10T11:34:00Z

/azp run

azure-pipelines · 2025-03-10T11:34:07Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mattleibow

Thanks for getting this PR up and iterating on it. So far CI has been green all the way.

This is looking done, except for 2 small issues:

the other 2 targets files also need the risc items
there are double files in the nuget (not because of this PR) and if you are able to just fix the wildcards

mattleibow · 2025-03-11T06:51:47Z

/azp run

azure-pipelines · 2025-03-11T06:51:54Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mattleibow · 2025-03-11T18:56:17Z

I see all the linux tests are failing with :

System.TypeInitializationException : The type initializer for 'SkiaSharp.SKImageInfo' threw an exception.
---- System.DllNotFoundException : Unable to load shared library 'libSkiaSharp' or one of its dependencies.

It could either be that the so is truly not there OR it may be the so is depending on something that is not there.

mattleibow · 2025-03-11T19:03:04Z

Actually, all of my CI is failing with that - even those with no changes... I wonder if the azure devops image is broken somehow.

Let me rebuild main and see what happens.

kasperk81 · 2025-03-11T19:03:23Z

the fix is #3194

mattleibow · 2025-03-12T03:01:29Z

Thanks for the work on this PR!

mattleibow · 2025-03-13T17:32:07Z

This build should be on the preview feed as nightly 19

aicynide · 2025-03-15T22:32:46Z

@kasperk81 please add linux-bionic build support also 🙏 for arm64, x64 and x86 Architectures
Please

kasperk81 · 2025-03-16T01:06:14Z

@aicynide, android has its own package https://www.nuget.org/packages/SkiaSharp.NativeAssets.Android

aicynide · 2025-03-16T02:28:04Z

@aicynide, android has its own package https://www.nuget.org/packages/SkiaSharp.NativeAssets.Android

Yes you are I think SkiaSharp my work under linux-bionic but issue is with sqlitePCL.raw libe_sqlite.so https://github.com/termux/termux-packages/actions/runs/13115509057/job/36588727595?pr=23074#step:6:2786
linux-bionic-{arch} is somewhat different from android. android-{arch} is for android but linux-bionic-{arch} is for Termux
I'm a termux package contributor and package maintainer, currently I'm working on porting jellyfin to termux and some dependencies of jellyfin like SQLITEPCL.raw nuget is available for linux-bionic

It would be a huge help if you can provide SQLITEPCL.raw 's libe_sqlite.so for linux-bionic

Thank you

mattleibow · 2025-03-16T03:36:08Z

@aicynide please open a new issue.

mattleibow · 2025-03-25T06:36:40Z

Hi folks,

The builds are now available on the preview feed: https://aka.ms/skiasharp-eap/index.json
Let me know if version 3.119.0-preview.0.38 is now working again for you.

As soon as you let me know, I can release this to NuGet and then hopefully a stable version shortly afterwards.

mono/SkiaSharp#3192 mono/SkiaSharp#3194 mono/SkiaSharp#3198 mono/SkiaSharp#3200

…18571) * SkiaSharp.props:Update it to support LoongArch64 and RISC-V 64 mono/SkiaSharp#3198 mono/SkiaSharp#3192 mono/SkiaSharp#3200 https://www.nuget.org/packages/SkiaSharp.NativeAssets.Linux/3.119.0-preview.1.2 * HarfBuzzSharp.props:Support LoongArch64 and RISC-V64 mono/SkiaSharp#3198 mono/SkiaSharp#3192 mono/SkiaSharp#3200 https://www.nuget.org/packages/HarfBuzzSharp.NativeAssets.Linux/8.3.1.1-preview.1.2 * Update SkiaSharp.props * Update HarfBuzzSharp.props --------- Co-authored-by: Maxwell Katz <maxkatz6@outlook.com>

github-project-automation Bot added this to SkiaSharp Backlog Mar 9, 2025

dotnet-policy-service Bot added the community ✨ label Mar 9, 2025

Add riscv64 build support

937cbbd

fix arm64

7de256d

mattleibow previously approved these changes Mar 10, 2025

View reviewed changes

Update native/linux-clang-cross/build.cake

aaa80d7

kasperk81 dismissed mattleibow’s stale review via aaa80d7 March 10, 2025 11:03

mattleibow approved these changes Mar 10, 2025

View reviewed changes

mattleibow requested changes Mar 10, 2025

View reviewed changes

github-project-automation Bot moved this to Changes Requested in SkiaSharp Backlog Mar 10, 2025

Fix so pattern

f0468c1

kasperk81 mentioned this pull request Mar 11, 2025

[BUG] vulkanmemallocator is missing snprintf header #3193

Closed

mattleibow approved these changes Mar 11, 2025

View reviewed changes

mattleibow reviewed Mar 11, 2025

View reviewed changes

Comment thread binding/Binding.Shared/LibraryLoader.cs

mattleibow reviewed Mar 11, 2025

View reviewed changes

Comment thread native/linux-clang-cross/build.cake

Merge branch 'main' into main

ba912b7

mattleibow merged commit e5f74e7 into mono:main Mar 12, 2025

github-project-automation Bot moved this from Changes Requested to Done in SkiaSharp Backlog Mar 12, 2025

kasperk81 mentioned this pull request Mar 13, 2025

Extend clang-cross to support alpine #3200

Merged

5 tasks

kasperk81 mentioned this pull request Mar 13, 2025

Add LoongArch64 build support #3198

Merged

4Darmygeometry added a commit to 4Darmygeometry/SteamTools that referenced this pull request Mar 29, 2025

Program.cs:SkiaSharp的龙架构和RISC-V支持已经主线化了

1c13f6b

mono/SkiaSharp#3192 mono/SkiaSharp#3194 mono/SkiaSharp#3198 mono/SkiaSharp#3200

4Darmygeometry mentioned this pull request Apr 2, 2025

Update SkiaSharp and HarfBuzzSharp version to support LA64 and RV64 AvaloniaUI/Avalonia#18571

Merged

3 tasks

Uh oh!

Conversation

kasperk81 commented Mar 9, 2025

Uh oh!

mattleibow commented Mar 9, 2025

Uh oh!

azure-pipelines Bot commented Mar 9, 2025

Uh oh!

mattleibow commented Mar 10, 2025

Uh oh!

azure-pipelines Bot commented Mar 10, 2025

Uh oh!

mattleibow commented Mar 10, 2025

Uh oh!

mattleibow commented Mar 10, 2025

Uh oh!

azure-pipelines Bot commented Mar 10, 2025

Uh oh!

mattleibow left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mattleibow commented Mar 11, 2025

Uh oh!

azure-pipelines Bot commented Mar 11, 2025

Uh oh!

mattleibow commented Mar 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mattleibow commented Mar 11, 2025

Uh oh!

kasperk81 commented Mar 11, 2025

Uh oh!

mattleibow commented Mar 12, 2025

Uh oh!

mattleibow commented Mar 13, 2025

Uh oh!

aicynide commented Mar 15, 2025

Uh oh!

kasperk81 commented Mar 16, 2025

Uh oh!

aicynide commented Mar 16, 2025

Uh oh!

mattleibow commented Mar 16, 2025

Uh oh!

mattleibow commented Mar 25, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

mattleibow commented Mar 11, 2025 •

edited

Loading