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
32bit test fails on when built with -D_TIME_BITS=64 #216
Comments
Additional patch for compiling in chroot environment invoked by setarch i686. |
How exactly do you build this? The
I get the same failure with --- meson.build
+++ meson.build
@@ -55,6 +55,9 @@ if cc.has_function('__fxstatat', prefix: '#include <sys/stat.h>')
add_project_arguments('-DHAVE_FXSTATAT', language: 'c')
endif
+add_project_arguments('-D_TIME_BITS=64', language: 'c')
+add_project_arguments('-D_FILE_OFFSET_BITS=64', language: 'c')
+
meson.add_dist_script(srcdir / 'getversion.sh')
So I think the I tried this in current Debian testing i386. So how exactly did you try this? |
emits following error as you saw:
|
So, first, add #undef _TIME_BITS in src/libumockdev-preload.c:
|
So fix that too:
|
Now ========================
|
You have to re-wrap every functions otherwise redirected to __ioctl_time64 et al in src/libumockdev-preload.c . |
Unsetting `_FILE_OFFSET_BITS` causes time_t to be treated as 32-bit instead of the 64-bit that we want with `-D_TIME_BITS=64`. Fixes #216 Co-Authored-By: Steve Langasek <steve.langasek@canonical.com>
Unsetting `_FILE_OFFSET_BITS` causes time_t to be treated as 32-bit instead of the 64-bit that we want with `-D_TIME_BITS=64`. Fixes #216 Co-Authored-By: Steve Langasek <steve.langasek@canonical.com> Co-Authored-By: Martin Pitt <mpitt@debian.org>
Unsetting `_FILE_OFFSET_BITS` causes time_t to be treated as 32-bit instead of the 64-bit that we want with `-D_TIME_BITS=64`. Fixes #216 Co-Authored-By: Steve Langasek <steve.langasek@canonical.com> Co-Authored-By: Martin Pitt <mpitt@debian.org>
on 32bit environment with -D_TIME_BITS=64 -D_FILE_OFFSET_BITS-64,
/usr/bin/meson test
fails as below.Attached patch seems to fix this, but a dirty hack;
maybe should left out as open issue.
patch-time64.patch
The text was updated successfully, but these errors were encountered: