[FFe] Accept new python-based Wubi for 9.04
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Wubi |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
Please give me the approval to point cdimage at the new python-based Wubi and umenu. I had intended to do this before feature freeze, but it fell off the radar.
The old Wubi code was written in NSIS, which proved extraordinarily difficult to debug and numerous bugs were discovered that would disappear upon further inspection. This new branch of Wubi is written in Python, making it easier to debug, and uses a simple ctypes wrapper around MFC, allowing it to still be quite small.
It is feature complete and Agostino has been kind enough to provide the following list of major bugs still outstanding:
1) Finalize english text strings (with proper substitutions), possibly
as close as possible to previous ones and test gettext in python (I have
set it up but never actually tested it), I still have a stings that are
developemnt ones.
2) Controls transparency in win32, mostly the issue is lack of decent
documentation on how to do that, I played with a few recipes I found on
the web but my trials were unsuccessful so far
3) Theming, as you pointed out XP themes need to be applied, I have
investigated how to do that:
http://
useful code in
http://
unk/src/win32/ but need to implement that
4) Bittorrent, it is there but I only tested it separately, I did ask
Steve to add torrents to the metalink, but they are not there, we might
want to exclude bittorrent from the build for this round (also to save
space).
5) CD-boot mode has to be done. All the subtasks are working, so it
mostly involves compiling a list of tasks and testing.
6) I did not include grub2 at the end, but I need to add back the code
to compile grub4dos from source (code is very similar to 8.10, already
in my tree, need to test and commit) as I used prebuilt grub4dos
binaries in the testing stages.
1, 5, 6 should be quick, 4 can be scraped for the time being (or we
could test it briefly and then decide according to the results), 2 and 3
are cosmetic changes. For the rest wubi-python should be already fully
functional and worked in my tests. I will try to finish most of the
above by this w/e.
A few known bugs: download resume on old downloads does not work, I do
not check whether another instance of wubi is already running (I should
add that), I need to check for admin rights, when running wubi on a
system with a pre-existing installation, the pre-existing uninstaller is
started but that window is not shown until the first wubi.exe is killed.
I need to make sure that 8.04/8.10 is detected and uninstalled before
proceeding with 9.04 installation. I have some code to show a "use local
ISO" button during the download progress dialog but that functionality
is not activated at the moment (and might stay that way).
Changed in wubi: | |
status: | Confirmed → Fix Released |
More from Agostino:
Yesterday I took care of #5 and #6. While I was on the code I remembered
that when dealing with a CD (as opposed to an ISO), at the moment I copy
the files instead of extracting the ISO. This was an old workaround for
#207137, but that approach is blocked by #243105. At the time I assumed
#243105 would be fixed and I am not sure of the current status of that.
I haven't tested yet the new Wubi with a real CD, but I would expect it
to fail because of that. If fixing #243105 is not viable I will have to
implement ISO extraction in python, which should not bee too difficult,
but is another item on the list.