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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix windows build #291

Merged

Conversation

JeremieSpiesser
Copy link

Hello 馃憢,
I had two errors when building sanzu for windows.

  1. 404 error when running pacman -Syu
  2. Error because the tarball of ffmpeg was removed from the repo (due to the xz drama)

This PR makes it possible to build again for windows by

  1. Changing the archlinux mirror used
  2. Updating the ffmpeg tarball

On the "latest" channel, there is no n4.x version anymore for ffmpeg so I used version n6.1. It seems to work fine from my limited testing (machine with an intel gpu using h264_qsv)

@serpilliere
Copy link
Contributor

Hi @JeremieSpiesser !

Thank you for the fix. We were aware of the ffmpeg version, but made some "time delay" for some old OS (think Centos7). But it may be time to update it now, as we are reaching the EOL.

Can you signoff the commit to pass the DCO?

@JeremieSpiesser JeremieSpiesser force-pushed the fix-windows-build-30-04-2024 branch 2 times, most recently from af4684d to 4703bc0 Compare May 6, 2024 08:05
Jeremie Spiesser added 2 commits May 22, 2024 08:35
Signed-off-by: Jeremie Spiesser <jeremie.spiesser@qarnot-computing.com>
Signed-off-by: Jeremie Spiesser <jeremie.spiesser@qarnot-computing.com>
@serpilliere serpilliere force-pushed the fix-windows-build-30-04-2024 branch from ee6fb90 to 2682901 Compare May 22, 2024 06:36
@serpilliere serpilliere merged commit 8a6761f into cea-sec:main May 22, 2024
10 of 11 checks passed
@serpilliere
Copy link
Contributor

Thank you for the fix @JeremieSpiesser !

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.

None yet

2 participants