From 3e0f49b846924a54df9115c038cf0a559a567db2 Mon Sep 17 00:00:00 2001 From: Roy Marples Date: Tue, 15 Jan 2008 09:24:50 +0000 Subject: rc_newer_than no longer tests if souce has data, making it a more logical function. As such, we add the existss function to check for existance and size when checking to see if the deptree needs an update. --- src/includes/rc-misc.h | 6 ++++++ src/librc/librc-depend.c | 11 ++++++++--- 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/src/includes/rc-misc.h b/src/includes/rc-misc.h index 68ebc80..ff32df9 100644 --- a/src/includes/rc-misc.h +++ b/src/includes/rc-misc.h @@ -111,6 +111,12 @@ static inline bool exists (const char *pathname) return (stat (pathname, &buf) == 0); } +static inline bool existss (const char *pathname) +{ + struct stat buf; + + return (stat (pathname, &buf) == 0 && buf.st_size != 0); +} char *rc_conf_value (const char *var); bool rc_conf_yesno (const char *var); char **env_filter (void); diff --git a/src/librc/librc-depend.c b/src/librc/librc-depend.c index f52c6de..4ccd7ee 100644 --- a/src/librc/librc-depend.c +++ b/src/librc/librc-depend.c @@ -606,7 +606,8 @@ bool rc_newer_than (const char *source, const char *target) char *path; int serrno = errno; - if (stat (source, &buf) != 0 || buf.st_size == 0) + /* We have to exist */ + if (stat (source, &buf) != 0) return (false); mtime = buf.st_mtime; @@ -618,11 +619,13 @@ bool rc_newer_than (const char *source, const char *target) if (mtime < buf.st_mtime) return (false); + /* If not a dir then reset errno */ if (! (dp = opendir (target))) { errno = serrno; return (true); } + /* Check if we're newer than all the entries in the dir */ while ((d = readdir (dp))) { if (d->d_name[0] == '.') continue; @@ -682,8 +685,10 @@ bool rc_deptree_update_needed (void) if (mkdir (depdirs[i], 0755) != 0 && errno != EEXIST) fprintf (stderr, "mkdir `%s': %s\n", depdirs[i], strerror (errno)); - /* Quick test to see if anything we use has changed */ - if (! rc_newer_than (RC_DEPTREE, RC_INITDIR) || + /* Quick test to see if anything we use has changed and we have + * data in our deptree */ + if (! existss (RC_DEPTREE) || + ! rc_newer_than (RC_DEPTREE, RC_INITDIR) || ! rc_newer_than (RC_DEPTREE, RC_CONFDIR) || ! rc_newer_than (RC_DEPTREE, RC_INITDIR_LOCAL) || ! rc_newer_than (RC_DEPTREE, RC_CONFDIR_LOCAL) || -- cgit v1.2.3