[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#140980: boot-floppies: confusing message box



#include <hallo.h>
David Kimdon wrote on Wed Apr 03, 2002 um 02:56:46PM:

> > > partitions I want in the boot menu?  I think I am choosing, so the above
> > > 'View : Show' menu should probably be change to read 'Choose'.
> > 
> > Means new functionality, we should not do this in the current phase.
> 
> No, I'm not asking for new functionality.  I'm asking for
> clarification of the message, it is confusing.  Someone needs to read

Hm. Okay. I choosed the menuBox function since it was the easiest way to
implement a such display option. Another way is a plain list of
partitions with description. No way to switch anything, pretty clear
message, IMHO. Patch attached, untested.

Gruss/Regards,
Eduard.
-- 
Kleines Lexikon der Computerwerbung:
"integriert" 
minderwertige Einzelteile vereint in einem katastrophalem Ganzen
Index: utilities/dbootstrap/bootconfig.c
===================================================================
RCS file: /cvs/debian-boot/boot-floppies/utilities/dbootstrap/bootconfig.c,v
retrieving revision 1.194
diff -u -r1.194 bootconfig.c
--- utilities/dbootstrap/bootconfig.c	2002/03/25 05:58:20	1.194
+++ utilities/dbootstrap/bootconfig.c	2002/04/03 23:28:29
@@ -2638,7 +2638,21 @@
            case 0:
               break;
            case 1:
-              menuBox(_("Following partitions are bootable: "), _("Other bootable partitions"), bootable, boot_number, 1);
+              { 
+                 char * buf;
+                 int i;
+                 buf = (char *) calloc (boot_number, 50);
+                 for(i=0; i< boot_number; i++) {
+                    strcat(buf, "\t");
+                    strcat(buf, bootable[i].tag);
+                    strcat(buf, "(");
+                    strcat(buf, bootable[i].string);
+                    strcat(buf, ")\n");
+                 }
+                 sprintf(prtbuf, _("The following partitions seem to be bootable and would be added to the boot menu:\n\n%s"), buf);
+                 problemBox(prtbuf, _("Other bootable partitions"));
+                 free(buf);
+              }
               goto redisplay_options;
            case 2:
               boot_number=0;

Reply to: