aboutsummaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
authorEgor Tensin <egor@tensin.name>2024-12-26 12:08:58 +0100
committerEgor Tensin <egor@tensin.name>2024-12-26 12:08:58 +0100
commit9f40d20e1c97e2c85e26a6a13ccf04e60d9f83f5 (patch)
treeb5a1c96518c96974e53a70e02d4e684efc198bed
parentupdate DEVELOPMENT.md (diff)
downloadcimple-9f40d20e1c97e2c85e26a6a13ccf04e60d9f83f5.tar.gz
cimple-9f40d20e1c97e2c85e26a6a13ccf04e60d9f83f5.zip
process: proc_ -> process_HEADmaster
-rw-r--r--src/ci.c8
-rw-r--r--src/ci.h4
-rw-r--r--src/process.c13
-rw-r--r--src/process.h15
-rw-r--r--src/protocol.c10
-rw-r--r--src/protocol.h6
-rw-r--r--src/server.c4
-rw-r--r--src/storage.c4
-rw-r--r--src/storage.h2
-rw-r--r--src/storage_sqlite.c2
-rw-r--r--src/storage_sqlite.h2
-rw-r--r--src/worker.c8
12 files changed, 41 insertions, 37 deletions
diff --git a/src/ci.c b/src/ci.c
index cdb2f21..d906fc4 100644
--- a/src/ci.c
+++ b/src/ci.c
@@ -31,13 +31,13 @@ static const char *ci_env[] = {
};
/* clang-format on */
-static int ci_run_script(const char *script, struct proc_output *result)
+static int ci_run_script(const char *script, struct process_output *result)
{
const char *args[] = {script, NULL};
- return proc_capture(args, ci_env, result);
+ return process_execute_and_capture(args, ci_env, result);
}
-int ci_run(struct proc_output *result)
+int ci_run(struct process_output *result)
{
for (const char **script = ci_scripts; *script; ++script) {
if (!file_exists(*script))
@@ -76,7 +76,7 @@ cleanup_repo:
return ret;
}
-int ci_run_git_repo(const char *url, const char *rev, struct proc_output *output)
+int ci_run_git_repo(const char *url, const char *rev, struct process_output *output)
{
char *oldpwd = NULL;
git_repository *repo = NULL;
diff --git a/src/ci.h b/src/ci.h
index 397f4a7..542243c 100644
--- a/src/ci.h
+++ b/src/ci.h
@@ -10,7 +10,7 @@
#include "process.h"
-int ci_run(struct proc_output *);
+int ci_run(struct process_output *);
/*
* This is a high-level function. It's basically equivalent to the following
@@ -25,6 +25,6 @@ int ci_run(struct proc_output *);
* rm -rf "$dir"
*
*/
-int ci_run_git_repo(const char *url, const char *rev, struct proc_output *);
+int ci_run_git_repo(const char *url, const char *rev, struct process_output *);
#endif
diff --git a/src/process.c b/src/process.c
index 580daaf..1c452ac 100644
--- a/src/process.c
+++ b/src/process.c
@@ -58,7 +58,7 @@ static int wait_for_child(pid_t pid, int *ec)
return 0;
}
-int proc_spawn(const char *args[], const char *envp[], int *ec)
+int process_execute(const char *args[], const char *envp[], int *ec)
{
pid_t child_pid = fork();
if (child_pid < 0) {
@@ -93,7 +93,8 @@ static int redirect_and_exec_child(int pipe_fds[2], const char *args[], const ch
return exec_child(args, envp);
}
-int proc_capture(const char *args[], const char *envp[], struct proc_output *result)
+int process_execute_and_capture(const char *args[], const char *envp[],
+ struct process_output *result)
{
static const int flags = O_CLOEXEC;
int pipe_fds[2];
@@ -137,9 +138,9 @@ close_pipe:
return ret;
}
-int proc_output_create(struct proc_output **_output)
+int process_output_create(struct process_output **_output)
{
- struct proc_output *output = calloc(1, sizeof(struct proc_output));
+ struct process_output *output = calloc(1, sizeof(struct process_output));
if (!output) {
log_errno("calloc");
return -1;
@@ -153,13 +154,13 @@ int proc_output_create(struct proc_output **_output)
return 0;
}
-void proc_output_destroy(struct proc_output *output)
+void process_output_destroy(struct process_output *output)
{
free(output->data);
free(output);
}
-void proc_output_dump(const struct proc_output *output)
+void process_output_dump(const struct process_output *output)
{
log("Process exit code: %d\n", output->ec);
log("Process output: %zu bytes\n", output->data_size);
diff --git a/src/process.h b/src/process.h
index 810010d..965cb5e 100644
--- a/src/process.h
+++ b/src/process.h
@@ -10,24 +10,25 @@
#include <stddef.h>
-struct proc_output {
+struct process_output {
int ec;
unsigned char *data;
size_t data_size;
};
/* The exit code is only valid if the functions returns a non-negative number. */
-int proc_spawn(const char *args[], const char *envp[], int *ec);
+int process_execute(const char *args[], const char *envp[], int *ec);
-/* Similarly, the contents of the proc_output structure is only valid if the function returns a
+/* Similarly, the contents of the process_output structure is only valid if the function returns a
* non-negative number.
*
* In that case, you'll need to free the output. */
-int proc_capture(const char *args[], const char *envp[], struct proc_output *result);
+int process_execute_and_capture(const char *args[], const char *envp[],
+ struct process_output *result);
-int proc_output_create(struct proc_output **);
-void proc_output_destroy(struct proc_output *);
+int process_output_create(struct process_output **);
+void process_output_destroy(struct process_output *);
-void proc_output_dump(const struct proc_output *);
+void process_output_dump(const struct process_output *);
#endif
diff --git a/src/protocol.c b/src/protocol.c
index d950004..0fa06f1 100644
--- a/src/protocol.c
+++ b/src/protocol.c
@@ -120,7 +120,7 @@ static const char *const finished_key_ec = "exit_code";
static const char *const finished_key_data = "output";
int request_create_finished_run(struct jsonrpc_request **request, int run_id,
- const struct proc_output *output)
+ const struct process_output *output)
{
int ret = 0;
@@ -153,12 +153,12 @@ free_request:
}
int request_parse_finished_run(const struct jsonrpc_request *request, int *_run_id,
- struct proc_output **_output)
+ struct process_output **_output)
{
int ret = 0;
- struct proc_output *output = NULL;
- ret = proc_output_create(&output);
+ struct process_output *output = NULL;
+ ret = process_output_create(&output);
if (ret < 0)
return ret;
@@ -187,7 +187,7 @@ int request_parse_finished_run(const struct jsonrpc_request *request, int *_run_
return ret;
free_output:
- proc_output_destroy(output);
+ process_output_destroy(output);
return ret;
}
diff --git a/src/protocol.h b/src/protocol.h
index 2277998..c496449 100644
--- a/src/protocol.h
+++ b/src/protocol.h
@@ -21,8 +21,10 @@ int request_parse_new_worker(const struct jsonrpc_request *);
int request_create_start_run(struct jsonrpc_request **, const struct run *);
int request_parse_start_run(const struct jsonrpc_request *, struct run **);
-int request_create_finished_run(struct jsonrpc_request **, int run_id, const struct proc_output *);
-int request_parse_finished_run(const struct jsonrpc_request *, int *run_id, struct proc_output **);
+int request_create_finished_run(struct jsonrpc_request **, int run_id,
+ const struct process_output *);
+int request_parse_finished_run(const struct jsonrpc_request *, int *run_id,
+ struct process_output **);
int request_create_get_runs(struct jsonrpc_request **);
int request_parse_get_runs(const struct jsonrpc_request *);
diff --git a/src/server.c b/src/server.c
index 89ffcb1..b917f7c 100644
--- a/src/server.c
+++ b/src/server.c
@@ -297,7 +297,7 @@ static int server_handle_cmd_finished_run(const struct jsonrpc_request *request,
int ret = 0;
int run_id = 0;
- struct proc_output *output;
+ struct process_output *output;
ret = request_parse_finished_run(request, &run_id, &output);
if (ret < 0)
@@ -312,7 +312,7 @@ static int server_handle_cmd_finished_run(const struct jsonrpc_request *request,
log("Marked run %d as finished\n", run_id);
free_output:
- proc_output_destroy(output);
+ process_output_destroy(output);
return ret;
}
diff --git a/src/storage.c b/src/storage.c
index 43e871f..54a62c2 100644
--- a/src/storage.c
+++ b/src/storage.c
@@ -18,7 +18,7 @@ typedef int (*storage_create_t)(struct storage *, const struct storage_settings
typedef void (*storage_destroy_t)(struct storage *);
typedef int (*storage_run_create_t)(struct storage *, const char *repo_url, const char *rev);
-typedef int (*storage_run_finished_t)(struct storage *, int repo_id, const struct proc_output *);
+typedef int (*storage_run_finished_t)(struct storage *, int repo_id, const struct process_output *);
typedef int (*storage_get_runs_t)(struct storage *, struct run_queue *);
typedef storage_get_runs_t storage_get_run_queue_t;
@@ -105,7 +105,7 @@ int storage_run_create(struct storage *storage, const char *repo_url, const char
return api->run_create(storage, repo_url, rev);
}
-int storage_run_finished(struct storage *storage, int run_id, const struct proc_output *output)
+int storage_run_finished(struct storage *storage, int run_id, const struct process_output *output)
{
const struct storage_api *api = get_api(storage->type);
if (!api)
diff --git a/src/storage.h b/src/storage.h
index c50444d..01ec25a 100644
--- a/src/storage.h
+++ b/src/storage.h
@@ -36,7 +36,7 @@ int storage_create(struct storage *, const struct storage_settings *);
void storage_destroy(struct storage *);
int storage_run_create(struct storage *, const char *repo_url, const char *rev);
-int storage_run_finished(struct storage *, int run_id, const struct proc_output *);
+int storage_run_finished(struct storage *, int run_id, const struct process_output *);
int storage_get_runs(struct storage *, struct run_queue *);
int storage_get_run_queue(struct storage *, struct run_queue *);
diff --git a/src/storage_sqlite.c b/src/storage_sqlite.c
index 6c8a26f..efefaf0 100644
--- a/src/storage_sqlite.c
+++ b/src/storage_sqlite.c
@@ -410,7 +410,7 @@ int storage_sqlite_run_create(struct storage *storage, const char *repo_url, con
}
int storage_sqlite_run_finished(struct storage *storage, int run_id,
- const struct proc_output *output)
+ const struct process_output *output)
{
struct prepared_stmt *stmt = &storage->sqlite->stmt_run_finished;
int ret = 0;
diff --git a/src/storage_sqlite.h b/src/storage_sqlite.h
index 2d82496..2a4df2b 100644
--- a/src/storage_sqlite.h
+++ b/src/storage_sqlite.h
@@ -24,7 +24,7 @@ int storage_sqlite_create(struct storage *, const struct storage_settings *);
void storage_sqlite_destroy(struct storage *);
int storage_sqlite_run_create(struct storage *, const char *repo_url, const char *rev);
-int storage_sqlite_run_finished(struct storage *, int id, const struct proc_output *);
+int storage_sqlite_run_finished(struct storage *, int id, const struct process_output *);
int storage_sqlite_get_runs(struct storage *, struct run_queue *runs);
int storage_sqlite_get_run_queue(struct storage *, struct run_queue *runs);
diff --git a/src/worker.c b/src/worker.c
index 5c0ab96..8ee0703 100644
--- a/src/worker.c
+++ b/src/worker.c
@@ -179,8 +179,8 @@ static int worker_do_run(struct worker *worker)
{
int ret = 0;
- struct proc_output *result = NULL;
- ret = proc_output_create(&result);
+ struct process_output *result = NULL;
+ ret = process_output_create(&result);
if (ret < 0)
return ret;
@@ -190,7 +190,7 @@ static int worker_do_run(struct worker *worker)
goto free_output;
}
- proc_output_dump(result);
+ process_output_dump(result);
struct jsonrpc_request *finished_request = NULL;
@@ -214,7 +214,7 @@ free_request:
jsonrpc_request_destroy(finished_request);
free_output:
- proc_output_destroy(result);
+ process_output_destroy(result);
run_destroy(worker->run);