chiark / gitweb /
[PATCH] clean up the logging patch a bit to make the option more like the other options.
authorgreg@kroah.com <greg@kroah.com>
Fri, 16 Jan 2004 06:00:58 +0000 (22:00 -0800)
committerGreg KH <gregkh@suse.de>
Wed, 27 Apr 2005 04:13:17 +0000 (21:13 -0700)
Also documented it and added it to the .spec file.

Makefile
README
logging.h
udev.spec

index 88c7cf0..f19354f 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -18,7 +18,7 @@
 
 # Set the following to control the use of syslog
 # Set it to `false' to remove all logging
-LOG = true
+USE_LOG = true
 
 # Set the following to `true' to log the debug
 # and make a unstripped, unoptimized  binary.
@@ -109,7 +109,7 @@ else
        CFLAGS+=-pipe
 endif
 
-ifeq ($(strip $(LOG)),true)
+ifeq ($(strip $(USE_LOG)),true)
        CFLAGS  += -DLOG
 endif
 
diff --git a/README b/README
index 123fce6..26032c3 100644 (file)
--- a/README
+++ b/README
@@ -30,18 +30,28 @@ To use:
   Note:
       There are a number of different flags that you can use when building
       udev.  They are as follows:
-       prefix - set this to the default root that you want udev to be
+       prefix
+               set this to the default root that you want udev to be
                installed into.  This works just like the 'configure --prefix'
                script does.  Default value is ''.  Only override this if you
                really know what you are doing.
-       USE_KLIBC - if set to 'true', udev is built and linked against the included
-               version of klibc.  Default value is 'false'.
-       DEBUG - if set to 'true', debugging messages will be sent to the syslog
-               as udev is run.  Default value is 'false'.
-       USE_DBUS -  if set to 'true', DBUS messages will be sent everytime udev
+       USE_KLIBC
+               if set to 'true', udev is built and linked against the
+               included version of klibc.  Default value is 'false'.
+       USE_LOG
+               if set to 'true', udev will emit messages to the syslog when
+               it creates or removes device nodes.  This is helpful to see
+               what udev is doing.  This is enabled by default.  Note, if you
+               are building udev against klibc it is recommended that you
+               disable this option (due to klibc's syslog implementation.)
+       USE_DBUS
+               if set to 'true', DBUS messages will be sent everytime udev
                creates or removes a device node.  This requires that DBUS
                development headers and libraries be present on your system to
                build properly.  Default value is 'false'.
+       DEBUG
+               if set to 'true', debugging messages will be sent to the syslog
+               as udev is run.  Default value is 'false'.
 
       So, if you want to build udev using klibc with debugging messages, you
       would do:
index 85016ad..1f21268 100644 (file)
--- a/logging.h
+++ b/logging.h
@@ -1,5 +1,5 @@
 /*
- * udev.h
+ * logging.h
  *
  * Userspace devfs
  *
index 0733180..575e3cd 100644 (file)
--- a/udev.spec
+++ b/udev.spec
@@ -5,6 +5,12 @@
 # it probably is not where you want it to be.
 %define klibc 1
 
+# if we want to have logging support in or not.
+# 0 - no logging  support
+# 1 - logging support
+# Note, it is not recommend if you use klibc to enable logging.
+%define log 0
+
 # if we want to build DBUS support in or not.
 # 0 - no DBUS support
 # 1 - DBUS support
@@ -45,6 +51,9 @@ make CC="gcc $RPM_OPT_FLAGS"  \
 %if %{klibc}
        USE_KLIBC=true          \
 %endif
+%if %{log}
+       USE_LOG=true            \
+%endif
 %if %{dbus}
        USE_DBUS=true           \
 %endif