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.

Sign up for GitHub

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

Jump to bottom

Fix current CI #408

Merged
merged 7 commits into from
Jun 29, 2024
Merged

Fix current CI #408

merged 7 commits into from
Jun 29, 2024

Conversation

gnattu
Copy link
Member

@gnattu gnattu commented Jun 28, 2024 •
edited
Loading

Our current CI is partially broken and this tries to fix it.

  • Debian Buster is coming to EOL and we are not building server packages for it. The GCC it uses also lacks certain SIMD intrinsics required for the tonemapx filter.
  • Harfbuzz dropped autoconf build in 9.0 and we are still using that.
  • gnu website is not reliable

Changes

  • Drop Debian Buster from CI build matrix.
  • Use more reliable Linux Kernel mirror (mirrors.kernel.org) to replace the fragile ftp.gnu.org and ftpmirror.gnu.org
  • Pin harfbuzz commit until we are ready to for its new version
  • Use googlesource.com and github mirror instead of git.savannah.gnu.org

Issues

#407 Would require these changes to have the pipeline succeed.

@gnattu
ci: drop debian buster
a470ec3 
Debian Buster is reaching EOL at the end of this month. We are no
longer buiding server packages for this debian version, and its
gcc is too old to support required simd intrinsics
@gnattu
ci: use kernel.org gnu mirro
5bdce2f 
The GNU main site and ftpmirror.gnu.org is not reliable and usually
fails. Use the more stable Linux Kernel mirror instead.
@gnattu
ci: pin harfbuzz commit for windows image build
f09a0a3 
harfbuzz dropped autoconf build in 9.0. Use the spcific commit
we are using for dependency before we are ready to switch to menson.
@gnattu gnattu requested a review from a team June 28, 2024 22:34
nyanmisaka
nyanmisaka reviewed Jun 29, 2024
View reviewed changes
docker-build-win64.sh Outdated Show resolved Hide resolved
gnattu and others added 2 commits June 29, 2024 13:28
@gnattu @nyanmisaka
Apply suggestions from code review
efc31ad 
Co-authored-by: Nyanmisaka <nst799610810@gmail.com>
@gnattu
ci: remove debian buster
cafffc2
@nyanmisaka nyanmisaka merged commit 4edf5f8 into jellyfin Jun 29, 2024
25 checks passed
@nyanmisaka nyanmisaka deleted the ci-fix branch June 29, 2024 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shadowghost Shadowghost Shadowghost approved these changes

@nyanmisaka nyanmisaka nyanmisaka approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gnattu @Shadowghost @nyanmisaka