jellyfin / jellyfin-ffmpeg

FFmpeg for Jellyfin
https://jellyfin.org
Other
478 stars 127 forks source link

Update CI to work with new repository layout #353

Closed joshuaboniface closed 6 months ago

joshuaboniface commented 6 months ago

Changes

  1. Use new path layout.
  2. Add major version calculation for use in path.
  3. Remove sha256sums.
  4. Drop non-LTS Ubuntus from builds (matches main server builds)

Issues N/A

nyanmisaka commented 6 months ago

There are two more leftovers: https://github.com/jellyfin/jellyfin-ffmpeg/blob/a1e482c4174d7d99d3769028c92b70bb47d780c4/.github/workflows/publish.yaml#L80-L81

nyanmisaka commented 6 months ago

The jellyfin-5.1 branch may also need to do the same thing, and we will need to maintain it for a while for backward compatibility reasons.

joshuaboniface commented 6 months ago

Done and done.

I see the MacOS build is failing, but I'm not sure if that's expected or not. Otherwise, hopefully all good!

nyanmisaka commented 6 months ago

@joshuaboniface FYI we will defer adding the noble(ubu2404) build target because upstream Debian is transitioning to glib2.0, which results in some dev libs being unavailable in the non-proposed repo. Hopefully they will be available around the time ubu2404 is released.

joshuaboniface commented 6 months ago

Sounds good @nyanmisaka yea I avoided adding it yet for the server CI too because the .NET repo isn't available for it yet :D