diff options
author | Paul Smith <psmith@gnu.org> | 2013-04-28 17:31:46 -0400 |
---|---|---|
committer | Paul Smith <psmith@gnu.org> | 2013-04-28 17:31:46 -0400 |
commit | 8b034b7d18a08d3a2378e5b6fc9fbdd1b658f142 (patch) | |
tree | 6e30929c6b4e65c68fb7b81faf1d48035cb605ec | |
parent | 93843ccd08a8d4dff64c5fb52701e150b35c959d (diff) | |
download | gunmake-8b034b7d18a08d3a2378e5b6fc9fbdd1b658f142.tar.gz |
Small cleanups and fixes.
-rw-r--r-- | filedef.h | 10 | ||||
-rw-r--r-- | job.c | 4 | ||||
-rw-r--r-- | makeint.h | 3 | ||||
-rw-r--r-- | tests/scripts/features/output-sync | 10 |
4 files changed, 15 insertions, 12 deletions
@@ -28,13 +28,9 @@ struct file const char *vpath; /* VPATH/vpath pathname */ struct dep *deps; /* all dependencies, including duplicates */ struct commands *cmds; /* Commands to execute for this target. */ - int command_flags; /* Flags OR'd in for cmds; see commands.h. */ const char *stem; /* Implicit stem, if an implicit rule has been used */ struct dep *also_make; /* Targets that are made by making this. */ - FILE_TIMESTAMP last_mtime; /* File's modtime, if already known. */ - FILE_TIMESTAMP mtime_before_update; /* File's modtime before any updating - has been performed. */ struct file *prev; /* Previous entry for same file name; used when there are multiple double-colon entries for the same file. */ @@ -59,7 +55,11 @@ struct file the same file. Otherwise this is null. */ struct file *double_colon; - short int update_status; /* Status of the last attempt to update, + FILE_TIMESTAMP last_mtime; /* File's modtime, if already known. */ + FILE_TIMESTAMP mtime_before_update; /* File's modtime before any updating + has been performed. */ + int command_flags; /* Flags OR'd in for cmds; see commands.h. */ + char update_status; /* Status of the last attempt to update, or -1 if none has been made. */ enum cmd_state /* State of the commands. */ { /* Note: It is important that cs_not_started be zero. */ @@ -659,7 +659,7 @@ acquire_semaphore (void) fl.l_type = F_WRLCK; fl.l_whence = SEEK_SET; - fl.l_start = 0; /* lock just one byte */ + fl.l_start = 0; fl.l_len = 1; if (fcntl (sync_handle, F_SETLKW, &fl) != -1) return &fl; @@ -976,7 +976,7 @@ reap_children (int block, int err) /* Search for a child matching the deceased one. */ lastc = 0; for (c = children; c != 0; lastc = c, c = c->next) - if (c->remote == remote && c->pid == pid) + if (c->pid == pid && c->remote == remote) break; if (c == 0) @@ -168,6 +168,9 @@ unsigned int get_path_max (void); (! INTEGER_TYPE_SIGNED (t) ? (t) 0 : ~ (t) 0 << (sizeof (t) * CHAR_BIT - 1)) #define INTEGER_TYPE_MAXIMUM(t) (~ (t) 0 - INTEGER_TYPE_MINIMUM (t)) +/* The maximum number of digits needed to represent the largest integer. */ +#define INTEGER_LENGTH sizeof("18446744073709551616") + #ifndef CHAR_MAX # define CHAR_MAX INTEGER_TYPE_MAXIMUM (char) #endif diff --git a/tests/scripts/features/output-sync b/tests/scripts/features/output-sync index babc08f..d1b743d 100644 --- a/tests/scripts/features/output-sync +++ b/tests/scripts/features/output-sync @@ -198,11 +198,11 @@ bar: end #MAKE#[1]: Leaving directory '#PWD#/bar' #MAKE#[1]: Entering directory '#PWD#/foo' Makefile:20: recipe for target 'foo-fail' failed -make[1]: Leaving directory '/home/psmith/src/make/make/tests/foo' -make[1]: Entering directory '/home/psmith/src/make/make/tests/foo' -make[1]: *** [foo-fail] Error 1 -make[1]: Leaving directory '/home/psmith/src/make/make/tests/foo' -make[1]: Entering directory '/home/psmith/src/make/make/tests/foo' +#MAKE#[1]: Leaving directory '/home/psmith/src/make/make/tests/foo' +#MAKE#[1]: Entering directory '/home/psmith/src/make/make/tests/foo' +#MAKE#[1]: *** [foo-fail] Error 1 +#MAKE#[1]: Leaving directory '/home/psmith/src/make/make/tests/foo' +#MAKE#[1]: Entering directory '/home/psmith/src/make/make/tests/foo' foo-fail: start foo-fail: end #MAKE#[1]: Leaving directory '#PWD#/foo' |