revert r1.11 of parse.y and create logic in conf_setup instead to deal

with the various scenarios of when to attempt a parse of the config,
load defaults, and when to warn and/or exit.  triggered by bogus warning
first noticed by sobrado@.

ok oga@
This commit is contained in:
okan 2011-08-22 16:18:05 +00:00
parent 912dd46a7e
commit 2dc8df110c
2 changed files with 14 additions and 10 deletions

22
conf.c
View File

@ -265,26 +265,32 @@ conf_clear(struct conf *c)
void
conf_setup(struct conf *c, const char *conf_file)
{
char *home;
struct stat sb;
int parse = 0;
conf_init(c);
if (conf_file == NULL) {
char *home = getenv("HOME");
if (home == NULL)
if ((home = getenv("HOME")) == NULL)
errx(1, "No HOME directory.");
(void)snprintf(c->conf_path, sizeof(c->conf_path), "%s/%s",
home, CONFFILE);
} else
if (stat(c->conf_path, &sb) == 0 && (sb.st_mode & S_IFREG))
parse = 1;
} else {
if (stat(conf_file, &sb) == -1 || !(sb.st_mode & S_IFREG))
errx(1, "%s: %s", conf_file, strerror(errno));
else
else {
(void)strlcpy(c->conf_path, conf_file,
sizeof(c->conf_path));
parse = 1;
}
}
conf_init(c);
if (parse_config(c->conf_path, c) == -1)
if (parse && (parse_config(c->conf_path, c) == -1))
warnx("config file %s has errors, not loading", c->conf_path);
}

View File

@ -462,8 +462,6 @@ pushfile(const char *name)
nfile->name = xstrdup(name);
if ((nfile->stream = fopen(nfile->name, "r")) == NULL) {
if (errno != ENOENT)
warn("%s", nfile->name);
free(nfile->name);
free(nfile);
return (NULL);