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

addpkg: svt-hevc, svt-vp9 #1578

Closed
wants to merge 1 commit into from

Conversation

TinySnow
Copy link
Contributor

These two package have same error: immintrin.h: No such file or directory.

I searched some talks, blogs, bugs or issues, finally ensured that this immintrin.h head file is for x86_64 architecture only.

Since I have no idea about how to avoid this troublesome error to let the package port successfully, so I just simply add these two packages to the blacklist.txt(below the Binary package that don't have RISC-V support) and REPORTED ITS UPSTREAM.

If there are any mistakes about the management, please dm me.

PERHAPS USEFUL LINKS:

UPSTREAM ISSUE LINKS:

@felixonmars
Copy link
Owner

Since they have sources available, they are definitely not "binary package"s and should not be added there.

IMHO they are just not ported yet, which doesn't belong to the blacklist.

@XieJiSS
Copy link
Collaborator

XieJiSS commented Aug 10, 2022

I suspect that at least the latter package can be built in compatibility mode, i.e. getting rid of AVX2/SSE. At least they do provide source code for this purpose: https://github.com/OpenVisualCloud/SVT-VP9/tree/d46acba4668817b2963fa55546a7b4f565c66120/Source/Lib/C_DEFAULT

@XieJiSS
Copy link
Collaborator

XieJiSS commented Aug 10, 2022

Also posting here to keep consistency:

I think removing https://github.com/OpenVisualCloud/SVT-VP9/blob/d46acba4668817b2963fa55546a7b4f565c66120/CMakeLists.txt#L227-L230 will make svt-vp9 build

@Xeonacid
Copy link
Collaborator

Close for now.

@Xeonacid Xeonacid closed this Aug 12, 2023
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.

4 participants