tools: commit missing support for _info ported flag.
authorRusty Russell <rusty@rustcorp.com.au>
Wed, 3 Feb 2016 02:49:57 +0000 (13:19 +1030)
committerRusty Russell <rusty@rustcorp.com.au>
Wed, 3 Feb 2016 02:49:57 +0000 (13:19 +1030)
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
doc/ccanlint.1.txt
tools/depends.c
tools/tools.h

index 26d46f4c7e1b52274e62c1799003de98c3e0d986..b2f1537a0c6c5e4e7b5059a3e2689b7acf78e1fa 100644 (file)
@@ -87,6 +87,10 @@ test, but happy:
   CCAN module must have '_info' file describing. No this score is 0.
   However, *ccanlint* question may help to write one.
 
   CCAN module must have '_info' file describing. No this score is 0.
   However, *ccanlint* question may help to write one.
 
+*info_ported*::
+  CCAN module '_info' can say 'ported' arg; if says '0' not ported, few tests
+  run.
+
 *depends_exist*::
   '_info' file CCAN other module without saying, must find. It is not score 0.
 
 *depends_exist*::
   '_info' file CCAN other module without saying, must find. It is not score 0.
 
index 871a3e8712368a649dc3d2d478e696087496bd57..53fed44bff722571b39d54a5f94a47a7cfe7e694 100644 (file)
@@ -227,7 +227,7 @@ get_all_deps(const void *ctx, const char *dir, const char *style,
        return deps;
 }
 
        return deps;
 }
 
-/* Can return NULL: _info may not support 'libs'. */
+/* Can return NULL: _info may not support prop. */
 static char **get_one_prop(const void *ctx, const char *dir, const char *prop,
                           char *(*get_info)(const void *ctx, const char *dir))
 {
 static char **get_one_prop(const void *ctx, const char *dir, const char *prop,
                           char *(*get_info)(const void *ctx, const char *dir))
 {
@@ -282,6 +282,48 @@ char **get_cflags(const void *ctx, const char *dir,
        return flags;
 }
 
        return flags;
 }
 
+static bool get_one_ported(const void *ctx, const char *dir,
+                          char *(*get_info)(const void *ctx, const char *dir))
+{
+       char **ported = get_one_prop(ctx, dir, "ported", get_info);
+
+       /* No news is good news. */
+       if (!ported || tal_count(ported) == 0)
+               return true;
+
+       if (tal_count(ported) != 1)
+               errx(1, "%s/_info ported gave %zu lines, not one",
+                    dir, tal_count(ported));
+               
+       if (streq(ported[0], "1"))
+               return true;
+       else if (streq(ported[0], "0"))
+               return false;
+       errx(1, "%s/_info ported gave invalid output '%s'", dir, ported[0]);
+}
+
+bool get_ported(const void *ctx, const char *dir, bool recurse,
+               char *(*get_info)(const void *ctx, const char *dir))
+{
+       if (!get_one_ported(ctx, dir, get_info))
+               return false;
+
+       if (recurse) {
+               size_t i;
+               char **deps = get_deps(ctx, dir, "depends", true, get_info);
+               for (i = 0; deps[i]; i++) {
+                       char *subdir;
+                       if (!strstarts(deps[i], "ccan/"))
+                               continue;
+
+                       subdir = path_join(ctx, find_ccan_dir(dir), deps[i]);
+                       if (!get_one_ported(ctx, subdir, get_info))
+                               return false;
+               }
+       }
+       return true;
+}
+
 char **get_libs(const void *ctx, const char *dir, const char *style,
                char *(*get_info)(const void *ctx, const char *dir))
 {
 char **get_libs(const void *ctx, const char *dir, const char *style,
                char *(*get_info)(const void *ctx, const char *dir))
 {
index cfbe871f9bfb0228016f1e2ab80f0085480592ae..7e2598c3167559df675c71df2f9d6ebcf624464a 100644 (file)
@@ -46,6 +46,9 @@ char **get_libs(const void *ctx, const char *dir, const char *style,
 char **get_cflags(const void *ctx, const char *dir,
                char *(*get_info)(const void *ctx, const char *dir));
 
 char **get_cflags(const void *ctx, const char *dir,
                char *(*get_info)(const void *ctx, const char *dir));
 
+bool get_ported(const void *ctx, const char *dir, bool recurse,
+               char *(*get_info)(const void *ctx, const char *dir));
+
 /* From tools.c */
 /* If set, print all commands run, all output they give and exit status. */
 extern bool tools_verbose;
 /* From tools.c */
 /* If set, print all commands run, all output they give and exit status. */
 extern bool tools_verbose;