/srv/irclogs.ubuntu.com/2010/09/06/#ubuntu-installer.txt

=== ara_ is now known as ara
evScottK, Riddell: http://paste.ubuntu.com/489138/ - partitioning page fixes.  Could either of you let me know how well that works for you?  I still have to add back the "use largest free space" option, but it should otherwise be complete, functional, and hopefully bug free.10:12
Riddellev: cool, I'll give it a test shortly10:25
evthanks!10:27
CIA-34ubiquity: evand * r4280 trunk/ (4 files in 3 dirs):11:44
CIA-34ubiquity: Fetch the translations for the release_notes_only and11:44
CIA-34ubiquity: update_installer_only strings (LP: #629627).11:44
=== ogra_ is now known as ogra
CIA-34ubiquity: evand * r4281 trunk/ (3 files in 3 dirs):11:58
CIA-34ubiquity: Don't use the same name for the error method and GTK label11:58
CIA-34ubiquity: (LP: #631046).11:58
CIA-34grub-installer: cjwatson * r864 timeout/ (debian/grub-installer.templates grub-installer): merge lp:~soren/grub-installer/ubuntu-boot-delay12:00
CIA-34grub-installer: cjwatson * r865 timeout/ (debian/grub-installer.templates grub-installer): boot_delay -> timeout, for consistency12:01
CIA-34grub-installer: cjwatson * r864 ubuntu/ (3 files in 2 dirs):12:05
CIA-34grub-installer: Add a preseedable grub-installer/timeout template to adjust the initial12:05
CIA-34grub-installer: > GRUB timeout (LP: #628418).12:05
CIA-34grub-installer: cjwatson * r865 ubuntu/grub-installer: consistent behaviour for grub-legacy12:07
CIA-34grub-installer: cjwatson * r866 ubuntu/debian/changelog: releasing version 1.55ubuntu212:08
CIA-34ubiquity: evand * r4282 trunk/ (debian/changelog ubiquity/plugins/ubi-usersetup.py):12:27
CIA-34ubiquity: Re-use the valid username check from user-setup in the GTK UI12:27
CIA-34ubiquity: (LP: #631046).12:27
NCommandercjwatson: hey, not sure if your around, but its been ~2 weeks since I sent mail to d-boot about the changes to libd-i. Any chance we can move towards getting that merged so I can knock that spec out? :-)13:09
NCommandercjwatson: or how to move this forward in general13:10
cjwatsonI'll look over it13:15
cjwatsonone of our major patch reviewers died recently which hasn't exactly helped13:15
cjwatson(replied)13:22
davmor2cjwatson, ev: did you every get the wubi issue resolved?13:35
davmor2was on hols last week13:35
evdavmor2: yes13:36
evshould work now13:36
davmor2ev: cool I'll give it a spin latter :)13:37
davmor2ev: what was it for future reference13:37
evgrub issue13:37
cjwatson  * wubildr.cfg needs to be run using GRUB's normal parser rather than its13:45
cjwatson    rescue parser.  Rather than embedding it directly as a configuration13:45
cjwatson    file, embed it as a memdisk and add wubildr-bootstrap.cfg which invokes13:45
cjwatson    it using normal mode (LP: #600578).13:45
Riddellev: hmm, it was all going well then failure, fault in my VM? http://people.canonical.com/~jriddell/tmp/partitioner-kubuntu-fault.png13:50
evcould be13:51
evRiddell: anything in /var/log/messages of interest?13:52
Riddellhttp://muse.19inch.net/~jr/tmp/debug  http://muse.19inch.net/~jr/tmp/syslog http://muse.19inch.net/~jr/tmp/messages13:53
evinteresting:13:54
evSep  6 12:40:20 ubuntu ubiquity: Resizing the filesystem on /dev/sda1 to 816486 (4k) blocks.13:54
evSep  6 12:40:20 ubuntu ubiquity: The filesystem on /dev/sda1 is now 816486 blocks long.13:54
evSep  6 12:40:20 ubuntu ubiquity:13:54
evSep  6 12:40:22 ubuntu ubiquity: resize2fs 1.41.12 (17-May-2010)13:54
evSep  6 12:40:22 ubuntu ubiquity: The filesystem is already 816486 blocks long.  Nothing to do!13:54
Riddellthat doesn't look healthy13:55
evRiddell: what steps did you take at the partitioner?13:55
evalso, can I have /var/log/partman13:55
Riddellhttp://muse.19inch.net/~jr/tmp/partman13:56
Riddellthe "Install side by side" step13:57
evokay, did you move the slider at all?14:03
Riddellev: yes14:06
CIA-34usb-creator: cjwatson * r320 allow-system-internal/ (9 files in 7 dirs):14:26
CIA-34usb-creator: Add an --allow-system-internal option (Unix only) to allow installation14:26
CIA-34usb-creator: to system-internal devices such as hard disks. This is useful when14:26
CIA-34usb-creator: preparing test USB images in KVM.14:26
CIA-34ubiquity: cjwatson * r4283 ubiquity/ (debian/changelog scripts/plugininstall.py ubiquity/misc.py):15:09
CIA-34ubiquity: Factor out common /proc/mounts handling into a new15:09
CIA-34ubiquity: ubiquity.misc.mount_info function.15:09
CIA-34ubiquity: evand * r4284 trunk/ (3 files in 3 dirs):15:41
CIA-34ubiquity: Do not cut off the text of the Try and Install buttons by setting15:41
CIA-34ubiquity: their width to just the largest initial width of the two15:41
CIA-34ubiquity: (LP: #629437).15:41
cjwatsonhmm.  the mtime of /casper-rw is getting hashed into apt-cdrom's identification15:43
cjwatsonthat's really not going to help matters15:43
cjwatsonI'd sort of like to move casper-rw to .disk or something, but then that would require casper changes and everyone would have to have a new version of usb-creator15:48
cjwatsonand we'd keep on getting bugs about broken OEM installs from people who haven't15:48
evassuming btrfs lands in 11.04, perhaps we should consider replacing the casper-rw machinery with something that leverages btrfs15:51
evand move things around then15:51
cjwatsonyou know, I'm half-tempted to use a dummy apt cdrom method15:51
cjwatsonall the media-change stuff is a pain and it would be good to be able to just say "no, damnit, /cdrom is it"15:52
cjwatsonor I suppose I could use LD_PRELOAD15:52
evworks for me15:52
cjwatsonLD_PRELOAD might actually be easier; the filename for the method executable comes from the URI scheme, so it's hard to substitute15:53
cjwatsonbut LD_PRELOADing C++ code, OMG15:55
evhaha15:55
cjwatsonand since casper uses apt-cdrom, anything else that uses apt (e.g. jockey in the live session) will have the same problem15:58
cjwatsonmoving casper-rw out of the root seems like the only feasible option15:59
cjwatsondoes .disk seem like a reasonable place for it?16:01
cjwatsonor the casper subdirectory, I suppose16:01
evindeed16:02
CIA-34usb-creator: cjwatson * r320 trunk/ (debian/changelog debian/control usbcreator/install.py): Use python-debian for Ubuntu release version comparison.16:15
cjwatsonev: is the str(self.target) stuff in usbcreator/install.py still necessary?  it seems to be cast to str in some contexts but not in others16:28
evcjwatson: not sure, that's purely PyQt being god-awful.  GTK knows how to handle unicode in a sane fashion.16:29
cjwatsonright, I know, but what I don't get is why the KDE frontend works at all since there are some places where self.target isn't cast16:31
cjwatsonself.target is only assigned to in one place, so maybe cast it there?16:33
evsure16:37
ScottKev: I can do some testing later today if you still need it.16:54
evScottK: please do.  I haven't had a chance to look at Riddell's bug, but I'm keen to see if it works for you.16:55
ScottKev: OK.  Should be able to give it a whack in about 90 minutes.16:56
evcool, thanks!16:57
NCommandercjwatson: thanks for your review. I agree with your suggestions and working on rewqorking the patch to incorperate it.17:16
evweird, indicator-session off the custom panel can't seem to talk to ConsoleKit over dbus, despite the session daemon running.17:16
cjwatsonev: should I land that usb-creator patch?17:24
evcjwatson: please do17:25
CIA-34usb-creator: cjwatson * r321 trunk/ (9 files in 7 dirs): merge lp:~cjwatson/usb-creator/allow-system-internal17:27
CIA-34ubiquity: evand * r4285 trunk/ (6 files in 6 dirs):17:51
CIA-34ubiquity: Update the panel to use the new location for the panel background,17:51
CIA-34ubiquity: and force a redraw when setting it.17:51
evdamn17:51
CIA-34ubiquity: evand * r4285 trunk/ (debian/changelog src/panel/panel.c):17:51
CIA-34ubiquity: Update the panel to use the new location for the panel background,17:51
CIA-34ubiquity: and force a redraw when setting it.17:51
* ev whistles, hopes no one noticed that17:51
evah, ck-launch-session sets XDG_SESSION_COOKIE, which is only going to picked up by metacity.  Rats.18:33
evright, home time18:33
ScottKRiddell: When you're testing ubiquity changes, how do you usually go about doing it?  Did you make a .deb with the patch that's somewhere reachable?18:33
evScottK: you can patch in the live environment, or in casper (setting break=bottom on the kernel command line: https://wiki.ubuntu.com/Installer/Development/Tips)18:34
evright, out18:34
ScottKThanks.18:34
RiddellScottK: in this case I patched the files, and just did sudo cp to move them into the right place http://muse.19inch.net/~jr/tmp/ubiquity/18:36
ScottKThanks.18:37
Riddellev, ScottK: i tried the change again on real hardware and it worked fine18:37
* ScottK will try to squeeze testing in around the other chores for the afternoon.18:37
Riddelland I've been getting write errors while upgrade testing today18:37
Riddellso i think my VM (or my laptop harddisk) is having problems rather than the patch18:38
=== JanC_ is now known as JanC
ScottKOK.18:41
ScottKMeh.  No ubiquity testing for me today.  My wife is back from her trip earlier than I'd expected.20:27
NCommandercjwatson: so I'm working on reworking the libd-i patch, and after some googling, I think it makes sense to implement the new guess function as a weak symbol (especially because this leaves Debian free to not implement it until its needed)21:06
NCommanderis that going to cause a problem, or should I rework my WIP patch to use strong symbols only21:07
cjwatsonNCommander: um, I'm not sure, will probably be easier to judge based on a patch which I can try out, since I always have to look up linker details21:37
NCommandercjwatson: ah :-). I don't think we need to touch the linker voodoo in libd-i if its a weak symbol, but once I have a new patch ready (probably by your tomorrow)21:40

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!