The patch is just here so I don't have to explain so much, I'll commit it,
possibly with modifications, depending on peoples thoughts.

The new install system can easily give people the option of immediately
installing stable/testing or unstable. 

Pros:
-gives the theoretical person with a slow modem connection the option of
installing unstable immediately rather than needing to install stable and then
upgrade.
-debootstrap supports it, why not give people the option, I like it that
Debian will allow you to do what you want, warn you a lot sometimes, but in the end
doesn't stand in your way.
-the brave souls who choose to install testing will help us to insure
testing is more or less always in a bootstrappable state.


Cons:
-providing support for people trying to install testing/unstable might be a
pain.
-even with all the warnings I put in, someone might install unstable without
meaning to, sad users is not good.



By the way, even if this discussion isn't all that important for boot-floppies
since it is being end of lifed, debian-installer has the same option, I wonder
what people think.


David



Index: extract_base.c
===================================================================
RCS file: /cvs/debian-boot/boot-floppies/utilities/dbootstrap/extract_base.c,v
retrieving revision 1.21.2.5
diff -u -r1.21.2.5 extract_base.c
--- extract_base.c      2001/03/04 08:20:52     1.21.2.5
+++ extract_base.c      2001/03/17 18:42:42
@@ -81,15 +81,27 @@
  * progress_execlog should not print so much that we print out of the box.
  *     This is ugly but it may be hard to know where we are going to wordwrap. 
  * figure out how to handle initial reboot.
+ * would like to set defaults on warnings differently.
  */ 
 
 
-#define BASE_ON_CD     0
-#define BASE_ON_NET    1
-
 int 
 debootstrap_extract_base (void) {
+  int rs;
   struct stat statbuf;
+  char *suite;
+  char *msg=_("Please choose the pool you would like to install your bases system 
+from.  It is strongly recommended that you choose \"stable\".");
+  char *title=_("Select a Distribution");
+#define SUITE_STABLE   100
+#define SUITE_TESTING  101
+#define SUITE_UNSTABLE         102
+#define NUM_CHOICES    3
+  struct d_choices choices[] = {
+      { _("stable"),  _("aka. \"potato\" - strongly recommended"), SUITE_STABLE },
+      {_("testing"), _("aka. \"woody\" - somewhat tested, not recommended"), 
+SUITE_TESTING },
+      {_("unstable"), _("aka. \"sid\" dangerous and not well tested"), SUITE_UNSTABLE 
+},
+  };
+
 
   /* Sanity Check */
   if (NAME_ISREG ("/target/etc/debian_version", &statbuf)) {
@@ -105,6 +117,33 @@
   disqtype=debootstrap;
   if ( choose_medium() ) return 1;
 
+  rs=menuBox(msg, title, choices, NUM_CHOICES, 1); 
+
+  switch(choices[rs].state){
+      case SUITE_STABLE:
+         {
+             suite="potato";
+             break;
+         }
+      case SUITE_TESTING:
+         {
+             suite="woody";
+             if (yesNoBox(_("You have chosen to install the \"testing\" distribution. 
+This distribution contains packages that are still being evaluated.  Bugs do exist 
+and some of them may be serious.  Unless you have a specific need you should say 
+\"no\" and choose the \"stable\" distribution. Are you sure you want to do this?"), 
+_("Warning")) == DLG_NO)
+                 return DLG_CANCEL;
+             break;
+         }
+      case SUITE_UNSTABLE:
+         {
+             suite="sid";
+             if (yesNoBox(_("You have chosen to install the unstable distribution. 
+This distribution contains new packages, some of which have not been tested at all.  
+If you proceed down this path you may end up with an unbootable system.  Bugs do 
+certainly exist and some of them may be extremely serious. Unless you have a specific 
+need you should say \"no\" and choose the \"stable\" distribution.  Are you sure you 
+want to do this?"), _("Warning")) == DLG_NO)
+                 return DLG_CANCEL;
+             break;
+
+         }
+      default:
+         return -1;
+  }
+
 
   if (!strcmp (Archive_Dir, "netfetch"))
     {
@@ -138,7 +177,8 @@
        }
        
        snprintf(prtbuf, sizeof prtbuf, 
-               "/usr/sbin/debootstrap woody /target http://%s:%d/%s", 
+               "/usr/sbin/debootstrap %s /target http://%s:%d/%s", 
+               suite,
                nf_state.server.hostname,
                nf_state.server.port,
                nf_state.path); 
@@ -147,7 +187,8 @@
     {
       /* archive is on the filesystem */
                snprintf(prtbuf, sizeof prtbuf, 
-               "/usr/sbin/debootstrap woody /target %s", 
+               "/usr/sbin/debootstrap %s /target %s",
+               suite,
                Archive_Dir); 
     }
     


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to