cvs main-menu bust?

[Richard Hirst <rhirst@linuxcare.com>]

This change:

@@ -304,20 +306,21 @@
        int ret, i;
        struct package_t *dep;
 
+       if (di_pkg_is_virtual(p)) {
+               if (!satisfy_virtual(p))
+                       return 0;
+       }
+
        for (i = 0; p->depends[i] != 0; i++) {
                if ((dep = p->depends[i]->ptr) == NULL)
                        continue;
                if (dep->status == installed)
                        continue;
-               if (di_pkg_is_virtual(dep)) {
-                       if (!satisfy_virtual(dep))
-                               return 0;
-               } else {
-                       /* Recursively configure this package */
-                       if (!config_package(dep))
-                               return 0;
-               }
+               /* Recursively configure this package */
+               if (!config_package(dep))
+                       return 0;
        }
+
        asprintf(&configcommand, DPKG_CONFIGURE_COMMAND " %s", p->package);
        ret = SYSTEM(configcommand);
        free(configcommand);


breaks main-menu for me.  Execute a Shell still works, but the other
entries on that first menu all just result in the main menu being
redisplayed, preceeded by

"Package libdebconf1 is already installed and configured"

This in on hppa.  Anyone else see this problem, or is it just me?

Richard