kmod: needs largefile
PATCH v2 1/1 kmod: needs largefile
Since version 20 it requires largefile support.
http://git.kernel.org/cgit/utils/kernel/kmod/kmod.git/commit/?id=481ad14550087c77c9df6d117c7a975a38a4ad33
- Disable --disable-largefile. We already had possible bugs in the index implementation if this were passed. It's now fixed, but it's never tested. Disable it.
Fix build failure:
configure: error: --disable-largefile is not supported by kmod package/pkg-generic.mk:172: recipe for target 'output/build/kmod-20/.stamp_configured' failed
Fixes:
http://autobuild.buildroot.net/results/f662d9230e4bc2da7dc56a2ffd493cc37254d377//
http://autobuild.buildroot.net/results/233ec8ee346e15d41658cffc6bbb4a6c9dd46746//
---
Notes:
v2:
- Add sob line
package/kmod/Config.in | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
package/kmod/Config.in +4 -1
@@ -1,6 +1,6 @@ config BR2_PACKAGE_KMOD bool "kmod" - depends on !BR2_STATIC_LIBS + depends on !BR2_STATIC_LIBS && BR2_LARGEFILE help handle kernel modules
@@ -16,3 +16,6 @@ config BR2_PACKAGE_KMOD_TOOLS modinfo, modprobe, rmmod). endif + +comment "kmod needs a toolchain w/ largefile" + depends on !BR2_LARGEFILE
Signature · 1 line
2.3.1
Thomas Petazzoni
AppliedPATCH v2 1/1 kmod: needs largefile
Dear Fabio Porcedda,
Quoted 11 lines · Fabio Porcedda
Since version 20 it requires largefile support.
http://git.kernel.org/cgit/utils/kernel/kmod/kmod.git/commit/?id=481ad14550087c77c9df6d117c7a975a38a4ad33
- Disable --disable-largefile. We already had possible bugs in the index implementation if this were passed. It's now fixed, but it's never tested. Disable it.Fix build failure:
configure: error: --disable-largefile is not supported by kmodpackage/pkg-generic.mk:172: recipe for target 'output/build/kmod-20/.stamp_configured' failed
Quoted 6 lines
Fixes:
http://autobuild.buildroot.net/results/f662d9230e4bc2da7dc56a2ffd493cc37254d377//
http://autobuild.buildroot.net/results/233ec8ee346e15d41658cffc6bbb4a6c9dd46746//Signed-off-by Fabio Porcedda <[email protected]>Cc Baruch Siach <[email protected]>
Applied, after doing some minor changes:
[Thomas:
- use two separate 'depends on' conditions for BR2_STATIC_LIBS and BR2_LARGEFILE, like we do in most packages.
- take this opportunity to also indicate the dependency on dynamic library in the comment, which was missing until now.]
Thanks!
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com