/srv/irclogs.ubuntu.com/2011/08/31/#ubuntu-installer.txt

CIA-31ubiquity: cjwatson * r4914 trunk/bin/ (oem-config-remove-gtk ubiquity ubiquity-dm): remove trailing whitespace09:17
CIA-31ubiquity: cjwatson * r4915 trunk/ (bin/ubiquity-dm debian/changelog): /apps/metacity/general/num_workspaces should be an int, not a boolean.09:22
CIA-31ubiquity: evand * r4916 trunk/ (debian/changelog ubiquity/plugins/ubi-language.py):11:15
CIA-31ubiquity: Don't try to connect signals on the release notes label in the oem-11:15
CIA-31ubiquity: config version of the language page, as it doesn't exist there.11:15
evbeen trying to fix the iconview wrapping issue today14:03
evwhile building a test harness for ubi-language at the same time14:04
evbut the former is proving quite annoying14:04
cjwatsonI'll be attacking bug 837681 once I get a current image sucked down14:04
ubot2`Launchpad bug 837681 in ubiquity "Automatic partitioning corrupts GUID partition table (GPT)" [Critical,New] https://launchpad.net/bugs/83768114:04
cjwatsonI fully expect it to do my head in14:06
stgraberev: can you look at http://paste.ubuntu.com/678921/ as a fix for bug 770320? I actually don't see any country in the current list having that problem. (Ethiopia is gone in Oneiric apparently)14:13
ubot2`Launchpad bug 770320 in ubiquity "Sorting non-English list: French installer sorts accented names at end of alphabet" [Low,Triaged] https://launchpad.net/bugs/77032014:13
stgraberI noticed that you used another locale aware sorting method for the initial language step, though I couldn't find a way to easily make it work for the keyboard14:14
stgraberalso, I'm not sure if using os.getenv('LANG','en_US.UTF-8') will get me the current locale (as in the one selected in ubi-language.py) or if I should use some other way of retrieving it14:14
evI'll have a look after this iconview stuff14:16
cjwatsondebian-installer/locale14:16
cjwatsonor debian-installer/language if you only want the language code14:16
stgraberJust noticed that this step already has a get_locale() function that seems to do the right thing. http://paste.ubuntu.com/678928/14:27
evfinally: http://paste.ubuntu.com/678954/ now to just figure out how to test for the text wrapping15:05
CIA-31ubiquity: evand * r4917 trunk/ (debian/changelog ubiquity/plugins/ubi-migrationassistant.py):15:11
CIA-31ubiquity: Construct a GtkTreeCellDataFunc with the correct number of arguments15:11
CIA-31ubiquity: in the migration assistant plugin (LP: #829987).15:11
jibelev, on the timezone screen, does ubiquity tries to download something from the net ?15:12
evyes15:13
evlots, actually15:13
evit hits geoip.ubuntu.com to find your current location if it can15:13
evit also his geoname-lookup.ubuntu.com every time you type in the box (after sufficient time has past)15:13
jibelok, that would explain why it hangs there sometimes. Could the call be non-blocking ?15:14
jibelwhen the connection is firewalled and these addresses are not authorized, ubiquity completely freezes15:15
evjibel: please file a bug for that15:44
evif you could provide a test case as well, that would be ace15:44
skaetcjwatson,  been able to poke at 837681 yet?   or is image still downloading?15:44
cjwatsonI'm working on constructing a test case15:45
cjwatsonsince it involves a double installation and faking the architecture, and since compiz hates me, and since I've had meetings, it's taking a little while15:45
jibelev, bug 83721715:46
ubot2`Launchpad bug 837217 in ubiquity "Time Zone selector hangs and ubiquity freezes if network connection is firewalled" [Medium,New] https://launchpad.net/bugs/83721715:46
skaetcjwatson,  gotcha.   Just trying to assess scope of impact and whether we can get away with release noting it, or its going to be another late night. :/15:47
cjwatsonI don't know15:49
evthanks jibel15:49
jibelon a netboot install what the task 'Ubuntu desktop USB' is supposed to install ?16:03
jibelit installs something, but for sure it is not a desktop.16:03
cjwatsonit should be equivalent to what's on the USB images (well, currently called DVD)16:03
cjwatsonbug on tasksel please?  I suspect it's only installing the extra packages and not ubuntu-desktop as well16:04
bdmurrayev: wrt bug 837287 should there be a panel or not?16:10
ubot2`Launchpad bug 837287 in ubiquity "live cd panel / top menu is cropped" [High,Fix released] https://launchpad.net/bugs/83728716:10
evbdmurray: there should be a panel in ubiquity 2.7.2416:13
superm1ev, seems that there is still a problem in 2.7.24 :(16:14
evrubbish16:14
superm1it's loading those indicators now finally but it doesn't feel like it's important enough to draw em'16:14
bdmurrayso 837287 shouldn't really be fixed then16:15
superm1no not yet16:16
CIA-31ubiquity: evand * r4918 trunk/ (debian/changelog ubiquity/plugins/ubi-language.py): Don't let the text wrap for the language names in oem-config.16:17
evI have a test case for that in progress16:17
bdmurrayI noticed the "OEM Configuration (temporary user)" seemed longer than lightdm(?) could handle when doing an oem install the other day16:39
cjwatsonlightdm should be fixed then, I'm pretty sure there are real people with names that long16:41
cjwatsonI saw several such just reviewing UDS sponsorship applications today16:42
cjwatson(particularly those from Spanish-speaking countries)16:42
cjwatsonev: if you have ubiquity stuff to upload for b1, don't wait for 837681, since I think it's probably not in ubiquity17:28
skaetev, would be good to get the migration-assistant bug fix included in any respins.17:31
* cjwatson -> dinner17:51
cjwatsonev: please let me know when you're around and if you intend to do more ubiquity work tonight; otherwise I intend to upload ubiquity from trunk after dinner17:52
stgrabercjwatson: I'll push my fix for bug 770320 then17:55
ubot2`Launchpad bug 770320 in ubiquity "Sorting non-English list: French installer sorts accented names at end of alphabet" [Low,Triaged] https://launchpad.net/bugs/77032017:55
stgraber(very far from critical, just wanted something easy to do this morning ;))17:55
stgrabercjwatson, ev: Actually, I have a question for one of you two before I push the fix for bug 770320. I'm a bit lost in ubi-language, trying to figure out if it's supposed to call locale-gen or not.18:37
ubot2`Launchpad bug 770320 in ubiquity "Sorting non-English list: French installer sorts accented names at end of alphabet" [Low,Triaged] https://launchpad.net/bugs/77032018:37
stgraberMy current fix uses locale.setlocale(locale.LC_ALL, '') which works fine as long as the locale exists, which isn't always the case18:38
stgrabercasper does a locale-gen for the locale selected in gfxboot so that covers a lot of cases18:38
stgraberbut when you then select another locale in ubiquity, I'd expect it to also call locale-gen, which localechooser is apparently supposed to do but in my case doesn't seem to happen18:38
stgraber(and so ubiquity explodes as I'm trying to do a setlocale() on a non-existing locale)18:39
stgraberso I'm wondering if it's a bug that locale-gen isn't called when exitting ubi-language or if that code is only supposed to be called much later (and so will make other plugins depending on the locale fail)18:40
cjwatsonstgraber: no, it's not supposed to be called, it takes too long and uses too much memory; the installer in general doesn't need a generated locale19:22
cjwatsone.g.19:22
cjwatson  * Set LANGUAGE as well as LANG when changing locale; this allows us to get19:22
cjwatson    correct debconf translations without needing to run locale-gen (closes:19:22
cjwatson    Malone #43071).19:22
ubot2`Launchpad bug 43071 in ubiquity "Ubiquity in Kubuntu gets killed by out-of-memory with 256 MB RAM" [High,Fix released] https://launchpad.net/bugs/4307119:22
cjwatsonI'm afraid you can't use collation ordering from the locale19:23
cjwatsonor at least not reliably; I suppose it's OK to use it if the locale is available anyway19:23
stgrabercjwatson: http://paste.ubuntu.com/679160/19:25
stgraberthat's a lot of fallback to implement to get the best possible locale...19:26
stgraber(even if we could have run locale-gen in ubi-language, the _locale module seems to keep a cache of what's valid, so we'd still need to restart ubiquity ...)19:26
cjwatsonyeah, that's a non-starter19:27
cjwatson(restarting ubiquity)19:27
cjwatsonsorting isn't a critical function, I think it's OK to have as much or as little in the way of fallback as you think worthwhile19:28
cjwatsonwhy choices.sort in one branch and choices=sorted() in the other?19:28
stgraberI'm just going to run ubi-console-setup one more time with a weird language just to make sure it works, but I think the current code should work fine (I'll probably move it to i18n later so we can use it at some other places)19:28
cjwatson(also, PEP-8 would be "choices = sorted(choices)")19:29
stgraberyeah, I just noticed that one, changed it to choices.sort() everywhere now19:29
cjwatsonhm, another thing, the effect of this is to add an .encode('utf-8') to each choice - is that absolutely guaranteed safe?19:31
cjwatsonstrikes me as potentially worrisome19:31
cjwatsonI wonder if it would be safer to encode in the sort key function19:31
cjwatsonso choices.sort(key=lambda c: locale.strxfrm(c.encode('utf-8'))) # or similar19:32
stgrabercjwatson: ok, tested with yet another language, seems to work fine. Pushed to lp:ubiquity19:40
stgrabercjwatson: I'm now starting to look at beta2 bugs for ubiquity, so I'm not expecting anything else to include before the next upload.19:45
cjwatsonok19:45
cjwatsonI'll go ahead and upload this lot the19:45
cjwatsonn19:45
cjwatsonstill working on the suspected parted bug :-/19:46
CIA-31ubiquity: cjwatson * r4920 trunk/ (d-i/manifest debian/changelog): Automatic update of included source packages: flash-kernel 2.28ubuntu29.19:52
CIA-31ubiquity: cjwatson * r4921 trunk/debian/changelog: releasing version 2.7.2519:58
superm1cjwatson, re all GPT installs being busted or not, i've managed to do an EFI install that re-used an existing ESP at the start of the disk with the 08/31 daily20:01
cjwatsonyeah, I'd got far enough to conclude it wasn't all of them20:04
cjwatsonbig gap between all and one though :)20:04
cjwatsonit stinks of a bug in parted's built-in gptsync, and I'll need to do most of the work of tracking it down before I can make a decent guess at the impact20:05
cjwatsonmy first test was non-EFI with the bios_grub partition at the start of the disk, and that seemed to partition fine20:05
=== kentb is now known as kentb-afk
stgraberthe Kubuntu manual partitioning issue is: http://paste.ubuntu.com/67925922:39
stgraberwill poke at it in a bit22:39

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