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

veracrypt-1.24-Hotfix1 fails to build on linux #529

Closed
david-geiger opened this issue Oct 28, 2019 · 9 comments
Closed

veracrypt-1.24-Hotfix1 fails to build on linux #529

david-geiger opened this issue Oct 28, 2019 · 9 comments

Comments

@david-geiger
Copy link

Hi,

latest release fails to build on Mageia Cauldron with:

/home/iurt/rpmbuild/BUILD/VeraCrypt-VeraCrypt_1.24-Hotfix1/src/Crypto/jitterentropy-base-user.h:52:16: error: conflicting declaration 'typedef uint64 __u64'

veracrypt_build_log.log

Regards,
David

@idrassi
Copy link
Member

idrassi commented Oct 28, 2019

Hi,

It looks like __u64 is already defined under Mageia but not on other distributions.
The fix is easy: just remove this line and replace all occurrences of __u64 by uint64.

I can push a commit for this but it would be helpful if you can first check that it works this way and maybe you could create a pull request for it.

@david-geiger
Copy link
Author

Hmmm it build on Mageia7 with GCC 8.3.1 but not on Cauldron with GCC 9.2.1, could it be the that?

@david-geiger
Copy link
Author

Also note that veracrypt 1.24 build fine on Cauldron, just the hotfix1 who doesn't build.

@idrassi
Copy link
Member

idrassi commented Oct 28, 2019

I have pushed a commit that should fix this error: 3fa636d

Can you pleas try it and report the result?
I think that indeed the issue is caused by gcc 9 usage.

@david-geiger
Copy link
Author

hmmm now it fails on aarch64 with:

In file included from ../Crypto/jitterentropy.h:47,
from ../Crypto/jitterentropy-base.c:67:
../Crypto/jitterentropy-base.c: In function 'jent_loop_shuffle':
../Crypto/jitterentropy-base-user.h:92:2: error: impossible constraint in 'asm'
92 | asm volatile("rdtsc" : EAX_EDX_RET(val, low, high));
| ^~~
make[1]: *** [/home/iurt/rpmbuild/BUILD/VeraCrypt-VeraCrypt_1.24-Hotfix1/src/Build/Include/Makefile.inc:25: ../Crypto/jitterentropy-base.o0] Error 1
make[1]: Leaving directory '/home/iurt/rpmbuild/BUILD/VeraCrypt-VeraCrypt_1.24-Hotfix1/src/Volume'
make[1]: *** Waiting for unfinished jobs....

@david-geiger
Copy link
Author

Same error on armv7h.

@idrassi
Copy link
Member

idrassi commented Oct 28, 2019

I have pushed a commit that should fix both failure: afe6b2f

Before this commit, the code always expected an x86/x86-64 platform.

@david-geiger
Copy link
Author

Excellent! now it build fine on all arches...
Thanks

@idrassi
Copy link
Member

idrassi commented Oct 29, 2019

Thank you for the confirmation.

@idrassi idrassi closed this as completed Oct 29, 2019
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

No branches or pull requests

2 participants