btrfsprogs: only install udev rules for udev >= 190
diff mbox

Message ID 57C62FC0.6020206@suse.com
State New
Headers show

Commit Message

Jeff Mahoney Aug. 31, 2016, 1:15 a.m. UTC
Prior to udev v190, there was no btrfs builtin helper.  Installing it on
systems with an older udev will cause problems.

Signed-off-by: Jeff Mahoney <jeffm@suse.com>
---

 configure.ac |    8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

Comments

David Sterba Sept. 1, 2016, 3:32 p.m. UTC | #1
On Tue, Aug 30, 2016 at 09:15:44PM -0400, Jeff Mahoney wrote:
> Prior to udev v190, there was no btrfs builtin helper.  Installing it on
> systems with an older udev will cause problems.
> 
> Signed-off-by: Jeff Mahoney <jeffm@suse.com>

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch
diff mbox

diff --git a/configure.ac b/configure.ac
index 97e89f2..8fd8f42 100644
--- a/configure.ac
+++ b/configure.ac
@@ -161,7 +161,13 @@  PKG_STATIC(UUID_LIBS_STATIC, [uuid])
 PKG_CHECK_MODULES(ZLIB, [zlib])
 PKG_STATIC(ZLIB_LIBS_STATIC, [zlib])
 
-UDEVDIR="$(pkg-config udev --variable=udevdir)"
+# udev v190 introduced the btrfs builtin and a udev rule to use it.
+# Our udev rule gives us the friendly dm names but isn't required (or valid)
+# on earlier releases.
+UDEVDIR=
+if pkg-config udev --atleast-version 190; then
+	UDEVDIR="$(pkg-config udev --variable=udevdir)"
+fi
 AC_SUBST(UDEVDIR)
 
 dnl lzo library does not provide pkg-config, let use classic way