[10:53] <njin> hello, bug 962972 critical
[10:53] <ubot2`> Launchpad bug 962972 in ubiquity "HPFS/NTFS not detected" [Undecided,New] https://launchpad.net/bugs/962972
[12:57] <CIA-32> ubiquity: cjwatson * r5325 trunk/ (debian/changelog ubiquity/misc.py):
[12:57] <CIA-32> ubiquity: ubiquity.misc.boot_device needs root privileges in order to talk to
[12:57] <CIA-32> ubiquity: parted_server.
[12:58] <cjwatson> stgraber: bug 962585 is a regression from your 'busybox tail' change.  Do you think it's better to use 'busybox tail' only if busybox is available (osextras.find_on_path) or to depend on busybox?
[12:58] <ubot2`> Launchpad bug 962585 in ubiquity "ubiquity crashed with GError in function(): Failed to execute child process "/bin/busybox" (No such file or directory)" [Undecided,New] https://launchpad.net/bugs/962585
[13:06] <stgraber> cjwatson: odd that ubuntustudio doesn't have busybox...
[13:06] <stgraber> cjwatson: I think we should either depend on it or check for it and hide the terminal if it's not there
[13:06] <cjwatson> busybox might come from friendly-recovery or something
[13:06] <stgraber> no point in showing the terminal if it's never going to update
[13:07] <cjwatson> I'm OKish with a Depends
[13:07] <stgraber> "busybox-static | busybox" should do it
[13:09] <cjwatson> that is actually odd, busybox-static is in standard
[13:09] <cjwatson> not that ubiquity should be relying on that
[13:09] <ogra_> wasnt it also a dep of initramfs-tools ?
[13:10] <cjwatson> yeah, I wonder if there's a path problem
[13:10] <ogra_> or was that busybox-initramfs
[13:10] <cjwatson> why are you hardcoding the path to busybox, anyway?
[13:11] <cjwatson> we should never hardcode paths to executables that are on the search path ...
[13:12] <stgraber> cjwatson: probably because the path to tail was hardcoded (not a good excuse, I know ;))
[13:16] <cjwatson> would need to set the GLib.SpawnFlags.SEARCH_PATH flag
[13:36] <cjwatson> stgraber: ah, I see the build bug that caused this
[13:36] <CIA-32> ubiquity: cjwatson * r5326 trunk/debian/ (changelog control):
[13:36] <CIA-32> ubiquity: Make ubiquity-frontend-gtk depend on busybox-static | busybox for now,
[13:36] <CIA-32> ubiquity: matching the 'busybox tail' workaround in ubiquity 2.9.32 (LP: #962585).
[13:37] <cjwatson> (added a livecd-rootfs task)
[14:18] <cjwatson> bdmurray: any more on that --debug unicode fix?  if you don't have time to finish polishing the test, I'd be happy to sort that out
[14:32] <davmor2> cjwatson: is it still shift to get the the grub menu?  I need to boot into oneiric on my one test box and everything I do to get  to is seems to fail(box has oneiric and precise onit)
[14:35] <cjwatson> nothing's changed there
[14:43] <davmor2> cjwatson: okay I've tried pressing shift during the bios screen, just after the bios screen,  repeatedly hit shift throughout and nothing nadda still boots into precise :(
[14:43] <cjwatson> sorry, can't help :(
[14:43] <cjwatson> some bioses seem to not pass it though
[14:43] <cjwatson> *through
[14:44] <barry> jodh: would it still be helpful to test the upstart ppa?
[14:44] <cjwatson> you can always comment out GRUB_HIDDEN_TIMEOUT=0 in /etc/default/grub and change GRUB_HIDDEN_TIMEOUT_QUIET to false, then run sudo update-grub; then you'll always see the menu
[14:44] <jodh> barry: certainly would!
[14:44] <jodh> barry: thanks
[14:45] <barry> jodh: i fortunately have a vm i can take disk snapshots of :)  let me give it a go and i'll let you know
[14:45] <jodh> barry: nice ;) ta.
[14:47] <davmor2> cjwatson: ta that will do,  I just thought this is running through a USB kvm I wonder if it is that the keyboard is temporarily disabled post boot up till the kernel is in play again,  I'll go for the menu mod
[14:47] <cjwatson> davmor2: kvm doesn't pass through enough modifier key state
[14:47] <cjwatson> it passes through press/release, but not whether it's held down or not, which is what grub needs
[14:48] <davmor2> cjwatson: ah that could explain it too
[14:48] <cjwatson> given that I've been told it has to have zero delay
[14:56] <bdmurray> cjwatson: I'd like to finish it.  The test was failing with the patch applied though…
[14:57] <cjwatson> Maybe I should do this ubiquity upload first then
[15:07] <CIA-32> ubiquity: cjwatson * r5327 trunk/ (d-i/manifest debian/changelog): Automatic update of included source packages: flash-kernel 2.28ubuntu42.
[15:10] <barry> jodh: other than shutting down, restarting, is there anything in particular you want me to try?
[15:10] <jodh> you could try restarting a few services and checking to make sure the number of fds in /proc/1/fd remains static.
[15:11] <barry> cool
[15:11] <jodh> barry: restart 1027 times say and check /proc/1/fd and check that cpu and memory don't grow.
[15:11] <CIA-32> ubiquity: cjwatson * r5328 trunk/debian/changelog: releasing version 2.10.1
[15:11] <barry> jodh: does it have to be exactly 1027 times? :)
[15:33] <jodh> barry: sorry, just meant a large number - as high as you like ;)
[15:36] <CIA-32> localechooser: cjwatson * r171 ubuntu/ (debian/changelog post-base-installer.d/05localechooser):
[15:36] <CIA-32> localechooser: Invert the set of locale categories set in case of a language/location
[15:36] <CIA-32> localechooser: conflict, so we now set LC_NUMERIC, LC_TIME, LC_MONETARY, LC_PAPER,
[15:36] <CIA-32> localechooser: LC_NAME, LC_ADDRESS, LC_TELEPHONE, LC_MEASUREMENT, and LC_IDENTIFICATION
[15:36] <CIA-32> localechooser: instead (LP: #926207).
[15:37] <barry> jodh: cool, thought so :)
[15:37] <barry> jodh: so i'm going to restart apparmor, ssh, and ntp in a loop and let it run while i get some lunch
[15:37] <jodh> barry: gr8, thx.
[15:54] <cjwatson> graaa, build failure
[15:54]  * cjwatson goes to sort that
[15:54] <cjwatson> oh well, I had another upload to do anyway :-)
[16:09] <CIA-32> ubiquity: cjwatson * r5329 trunk/ (debian/changelog tests/test_misc.py): Fix test_misc.GrubDefaultTests.test_avoid_cdrom.
[16:10] <CIA-32> localechooser: cjwatson * r172 ubuntu/post-base-installer.d/05localechooser: set LC_* to correct value
[16:33] <CIA-32> localechooser: cjwatson * r173 ubuntu/debian/changelog: releasing version 2.39ubuntu2
[16:35] <CIA-32> ubiquity: cjwatson * r5330 trunk/ (debian/changelog scripts/localechooser-apply):
[16:35] <CIA-32> ubiquity: Invert the set of locale categories set in case of a language/location
[16:35] <CIA-32> ubiquity: conflict, so we now set LC_NUMERIC, LC_TIME, LC_MONETARY, LC_PAPER,
[16:35] <CIA-32> ubiquity: LC_NAME, LC_ADDRESS, LC_TELEPHONE, LC_MEASUREMENT, and LC_IDENTIFICATION
[16:35] <CIA-32> ubiquity: instead (LP: #926207).
[16:40] <barry> jodh: i'm going to let this continue to run but so far, everything looks good.  both fd count and mem usage for pid 1 look rock solid
[16:41] <bdmurray> regarding bug 944191 would duplicates have similar tracebacks?
[16:41] <ubot2`> Launchpad bug 944191 in ubiquity "Installation program crashed while typing username" [Undecided,Fix released] https://launchpad.net/bugs/944191
[16:42] <jodh> barry: thanks for the feedback!
[16:43] <barry> jodh: np.  let me know if there's anything else you need, otherwise when this finishes if all looks good, i'll revert the disk snapshot
[16:43] <cjwatson> bdmurray: yes, should be the same
[16:44] <cjwatson> that particular race should trigger exclusively on /target/var
[16:44] <bdmurray> what about /target/var/lock?
[16:44] <cjwatson> nope, that's a different bug, which I think I've fixed separately.  bug# - is it recent?
[16:45] <cjwatson> more likely to be bug 891711, at least if it's from before ubiquity 2.9.19
[16:45] <ubot2`> Launchpad bug 891711 in ubiquity "Fails to copy directory over symlink (e.g. /var/lock when downgrading from 11.10 to 11.04)" [High,Confirmed] https://launchpad.net/bugs/891711
[16:45] <cjwatson> (that's only Confirmed in oneiric, silly bot)
[16:46] <bdmurray> bug 920078
[16:46] <ubot2`> Launchpad bug 920078 in ubiquity "ubuntu 11.04 installation error" [Undecided,New] https://launchpad.net/bugs/920078
[16:46] <cjwatson> dup of 891711
[16:47] <cjwatson> the /target/var/lock case is particularly prevalent for attempted "upgrades" from 11.10 to 11.04
[16:51] <cjwatson> marked as such now
[16:56] <CIA-32> ubiquity: cjwatson * r5331 trunk/ (d-i/manifest debian/changelog): Automatic update of included source packages: localechooser 2.39ubuntu2.
[16:56] <bdmurray> http://paste.ubuntu.com/896666/ there is my test again
[16:56] <bdmurray> and it fails with the patch I have for debug
[16:57] <bdmurray> however when I tested the patch by installing without an issue
[16:57] <cjwatson> ok, can you pastebin the entire diff you have and I'll try applying it?
[16:58] <bdmurray> http://paste.ubuntu.com/896671/
[16:59] <cjwatson> ok, building, few minutes
[17:20] <cjwatson> bdmurray: ah, I think I see, the write implementation is wrong.  let me try and come up with something better
[17:36] <cjwatson> grr, distressingly painful to create an object that works in just the right way
[17:37] <bdmurray> might it have something to do with the language being french when the system is started?
[17:37] <cjwatson> no, I've got it now
[17:38] <cjwatson> try http://paste.ubuntu.com/896731/ ?
[17:38] <cjwatson> some minor rearrangement but the main difference was adding the isinstance check to emulate Python's built-in file object behaviour more closely
[17:39] <cjwatson> (the rearrangement was just to avoid having to add that same check in two places)
[17:39] <cjwatson> http://paste.ubuntu.com/896733/ if you want just the test
[17:39] <bdmurray> right, I'd experimented with isinstance w/o success
[17:42] <cjwatson> for me, that passes with your ubiquity/filteredcommand.py change (with a traceback that IIRC matches the bug - I've forgotten which bug you were working on for this), and if I shelve that bit then it fails
[17:44] <bdmurray> its bug 960278
[17:44] <ubot2`> Launchpad bug 960278 in ubiquity "ubiquity crashed with UnicodeEncodeError in debug(): 'ascii' codec can't encode character u'\xe9' in position 55: ordinal not in range(128)" [Low,Triaged] https://launchpad.net/bugs/960278
[17:45] <cjwatson> I get the same exception, then
[17:46] <cjwatson> so I think that's good - do you want to do the branch dance or shall I just apply it in your name here?
[17:46] <cjwatson> (a changelog entry would be good at least, though)
[17:46] <bdmurray> I'll push a branch
[17:47] <cjwatson> ok, cool
[17:50] <bdmurray> thanks for the help
[17:52] <cjwatson> np
[17:52] <cjwatson> tell me the branch URL and I'll include it in ubiquity 2.10.2
[17:54] <bdmurray> https://code.launchpad.net/~brian-murray/ubiquity/bug-960278
[17:59] <cjwatson> pulled, thanks
[17:59] <CIA-32> ubiquity: cjwatson * r5332 trunk/ (3 files in 3 dirs): UTF-8-encode debug messages which are written to stderr (LP: #960278)
[17:59] <cjwatson> (hm, cia-submit doesn't honour author)
[18:05] <CIA-32> ubiquity: cjwatson * r5333 trunk/debian/changelog: releasing version 2.10.2