herd4-v8-comments

List of suggestions for Wubi. ** = high priority.

MENU.LST

* There should be a single menu entry in menu.lst. Now there are at least 3 (Ubuntu, Ubuntu (debug), Ubuntu Dynamically Added). In the new lupin code there is no distinction between debug and normal mode. The dynamic menu is supposed to replace the static menus. Static menus are provided only as an example (I will comment them out in lupin to make that point clear).
* I am not sure it is a good idea to append items to the menu. If you really want to append, you should pre-pend or change the grub default since otherwise the default menu selection is going to be wrong. In single installation mode, appending to the menu is mostly useless, it is safe enough to overwrite menu.lst (maybe create a backup copy).
** The find command in the dynamically generated menu is wrong, --set-root should come before the filename.
** The kernel parameters are wrong, see menu.lst within lupin for the parameters to be used
* Make sure line end chars in menu.lst are windows compatible (CR+LF)
* Hiddenmenu should be uncommented in menu.lst, since there should be a single item in the menu, there is no point in showing the menu at all.

PRESEED.CFG

** Strings in preseed.cfg mus be replaced by Wubi. Put some placeholder for each variable like STRLAYOUTCODE, and use string substitution to replace only the placeholder. In particular need to be replaced:
  d-i console-setup/layoutcode string STRLAYOUTCODE
  d-i time/zone string STRTIMEZONE
** Username and password are inserted incorrectly, the full line is replaced. This will probably corrupt the preseed file. Again use placeholder variables and replace those placeholders while keeping the rest of the line intact. This is what I get:
  "# To create a normal user account.
  ago
  ago
  # Normal user's password, either in clear text
  mypassword
  mypassword"
** password should be hashed! no clear text password, this is a major issue.

WUBI GUI

* Wubi icon should simply be the ubuntu logo, possibly in yellow, no text in the icon, icons do not have labels.
* language selector should be included in main dialog
* normal/expert selector can be removed
* remove timezone selector, things should either be enabled and working or not visible at all. Timezone should be autodetected, I am not sure there is any need for a selector.
* The final message: "Install complete. Do you wish to test the system to test ubuntu?" is misleading. It should read: "You need to reboot to complete the installation. Do you want to reboot now?" Ideally it should be within the wizard dialog not as a popup.
* Wubi looks for ISOs only under C:.
* If the ISO is detected, there is a dialog that is redundant.
* if there are multiple partitions, check free space on all of them, and default to the partition with biggest free space, give the option to select the drive.
* minimum installation size can probably be reduced to 4-5GB, but default image should be 6GB if there is enough space
* username should be auto-detected, don't use "ubuntu"

MISC

* Malone should be activated, it's not possible to submit bugs at the moment
* ISO should be under /wubi/install