From 39dabc84cfd5e8a8b2b94ef76430a4c30e5d2165 Mon Sep 17 00:00:00 2001
From: "Fabio M. Di Nitto" <fdinitto@redhat.com>
Date: Wed, 14 Oct 2009 04:10:41 +0000
Subject: [PATCH] configure.in: don't set default file owner and group

Per discussion on lvm-devel mailing list and part of debian patch set,
don't set defaults for owner and group, since nobody seems to use them, and
still allow override.
---
 WHATS_NEW    | 1 +
 configure    | 8 ++++----
 configure.in | 8 ++++----
 3 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/WHATS_NEW b/WHATS_NEW
index 5f06333a1..6ff56b9cc 100644
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,5 +1,6 @@
 Version 2.02.54 -
 =====================================
+  Set default owner and group to null.
   Add dmeventd.static to the build.
   Disable realtime support code by default.
   Make clvmd return 0 on success rather than 1.
diff --git a/configure b/configure
index eedc60cd1..7da2d3ced 100755
--- a/configure
+++ b/configure
@@ -1531,8 +1531,8 @@ Optional Features:
 Optional Packages:
   --with-PACKAGE[=ARG]    use PACKAGE [ARG=yes]
   --without-PACKAGE       do not use PACKAGE (same as --with-PACKAGE=no)
-  --with-user=USER        Set the owner of installed files [USER=root]
-  --with-group=GROUP      Set the group owner of installed files [GROUP=root]
+  --with-user=USER        Set the owner of installed files [USER=]
+  --with-group=GROUP      Set the group owner of installed files [GROUP=]
   --with-device-uid=UID   Set the owner used for new device nodes [UID=0]
   --with-device-gid=UID   Set the group used for new device nodes [GID=0]
   --with-device-mode=MODE Set the mode used for new device nodes [MODE=0600]
@@ -9078,7 +9078,7 @@ $as_echo "$STATIC_LINK" >&6; }
 ################################################################################
 { $as_echo "$as_me:$LINENO: checking file owner" >&5
 $as_echo_n "checking file owner... " >&6; }
-OWNER="root"
+OWNER=""
 
 
 # Check whether --with-user was given.
@@ -9096,7 +9096,7 @@ fi
 ################################################################################
 { $as_echo "$as_me:$LINENO: checking group owner" >&5
 $as_echo_n "checking group owner... " >&6; }
-GROUP="root"
+GROUP=""
 
 # Check whether --with-group was given.
 if test "${with_group+set}" = set; then
diff --git a/configure.in b/configure.in
index fb99c9f5d..834fd6eea 100644
--- a/configure.in
+++ b/configure.in
@@ -144,10 +144,10 @@ AC_PREFIX_DEFAULT(/usr)
 ################################################################################
 dnl -- Setup the ownership of the files
 AC_MSG_CHECKING(file owner)
-OWNER="root"
+OWNER=""
 
 AC_ARG_WITH(user,
-  [  --with-user=USER        Set the owner of installed files [[USER=root]] ],
+  [  --with-user=USER        Set the owner of installed files [[USER=]] ],
   [ OWNER="$withval" ])
 AC_MSG_RESULT($OWNER)
 
@@ -158,9 +158,9 @@ fi
 ################################################################################
 dnl -- Setup the group ownership of the files
 AC_MSG_CHECKING(group owner)
-GROUP="root"
+GROUP=""
 AC_ARG_WITH(group,
-  [  --with-group=GROUP      Set the group owner of installed files [[GROUP=root]] ],
+  [  --with-group=GROUP      Set the group owner of installed files [[GROUP=]] ],
   [ GROUP="$withval" ])
 AC_MSG_RESULT($GROUP)