-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Compilation error when building against linux-rolling-stable branch headers #1123
Comments
This looks like configure script failed to detect that these definitions are present and the fallback definitions in LTP are enabled unexpectedly. Are you sure that you have run the configure script against the right headers? What is the |
Also, make sure you always run before compilation:
|
My build script: cp --archive --update --reflink=auto ltp "$BUILDDIR"
cd "$BUILDDIR/ltp"
make -j "$(nproc)" autotools
./configure
make -j "$(nproc)"
make install (Unrelated but it sucks that I can't do an out of tree build, it'd be great if that was possible) ./configure output:
|
Strange. BTW out of tree build might have some minor bugs, but generally it should be working. See https://github.com/linux-test-project/ltp/blob/master/INSTALL#L75 |
The configure check for the structure looks like:
We are not looking under |
@metan-ucw Something about ./configure is still going wrong:
|
@DaanDeMeyer this really sounds like a system paths are not set up properly, e.g. compiler looks into a different path than |
@metan-ucw It seems the xfs headers are broken somehow:
|
@metan-ucw |
@DaanDeMeyer that's really strange I did look at the git repository of xfs-progs and there isn't a single use of |
@metan-ucw This is on Fedora 39 with |
@wangli5665 Can you please have a look? Is it #1124 really needed? @DaanDeMeyer FYI we have Fedora 39 build: https://github.com/linux-test-project/ltp/actions/runs/7653264167/job/20854753494 |
This build already seems to differ in that it uses clang, I'm using gcc |
@DaanDeMeyer I just installed Fedora-39 VM and the configure script works as expected both with and without xfsprogs-devel installed. I can't reproduce the failure at all. |
On Fedora-39:
|
@metan-ucw I got the distro version wrong, the build is done in Fedora rawhide. This is with gcc 14. Reproduce instructions:
conftest.c from configure:
Results:
|
Given that rawhide is not yet released version it may be as well bug in the fedora. I suppose that something in the headers that are included from the Either way we do have a policy in LTP that we do not accept patches against unreleased codebases unless it's 100% clear that the change will make it into the next release. |
Thanks for trying to fix this, @DaanDeMeyer. Worst case we'll just have to wait until Rawhide is released, I guess and until then we might have to skip testing with LTP in mkosi-kernel. |
Any news here? |
Still broken on Rawhide, but given the effort to get fixes merged I kinda gave up on LTP |
Sorry for missing this issue, isn't this has been resolved by b3630b5 ? More discussion: https://lists.linux.it/pipermail/ltp/2024-June/038867.html |
The text was updated successfully, but these errors were encountered: