/* FIXME: recursive ccanlint if they ask for it. */
static bool expect_obj_file(const char *dir)
{
- char *olddir;
struct manifest *dep_man;
bool has_c_files;
- olddir = talloc_getcwd(dir);
- if (!olddir)
- err(1, "Getting current directory");
-
- /* We will fail below if this doesn't exist. */
- if (chdir(dir) != 0)
- return false;
-
- dep_man = get_manifest(dir);
- if (chdir(olddir) != 0)
- err(1, "Returning to original directory '%s'", olddir);
- talloc_free(olddir);
+ dep_man = get_manifest(dir, dir);
/* If it has C files, we expect an object file built from them. */
has_c_files = !list_empty(&dep_man->c_files);
return has_c_files;
}
-static void *check_depends_built(struct manifest *m)
+static void *check_depends_built(struct manifest *m,
+ bool keep,
+ unsigned int *timeleft)
{
struct ccan_file *i;
struct stat st;
char *report = NULL;
list_for_each(&m->dep_dirs, i, list) {
- char *objfile;
-
- if (!expect_obj_file(i->name))
+ if (!expect_obj_file(i->fullname))
continue;
- objfile = talloc_asprintf(m, "%s.o", i->name);
- if (stat(objfile, &st) != 0) {
+ i->compiled = talloc_asprintf(i, "%s.o", i->fullname);
+ if (stat(i->compiled, &st) != 0) {
report = talloc_asprintf_append(report,
"object file %s\n",
- objfile);
- } else {
- struct ccan_file *f = new_ccan_file(m, objfile);
- list_add_tail(&m->dep_objs, &f->list);
- }
-
+ i->compiled);
+ i->compiled = NULL;
+ }
}
/* We may need libtap for testing, unless we're "tap" */
if (!streq(m->basename, "tap")
&& (!list_empty(&m->run_tests) || !list_empty(&m->api_tests))) {
- if (stat("../tap.o", &st) != 0) {
+ char *tapobj = talloc_asprintf(m, "%s/ccan/tap.o", ccan_dir);
+ if (stat(tapobj, &st) != 0) {
report = talloc_asprintf_append(report,
- "object file ../tap.o"
- " (for tests)\n");
+ "object file %s"
+ " (for tests)\n",
+ tapobj);
}
}
}
struct ccanlint depends_built = {
- .name = "CCAN dependencies are built",
+ .key = "depends-built",
+ .name = "Module's CCAN dependencies are already built",
.total_score = 1,
.check = check_depends_built,
.describe = describe_depends_built,