Skip to content

Support Windows ARM64 #13

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

Merged
16 commits merged into from
Jun 3, 2021
Merged

Support Windows ARM64 #13

16 commits merged into from
Jun 3, 2021

Conversation

ghost
Copy link

@ghost ghost commented Jun 2, 2021

  • Cherry-pick commits for .NET Core/.NET 5 support from master
  • Cherry-pick test fixes from master
  • Enable ARM64 libgit2 loading

tmds and others added 16 commits June 2, 2021 12:18
…ad packed native libraries.

LibGit2Sharp includes a number of native libraries that are built on different OSes.
These native libraries have a dependency on OpenSSL and the system c-library.

On x64, in case the Linux flavor is not known, a fallback is performed to the 'linux-x64' native library.
This library is built with a dependency on OpenSSL 1.0.
OpenSSL 1.0 is deprecated by OpenSSL 1.1, so on more recent version of Linux, the 'linux-x64' library fails to load.

On arm64, native libraries are currently included for debian.* (OpenSSL 1.1) and ubuntu.18.04 (OpenSSL 1.0).
Loading on other distros will fail.

In both cases LibGit2Sharp is probably including a library that works, but the default resolution logic
is not able to find it.

By using the 'NativeLibrary' class (.NET Core 3.0) we can extend the resolution logic, and try to load
the other native libraries that are packed with LibGit2Sharp.
libgit2 validates objects when signing; update our test to use a valid
commit and tree object.
@ghost ghost force-pushed the A-Ovchinnikov/win-arm64-take2 branch from 96fd648 to f154641 Compare June 2, 2021 13:45
@ghost ghost marked this pull request as ready for review June 2, 2021 13:47
@ghost ghost requested a review from MarcellvanRooyen June 2, 2021 13:47
Copy link
Collaborator

@MarcellvanRooyen MarcellvanRooyen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

@ghost ghost merged commit 2504c52 into mendix/releases Jun 3, 2021
@ghost ghost deleted the A-Ovchinnikov/win-arm64-take2 branch June 3, 2021 09:54
ghost pushed a commit that referenced this pull request Jul 18, 2022
* Cherry-pick some commits from `master` branch
* Delete Travis, AppVeyor and Azure Pipelines CI
* Enable loading arm64 libgit2
* Use .NET 5.0 for tests

Co-authored-by: Tom Deseyn <tom.deseyn@gmail.com>
Co-authored-by: Brandon Ording <bording@gmail.com>
Co-authored-by: Andrew Arnott <andrewarnott@gmail.com>
Co-authored-by: Frederik Carlier <frederik.carlier@quamotion.mobi>
Co-authored-by: Edward Thomson <ethomson@edwardthomson.com>
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants