Bug 8851 - Make sure fio can compile with libaio support if it the package is installed
Summary: Make sure fio can compile with libaio support if it the package is installed
Status: RESOLVED FIXED
Alias: None
Product: buildroot
Classification: Unclassified
Component: Other (show other bugs)
Version: unspecified
Hardware: All Linux
: P5 normal
Target Milestone: ---
Assignee: unassigned
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-04-15 01:56 UTC by Charles
Modified: 2016-04-19 21:40 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Charles 2016-04-15 01:56:37 UTC 
if libaio is being used, then make fio depend on it so that it gets compiled with aio support.

diff --git a/package/fio/fio.mk b/package/fio/fio.mk
index 08fef5f..384be2e 100644
--- a/package/fio/fio.mk
+++ b/package/fio/fio.mk
@@ -9,6 +9,10 @@ FIO_SITE = git://git.kernel.dk/fio.git
 FIO_LICENSE = GPLv2 + special obligations
 FIO_LICENSE_FILES = COPYING
 
+ifeq ($(BR2_PACKAGE_LIBAIO),y)
+FIO_DEPENDENCIES += libaio
+endif
+
 define FIO_CONFIGURE_CMDS
        (cd $(@D); ./configure --cc="$(TARGET_CC)" --extra-cflags="$(TARGET_CFLAGS)")
 endef
Comment 1 Thomas Petazzoni 2016-04-15 07:32:51 UTC 
Thanks, looks good. Could you submit this as a proper patch on our mailing list, so that we can merge it through the usual process? Thanks!
Comment 2 Thomas Petazzoni 2016-04-19 21:40:01 UTC 
Fixed by https://git.buildroot.net/buildroot/commit/?id=71122f537d2ff93f206818c8d91e06f55fe2b5e2, thanks for your suggestion!