X-Git-Url: https://git.ozlabs.org/?a=blobdiff_plain;f=tools%2Fccanlint%2Fcompulsory_tests%2Fbuild.c;h=9d697a303a57399a1754da32db7dfc0c20d1ffa0;hb=2926cafb52b9d95646d9dafa877d53f2368d8b2c;hp=7c73f0d4f834c98519dc68ff18420cb935d4d6cc;hpb=a40b318e7a07a452ae7456053727bd11b2fa49b4;p=ccan

diff --git a/tools/ccanlint/compulsory_tests/build.c b/tools/ccanlint/compulsory_tests/build.c
index 7c73f0d4..9d697a30 100644
--- a/tools/ccanlint/compulsory_tests/build.c
+++ b/tools/ccanlint/compulsory_tests/build.c
@@ -13,6 +13,7 @@
 #include <err.h>
 #include <string.h>
 #include <ctype.h>
+#include "build.h"
 
 static const char *can_build(struct manifest *m)
 {
@@ -33,12 +34,27 @@ static char *obj_list(const struct manifest *m)
 	return list;
 }
 
+char *build_module(struct manifest *m, bool keep, char **errstr)
+{
+	char *name = link_objects(m, m->basename, false, obj_list(m), errstr);
+	if (name) {
+		if (keep) {
+			char *realname = talloc_asprintf(m, "%s.o", m->dir);
+			/* We leave this object file around, all built. */
+			if (!move_file(name, realname))
+				err(1, "Renaming %s to %s", name, realname);
+			name = realname;
+		}
+	}
+	return name;
+}
+
 static void do_build(struct manifest *m,
 		     bool keep,
 		     unsigned int *timeleft,
 		     struct score *score)
 {
-	char *filename, *errstr;
+	char *errstr;
 
 	if (list_empty(&m->c_files)) {
 		/* No files?  No score, but we "pass". */
@@ -47,28 +63,22 @@ static void do_build(struct manifest *m,
 		return;
 	}
 
-	filename = link_objects(m, m->basename, false, obj_list(m), &errstr);
-	if (!filename) {
-		score->error = "The object file didn't build";
+	m->compiled = build_module(m, keep, &errstr);
+	if (!m->compiled) {
 		score_file_error(score, NULL, 0, errstr);
 		return;
 	}
 
-	if (keep) {
-		char *realname = talloc_asprintf(m, "%s.o", m->dir);
-		/* We leave this object file around, all built. */
-		if (!move_file(filename, realname))
-			err(1, "Renaming %s to %s", filename, realname);
-	}
 	score->pass = true;
 	score->score = score->total;
 }
 
-struct ccanlint build = {
-	.key = "build",
+struct ccanlint module_builds = {
+	.key = "module_builds",
 	.name = "Module can be built from object files",
 	.check = do_build,
 	.can_run = can_build,
+	.needs = "objects_build"
 };
 
-REGISTER_TEST(build, &depends_built, NULL);
+REGISTER_TEST(module_builds);