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

update jellyfin 10.8.1 #5335

Closed
wants to merge 8 commits into from
Closed

update jellyfin 10.8.1 #5335

wants to merge 8 commits into from

Conversation

stevenliuit
Copy link
Contributor

Description

update jellyfin 10.8.1
update dotnet 6.0.301

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Bug fix
  • New Package
  • Package update
  • Includes small framework changes
  • This change requires a documentation update (e.g. Wiki)

@publicarray
Copy link
Member

Nice work, Thanks for your contribution. I would like to have .NET 6 be fixed for non x64 platforms. See #5315 The DSM libs are too old for the new .NET versions.

@ta264 has been working on a chroot/bubblewrap/light wrapper to make *arr packages run with .NET 6: https://github.com/Servarr/spksrc/commits/master

@ta264
Copy link
Contributor

ta264 commented Jul 3, 2022

I think the bubblewrap workaround is good for the everything but armada370. I tried to compile a version that would support that cpu but didn't manage. I think that (and therefore the ds213j) is dead for .NET apps. Otherwise I think all arm platforms are working in my packages. If you're happy with the approach @publicarray I can start making some merge requests here

@stevenliuit
Copy link
Contributor Author

I think the bubblewrap workaround is good for the everything but armada370. I tried to compile a version that would support that cpu but didn't manage. I think that (and therefore the ds213j) is dead for .NET apps. Otherwise I think all arm platforms are working in my packages. If you're happy with the approach @publicarray I can start making some merge requests here

No problem, I think @publicarray's idea is pretty cool.

@publicarray
Copy link
Member

publicarray commented Jul 3, 2022

No problem, I think @publicarray's idea is pretty cool.

I really can't take any credit for this, it's all @ta264's excellent work and idea.

@ta264

Yea thanks, I acknowledge that ideally it is Synology who should fix it , but we would wait an eternity until that happens. I think bubblewrap is the best way forward, it should also help in the future when who knows what DSM update will break something again. Thanks again for working on this it's the best we have and I welcome any PRs you want to merge.

I can also make you a member if this will make things easier for your packages. (I also respect if you want to start hosting your own but this would make it easier for existing users)

@hgy59 Any thoughts or objections?

@stevenliuit
Copy link
Contributor Author

No problem, I think @publicarray's idea is pretty cool.

I really can't take any credit for this, it's all @ta264's excellent work and idea.

@ta264

Yea thanks, I acknowledge that ideally it is Synology who should fix it , but we would wait an eternity until that happens. I think bubblewrap is the best way forward, it should also help in the future when who knows what DSM update will break something again. Thanks again for working on this it's the best we have and I welcome any PRs you want to merge.

I can also make you a member if this will make things easier for your packages. (I also respect if you want to start hosting your own but this would make it easier for existing users)

@hgy59 Any thoughts or objections?

The jellyfin official website has updated the version of dotnet in 10.8.1
#7984 [@crobibero] Upgrade to dotnet 6.0.6, update remaining dependencies
this url: jellyfin/jellyfin#7984

update list:
https://github.com/dotnet/core/blob/main/release-notes/6.0/6.0.6/6.0.6.md
Also updates
SkiaSharp
SkiaSharp.NativeAssets.Linux
Sharpcompress

@stevenliuit
Copy link
Contributor Author

@publicarray Can we get a first release? @ta264 hasn't committed the code yet.

@publicarray
Copy link
Member

Continued here: #5382

@publicarray publicarray closed this Aug 5, 2022
@publicarray publicarray added the dotnet Related to dotnet (core) label Aug 6, 2022
@stevenliuit stevenliuit deleted the updatejellyfin branch August 23, 2022 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dotnet Related to dotnet (core)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants