From 300c03203dd5f9f525dc330d24bacc0e07f57766 Mon Sep 17 00:00:00 2001 From: William Hubbs Date: Mon, 26 Mar 2012 15:04:40 -0500 Subject: librc: Do not output error messages from within the library. This fixes a compile issue. Also, it is cleaner to have the client output error messages as opposed to having the library do this. Reported-by: Ewoud Kohl van Wijngaarden X-Gentoo-Bug: 409743 X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=409743 --- src/librc/librc-misc.c | 12 +++--------- src/rc/rc.c | 7 +++++++ 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/src/librc/librc-misc.c b/src/librc/librc-misc.c index 06f97e7..b907c5c 100644 --- a/src/librc/librc-misc.c +++ b/src/librc/librc-misc.c @@ -29,7 +29,6 @@ */ #include "librc.h" -#include "einfo.h" bool rc_yesno(const char *value) @@ -139,15 +138,13 @@ rc_proc_getent(const char *ent) if (!exists("/proc/cmdline")) return NULL; - if (!(fp = fopen("/proc/cmdline", "r"))) { - eerror("failed to open `/proc/cmdline': %s", strerror(errno)); + if (!(fp = fopen("/proc/cmdline", "r"))) return NULL; - } proc = NULL; i = 0; if (rc_getline(&proc, &i, fp) == -1 || proc == NULL) - eerror("rc_getline: %s", strerror(errno)); + return NULL; if (proc != NULL) { len = strlen(ent); @@ -394,13 +391,10 @@ rc_conf_value(const char *setting) atexit(_free_rc_conf); #endif - /* Support old configs, but complain about it. */ + /* Support old configs. */ if (exists(RC_CONF_OLD)) { old = rc_config_load(RC_CONF_OLD); TAILQ_CONCAT(rc_conf, old, entries); - ewarn("Your system still has %s", RC_CONF_OLD); - ewarn("Please migrate to the appropriate settings in %s", RC_CONF); - ewarn("and delete %s.", RC_CONF_OLD); #ifdef DEBUG_MEMORY free(old); #endif diff --git a/src/rc/rc.c b/src/rc/rc.c index 4cf0374..2590048 100644 --- a/src/rc/rc.c +++ b/src/rc/rc.c @@ -793,6 +793,13 @@ main(int argc, char **argv) env_filter(); env_config(); + /* complain about old configuration settings if they exist */ + if (exists(RC_CONF_OLD)) { + ewarn("%s still exists on your system and should be removed.", + RC_CONF_OLD); + ewarn("Please migrate to the appropriate settings in %s", RC_CONF); + } + argc++; argv--; while ((opt = getopt_long(argc, argv, getoptstring, -- cgit v1.2.3