/srv/irclogs.ubuntu.com/2007/07/02/#ubuntu-installer.txt

=== ubuntulog [i=ubuntulo@trider-g7.fabbione.net] has joined #ubuntu-installer
=== Topic for #ubuntu-installer: Development of d-i and ubiquity in Ubuntu | http://wiki.ubuntu.com/InstallerDevelopment
=== Topic (#ubuntu-installer): set by cjwatson at Fri Jan 5 15:12:40 2007
=== #ubuntu-installer [freenode-info] please register your nickname...don't forget to auto-identify! http://freenode.net/faq.shtml#nicksetup
=== cjwatson_ [n=cjwatson@82-69-40-219.dsl.in-addr.zen.co.uk] has joined #ubuntu-installer
=== cjwatson1 [n=cjwatson@82-69-40-219.dsl.in-addr.zen.co.uk] has joined #ubuntu-installer
=== Starting logfile irclogs/ubuntu-installer.log
=== ubuntulog [i=ubuntulo@ubuntu/bot/ubuntulog] has joined #ubuntu-installer
=== Topic for #ubuntu-installer: Development of d-i and ubiquity in Ubuntu | http://wiki.ubuntu.com/InstallerDevelopment
=== Topic (#ubuntu-installer): set by cjwatson at Fri Jan 5 15:12:40 2007
=== blackskad [n=blackska@d54C4A53D.access.telenet.be] has joined #ubuntu-installer
CIA-26oem-config: cjwatson * r300 oem-config/ (d-i/manifest debian/changelog): * Automatic update of included source packages: console-setup 1.16ubuntu4.09:43
CIA-26oem-config: cjwatson * r301 oem-config/debian/changelog: releasing version 1.1610:03
cjwatsonevand: hw-detect 1.53ubuntu1 uploaded11:00
cjwatsonevand: could you push your bzr branches for base-installer and main-menu?11:00
CIA-26ubiquity: cjwatson * r2109 ubiquity/ (87 files in 5 dirs): (log message trimmed)01:24
CIA-26ubiquity: * If oem-config/enable is true, then:01:24
CIA-26ubiquity:  - Adjust title bar to indicate that Ubiquity is running in OEM mode.01:24
CIA-26ubiquity:  - Hardcode the user's full name to "OEM Configuration (temporary user)",01:24
CIA-26ubiquity:  the username to "oem", and the UID to 29999.01:24
CIA-26ubiquity:  - Install the appropriate oem-config-FRONTEND package on the target01:24
CIA-26ubiquity:  system.01:24
CIA-26ubiquity: cjwatson * r2110 ubiquity/ubiquity/frontend/ (gtk_ui.py kde_ui.py noninteractive.py): use new BaseFrontend.debconf_communicator method01:29
CIA-26oem-config: cjwatson * r302 oem-config/ (11 files in 6 dirs):01:46
CIA-26oem-config: * Move .glade and .ui files to a new top-level gui directory in the source01:46
CIA-26oem-config:  package, and install them under /usr/share/oem-config/glade and01:46
CIA-26oem-config:  /usr/share/oem-config/qt respectively.01:46
CIA-26oem-config: cjwatson * r303 oem-config/ (configure configure.ac): bump to 1.1701:48
CIA-26oem-config: cjwatson * r304 oem-config/lib/ (8 files in 3 dirs): PEP-8 import spacing01:53
CIA-26oem-config: cjwatson * r305 oem-config/ (8 files in 3 dirs):01:56
CIA-26oem-config: * Rename gtk-ui to gtk_ui and kde-ui to kde_ui, to make it easier to01:56
CIA-26oem-config:  subclass them.01:56
CIA-26ubiquity: cjwatson * r2111 ubiquity/ubiquity/frontend/gtk_ui.py: whitespace02:28
CIA-26oem-config: cjwatson * r306 oem-config/ (8 files in 3 dirs):02:31
CIA-26oem-config: * Break up the glade file into multiple pieces to make updates to single02:31
CIA-26oem-config:  pages a lot easier (port of work done in ubiquity by Mario Limonciello).02:31
CIA-26oem-config: cjwatson * r307 oem-config/ (4 files in 3 dirs):02:38
CIA-26oem-config: * Drop into pdb.post_mortem on non-SyntaxError exceptions if the --pdb02:38
CIA-26oem-config:  option is given and oem-config is running from a terminal.02:38
CIA-26oem-config: cjwatson * r308 oem-config/ (debian/changelog gui/glade/step_language.glade):02:41
CIA-26oem-config: * Fix title of language page: "Choose language and location" no longer02:41
CIA-26oem-config:  makes sense now that the location is computed from the timezone.02:41
CIA-26oem-config: cjwatson * r309 oem-config/gui/glade/ (4 files): expand -i02:45
CIA-26oem-config: cjwatson * r310 oem-config/gui/glade/ (4 files): load/save in glade-3 3.3.1-0ubuntu102:56
CIA-26ubiquity: cjwatson * r2112 ubiquity/gui/glade/ (stepLanguage.glade stepReady.glade): load/save in glade-3 3.3.1-0ubuntu103:00
evandcjwatson: Thanks! http://people.ubuntu.com/~evand/bzr/03:01
CIA-26oem-config: cjwatson * r311 oem-config/ (5 files in 2 dirs): * Minor UI tweaks to sync up with ubiquity.03:05
evandcjwatson: Are you OK with me merging the mythbuntu stuff to the ~ubuntu-installer branch?03:53
cjwatsongo ahead03:54
evandthanks03:55
=== superm1 [n=superm1@ubuntu/member/superm1] has joined #ubuntu-installer
cjwatsonevand: base-installer uploaded; minor point, you forgot -v1.79ubuntu1 when building04:06
evandwhoops04:06
evandthanks04:06
evandsuperm1: pushing the merge shortly.  Just checking to make sure everything works ok in VMWare.04:09
superm1great evand04:09
superm1my worry with it was the stuff in the d-i/source folder if thats okay to have as is (since the rest is populated dynamically)04:09
cjwatsond-i/source isn't in bzrr04:10
cjwatsonbzr04:10
cjwatsonso it doesn't matter ...04:10
superm1cjwatson, well there was maybe 4 scripts or so in my branch for the mythbuntu specific stuff04:11
superm1that I made sure were added in the last commit of mine04:11
evandthere's nothing in d-i/source in your branch04:12
evandbut as cjwatson says, d-i/source isn't versioned04:12
cjwatsonubiquity-local scripts should live in scripts/04:12
evandand I just hit it in the build04:13
superm1evand, are you sure you pulled the latest revision of my branch?  There should be a few things in there04:13
evandpositive04:13
evandat any rate, as cjwatson says04:14
evandthey shouldn't be in there04:14
evandthat gets rm -rf'ed in a maintainer-clean anyway04:14
superm1okay then i'll have to move them over04:14
superm1thats good to know04:14
evandthanks04:14
cjwatsonsuperm1: codebrowse.launchpad.net agrees04:14
cjwatsonevand: I'm going to rebuild the main-menu upload, since you included .bzr in the tarball, which I'd prefer not to be there04:23
cjwatsonthe actual diff looks ok04:23
evandhrm, not sure how I missed that one04:24
evandok, thanks04:24
superm1evand, okay i pushed moving all the scripts to the scripts/ directory in the branch.  Its in rev 2111 of my branch04:26
cjwatson files removed:  d-i/source/04:27
cjwatsonplease don't do that04:27
cjwatsonthe directory is revision-controlled, just not its contents ...04:27
cjwatson(sorry I was unclear earlier)04:28
superm1ah, when i was doing bzr add it started putting things inside of it into my commit so i took it off04:28
superm1er k then i'll resolve that04:28
cjwatsonum, I wonder how to resolve it in a way that doesn't result in confusion on later merges04:29
superm1can i revert changes on a merge?04:29
cjwatsonyou could try 'bzr merge -r2111..2110 d-i/source'04:30
cjwatsonthe correct result should be that the directory is restored with the same id (as shown by 'bzr ls --show-ids')04:31
superm1so i then move the contents of source back into it? (They were moved source.moved)04:32
cjwatsonthe contents don't matter; you can restore them with 'debian/rules update; bzr revert debian/changelog' anyway04:32
superm1oh right.04:32
cjwatsonor actually 'debian/rules update-local'04:32
superm1okay 2112 should have the directory re-added with that id again04:36
CIA-26ubiquity: cjwatson * r2113 ubiquity/ (d-i/get-sources debian/changelog):04:37
CIA-26ubiquity: * Add a warning in d-i/source/README about modifying files in this04:37
CIA-26ubiquity:  directory.04:37
evandok04:37
CIA-26oem-config: cjwatson * r312 oem-config/ (d-i/get-sources debian/changelog):04:39
CIA-26oem-config: * Add a warning in d-i/source/README about modifying files in this04:39
CIA-26oem-config:  directory.04:39
cjwatsonevand: hmm, main-menu is confusing. looks like you maybe merged one revision more than the bzr log claims04:41
cjwatsonI'll upload it anyway, should be fine, just a bit weird04:42
cjwatsonI think also joeyh uploaded it without waiting for all the i18n machinery to grind04:42
evandhrm04:42
cjwatsonthere are bots that trawl d-i svn and update all the po and pot files from master versions04:43
evandneat04:43
cjwatsonuploaded04:44
evandthanks04:44
=== GaryvdM [n=chatzill@mtngprs5.mtn.co.za] has joined #ubuntu-installer
GaryvdMPlease will someone help me. I have installed PostgreSQL on my kubuntu edgy box, but when I try run pg_ctl, it says Command not found. Where would pg_ctl normaly be installed on the fs, or how I can find out?05:19
evandGaryvdM: This is not a support channel.  Please /join #ubuntu05:19
GaryvdMok - sorry05:19
evandNo worries :)05:19
evandsuperm1_: what was your reason for the changes to ubiquity-frontend-gtk.install and ubiquity.install.any?05:33
=== GaryvdM [n=chatzill@mtngprs5.mtn.co.za] has left #ubuntu-installer []
evandthey strike me as unnecessary, but perhaps I'm missing the obvious05:34
=== blackskad [n=blackska@d54C4A53D.access.telenet.be] has joined #ubuntu-installer
cjwatsonhmm, partman-loop might not be as hard to write as I thought; I had been misremembering the order in which mount.d scripts are rrun06:02
cjwatsonrun06:02
=== superm1 [i=malimonc@ubuntu/member/superm1] has joined #ubuntu-installer
superm1evand, they were because the * would automatically add some of my files06:25
superm1into ubiquity-frontend-gtk06:25
superm1which would be undesirable06:25
evandgotcha, thanks06:27
CIA-26ubiquity: evand * r2114 ubiquity/ (32 files in 7 dirs): * Merge in mythbuntu alpha 2 changes.09:27
evandI made some minor changes here and there.  I moved the mythbuntu scripts into a subdirectory.  I changed the wording on the copyright ever so slightly (so please look and let me know if you take issue with that)09:29
evandoh and as I myself learned, you want to revert d-i/manifest and the changes to changelog when you build unless you're building the release.09:29
evandI walked through the interface and everything looks like it's still working.09:32
=== cjwatson_ [n=cjwatson@82-69-40-219.dsl.in-addr.zen.co.uk] has joined #ubuntu-installer
CIA-26oem-config: cjwatson * r313 oem-config/ (debian/changelog finish-install.d/07oem-config-user): * Fix desktop file installation harder.10:07
superm1great evand thanks. i'll merge them back to my branch when I get home later and make sure everything is still working right10:14
superm1and looks good10:14
superm1is there anything that will need to be done to make sure translators can translate our UI elements?10:15
superm1other than have them in the templates file?10:15
evandsuperm1: debian/ubiquity.templates10:19
superm1evand, but since the templates for the mythbuntu ui are in a sep. template file, will that matter?10:20
superm1or will it be submitted just the same10:20
cjwatson_it needs to go in debian/po/POTFILES.in10:20
superm1ah okay10:20
superm1well i'm going to scrub to make sure i have all the right ones that need translation first10:20
cjwatson(and then run debconf-updatepo and commit)10:20
superm1before adding there then10:20
cjwatsonsuperm1: hmm, I don't like that summary template, it's massive overkill10:23
cjwatsonnot to mention crazy indenting10:23
superm1by overkill you mean too much is presented?10:23
cjwatsonI think translators will have a lot of trouble with that, and it should be cut down10:23
cjwatsonyes10:23
superm1alright I can pull out a lot of the details then10:23
cjwatsonpeople will only read a few lines of summary - and it will actually push partitioning information off the screen, which is a lot more important than your mythtv theme!10:24
cjwatsonit also displays passwords which is a big no-no10:24
superm1ah i see10:24
superm1i'll clean that up then tonight10:25
cjwatsonthere are some places where you have _Description: lines wrapped10:25
cjwatsontechnically the format is actually:10:25
cjwatson_Description: DESCRIPTION10:25
cjwatson EXTENDED_DESCRIPTION10:25
cjwatsonso wrapping from the first line pushes it into a different field - looking at the cases in question I would suggest just putting them on a very long single line10:25
superm1how are you supposed to present a multiline description then?10:25
superm1with \n's in the field?10:25
cjwatsondebconf will wrap it for you10:26
cjwatsonhardcoded newlines are ignored unless you really try hard anyway ...10:26
cjwatsonactually, it might even be better to do this:10:26
cjwatson_Description:10:26
superm1but if its going to override somewhere in the glade that is already multiline10:26
cjwatson some text that is10:26
cjwatson wrapped10:26
cjwatsonsuperm1: none of the text I'm talking about benefits from carefully positioned \n characters10:27
superm1i see10:27
cjwatsonwant me to just commit what I mean?10:28
cjwatsonI don't seem to be explaining it very well10:28
superm1sure that might be best :)10:28
cjwatsonyou realise that putting a newline in a <property name="label"> doesn't actually result in a newline in the final display, right?10:29
superm1I had thought it did..... there were several cases that when editing within glade-3 that i'd just hit enter and start another line in the label text field10:30
cjwatsonor perhaps another explanation10:30
cjwatson_Description: An RSS feed news reader plugin for MythTV that keeps you up-to-date on your10:30
cjwatson favorite news topics.10:30
cjwatsonthis results in the following in the debconf database:10:30
cjwatsonDescription: An RSS feed news reader plugin for MythTV that keeps you up-to-date on your10:30
cjwatsonExtended_description: favorite news topics.10:30
superm1ah and that will completely break it10:30
superm1i see10:30
cjwatsonand results in "An RSS feed news reader plugin for MythTV that keeps you up-to-date on your" and "favorite news topics." being separate translatable chunks10:30
cjwatsonsuperm1: oh, well if it does then it isn't what you want :)10:31
cjwatsonyou don't want to hardcode newlines in the glade file - you have no idea how wide the screen is10:31
superm1then what is the most appropriate way to do it in glade?10:31
superm1just keep things short ;)?10:32
cjwatsondon't press Enter10:32
cjwatsonset the line-wrap property and gtk will wrap stuff for you10:32
superm1how does it know how wide i'm going to allow then?10:32
cjwatsonyou don't do the allowing, gtk does :)10:33
superm1I think i'll need to toy with it a bit then, because when I was doing really long lines the first time around, the GUI window would expand to get wider10:34
cjwatsonyou probably forgot to turn line-wrap on for that widget10:34
superm1probably then10:34
cjwatsonor else you had it in single-line mode (there are two related properties there)10:34
CIA-26ubiquity: cjwatson * r2115 ubiquity/debian/ubiquity-frontend-mythbuntu.templates: fix incorrect description wrapping10:34
cjwatsongtk doesn't line-wrap optimally (it tends to use much less than the full width) but I expect that will be fixed eventually and it's better for us to live with that than employ dubious workarounds10:35
superm1is the full width then determined by the 'alignment' widgets?10:35
superm1i saw those in use in a lot of your pages, but wasn't sure when was appropriate to be using them10:36
cjwatsonnot really; we use GtkAlignments to control things like indentation and spacing10:39
superm1oh ok10:39
cjwatsonyou can wrap an alignment around any non-top-level widget to say "give this a left margin of 12 pixels"10:39
cjwatsonwhich is good for fine details of page layout10:39
cjwatsonhmm, I notice mythbuntu-setup does xhost +$HOSTNAME but then goes on to use xauth. Do you really need both? Just xauth would be bette10:39
cjwatsonr10:39
superm1because the hostname is updated10:40
superm1to the new one chosen during install10:40
superm1originally it was just done via xauth10:40
superm1but I realized that mysql was getting messed up because the wrong hostname was being used10:41
superm1(not the one that is post install)10:41
cjwatsonthat would break xhost, not xauth, wouldn't it?10:41
cjwatsonI'm just puzzled by why you turn on two different X authentication mechanisms10:42
superm1well xhost was the one that i added later10:42
superm1it was not necessary10:42
superm1until i realized that10:42
cjwatsonwould be better to call apt-install on the packages you want than the manifest-mythbuntu trick10:42
superm1apt-install for removal?10:43
superm1the manifest mythbuntu trick calculates removals, additions are done via the same mechanism as the language pack install10:44
cjwatsonno, I mean it would be better to invert it and state what you want to keep using apt-install10:44
cjwatsonthen you wouldn't have to override that whole method from scripts/install.py10:44
superm1oh that's an interesting way to think about it10:44
superm1didn't consider that as a possibility even10:44
cjwatsonthat's the usual way :-)10:45
superm1i was feeling intuitive with my regex magic :)10:45
superm1indeed though, the less methods that need to be overriden, the better10:46
CIA-26ubiquity: cjwatson * r2116 ubiquity/ubiquity/components/mythbuntu_services.py: no need to import both os and os.path10:49
cjwatson+from ubiquity.misc import *10:50
cjwatson+10:50
cjwatson+from ubiquity import misc10:50
=== cjwatson blinks
CIA-26ubiquity: cjwatson * r2117 ubiquity/ubiquity/frontend/mythbuntu_ui.py: python doesn't need trailing semicolons10:51
superm1ah the things that i easily overlook... thanks for catching these10:52
cjwatsonnot a problem10:54
CIA-26ubiquity: cjwatson * r2118 ubiquity/bin/ubiquity-wrapper: slightly more idiomatic10:55
CIA-26ubiquity: cjwatson * r2119 ubiquity/debian/ubiquity-frontend-mythbuntu.postinst: unnecessary postinst10:57
CIA-26ubiquity: cjwatson * r2120 ubiquity/ubiquity/components/ (mythbuntu_install.py mythbuntu_summary.py summary.py): avoid some duplication10:58
superm1that postinst was necessary I had thought, it registers all the things in the templates file10:59
superm1with debconf10:59
superm1within the #DEBHELPER# tag10:59
cjwatsonsuperm1: whoops, you're quite right11:01
cjwatsonI'll revert that11:01
CIA-26ubiquity: cjwatson * r2121 ubiquity/ubiquity/frontend/mythbuntu_ui.py: tidy up imports11:01
superm1and in 2120, will the default /usr/share/ubiquity/install.py be called before getting to updating prep[0] ?11:02
superm1or will that work appropriately11:02
cjwatsonphone11:03
CIA-26ubiquity: cjwatson * r2122 ubiquity/ (debian/changelog gui/glade/stepUserInfo.glade):11:03
CIA-26ubiquity: * GTK frontend:11:03
CIA-26ubiquity:  - Fix full-name error reason widget, and make the error reasons11:03
CIA-26ubiquity:  selectable.11:03
CIA-26ubiquity: cjwatson * r2123 ubiquity/debian/ubiquity-frontend-mythbuntu.postinst: revert r2119, I was too wrong for words11:04
superm1and in 2121, don't the debconf imports still need to happen?  so that things like debconf_progress_start() can be called11:05
cjwatsonsuperm1: 2120> the only change I could see in mythbuntu_install's prepare method was that you returned a slightly different list11:17
superm1i'll double check it later this evening then11:18
cjwatsonsuperm1: so that's easily done, just call the superclass' prepare method, modify the list you get back, and return that11:18
superm1make sure it works as expected still11:18
cjwatsonsure, definitely11:18
cjwatsonsuperm1: 2121> python imports are very simple: they do some initialisation (often none) and bind names11:18
cjwatsonsuperm1: well-behaved python modules tend to do no initialisation you actually care about11:18
cjwatsonsuperm1: so, to a good first approximation, you only need to import something if you use something from its namespace11:19
cjwatsonsuperm1: those names weren't used, so I deleted the imports11:19
superm1i see11:19
cjwatson(and I happened to know that DebconfCommunicator definitely wasn't needed any more)11:19
superm1okay makes sense :)11:19
cjwatsondebconf_progress_start is implemented in a different file, which imports everything it needs itself to do so11:20
CIA-26ubiquity: cjwatson * r2124 ubiquity/ubiquity/frontend/mythbuntu_ui.py: come to think of it, we don't need to import gtk.glade either11:22

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