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

Python fails to build on x86 Debian Non-Debug with X 3.x: error on Hacl_Hash_Blake2s_Simd128.c #125535

Closed
vstinner opened this issue Oct 15, 2024 · 8 comments
Labels
build The build process and cross-build

Comments

@vstinner
Copy link
Member

Build: https://buildbot.python.org/#/builders/1245/builds/3269

gcc -c -I./Modules/_hacl -I./Modules/_hacl/include -D_BSD_SOURCE -D_DEFAULT_SOURCE -fno-strict-overflow -Wsign-compare
 -DNDEBUG -g -O3 -Wall -UNDEBUG   -std=c11 -Wextra -Wno-unused-parameter -Wno-missing-field-initializers -Wstrict-prototypes
 -Werror=implicit-function-declaration -fvisibility=hidden  -I./Include/internal -I./Include/internal/mimalloc  -I. -I./Include
 -fPIC -msse -msse2 -msse3 -msse4.1 -msse4.2 -DHACL_CAN_COMPILE_VEC128
 -o Modules/_hacl/Hacl_Hash_Blake2s_Simd128.o ./Modules/_hacl/Hacl_Hash_Blake2s_Simd128.c

In file included from ./Modules/_hacl/internal/Hacl_Hash_Blake2s_Simd128.h:40,
                 from ./Modules/_hacl/Hacl_Hash_Blake2s_Simd128.c:26:
./Modules/_hacl/internal/../Hacl_Hash_Blake2s_Simd128.h:55:3: error: unknown type name ‘Lib_IntVector_Intrinsics_vec128’
   55 |   Lib_IntVector_Intrinsics_vec128 *fst;
      |   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

cc @gpshead

@vstinner
Copy link
Member Author

cc @zware, owner of the buildbot worker.

@vstinner
Copy link
Member Author

Same error on x86 Debian Installed with X 3.x: https://buildbot.python.org/#/builders/1244/builds/3253

@vstinner
Copy link
Member Author

"x86 Debian Non-Debug with X 3.x" and "x86 Debian Installed with X 3.x" workers are 32-bit machines, whereas Hacl_Hash_Blake2s_Simd128.c requires Lib_IntVector_Intrinsics_vec128 and Lib_IntVector_Intrinsics_vec128 is only defined on 64-bit architectures (#if defined(__x86_64__) || defined(_M_X64)).

@zware
Copy link
Member

zware commented Oct 15, 2024

Duplicate of gh-125244, but this will make searching for it a bit easier :)

@zware zware closed this as not planned Won't fix, can't repro, duplicate, stale Oct 15, 2024
@zware
Copy link
Member

zware commented Oct 15, 2024

Oh, I didn't realize that's a PR, not an issue. So this might as well stay open.

@zware zware reopened this Oct 15, 2024
@vstinner
Copy link
Member Author

Good that a PR was proposed to fix the issue!

@hugovk
Copy link
Member

hugovk commented Oct 15, 2024

Removing release-blocker because 32-bit Linux isn't a tiered platform: https://peps.python.org/pep-0011/

@zware
Copy link
Member

zware commented Oct 17, 2024

This is now resolved with the merge of GH-125244.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build The build process and cross-build
Projects
Development

No branches or pull requests

4 participants