-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
x264: update to 0.164.3108 #27814
base: master
Are you sure you want to change the base?
x264: update to 0.164.3108 #27814
Conversation
- add maintainer Closes: https://trac.macports.org/ticket/62079
Notifying maintainers: |
multimedia/x264/Portfile
Outdated
# instead we can use the previous ppc assembly, which does compile and provides the same functions | ||
patchfiles-append patch-x264-older-ppc-code.diff \ | ||
patch-x264-apple-silicon.diff | ||
platform darwin powerpc { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does not touch any non-powerpc code, why not keep it unconditionally?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It changes the version string. It's also specific to Apple PPC not PPC generally.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not think that the code compliant with ISA 2.02 won’t compile or execute on the current POWER arch. VSX is an extension, it can be disabled when compiling for VSX-capable CPUs (GCC has -mno-vsx
flag for that).
It is also not Apple-specific, at least from the comment in the portfile. Both Linux and *BSD support Power arch without VSX insns support.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Presumably the performance will be worse? It seems this issue has been reported upstream and there's a simpler patch there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@barracuda156 or @kencu Can you try the updated patch (based on this comment) on PPC?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I certainly can test, but only in about a week, I got no access to my hardware until then. From comments it seems that verifying the build is not sufficient, we also need to test encoding/decoding.
Hopefully someone of Gentoo people verified that, but from the patch itself it is not clear whether something actually defines WORDS_BIGENDIAN
on a BE platform (it is not a standard macro defined by the compiler, AFAIR).
Description
Closes: https://trac.macports.org/ticket/62079
Tested on
macOS 15.3.1 24D70 arm64
Xcode 16.2 16C5032a
Verification
Have you
port lint
?sudo port -vst install
?