/srv/irclogs.ubuntu.com/2012/03/23/#ubuntu-installer.txt

njinhello, bug 962972 critical10:53
ubot2`Launchpad bug 962972 in ubiquity "HPFS/NTFS not detected" [Undecided,New] https://launchpad.net/bugs/96297210:53
CIA-32ubiquity: cjwatson * r5325 trunk/ (debian/changelog ubiquity/misc.py):12:57
CIA-32ubiquity: ubiquity.misc.boot_device needs root privileges in order to talk to12:57
CIA-32ubiquity: parted_server.12:57
cjwatsonstgraber: 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/96258512:58
stgrabercjwatson: odd that ubuntustudio doesn't have busybox...13:06
stgrabercjwatson: I think we should either depend on it or check for it and hide the terminal if it's not there13:06
cjwatsonbusybox might come from friendly-recovery or something13:06
stgraberno point in showing the terminal if it's never going to update13:06
cjwatsonI'm OKish with a Depends13:07
stgraber"busybox-static | busybox" should do it13:07
cjwatsonthat is actually odd, busybox-static is in standard13:09
cjwatsonnot that ubiquity should be relying on that13:09
ogra_wasnt it also a dep of initramfs-tools ?13:09
cjwatsonyeah, I wonder if there's a path problem13:10
ogra_or was that busybox-initramfs13:10
cjwatsonwhy are you hardcoding the path to busybox, anyway?13:10
cjwatsonwe should never hardcode paths to executables that are on the search path ...13:11
stgrabercjwatson: probably because the path to tail was hardcoded (not a good excuse, I know ;))13:12
cjwatsonwould need to set the GLib.SpawnFlags.SEARCH_PATH flag13:16
cjwatsonstgraber: ah, I see the build bug that caused this13:36
CIA-32ubiquity: cjwatson * r5326 trunk/debian/ (changelog control):13:36
CIA-32ubiquity: Make ubiquity-frontend-gtk depend on busybox-static | busybox for now,13:36
CIA-32ubiquity: matching the 'busybox tail' workaround in ubiquity 2.9.32 (LP: #962585).13:36
cjwatson(added a livecd-rootfs task)13:37
cjwatsonbdmurray: 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 out14:18
davmor2cjwatson: 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:32
cjwatsonnothing's changed there14:35
davmor2cjwatson: 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
cjwatsonsorry, can't help :(14:43
cjwatsonsome bioses seem to not pass it though14:43
cjwatson*through14:43
barryjodh: would it still be helpful to test the upstart ppa?14:44
cjwatsonyou 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 menu14:44
jodhbarry: certainly would!14:44
jodhbarry: thanks14:44
barryjodh: i fortunately have a vm i can take disk snapshots of :)  let me give it a go and i'll let you know14:45
jodhbarry: nice ;) ta.14:45
davmor2cjwatson: 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 mod14:47
cjwatsondavmor2: kvm doesn't pass through enough modifier key state14:47
cjwatsonit passes through press/release, but not whether it's held down or not, which is what grub needs14:47
davmor2cjwatson: ah that could explain it too14:48
cjwatsongiven that I've been told it has to have zero delay14:48
bdmurraycjwatson: I'd like to finish it.  The test was failing with the patch applied though…14:56
cjwatsonMaybe I should do this ubiquity upload first then14:57
CIA-32ubiquity: cjwatson * r5327 trunk/ (d-i/manifest debian/changelog): Automatic update of included source packages: flash-kernel 2.28ubuntu42.15:07
barryjodh: other than shutting down, restarting, is there anything in particular you want me to try?15:10
jodhyou could try restarting a few services and checking to make sure the number of fds in /proc/1/fd remains static.15:10
barrycool15:11
jodhbarry: restart 1027 times say and check /proc/1/fd and check that cpu and memory don't grow.15:11
CIA-32ubiquity: cjwatson * r5328 trunk/debian/changelog: releasing version 2.10.115:11
barryjodh: does it have to be exactly 1027 times? :)15:11
jodhbarry: sorry, just meant a large number - as high as you like ;)15:33
CIA-32localechooser: cjwatson * r171 ubuntu/ (debian/changelog post-base-installer.d/05localechooser):15:36
CIA-32localechooser: Invert the set of locale categories set in case of a language/location15:36
CIA-32localechooser: conflict, so we now set LC_NUMERIC, LC_TIME, LC_MONETARY, LC_PAPER,15:36
CIA-32localechooser: LC_NAME, LC_ADDRESS, LC_TELEPHONE, LC_MEASUREMENT, and LC_IDENTIFICATION15:36
CIA-32localechooser: instead (LP: #926207).15:36
barryjodh: cool, thought so :)15:37
barryjodh: so i'm going to restart apparmor, ssh, and ntp in a loop and let it run while i get some lunch15:37
jodhbarry: gr8, thx.15:37
cjwatsongraaa, build failure15:54
* cjwatson goes to sort that15:54
cjwatsonoh well, I had another upload to do anyway :-)15:54
CIA-32ubiquity: cjwatson * r5329 trunk/ (debian/changelog tests/test_misc.py): Fix test_misc.GrubDefaultTests.test_avoid_cdrom.16:09
CIA-32localechooser: cjwatson * r172 ubuntu/post-base-installer.d/05localechooser: set LC_* to correct value16:10
CIA-32localechooser: cjwatson * r173 ubuntu/debian/changelog: releasing version 2.39ubuntu216:33
CIA-32ubiquity: cjwatson * r5330 trunk/ (debian/changelog scripts/localechooser-apply):16:35
CIA-32ubiquity: Invert the set of locale categories set in case of a language/location16:35
CIA-32ubiquity: conflict, so we now set LC_NUMERIC, LC_TIME, LC_MONETARY, LC_PAPER,16:35
CIA-32ubiquity: LC_NAME, LC_ADDRESS, LC_TELEPHONE, LC_MEASUREMENT, and LC_IDENTIFICATION16:35
CIA-32ubiquity: instead (LP: #926207).16:35
barryjodh: 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 solid16:40
bdmurrayregarding 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/94419116:41
jodhbarry: thanks for the feedback!16:42
barryjodh: np.  let me know if there's anything else you need, otherwise when this finishes if all looks good, i'll revert the disk snapshot16:43
cjwatsonbdmurray: yes, should be the same16:43
cjwatsonthat particular race should trigger exclusively on /target/var16:44
bdmurraywhat about /target/var/lock?16:44
cjwatsonnope, that's a different bug, which I think I've fixed separately.  bug# - is it recent?16:44
cjwatsonmore likely to be bug 891711, at least if it's from before ubiquity 2.9.1916: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/89171116:45
cjwatson(that's only Confirmed in oneiric, silly bot)16:45
bdmurraybug 92007816:46
ubot2`Launchpad bug 920078 in ubiquity "ubuntu 11.04 installation error" [Undecided,New] https://launchpad.net/bugs/92007816:46
cjwatsondup of 89171116:46
cjwatsonthe /target/var/lock case is particularly prevalent for attempted "upgrades" from 11.10 to 11.0416:47
cjwatsonmarked as such now16:51
CIA-32ubiquity: cjwatson * r5331 trunk/ (d-i/manifest debian/changelog): Automatic update of included source packages: localechooser 2.39ubuntu2.16:56
bdmurrayhttp://paste.ubuntu.com/896666/ there is my test again16:56
bdmurrayand it fails with the patch I have for debug16:56
bdmurrayhowever when I tested the patch by installing without an issue16:57
cjwatsonok, can you pastebin the entire diff you have and I'll try applying it?16:57
bdmurrayhttp://paste.ubuntu.com/896671/16:58
cjwatsonok, building, few minutes16:59
cjwatsonbdmurray: ah, I think I see, the write implementation is wrong.  let me try and come up with something better17:20
cjwatsongrr, distressingly painful to create an object that works in just the right way17:36
bdmurraymight it have something to do with the language being french when the system is started?17:37
cjwatsonno, I've got it now17:37
cjwatsontry http://paste.ubuntu.com/896731/ ?17:38
cjwatsonsome minor rearrangement but the main difference was adding the isinstance check to emulate Python's built-in file object behaviour more closely17:38
cjwatson(the rearrangement was just to avoid having to add that same check in two places)17:39
cjwatsonhttp://paste.ubuntu.com/896733/ if you want just the test17:39
bdmurrayright, I'd experimented with isinstance w/o success17:39
cjwatsonfor 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 fails17:42
bdmurrayits bug 96027817: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/96027817:44
cjwatsonI get the same exception, then17:45
cjwatsonso 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
bdmurrayI'll push a branch17:46
cjwatsonok, cool17:47
bdmurraythanks for the help17:50
cjwatsonnp17:52
cjwatsontell me the branch URL and I'll include it in ubiquity 2.10.217:52
bdmurrayhttps://code.launchpad.net/~brian-murray/ubiquity/bug-96027817:54
cjwatsonpulled, thanks17:59
CIA-32ubiquity: 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)17:59
CIA-32ubiquity: cjwatson * r5333 trunk/debian/changelog: releasing version 2.10.218:05

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