lkml.org 
[lkml]   [2004]   [Feb]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    SubjectRe: [ANNOUNCE] udev 016 release
    From
    Date
    On Tue, 2004-02-03 at 22:13, Greg KH wrote:

    Once again, patch to make logging a config option.

    Reason for this (since you asked for it =):
    - In our setup it is easy (although still annoying) .. just
    edit the ebuild, add logging support (or remove it) and rebuild.
    For say a binary distro, having the logging is useful for debugging
    some times, but its more a once of, or rare thing, as you do not
    add or change config files every day. Sure, we can have logging
    by default, but many do not want ~300 lines of extra debugging in
    their logs is not pleasant, and they will complain. Rebuilding
    the package for that binary package (given the users it is targeted
    to) is usually not within most users grasp.


    Thanks,

    --
    Martin Schlemmer
    diff -uprN udev-016/Makefile udev-016.logging/Makefile
    --- udev-016/Makefile 2004-02-03 22:55:30.201851776 +0200
    +++ udev-016.logging/Makefile 2004-02-03 23:03:55.869978536 +0200
    @@ -239,6 +239,7 @@ udev_version.h:
    @echo \#define UDEV_CONFIG_FILE \"$(configdir)\udev.conf\" >> $@
    @echo \#define UDEV_RULES_FILE \"$(configdir)\udev.rules\" >> $@
    @echo \#define UDEV_PERMISSION_FILE \"$(configdir)\udev.permissions\" >> $@
    + @echo \#define UDEV_LOG_DEFAULT \"yes\" >> $@
    @echo \#define UDEV_BIN \"$(DESTDIR)$(sbindir)/udev\" >> $@
    @echo \#define UDEVD_BIN \"$(DESTDIR)$(sbindir)/udevd\" >> $@
    @echo \#define UDEVD_SOCK \"$(udevdir)/\.udevd.sock\" >> $@
    diff -uprN udev-016/etc/udev/udev.conf.in udev-016.logging/etc/udev/udev.conf.in
    --- udev-016/etc/udev/udev.conf.in 2004-02-03 22:58:34.939767344 +0200
    +++ udev-016.logging/etc/udev/udev.conf.in 2004-02-03 22:56:46.535247336 +0200
    @@ -29,3 +29,6 @@ default_owner="root"
    # explicit match in the permissions file
    default_group="root"

    +# udev_log - set to "yes" if you want logging, else "no"
    +udev_log="yes"
    +
    diff -uprN udev-016/logging.h udev-016.logging/logging.h
    --- udev-016/logging.h 2004-02-02 11:34:49.000000000 +0200
    +++ udev-016.logging/logging.h 2004-02-03 23:04:51.204566400 +0200
    @@ -33,6 +33,9 @@
    #include <stdarg.h>
    #include <syslog.h>

    +#include "udev.h"
    +#include "udev_version.h"
    +
    #undef info
    #define info(format, arg...) \
    do { \
    @@ -62,6 +65,9 @@ static inline void log_message (int leve
    {
    va_list args;

    + if (0 != strncmp(udev_log_str, UDEV_LOG_DEFAULT, BOOL_SIZE))
    + return;
    +
    va_start(args, format);
    vsyslog(level, format, args);
    va_end(args);
    diff -uprN udev-016/udev.h udev-016.logging/udev.h
    --- udev-016/udev.h 2004-01-26 22:24:42.000000000 +0200
    +++ udev-016.logging/udev.h 2004-02-03 23:03:25.682567720 +0200
    @@ -32,6 +32,8 @@
    #define OWNER_SIZE 30
    #define GROUP_SIZE 30
    #define MODE_SIZE 8
    +#define BOOL_SIZE 5 /* 'yes', 'no' and possibly 'true' or 'false'
    + in future */

    struct udevice {
    char name[NAME_SIZE];
    @@ -72,5 +74,6 @@ extern char udev_rules_filename[PATH_MAX
    extern char default_mode_str[MODE_SIZE];
    extern char default_owner_str[OWNER_SIZE];
    extern char default_group_str[GROUP_SIZE];
    +extern char udev_log_str[BOOL_SIZE];

    #endif
    diff -uprN udev-016/udev_config.c udev-016.logging/udev_config.c
    --- udev-016/udev_config.c 2004-02-03 23:00:40.438688608 +0200
    +++ udev-016.logging/udev_config.c 2004-02-03 22:56:46.649230008 +0200
    @@ -48,6 +48,7 @@ char udev_config_filename[PATH_MAX+NAME_
    char default_mode_str[MODE_SIZE];
    char default_owner_str[OWNER_SIZE];
    char default_group_str[GROUP_SIZE];
    +char udev_log_str[BOOL_SIZE];


    static void init_variables(void)
    @@ -60,6 +61,7 @@ static void init_variables(void)
    strfieldcpy(udev_config_filename, UDEV_CONFIG_FILE);
    strfieldcpy(udev_rules_filename, UDEV_RULES_FILE);
    strfieldcpy(udev_permissions_filename, UDEV_PERMISSION_FILE);
    + strfieldcpy(udev_log_str, UDEV_LOG_DEFAULT);
    }

    #define set_var(_name, _var) \
    @@ -156,6 +158,7 @@ static int parse_config_file(void)
    set_var("default_mode", default_mode_str);
    set_var("default_owner", default_owner_str);
    set_var("default_group", default_group_str);
    + set_var("udev_log", udev_log_str);
    }
    dbg_parse("%s:%d:%Zd: error parsing '%s'", udev_config_filename,
    lineno, temp - line, temp);
    @@ -191,6 +194,7 @@ static void get_dirs(void)
    dbg_parse("udev_db_filename = %s", udev_db_filename);
    dbg_parse("udev_rules_filename = %s", udev_rules_filename);
    dbg_parse("udev_permissions_filename = %s", udev_permissions_filename);
    + dbg_parse("udev_log_str = %s", udev_log_str);
    parse_config_file();

    dbg_parse("udev_root = %s", udev_root);
    @@ -198,6 +202,7 @@ static void get_dirs(void)
    dbg_parse("udev_db_filename = %s", udev_db_filename);
    dbg_parse("udev_rules_filename = %s", udev_rules_filename);
    dbg_parse("udev_permissions_filename = %s", udev_permissions_filename);
    + dbg_parse("udev_log_str = %s", udev_log_str);
    }

    void udev_init_config(void)[unhandled content-type:application/pgp-signature]
    \
     
     \ /
      Last update: 2005-03-22 14:00    [W:4.164 / U:0.512 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site