aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
-rw-r--r--src/build.c33
-rw-r--r--src/build.h15
-rw-r--r--src/redo-ifchange.c3
-rw-r--r--src/redo.c2
4 files changed, 23 insertions, 30 deletions
diff --git a/src/build.c b/src/build.c
index 2b67fff..4629b4e 100644
--- a/src/build.c
+++ b/src/build.c
@@ -19,6 +19,7 @@
#define _FILENAME "build.c"
#include "dbg.h"
+#define HASHSIZE 20
const char do_file_ext[] = ".do";
const char default_name[] = "default";
@@ -28,9 +29,15 @@ const char redo_root[] = "REDO_ROOT";
const char redo_parent_target[] = "REDO_PARENT_TARGET";
const char redo_magic[] = "REDO_MAGIC";
-#define HASHSIZE 20
+static char *get_do_file(const char *target);
+static char **parse_shebang(char *target, char *dofile, char *temp_output);
+static char **parsecmd(char *cmd, size_t *i, size_t keep_free);
+static char *get_relpath(const char *target);
+static char *get_dep_path(const char *target);
+static void write_dep_hash(const char *target);
+static bool dependencies_changed(char buf[], size_t read);
+
-/* TODO: more useful return codes? */
int build_target(const char *target) {
assert(target);
@@ -142,7 +149,7 @@ int build_target(const char *target) {
/* Read and parse shebang and return an argv-like pointer array containing the
arguments. If no valid shebang could be found, assume "/bin/sh -e" instead */
-char **parse_shebang(char *target, char *dofile, char *temp_output) {
+static char **parse_shebang(char *target, char *dofile, char *temp_output) {
FILE *fp = fopen(dofile, "rb+");
if (!fp)
fatal(ERRM_FOPEN, dofile)
@@ -176,7 +183,7 @@ char **parse_shebang(char *target, char *dofile, char *temp_output) {
}
/* Returns the right do-file for target */
-char *get_do_file(const char *target) {
+static char *get_do_file(const char *target) {
assert(target);
/* target + ".do" */
char *temp = concat(2, target, do_file_ext);
@@ -202,7 +209,7 @@ char *get_do_file(const char *target) {
/* Breaks cmd at spaces and stores a pointer to each argument in the returned
array. The index i is incremented to point to the next free pointer. The
returned array is guaranteed to have at least keep_free entries left */
-char **parsecmd(char *cmd, size_t *i, size_t keep_free) {
+static char **parsecmd(char *cmd, size_t *i, size_t keep_free) {
assert(cmd);
size_t argv_len = 16;
char **argv = safe_malloc(argv_len * sizeof(char*));
@@ -238,7 +245,7 @@ char **parsecmd(char *cmd, size_t *i, size_t keep_free) {
/* Custom version of realpath that doesn't fail if the last part of path
doesn't exit and allocates memory for the result itself */
-char *xrealpath(const char *path) {
+static char *xrealpath(const char *path) {
char *dirc = safe_strdup(path);
char *dname = dirname(dirc);
char *absdir = realpath(dname, NULL);
@@ -252,7 +259,7 @@ char *xrealpath(const char *path) {
}
/* Return the relative path against REDO_ROOT of target */
-char *get_relpath(const char *target) {
+static char *get_relpath(const char *target) {
assert(getenv(redo_root));
assert(target);
@@ -268,7 +275,7 @@ char *get_relpath(const char *target) {
}
/* Return the dependency file path of target */
-char *get_dep_path(const char *target) {
+static char *get_dep_path(const char *target) {
assert(target);
assert(getenv(redo_root));
@@ -321,7 +328,7 @@ void add_dep(const char *target, int indent) {
}
/* Hash target, storing the result in hash */
-void hash_file(const char *target, unsigned char (*hash)[HASHSIZE]) {
+static void hash_file(const char *target, unsigned char (*hash)[HASHSIZE]) {
FILE *in = fopen(target, "rb");
if (!in)
fatal(ERRM_FOPEN, target);
@@ -339,7 +346,7 @@ void hash_file(const char *target, unsigned char (*hash)[HASHSIZE]) {
}
/* Calculate and store the hash of target in the right dependency file */
-void write_dep_hash(const char *target) {
+static void write_dep_hash(const char *target) {
unsigned char hash[SHA_DIGEST_LENGTH];
unsigned magic = atoi(getenv(redo_magic));
@@ -361,7 +368,7 @@ void write_dep_hash(const char *target) {
free(dep_path);
}
-bool dependencies_changed(char buf[], size_t read) {
+static bool dependencies_changed(char buf[], size_t read) {
char *ptr = buf;
for (size_t i = 0; i < read; ++i) {
if (!buf[i]) {
@@ -410,9 +417,7 @@ bool has_changed(const char *target, int ident, bool is_sub_dependency) {
free(dep_path);
- unsigned magic = *(unsigned *) buf;
-
- if (magic == (unsigned) atoi(getenv(redo_magic)))
+ if (*(unsigned *) buf == (unsigned) atoi(getenv(redo_magic)))
return is_sub_dependency;
unsigned char hash[HASHSIZE];
diff --git a/src/build.h b/src/build.h
index 0b34241..5621c42 100644
--- a/src/build.h
+++ b/src/build.h
@@ -2,20 +2,9 @@
#define __RBUILD_H__
#include <stdbool.h>
-#include <stddef.h>
-char *realpath(const char *path, char *resolved_path);
-int setenv(const char *name, const char *value, int overwrite);
-extern char *get_do_file(const char *target);
-extern int build_target(const char *target);
-extern char **parse_shebang(char *target, char *dofile, char *temp_output);
-extern char **parsecmd(char *cmd, size_t *i, size_t keep_free);
-extern char *get_relpath(const char *target);
-extern char *get_dep_path(const char *target);
extern void add_dep(const char *target, int indent);
-extern void write_dep_hash(const char *target);
-extern bool has_changed(const char *target, int ident, bool is_sub_dependency);
-extern bool dependencies_changed(char buf[], size_t read);
-
+extern bool has_changed(const char *target, int ident, bool is_sub_dependency);
+extern int build_target(const char *target);
#endif
diff --git a/src/redo-ifchange.c b/src/redo-ifchange.c
index 1efe0a5..0b4c1af 100644
--- a/src/redo-ifchange.c
+++ b/src/redo-ifchange.c
@@ -5,8 +5,7 @@
#include "dbg.h"
int main(int argc, char *argv[]) {
- int i;
- for (i = 1; i < argc; ++i) {
+ for (int i = 1; i < argc; ++i) {
/*debug("Testing if %s is up-to-date ...\n", argv[i]);*/
if (has_changed(argv[i], 'c', false)) {
/*printf("=> no\n");*/
diff --git a/src/redo.c b/src/redo.c
index 76d8a86..b70cad7 100644
--- a/src/redo.c
+++ b/src/redo.c
@@ -21,7 +21,7 @@ int main(int argc, char *argv[]) {
/* create .redo directory */
if (mkdir(".redo/deps", 0744))
if (errno != EEXIST) /* TODO: unsafe, dir could be a file or broken symlink */
- fatal(ERRM_MKDIR, ".redo");
+ fatal(ERRM_MKDIR, ".redo/deps");
/* set REDO_ROOT */
char *cwd = getcwd(NULL, 0);
generated by cgit v1.2.3-70-g09d2 (git 2.51.2) at 2025-11-04 01:20:56 +0000