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

lttng-modules-dkms fails to build #12751

Closed
ranweiler opened this issue Jun 29, 2019 · 2 comments · Fixed by #12753
Closed

lttng-modules-dkms fails to build #12751

ranweiler opened this issue Jun 29, 2019 · 2 comments · Fixed by #12753

Comments

@ranweiler
Copy link
Contributor

System

  • xuname:
Void 4.19.56_1 x86_64 GenuineIntel notuptodate rrrdFFFF
  • package:
    lttng-modules-dkms-2.10.9_1

Expected behavior

When installing the package, the kernel modules build without error for installed kernels with headers.

Actual behavior

On installation, the kernel modules fail to build. The xbps console output includes:

Building DKMS module 'lttng-modules-2.10.9' for kernel-4.19.56_1... FAILED!
DKMS module 'lttng-modules-2.10.9' failed to build, please check /var/lib/dkms
for errors in the log file.
[...]
lttng-modules-dkms-2.10.9_1: installed successfully.

If we inspect the build log, we see:

$ cat /var/lib/dkms/lttng-modules/2.10.9/build/make.log
DKMS make.log for lttng-modules-2.10.9 for kernel 4.19.56_1 (x86_64)
Sat 29 Jun 2019 04:14:34 PM PDT
make: Entering directory '/usr/src/kernel-headers-4.19.56_1'
scripts/Makefile.build:45: /var/lib/dkms/lttng-modules/2.10.0/build/Makefile: No such file or directory
make[1]: *** No rule to make target '/var/lib/dkms/lttng-modules/2.10.0/build/Makefile'.  Stop.
make: *** [Makefile:1517: _module_/var/lib/dkms/lttng-modules/2.10.0/build] Error 2
make: Leaving directory '/usr/src/kernel-headers-4.19.56_1'

Steps to reproduce the behavior

Attempt to install the package via xbps-install -S lttng-modules-dkms.

Comments

The problem appears to be the first line of the dkms.conf file. It doesn't match the actual package/source version, so it ends up referring to a (in most cases) nonexistent directory.

I created a new (rev-bumped) package locally with that srcpkg file edited to read PACKAGE_VERSION=2.10.9. The module then builds as expected, and I was able to load it and trace kernel events.

I can open a PR with that narrow (manual) fix, but maybe there's a better way, that will ensure the file is auto-updated in the future.

@ranweiler
Copy link
Contributor Author

AFAICT, this should have been broken for a while (since it was introduced?). But maybe there's something else that changed that made things work before. Or, maybe the old (2.10.0) src directories were always on the package maintainer's system, so the incorrect package version never caused a failure (but instead quietly built an old version repeatedly). The build failure doesn't cause an XBPS package installation failure, either, so it wouldn't be caught in a CI system that only checked for that.

cc @maxice8, who probably has some insight here.

@maxice8
Copy link
Contributor

maxice8 commented Jun 30, 2019

I don't work on Void Linux anymore.

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 a pull request may close this issue.

2 participants