[09:29] <evand> mpt: Still working on it, but how is this so far? (http://evalicious.com/tmp/usb-creator.png)
[09:29] <evand> the slider will definitely get a human readable value on the right
[09:31] <mpt> evand, wow, nice work
[09:31] <mpt> The listboxes should have a border
[09:32] <mpt> "Other…" probably should be on the left rather than the right (if only to reduce the odd gap above the "USB disk to use:" label)
[09:33] <evand> noted.  Yeah, the gap is a bug in gtk, unfortunately.
[09:33] <evand> http://bugzilla.gnome.org/show_bug.cgi?id=101968
[09:34] <mpt> ah, yes, that famous bug
[09:34] <mpt> but no, I meant above the "USB disk to use:" label, not above the source listbox
[09:34] <evand> ah, whoops
[09:36] <mpt> "Other…" should have 12px inside horizontal padding (i.e. between the left/right of the label and the left/right edge of the button)
[09:36] <mpt> (and Glade should do that by default, dagnammit)
[09:38] <evand> heh
[09:41] <mpt> Ideally "How" should be aligned exactly with "Stored", but that's currently practically impossible in GTK <http://bugzilla.gnome.org/show_bug.cgi?id=454248>
[09:41] <evand> yeah, I tried to do that as best I could
[09:41] <mpt> but "How" should at least not be to the *left* of "Stored" in Ubuntu's default theme
[09:42] <evand> but packing it into the option button posed two problems
[09:42] <evand> one, it doesn't fully fill the remaining space to the right (which I ran into with another spec in the previous cycle)
[09:42] <evand> and two, the button then becomes centered on the combined label ("Stored in reserved") and slider
[09:43] <xivulon> cjwatson, evand, please have a look at 268123
[09:43] <mpt> yeah, ugly
[09:43] <xivulon> it's a blocker for wubi and cking is out of ideas
[09:43] <mpt> evand, System > Preferences > Appearance > Visual Effects has the same problem
[09:45] <mpt> evand, looks like "Make Startup Disk" has more margin to its right than the rest of the window does
[09:46] <evand> hrm, well spotted
[09:46] <mpt> And finally, "Size" and "Free Space" should be the same width (though I guess that's probably also really difficult)
[09:47] <evand> I can always give it a shot and make a note of it if I don't find a solution for that in time
[09:48] <evand> thanks for the pointers.  I'm going to take a short nap then get back to finishing these hopefully before the UI freeze cut off time, whenever that may be.
[09:48] <evand> do let me know if anything else comes to mind
[09:48] <evand> either here or in email
[09:49] <evand> xivulon: no idea without digging deeply into it, and I unfortunately don't have the time for that right now with the impending freeze
[09:49] <mpt> evand, what happens when either of those lists has more than one item?
[09:49] <persia> evand: I'm preseeding ubiquity, and am encountering a loop when partitioning.  Would you have time to review/troubleshoot a preseed file?  http://paste.ubuntu.com/45684/
[09:50] <mpt> evand, does the window resize, or does the list get a teeny tiny scrollbar?
[09:50] <cjwatson> xivulon: OK, I think I've got it
[09:50] <xivulon> cjwatson you are a *
[09:50] <evand> mpt: I was going to pack a scrollbar around them.  Thoughts?
[09:51] <mpt> I don't understand this GTK idea of a scrollbar being an independent entity
[09:51] <mpt> makes my brain hurt
[09:51] <evand> persia: select_disk looks wrong, but don't quote me on that.  Unfortunately I quite need to get to bed.
[09:51] <mpt> but yes, they should have vertical scrollbars :-)
[09:52] <evand> heh
[09:52] <evand> persia: if cjwatson doesn't reply before I come back I'll take a closer look then
[09:52] <persia> evand: OK.  Thanks.  select_disk came from debconf-get-selections on the livecd, but that doesn't make it right :)
[09:52] <cjwatson> you should never preseed partman-auto/select_disk
[09:52] <cjwatson> start out with the installation guide rather than with debconf-get-selections
[09:53] <persia> OK.
[09:53] <cjwatson> d-i partman-auto/init_automatically_partition select some_device
[09:53] <cjwatson> d-i partman-auto/disk string /dev/sdb
[09:54] <cjwatson> d-i partman-auto/method string regular
[09:54] <cjwatson> d-i partman-auto/choose_recipe select atomic
[09:54] <cjwatson> and in general everything else should have d-i in the first field rather than console-setup or ubiquity (subtle debconf ownership stuff)
[09:55] <persia> Oops :)
[09:56] <cjwatson> one of these days I'll write some kind of Pure Magic heuristic tool that figures out the preseed file you actually wanted to write
[09:56] <evand> oh, mpt before I depart.  Is disabling rows (which I need to figure out how exactly to do in a treeview) where the CD will never fit and adding a warning icon to rows where it might sufficiently self-explanitory, or should I add (dare I suggest) a tooltip or some other aid?
[09:57] <persia> cjwatson: Thank you.
[09:57] <cjwatson> xivulon: uploaded; *should* work ...
[09:57] <evand> cjwatson: kickstart ;)
[09:58] <cjwatson> heh
[09:58] <cjwatson> doesnae work with ubiquity though
[09:58] <xivulon> cjwatson thanks, cannot test now though, davmor2 can you please regenerate the initrd and see if it works?
[09:58] <mpt> evand, ah, I didn't make a complete suggestion for that, did I
[09:59] <cjwatson> davmor2: err, wait a couple of hours for the new busybox-initramfs to arrive
[09:59] <mpt> hmm
[10:01] <mpt> evand, I think a line of text underneath the USB disk list, describing the situation for the selected disk:
[10:02] <mpt> "<disk name> has enough free space for <OS version>."
[10:02] <mpt> "<disk name> contains too many files to fit <OS version> alongside."
[10:02] <mpt> "<disk name> is too small for <OS version>."
[10:05] <mpt> Actually, better for the second case:
[10:06] <mpt> "<disk name> is too full to fit <OS version> (N MB more needed).     ( Open )"
[10:06] <davmor2> Hi guys not going to be around much today I can probably test it tonight or tomorrow at some point
[10:06] <mpt> (where "Open" opens the disk in Nautilus)
[10:07] <xivulon> #ubuntu-testing
[10:07] <cjwatson> mpt: on bug 254652, would "Installation Failed (Faulty CD)" work?
[10:08] <cjwatson> mpt: I'd like to put the "damnit it wasn't the installer's fault" up top, if possible
[10:09] <mpt> cjwatson, as long as the reason is prominent in the window, I don't think it needs to be wedged into the title too
[10:09] <cjwatson> hm, ok, I was just concerned that people wouldn't bother reading the more verbose text
[10:10] <cjwatson> but if you're sure, ok
[10:10] <mpt> In alerts I think people are more likely to read the primary text than the window title
[10:10] <mpt> (to the point where most alerts shouldn't bother with titles, unless they have no parent window)
[10:11] <cjwatson> I'll make the primary text a little more verbose as well; "This particular error is due to" -> "This is due to"
[10:11] <cjwatson> little LESS verbose
[10:12] <mpt> great
[10:16] <CIA-46> ubiquity: cjwatson * r2809 ubiquity/debian/ (81 files in 2 dirs):
[10:16] <CIA-46> ubiquity: Fix window title of bulk copy failure alerts, and remove a bit of
[10:16] <CIA-46> ubiquity: unnecessary verbosity from the primary text (LP: #254652).
[10:58] <persia> I have a two-line patch for partman-partitioning/lib/disk-label.sh  Where's the best place to try to push it for review?
[11:01] <cjwatson> branch of lp:~ubuntu-core-dev/partman-partitioning/ubuntu, or attached to a bug if you can't be bothered :)
[11:01] <persia> I don't mind branching.
[11:12] <persia> I'm also getting a repeatable crash of partman, even with the now fixed preseed.  I'm not finding anything informative in /var/log/installer/debug, /var/log/partman or /var/log/syslog.  When running ubiquity under strace -f, I am unable to find the reported exit value of 10.
[11:13] <persia> Branch with 2-line patch for review is lp:~persia/partman-partitioning/ubuntu
[11:21] <cjwatson> partman behaves oddly if partially preseeded. Are you using ubiquity --debug? (/var/log/installer/debug isn't much use without that.)
[11:22] <cjwatson> that partman-partitioning branch is fine; I'll merge it
[11:22] <persia> I was using just "-d".  Does it need the entire word?
[11:23] <cjwatson> -d is fine too
[11:23] <cjwatson> can I see the debug file?
[11:23] <persia> Yeah, I was afraid my problem was with preseeding :(
[11:24] <cjwatson> though cf. bug 268593, there might be a more recently-introduced bug than the one I'm thinking of
[11:24] <persia> Take me a bit to pull it out: it's on a KVM instance with no functional mouse :)
[11:24] <cjwatson> scp to the host?
[11:25] <persia> Actually, bug 268593 looks exactly like what I'm finding.
[11:26] <cjwatson> you might try not preseeding partman-auto/init_automatically_partition
[11:26] <cjwatson> it shouldn't be needed if you've preseeded partman-auto/disk and partman-auto/method
[11:26] <cjwatson> (I don't think ...)
[11:26] <cjwatson> partman-partitioning uploaded, though CIA is sulking and refuses to report it
[11:26] <persia> I'll give that a shot when I've extracted the log.
[11:27] <persia> Thanks.  The next alternate CD might actually work (assuming it builds today)
[11:31] <persia> /var/log/installer/debug is http://paste.ubuntu.com/45721/
[11:31] <CIA-46> partman-partitioning: cjwatson * r683 ubuntu/debian/changelog: releasing version 59ubuntu5
[11:31] <CIA-46> partman-partitioning: cjwatson * r682 ubuntu/ (debian/changelog lib/disk-label.sh): merge from lp:~persia/partman-partitioning/ubuntu
[11:32] <persia> http://paste.ubuntu.com/45723/ is the associated /var/log/partman
[11:33] <cjwatson> oh, not quite the same thing
[11:34] <cjwatson> basically the problem is that ubiquity's state machine drove partman the wrong way and ended up backing up all the way out
[11:34] <cjwatson> did you interact with partman at all, or was it entirely automatic?
[11:34] <cjwatson> from the timestamps I'm guessing the latter unless you were super-quick
[11:34] <persia> entirely automatic: X doesn't recognise the KVM keyboard or mouse on the image I'm using.
[11:35] <persia> (which makes it a little tricky to "Try Again")
[11:35] <cjwatson> usually the process for debugging this kind of thing is to bring up two windows, one with partman and one with ubiquity's partman state machine, and do a tedious manual trace
[11:37] <persia> Oh well.  I was hoping for a silver bullet.
[11:38] <persia> I'll dig through by hand then.  Running `sudo partman` generates some errors that udpkg isn't found, and one that apt-install isn't found, but formats the disk as requested and mounts it to both swap and /target
[11:40] <cjwatson> oh, yeah, you can't realistically run partman from a shell
[11:40] <cjwatson> I wouldn't bother with that, it'll be too different
[11:40] <persia> I was just happy it worked : it confirmed for me that the patch you recently merged was sufficient.
[11:41] <persia> But yes, it doesn't really help debug ubiquity.
[11:47] <cjwatson> Hm. OK, the problem is that when ubiquity is in automatic mode it immediately responds to all debconf INPUT commands for questions that are marked as seen with 30 question skipped.
[11:47] <cjwatson> This is done at a very low level, and is across the board.
[11:48] <cjwatson> However, ubiquity's partman state machine relies on being able to go back and forward a lot, which involves it being informed about INPUT commands for questions that are marked as seen.
[11:49] <cjwatson> --automatic defeats this right now and so everything breaks.
[11:49] <persia> Essentially, partman doesn't work with --automatic then?
[11:50] <cjwatson> Right. This might not be too hard to fix, though; all ubiquity should need to do is explicitly mark questions as unseen when it wants to be notified of them again.
[11:50] <persia> What if I preseeded *every* internal value set during the desktop install?
[11:51] <cjwatson> You can't do that with partman
[11:51] <persia> Oh, that doesn't sound so bad.  Is that intrusive enough to warrant delay, or something that could be done sooner?
[11:51] <persia> I was afraid of that.  Perhaps the best solution is to go fix X :)
[11:52] <cjwatson> Preseeding only works with straight-through paths, and partman repeatedly asks the same questions with different parameters
[11:52] <cjwatson> That's why there's this completely separate partman-auto/{disk,method} approach for preseeding
[11:52] <persia> Oh, so partman-auto is intended to inform partman without blocking the queries?
[11:53] <cjwatson> Ish
[11:53] <cjwatson> It bypasses all the interactive stuff
[11:54] <cjwatson> But ubiquity does its own scanning in order to be able to build a UI
[11:54] <cjwatson> Of course, that isn't necessary if partman is preseeded, so we could also make ubiquity check for that ...
[11:54] <cjwatson> I have to go and do some other things, but I'm happy to work on this. It's a generic bug, not lpia-specific or anything
[11:55] <persia> OK.  No rush, as I only encounter it trying to work around something else :)
[11:55] <persia> Please let me know if I can help test, but I'll admit to only limited understanding of the ubiquity and d-i code.
[11:55] <cjwatson> All this ubiquity/partman state machine stuff is my fault anyway
[11:58] <persia> Well, you're also the person to thank for heaps of it working as well :)
[12:10] <cjwatson> Hmm, of course part of the problem here is that questions are getting marked as seen even though they were purely internal and never shown to the user
[12:10] <cjwatson> that really can't be helping
[12:39] <persia> OK.  With X hacked to work a bit better, I can now reliably reproduce something more akin to bug 268593, at least for /var/log/installer/debug and /var/log/partman
[15:07] <saispo> cjwatson1: do you think it's possible to build an i386 and an amd64 cd with extra repository where packages are in the same directory ? must tweak apt-ftparchive no ?
[15:22] <saispo> cjwatson1: you use dak or reprepro for building archives ?
[15:25] <cjwatson1> should be perfectly possible
[15:25] <cjwatson> Ubuntu as a whole uses Soyuz; cdimage uses apt-ftparchive
[15:30] <saispo> cjwatson: ok, thanks, will search how... :)
[16:34] <davmor2> cjwatson: right I'm back home now is it possible to test wubi now?
[16:35] <cjwatson> you'd need a CD for an actual wubi test, but it ought to be possible to upgrade busybox-initramfs, run 'sudo update-initramfs -u', and see if it's possible to mount an ntfs partition from the initramfs
[16:35] <cjwatson> I can't help you with the procedure right now though
[16:35] <cjwatson> if you need a CD, that should happen tomorrow
[16:37] <davmor2> cjwatson: cool does that mean the landscape issue is resolved now?
[16:43] <cjwatson> by means of removing landscape-client from ubuntu-desktop for the moment, yes
[16:51] <davmor2> cjwatson: Ah cool.  I'll leave for in the morning then I think as I believe it will dl the iso if the cd isn't available anway
[17:44] <cjwatson> evand: you have a pile of new bugs from me; 269044, 269048, and 269050 should be low-hanging fruit
[17:44] <cjwatson> (at least)
[17:48] <evand> cjwatson: much appreciated.  Gave a quick look over and I'll implement/integrate fixes once I'm done working in mpt's suggestions.
[17:48] <cjwatson> I'm just trying to get to the point where successful runs are actually possible
[17:48] <evand> weird, it's been working fine for me.
[17:48] <evand> but that's probably to be expected
[17:49] <cjwatson> I don't see how install.py could possibly work
[17:50] <evand> indeed, I'm quite surprised given your bug report, but I've seen it happen.
[17:50] <cjwatson> you aren't being fooled due to bug 269035?
[17:51] <evand> hrm, possibly.  Still, I've watched it fill the disk.
[17:52] <cjwatson> oh, it fills the disk OK
[17:52] <cjwatson> the failures in install.py are after bulk copying
[17:53] <evand> ahhh
[18:51] <kirkland> hey guys, what's the traditional way of getting important early boot messages to either a) be splash-printed, or b) kill splash so that the console shows?
[18:51] <kirkland> cf: https://bugs.launchpad.net/ubuntu/+source/initramfs-tools/+bug/268873
[18:52] <kirkland> just a pointer to some other code would suffice.....
[18:54] <cjwatson> usplash has a TEXT-URGENT message type; see usplash_write(8)
[18:55] <kirkland> cjwatson: thanks
[19:13] <kirkland> cjwatson: it's not immediately obvious to me as to how I might "usplash_read()", in that I have a question which asks, "Do you wish to start the degraded RAID?" which times out after 15 seconds
[19:13] <kirkland> cjwatson: if usplash is enabled, that's not shown, and the timeout happens
[19:14] <kirkland> cjwatson: i see how I could use usplash_write()... but not how I might grab that y/n input
[19:14] <kirkland> cjwatson: would it be reasonable for me to just kill usplash if running before displaying that text and prompting?
[19:17] <cjwatson> there's an INPUTTIMEOUT command
[19:18] <cjwatson> scripts/casper-functions in casper has an example of how to use the INPUT* family
[19:18] <cjwatson> please don't kill usplash
[19:19] <cjwatson>         if [ -x /sbin/usplash_write ]; then
[19:19] <cjwatson>                 /sbin/usplash_write "INPUTENTER $@"
[19:19] <cjwatson>                 read nunya < /dev/.initramfs/usplash_outfifo
[19:19] <cjwatson>         fi
[19:19] <cjwatson> though with a different command and actually doing something with the input
[19:27] <kirkland> cjwatson: thanks!
[19:34]  * evand thinks he's going to go blind staring at d-feet's supposedly pretty printed output.
[21:38] <cjwatson> persia: I'm fixing the lpia CD build failure; it's because the kernel team yet again forgot to tell me about an ABI bump.
[21:38] <cjwatson> I have complained about this before
[22:02] <superm1> cjwatson, perhaps make it the the responsibility of whomever uploads an ABI bump to update platform.intrepid in the future?
[22:03] <kirkland> cjwatson: solved the usplash problem, with a pointer from kees...  there was a missing chvt(1) call in initramfs-tools when calling the failure hooks
[22:04] <kirkland> cjwatson: one-line patch attached to https://bugs.launchpad.net/ubuntu/+source/initramfs-tools/+bug/268873 ...  kees may sponsor for me
[23:57] <xivulon> cjwatson, re 268123 partial progress:
[23:57] <xivulon> mount -t ntfs -> no such device
[23:57] <xivulon> mount.ntfs -> OK