threads / patch / 4086

patchconfig: if mtime (or size) of the config file changed since last read, reread it

Subject: [PATCH] config: if mtime (or size) of the config file changed since last read, reread it

## tl;dr

4 messages between May 6, 2006 and May 7, 2006. Diffs are folded; open one to read it.

replies: 3people: 3as markdown or json

Johannes Schindelin· May 6, 2006, 23:26 UTC · lore
Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
---
	This is extremely paranoic, I know.
 config.c |   16 ++++++++++------
 1 files changed, 10 insertions(+), 6 deletions(-)
Show changes to config.c +10 −6
diff --git a/config.c b/config.c
index 05d4d8c..6765186 100644
--- a/config.c
+++ b/config.c
@@ -13,6 +13,7 @@ #define MAXNAME (256)
 static const char *contents = NULL;
 static int config_length = 0, config_offset = 0;
 static const char *config_file_name;
+static time_t config_file_mtime = 0;
 static int config_linenr;
 static int get_next_char(void)
 {
@@ -255,23 +256,26 @@ int git_default_config(const char *var, 
 int git_config_from_file(config_fn_t fn, const char *filename)
 {
 	int ret, in_fd;
+	struct stat st;
 
 	config_offset = 0;
 
+	in_fd = open(filename, O_RDONLY);
+	fstat(in_fd, &st);
+
 	if (contents) {
-		if (!strcmp(config_file_name, filename))
+		if (!strcmp(config_file_name, filename)
+				&& config_file_mtime == st.st_mtime
+				&& config_length == st.st_size) {
+			close(in_fd);
 			return git_parse_file(fn);
+		}
 		munmap((char*)contents, config_length);
 		free((char*)config_file_name);
 	}
 
-	in_fd = open(filename, O_RDONLY);
-
 	ret = -1;
 	if (in_fd > 0) {
-		struct stat st;
-
-		fstat(in_fd, &st);
 		config_length = st.st_size;
 		contents = mmap(NULL, config_length, PROT_READ, MAP_PRIVATE,
 				in_fd, 0);

-- 
1.3.1.g5545a
Johannes Schindelin· May 6, 2006, 23:45 UTC · re: Johannes Schindelin · lore

Re: [PATCH] config: if mtime (or size) of the config file changed since last read, reread it

Hi,
sorry: bad patch. This is needed on top.
Show changes to config.c +4 −3
diff --git a/config.c b/config.c
index 6765186..452b587 100644
--- a/config.c
+++ b/config.c
@@ -261,6 +261,10 @@ int git_config_from_file(config_fn_t fn,
 	config_offset = 0;
 
 	in_fd = open(filename, O_RDONLY);
+	if (in_fd < 0 && ENOENT != errno )
+		die("opening %s: %s", config_file_name,
+				strerror(errno));
+
 	fstat(in_fd, &st);
 
 	if (contents) {
@@ -288,9 +292,6 @@ int git_config_from_file(config_fn_t fn,
 	} else {
 		contents = NULL;
 		config_length = 0;
-		if (in_fd < 0 && ENOENT != errno )
-			die("opening %s: %s", config_file_name,
-					strerror(errno));
 	}
 
 	return ret;
Jan-Benedict Glaw· May 7, 2006, 07:30 UTC · re: Johannes Schindelin · lore

Re: [PATCH] config: if mtime (or size) of the config file changed since last read, reread it

On Sun, 2006-05-07 01:45:22 +0200, Johannes Schindelin <Johannes.Schindelin@gmx.de> wrote:
Show 9 quoted lines
> diff --git a/config.c b/config.c
> index 6765186..452b587 100644
> --- a/config.c
> +++ b/config.c
> @@ -261,6 +261,10 @@ int git_config_from_file(config_fn_t fn,
>  	config_offset = 0;
>  
>  	in_fd = open(filename, O_RDONLY);
> +	if (in_fd < 0 && ENOENT != errno )

I admit that I don't like the (constant -operator- variable) notation, but mixing both in one line..?

MfG, JBG
-- 
Jan-Benedict Glaw       jbglaw@lug-owl.de    . +49-172-7608481             _ O _
"Eine Freie Meinung in  einem Freien Kopf    | Gegen Zensur | Gegen Krieg  _ _ O
 für einen Freien Staat voll Freier Bürger"  | im Internet! |   im Irak!   O O O
ret = do_actions((curr | FREE_SPEECH) & ~(NEW_COPYRIGHT_LAW | DRM | TCPA));
Junio C Hamano· May 7, 2006, 08:42 UTC · re: Jan-Benedict Glaw · lore

Re: [PATCH] config: if mtime (or size) of the config file changed since last read, reread it

Jan-Benedict Glaw <jbglaw@lug-owl.de> writes:
>> +	if (in_fd < 0 && ENOENT != errno )
>
> I admit that I don't like the (constant -operator- variable) notation,
> but mixing both in one line..?
Lol.
I would have written it as (in_fd < 0 && errno != ENOENT) BTW.

← back to recent threads