Patch from Joey Adams:
authorRusty Russell <rusty@rustcorp.com.au>
Sun, 29 Mar 2009 05:53:19 +0000 (16:23 +1030)
committerRusty Russell <rusty@rustcorp.com.au>
Sun, 29 Mar 2009 05:53:19 +0000 (16:23 +1030)
(15:27:11) joeyadams: ccanlint's handle_no_tests apparently has a bug.  It makes the test directory with
0600 permissions, so it can't be used.  the x bit provides directory listing privilege
(15:27:25) joeyadams: this gives me Permission denied when I try to run it
(15:28:34) joeyadams: Changing the opcode to 0700 makes the creation of run.c work

tools/ccanlint/has_tests.c

index 320478b38da61424727c093bece2032c55c5528f..0866869570c2e71b1f21a3d7eff5574fec33db51 100644 (file)
@@ -80,7 +80,7 @@ static void handle_no_tests(struct manifest *m, void *check_result)
        if (!ask("Should I create a template test/run.c file for you?"))
                return;
 
        if (!ask("Should I create a template test/run.c file for you?"))
                return;
 
-       if (mkdir("test", 0600) != 0) {
+       if (mkdir("test", 0700) != 0) {
                if (errno != EEXIST)
                        err(1, "Creating test/ directory");
        }
                if (errno != EEXIST)
                        err(1, "Creating test/ directory");
        }