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

[dotnet core] build for linux-musl-arm64 #1174

Merged
merged 9 commits into from
Aug 15, 2022
Merged

[dotnet core] build for linux-musl-arm64 #1174

merged 9 commits into from
Aug 15, 2022

Conversation

bfren
Copy link
Contributor

@bfren bfren commented Apr 28, 2022

See #1173

@ektrah
Copy link
Contributor

ektrah commented Apr 30, 2022

Nice work 😃

The artifact (libsodium.so) also needs to be added to libsodium.pkgproj so it gets included in the NuGet package.

/cc @SeanMollet @enclave-alistair

@bfren
Copy link
Contributor Author

bfren commented Apr 30, 2022

Done - I think!

@ektrah
Copy link
Contributor

ektrah commented May 2, 2022

libsodium.pkgproj looks good to me. While you're at it, you could also increment the version number in the same file.

Now someone needs to go ahead and test if the NuGet package actually works on their musl arm64 machine 😊

@SeanMollet
Copy link
Contributor

@ektrah That can be tested in a docker container. I might have some time later this week to build a proper test script for it.

@bfren
Copy link
Contributor Author

bfren commented May 2, 2022

libsodium.pkgproj looks good to me. While you're at it, you could also increment the version number in the same file.

Happy to bump the version - to 1.0.19.0?

@ektrah
Copy link
Contributor

ektrah commented May 2, 2022

Happy to bump the version - to 1.0.19.0?

That would be 1.0.18.3 (the third revision of the NuGet package containing pre-compiled binaries of libsodium 1.0.18).

@bfren
Copy link
Contributor Author

bfren commented May 12, 2022

@SeanMollet I'm happy to help with writing test scripts - I presume I need to build the package locally to test it? The CI process doesn't appear to publish it anywhere as far as I can tell.

@jedisct1 jedisct1 changed the title Building for linux-musl-arm64 [dotnet core] build for linux-musl-arm64 Jun 5, 2022
@bfren
Copy link
Contributor Author

bfren commented Aug 14, 2022

Is there anything more I can do to help get this finalised and merged?

@MichalPetryka
Copy link

Could linux-musl-arm and linux-s390x be handled too?

@ektrah
Copy link
Contributor

ektrah commented Aug 14, 2022

@MichalPetryka, .NET doesn’t run on s390x. (Note that this particular PR is about the packaging of libsodium for .NET, not libsodium in general.)

@MichalPetryka
Copy link

@MichalPetryka, .NET doesn’t run on s390x. (Note that this particular PR is about the packaging of libsodium for .NET, not libsodium in general.)

It does with community builds, fedora package repository does provide s390x builds of it.

@bfren
Copy link
Contributor Author

bfren commented Aug 15, 2022

I'd be happy to do what I did for arm64 to add an arm build - would the maintainers prefer another PR for that? The issue now is testing though, which I'm happy to help with if I can, and if someone wants to give me a steer.

@jedisct1 jedisct1 merged commit be10bae into jedisct1:next Aug 15, 2022
jedisct1 pushed a commit that referenced this pull request Aug 15, 2022
* Adding aarch64 build

* Removing workflow_dispatch

* Re-adding x64

* Disable libgcrypt hash - #1173

* Adding sudo to gcrypt disable - #1173

* Remove sudo, put gcrypt disable before building source list - #1173

* Adding linux-musl-arm64 build to NuGet package - #1174

* Explain the gcrypt configuration hack
Repository owner locked and limited conversation to collaborators Nov 13, 2022
@bfren bfren deleted the next branch November 23, 2022 17:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants