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

AVRO-3906: Add .NET 8.0 support #2589

Merged
merged 4 commits into from
Nov 22, 2023
Merged

AVRO-3906: Add .NET 8.0 support #2589

merged 4 commits into from
Nov 22, 2023

Conversation

zcsizmadia
Copy link
Contributor

What is the purpose of the change

Add .NET 8.0 support (AVRO-3906)

Verifying this change

This change is already covered by existing tests.

Documentation

  • Does this pull request introduce a new feature? (no)
  • If yes, how is the feature documented? (not applicable)

@zcsizmadia zcsizmadia changed the title AVRO-3906: Add NET8.0 support AVRO-3906: Add .NET 8.0 support Nov 19, 2023
@zcsizmadia
Copy link
Contributor Author

It seems like there is an issue with the C# ARM64 build if the XZ library package Joveler.Compression.XZ is bumped to the latest. The latest package works on my dev environment and succeeds with the standard github worklows, except on ARM64. Added comment about it in the versions.props.

lang/csharp/versions.props Show resolved Hide resolved
@zcsizmadia
Copy link
Contributor Author

@martin-g This PR fixes AVRO-3909. I wanted to wait for people give some suggestion before I merge, however this PR is fairly simple, so I can go ahead and merge it now to resolve the Newtonsoft vulnerability warning.

@zcsizmadia
Copy link
Contributor Author

Fyi, I created #2523 back in September asa preparation for the NET 8 analyzer warning about it, however it wont be needed any more, I will reject that PR once this is merged

@zcsizmadia zcsizmadia merged commit 5a279a9 into apache:main Nov 22, 2023
18 checks passed
@zcsizmadia zcsizmadia deleted the migrate-to-net8 branch November 22, 2023 13:56
@zcsizmadia
Copy link
Contributor Author

zcsizmadia commented Nov 22, 2023

Merged. @martin-g if you rebase those failing PRs with main it should be ok now.

RanbirK pushed a commit to RanbirK/avro that referenced this pull request May 13, 2024
* Add NEt8.0 support

* Add liblzma to ARM 64 C#

* Restore XZ library version

* Add comment for Joveler.Compression.XZ ARM64 issue

---------

Co-authored-by: Zoltan Csizmadia <[email protected]>
@stfnzl
Copy link

stfnzl commented Jun 21, 2024

Nice work! Definitely helps!
Is there a plan to release this on NuGet as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants