The recent option parsing changes broke the multiplexer, because the "toybox"

command was trying to parse their options and failing. (Commands
that aren't just disabled but have no option string even in the allyesconfig
case need an OPTSTR of 0.)

This is probably only a partial fix, there's some design work to think through
here.

diff --git a/scripts/mkflags.c b/scripts/mkflags.c
index 27fc517..39b935b 100644 (file)
--- a/scripts/mkflags.c
+++ b/scripts/mkflags.c
@@ -122,7 +122,7 @@ int main(int argc, char *argv[])
 
   for (;;) {
     struct flag *flist, *aflist, *offlist;
-    char *mgaps;
+    char *mgaps = 0;
     unsigned bit;
 
     *command = *flags = *allflags = 0;
@@ -141,11 +141,13 @@ int main(int argc, char *argv[])
 
     bit = 0;
     printf("// %s %s %s\n", command, flags, allflags);
-    mgaps = mark_gaps(flags, allflags);
+    if (*flags != ' ') mgaps = mark_gaps(flags, allflags);
+    else if (*allflags != ' ') mgaps = allflags;
     // If command disabled, use allflags for OLDTOY()
-    printf("#undef OPTSTR_%s\n#define OPTSTR_%s \"%s\"\n",
-            command, command, strcmp(flags, " ") ? mgaps : allflags);
-    free(mgaps);
+    printf("#undef OPTSTR_%s\n#define OPTSTR_%s ", command, command);
+    if (mgaps) printf("\"%s\"\n", mgaps);
+    else printf("0\n");
+    if (mgaps != allflags) free(mgaps);
 
     flist = digest(flags);
     offlist = aflist = digest(allflags);