[00:42] <CIA-28> wubi: Agostino Russo * r113 trunk/ (11 files in 5 dirs): Set the application icon (LP: #341631)
[00:56] <CIA-28> wubi: Agostino Russo * r114 trunk/ (3 files in 3 dirs): Fixed typo (LP: #355923)
[01:41] <CIA-28> wubi: Agostino Russo * r115 trunk/ (7 files in 3 dirs): Make enough space for translated subtitle (LP: #351519)
[10:02] <davmor2> Morning evand
[10:02] <evand> hello
[10:02] <davmor2> r108 still doesn't uninstall :(
[10:03] <evand> davmor2: can you try r115 from http://people.ubuntu.com/~evand/wubi/jaunty
[10:04] <davmor2> evand: what version is on the current cds?
[10:04] <evand> oh, right
[10:04] <evand> why do I keep forgetting that
[10:04] <evand> nevermind
[10:04] <evand> waiting on the release team to approve r115 which xivulon thinks will fix the bug
[10:05] <davmor2> evand: It might of been 115 I try on friday then hang on
[10:05] <evand> it's not on the CDs yet
[10:06] <davmor2> no it was 108 I tried
[10:06] <davmor2> 115 I can add as it does effect the rest of the cd :)
[10:07] <davmor2> I just use isomaster rename wubi-r115.exe to wubi.exe remove the one from the iso and drop the new one in :)
[10:30] <davmor2> evand: I'm just refreshing vista then I'm trying 115 on cd
[10:43] <evand> neat, I wasn't aware of quoting the delimiter in here-documents.  I really need to give the specification a more thorough read.
[10:44] <evand> Thanks cjwatson for that and the rest of the fixes you uploaded in my absence over the past week.  Very much appreciated.
[10:45] <cjwatson> no worries, was a bit of a busy week ;-)
[10:46] <evand> heh, indeed.  I noticed the massive changelogs for ubiquity and others.
[10:50] <davmor2> cjwatson:  ubiquity is a much nicer size now that it isn't in mini mode :)
[10:50] <davmor2> and London is in the southeast Yay
[10:51] <cjwatson> there are still a few fairly significant problems. The ones on my list at the moment are bug 351987, bug 339898, bug 353090, and bug 354534
[10:51] <davmor2> evand, cjwatson: I got missing text on the checkboxes on step 5
[10:52] <cjwatson> davmor2: yes, that's bug 353090
[10:52] <davmor2> cjwatson: Just saw :)
[10:53] <davmor2> bug 339898 is still in effect too
[10:53] <evand> cjwatson: okay, I'm currently working on the better slider for resizing partitions.  After that I'll tackle the m-a bug and go from there.
[10:53] <evand> s/better/better looking/
[10:54] <davmor2> evand: what wrong with the current one it is far more obvious than the older one
[10:54] <evand> davmor2: it uses just a black box to indicate the handle
[10:54] <cjwatson> davmor2: 339898> I believe I just mentioned that right above
[10:55] <davmor2> cjwatson: yes I was just saying it is still in effect on today's iso
[10:56] <evand> incidentally today's ISO has a fairly out of date livefs.  That's not implying any bugs are fixed in a newer version though.
[10:56] <davmor2> evand: :(
[10:57] <evand> err nevermind, that's only for amd64
[10:57] <davmor2> :) I used 32bit
[10:57] <davmor2> evand: right r115 install just complete cd eject fix still works I'll try uninstall now
[11:00] <davmor2> evand: the naming issue is still in effect (fusa still lists davmor2 even though tester is the username)
[11:00] <evand> indeed, I don't think anyone has tackled that one yet
[11:01] <davmor2> evand: I think xivulon put a fix on the bug to remind himself iirc but wanted to concentrate on uninstall and cd fix
[11:01] <evand> ok
[11:02] <davmor2> I think it's goning to use fullname rather than username from windows (if that makes sense while I look for the bug)
[11:05] <davmor2> evand: 115 threw up:  Internal Error   Error deleting directory
[11:05] <evand> davmor2: please update the bug with new log files
[11:05] <davmor2> evand: will do
[11:07] <ogra> cjwatson, so m last remaining prob i had to work around with ubiquity installs yesterday was that i end up with casper/filesystem.kernel-imx51 instead of casper/vmlinuz on the armel iso, i couldnt find where in cdimage the renaming happens, do you have a pointer for me ?
[11:07] <cjwatson> evand: I just noticed belatedly that the timezone map image includes country borders. Have we checked that it does not include disputed borders?
[11:07] <ogra> s/m/my/
[11:07] <davmor2> evand: bug 347449 for the name change thing
[11:07] <cjwatson> ogra: tools/boot/jaunty/* IIRC
[11:07] <ogra> gracias
[11:08] <evand> cjwatson: checking now
[11:08] <cjwatson> evand: oh, never mind me, they aren't country borders, are they? they're timezone borders
[11:08] <cjwatson> sorry, panicked for a second :)
[11:10] <evand> indeed, though hrm, we seem to be missing the image for UTC+5.5
[11:11] <ogra> ah, thats debian-cd ... that makes it somewhat clear why i didnt find it in my srv/cdimage... checkout
[11:11]  * evand digs for that
[11:14] <ogra> cjwatson, hmm, that looks like i could actually add all my redboot stuff from the builder script there as well
[11:15] <CIA-28> ubiquity: evand * r3181 jaunty-beta/ (pixmaps/timezone/timezone_5.5.png debian/changelog): Add the missing UTC+5.5 time zone image.
[11:15] <persia> ogra, Just make sure to track subarch when you stick it there.
[11:16] <ogra> persia, yeah, though the vfat converter seemingly needs to go elsewhere
[11:16] <davmor2> evand: Logs up on bug 341605
[11:17]  * evand checks the rest of the time zone images to make sure he didn't miss any others
[11:17] <evand> davmor2: thanks
[11:19] <davmor2> evand: That's weird looking at the system and the log it seems to of been successful :(
[11:19] <persia> ogra, I'd probably stick it in tools/vfat2sd or some such.
[11:19] <ogra> persia, i'm not sure it actually belongs in debian-cd
[11:20] <ogra> we used to keep the wrapper scripts in cdimage before
[11:20] <ogra> but then adding redboot wouldnt makes sense in the debian-cd tree though
[11:20] <ogra> *make
[11:21]  * ogra really wonders where it would fit best
[11:33] <cjwatson> ogra: redboot> well, that's what I was trying to say before ;-)
[11:34] <cjwatson> ogra: vfat conversion is done by tools/make-vfat-img in debian-cd
[11:34] <cjwatson> (at the moment, for other vfat targets)
[11:34]  * ogra goes to take a look 
[11:34] <cjwatson> evand: can we not use the jaunty-beta branch?
[11:34] <ogra> the prob is that i need the special partitioning that leaves 20M spare space before the first partition starts
[11:35] <cjwatson> I don't think there's any reason to do so - we can use trunk
[11:35] <ogra> http://paste.ubuntu.com/145325/ btw ...
[11:37] <persia> ogra, Are you sure you don't want that in tools/boot/jaunty/boot-armel+imx51 ?
[11:37] <ogra> persia, well, for jaunty it suffices, we dont support any other subarch for livefs builds
[11:38] <persia> I guess.  I just like to be tidy.
[11:39]  * ogra scratches haed about "/sbin/mkdosfs -C $outfile $size" ... does that mean i used dd since forever for no reason to create vfat images ??
[11:39] <evand> cjwatson: sure thing, I thought you had switched to just using jaunty-beta for the time being and was trying to mimic that behavior.
[11:39] <evand> noted though
[11:39] <ogra> oh man
[11:40] <cjwatson> evand: usually I've just used that kind of branch for really deep-freeze periods
[11:42] <persia> ogra, Oh, and if you do the other bit, that would probably be in tools/boot/jaunty/post-boot-armel+imx51 for the fconfig stuff (if you're doing it in debian-cd).
[11:43] <ogra> persia, right, the closer i look at debian-cd the more i agree we should add everything there
[11:43] <ogra> the code sacesme in its cleanness ...
[11:43] <ogra> *scares me
[11:44] <evand> ok
[11:45] <cjwatson> ogra: *sigh*
[11:46] <ogra> cjwatson, ?
[11:46] <cjwatson> ogra: oh, hang on, are you saying it's clean, or being sarcastic?
[11:46] <cjwatson> sorry, I think I misread :)
[11:46] <ogra> heh
[11:46] <ogra> no, no sarcasm here
[11:46] <cjwatson> ah, ok. most people complain about debian-cd so I guess I had a hardwired expectation :)
[11:47] <ogra> it just shows me how much i miss in my knowledge
[11:47] <persia> debian-cd is fiendishly complex, but very well organised for what it needs to do.
[11:47] <ogra> /sbin/mkdosfs -C $outfile $size <- is so trivial ... i'm creating vfat images since years and never discovered it
[11:48] <ogra> stuff like that is scattered all over
[12:14] <cjwatson> I'm not sure that post-boot-* is currently applied to vfat images
[12:15] <cjwatson> rather, I think it might be currently applied to the ISO image before conversion to vfat, which may not be very helpful
[12:15] <persia> cjwatson, It is: the syslinux call lives there.
[12:15] <cjwatson> ah, yes, you're right. Good.
[12:16] <persia> My thought that was for images that belonged on SD cards, we probably didn't want the syslinux call, and would instead do whatever magic is required for the desired format.
[12:18] <cjwatson> yes
[12:48] <CIA-28> partman-ext3: cjwatson * r753 ubuntu/ (debian/changelog valid_filesystems/ext4): Make valid_filesystems/ext4 executable (LP: #354851).
[12:49] <ogra> cjwatson, btw, i think we can/should keep the gernam keymap as is ... http://forum.ubuntuusers.de/topic/welches-deutsche-tastaturlayout-benutzt-du/
[12:50] <cjwatson> ok
[12:50] <cjwatson> thanks
[12:50] <ogra> on the ML there is a minimal majority for nodeadkeys, forum shows exactly the opposite
[12:50] <ogra> the most valid point is that win uses deadkeys by default so people coming from there find a familiar keymap
[12:52] <cjwatson> mm, yes, that's fairly solid
[12:54] <ogra> what i noticed is that many people pointed out that the description should be better in the UI (no idea how to do that without making it several sentences per map though)
[12:54] <CIA-28> partman-ext3: cjwatson * r754 ubuntu/debian/changelog: releasing version 56ubuntu2
[13:27] <xivulon> davmor2 hi
[13:33] <xivulon> your log in http://launchpadlibrarian.net/24859557/C%3A%5CUsers%5Cdavmor2%5CLocal%20Settings%5Ctemp%5Cwubi-9.04-rev115.log looks good to me
[13:34] <xivulon> it's not clear from your post whether you wanted to report some issues or confirm that everything is ok
[13:35] <xivulon> cjwatson, I edited those c files, for the leak, actually str.c>concat function had the cleanup code so that was ok
[13:46] <davmor2> xivulon: I got an error sorry 115 threw up:  Internal Error   Error deleting directory
[13:47] <davmor2> but the directory seems to be removed so not sure why
[13:49] <lool> ogra, cjwatson, persia: Hmm it's unfortunate I missed the discussion here
[13:52] <lool> ogra: I discussed this with cjwatson last week and today, and want to share a couple of things
[13:53] <lool> First cjwatson advised using vfat images + boot-armel; this is what I've enabled in the commits merged friday and I pursued this today; you can check antimony:~lool/cdimage for the next ones
[13:53] <lool> Then I asked about using a subarch instead of armel, i.e. armel+imx51, but cjwatson said we should simply use armel this cycle and revisit the subarch thing for the next one
[13:54] <lool> The immediate problem I've hit with the SUBARCH approach is that the live contents wasn't at the proper URL
[13:54] <persia> lool, Did you compare to the powerpc+ps3 scripts?
[13:55] <lool> Yes
[13:56] <lool> persia: If cjwatson tells me we don't want a SUBARCH yet, I'll just take his word for it
[13:56] <lool> I'm happy that we can help each other on this topic, but I fear we're duplicating work or stepping on each others' toes here   :6/
[13:56] <lool> :-/
[13:57] <lool> Blah it's not even per flavour, it's per board
[13:57] <persia> I'm just commenting :)
[13:57] <ogra> lool, sorry, my machine crashed
[13:57] <xivulon> davmor2, can you try with a clean installation? might be due to a previous version uninstaller being run
[13:58] <xivulon> does usb-creator extract the files from the ISO or does it copy the ISO itself?
[13:59] <persia> xivulon, Extract the contents
[14:00] <davmor2> xivulon: that was a clean install
[14:01] <davmor2> xivulon: But I'll do a fresh install of vista might take all day though :)
[14:03] <ogra> lool, wrt subarc, currently the livefs contains the imx51 kernel anyway, so we would need a subarch setting in livecd-rootfs as well
[14:03] <ogra> which would then end up in the right location again but have differeently called files
[14:04] <xivulon> davmor2, please make it clear in the bug report, I will investigate it tonight
[14:05] <davmor2> xivulon: Will do although I'm going to do a clean install and updates and see if there is something lingering still before I do
[14:05] <cjwatson> lool: not so much "don't want", but I think it's not worth the work
[14:05] <cjwatson> (subarch)
[14:05] <cjwatson> superm1: oh, wow, I found out why grub2 is breaking
[14:06] <ogra> cjwatson, well, we're very likely to get more armel subarches at least for live images in KK
[14:06] <cjwatson> superm1: when the grub package is removed, it calls db_purge (as is usual); this ends up accidentally purging all templates owned by ubiquity
[14:06] <cjwatson> ogra: right, absolutely for karmic, I was just talking about jaunty
[14:09] <xivulon> if usb creator extracts the files, fixin #299001 is blocked by #243105
[14:10] <xivulon> cjwatson is #243105 fixed by any chance? I doubt it would have been tested though.
[14:10] <xivulon> hi evand
[14:10] <xivulon> was discussing 299001
[14:12] <xivulon> evand you might want to remove the if block in pylaucher.c line 109, as the ! is wrong and the warning is unneded anyway
[14:12] <cjwatson> xivulon: sorry, no, at least not by me. I don't think I'm likely to get to this, certainly not for jaunty
[14:14] <xivulon> evand pylauncher.c, line 81-84, replace with: delete_directory(targetdir);
[14:15] <xivulon> cjwatson, np. evand, fyi #299001 is blocked by #24310
[14:15] <xivulon> 5
[14:15] <xivulon> you might also want to fix a typo in 355928
[14:18] <CIA-28> wubi: evand * r116 trunk/ (debian/changelog src/pylauncher/pylauncher.c): Properly test the return value of delete_directory.
[14:19] <lool> Actually we don't need boot-armel, only post-boot-armel
[14:20] <lool> Hmm sorry, we do need it for the casper kernel copy
[14:21] <persia> Well, both is good.  boot-armel+imx51 places the right kernel, and post-boot-armel+imx51 would stuff things in the right spots for redboot.
[14:21] <xivulon> evand no need to test the result of delete_directory at all, it's not a critical error will simply leave a folder in the temp directory, I had the test in for debugging purposes mostly
[14:21] <persia> Right.
[14:21] <xivulon> just delete the temp directory and take no action on errors
[14:30] <xivulon> evand, thx for the fix by the way
[14:30] <ogra> persia, the prob is, where do we create the special partitioned image
[14:31] <ogra> post-boot-armel+imx51 only makes sense on the image where we have the special partitioning ... and append the vfat at the end
[14:32] <persia> ogra, Hrm.  Depends on whether it's a new type of image (SD) or just a different way to package a VFAT.
[14:33] <persia> In the latter case, I'd suggest that creating the SD image from the vfat would be a post-boot action.  In the former case, you probably have to special-case Makefile, which is harder.
[14:33] <ogra> we need: a blank image with 20M spare space at the beginning and a part. table (which means we need to know the size of the vfat at this point)
[14:33] <ogra> then we need to add the bootloader, kernel, initramfs and redboot config to the 20M
[14:33] <ogra> then we dd the vfat to the end of that
[14:34] <persia> Right, and I don't think we actually know the vfat size until post-boot.
[14:34] <ogra> we do know the vfat size right after we called tools/make-vfat-img
[14:35] <persia> Except aren't we going to stuff a kernel into it in boot?
[14:35] <ogra> no
[14:35] <ogra> the kernel needs to be in casper
[14:35] <ogra> and casper only
[14:35] <ogra> not boot
[14:36] <ogra> oh, you mean boot-armel
[14:36]  * ogra thought /boot ... sorry
[14:36] <persia> RIght.
[14:36] <persia> I should have said boot-* and post-boot-*
[14:36] <ogra> so, boot would call tools/make-vfat-img at the end ...
[14:37] <persia> No.
[14:37] <ogra> post-boot gets the size and the vfat location as args and calls make-sd-img which creates the empty partitioned image
[14:37] <persia> So, you can define an image type to be VFAT.
[14:38] <persia> So you create a VFAT image of armel+imx51
[14:38] <persia> Then, in boot-armel+imx51, you add the right kernel
[14:38] <persia> Then, in post-boot-armel+imx51, you stuff that vfat into the SD image.
[14:38] <superm1> cjwatson, wow that's quite a find.  why are the ubiquity owned templates getting cleared though when purging the 'grub' package?  shouldn't only grub templates be cleared then?
[14:38] <ogra> persia, we need the kernel twice
[14:39] <ogra> once to stuff it into the vfat and once to put it into redboot
[14:39] <persia> RIght, once in boot-armel+imx51 and once in post-boot-armel+imx51
[14:39] <lool> ogra: It's not clear to me whether you're picking this up or whether I should continue what I was working on
[14:39] <ogra> and by the looks of it that happens in two different places
[14:39] <ogra> lool, what do you like to do ?
[14:40] <ogra> i told you last week that i will focus on it this complete week
[14:40] <ogra> but if you want to keep it, keep it and i'll start digging in gnome-keyring
[14:40] <lool> I'm in the middle of writing the boot- / post-boot- stuff, so since I had to learn all this stuff I prefer continuing
[14:40] <ogra> lool, i'm at least familiar with my script already
[14:41] <lool> Well I kind of know what the script does *cough*
[14:41] <ogra> right
[14:42] <cjwatson> superm1: normally speaking grub.postrm would be talking to a normal debconf frontend which knows that the relevant "owner" is grub. However, in the case of ubiquity, there's a single top-level debconf frontend whose owner is ubiquity, and all debconf activity is proxied through to that
[14:42] <cjwatson> anyway, it's an easy fix - just trying to confirm whether the rest of it works after that
[14:42] <superm1> cjwatson, ah i see.
[14:45] <_MMA_> I'm sure this has been put out there somewhere. Has the ext4 format option been removed from all the Jaunty Alt install disks?
[14:49] <cjwatson> _MMA_: bug 354851, fixed today
[14:49] <cjwatson> it was a merge accident
[14:50] <_MMA_> cjwatson: Oh ok. Cool. Just thought it was a case where I missed some email about a change. :)
[14:50] <cjwatson> or actually an upstream commit accident
[14:53] <CIA-28> ubiquity: cjwatson * r3181 ubiquity/ (debian/changelog ubiquity/debconffilter.py):
[14:53] <CIA-28> ubiquity: Don't pass the debconf PURGE command through debconffilter, since we
[14:53] <CIA-28> ubiquity: don't know the owner of the package calling it (LP: #354534).
[14:54] <CIA-28> oem-config: cjwatson * r648 trunk/ (debian/changelog lib/debconffilter.py):
[14:54] <CIA-28> oem-config: Don't pass the debconf PURGE command through debconffilter, since we
[14:54] <CIA-28> oem-config: don't know the owner of the package calling it (see LP #354534).
[14:56] <cjwatson> evand: do you remember what you meant by this comment in scripts/install.py?
[14:56] <cjwatson>         # Less than ideal.  Since we cannot know which bootloader we'll need
[14:56] <cjwatson>         # at file copy time, we should figure out why grub still fails when
[14:56] <cjwatson>         # apt-install-direct is present during configure_bootloader (code
[14:56] <cjwatson>         # removed).
[14:56] <cjwatson> evand: fixing the grub2 case properly seems to involve making apt-install-direct be present during configure_bootloader, so ...
[14:56] <CIA-28> ubiquity: evand * r3182 ubiquity/ (debian/changelog ubiquity/segmented_bar.py):
[14:56] <CIA-28> ubiquity: Draw something resembling a slider, rather than a black box, for the
[14:56] <CIA-28> ubiquity: slider on the partition bar.
[14:57] <evand> hrm
[14:59] <cjwatson> evand: also could you merge your tz fix from jaunty-beta?
[14:59] <evand> I don't recall the exact circumstances around the latter part of that, and the first part is just that we need to account for the bootloader in the blacklist as we copy files before we make a bootloader choice
[14:59] <evand> cjwatson: will do
[15:01] <evand> sorry about that, my trunk branch was woefully out of date and new runs of bzr pull exited successfully because my branch was a little broken due to a bad internet connection in dubai.
[15:01] <cjwatson> I was mostly wondering what the "(code removed)" was about
[15:01] <cjwatson> ah
[15:02] <evand> so it looked like ubiquity was still at 1.12.0 in trunk
[15:02] <evand> hrm
[15:03] <cjwatson> I'm trying it out now to see what happens :)
[15:03] <evand> heh, ok
[15:04] <cjwatson> maybe you meant that grub doesn't get reinstalled after its files have been removed or something
[15:05] <evand> I think I meant grub-install crashes, but I'm really not confident in that
[15:12] <CIA-28> ubiquity: evand * r3183 ubiquity/ (pixmaps/timezone/timezone_5.5.png debian/changelog): Add the missing UTC+5.5 time zone image.
[15:14] <cjwatson> hmm. grub-pc.postinst fails due to some kind of problem with debconf's fds not being connected up properly.
[15:16] <CIA-28> wubi: evand * r117 trunk/ (debian/changelog src/pylauncher/pylauncher.c):
[15:16] <CIA-28> wubi: Ignore any failure in delete_directory as it is just working with temporary
[15:16] <CIA-28> wubi: directories.
[15:21] <cjwatson> maybe the problem is actually that we aren't quite disconnecting from debconf properly
[15:49] <cjwatson> superm1: aha, I just did a working grub2 installation
[15:50] <superm1> cjwatson, awesome!
[15:56] <CIA-28> ubiquity: cjwatson * r3184 ubiquity/ (compat/apt-install debian/changelog):
[15:56] <CIA-28> ubiquity: Unset DEBCONF_REDIR in our apt-install shim so that starting up another
[15:56] <CIA-28> ubiquity: debconf frontend will actually work, as needed when installing grub-pc.
[15:56] <CIA-28> ubiquity: Also set APT_LISTCHANGES_FRONTEND=none just in case.
[15:56] <CIA-28> ubiquity: cjwatson * r3185 ubiquity/ (debian/changelog scripts/install.py):
[15:56] <CIA-28> ubiquity: Install boot loader packages immediately rather than queueing them until
[15:56] <CIA-28> ubiquity: later. Boot loader installers typically need them immediately.
[16:14] <cjwatson> lool: thanks for that isolinux wrapper of yours; very handy
[16:15] <lool> Glad it halps
[16:15] <lool> *helps
[16:15] <lool> Oh cool grub2 works
[16:16] <cjwatson> lool: should the ubuntu-mid gfxboot text be brown, like Ubuntu's?
[16:17] <cjwatson> and UNR for that matter
[16:19] <lool> cjwatson: I have no objection, and it would be good if it's consistent
[16:21] <cjwatson> lool: ok, thanks; done
[16:54] <davmor2> xivulon: still upgrading
[17:24] <xivulon> davmor2?
[17:25] <davmor2> xivulon: vista
[17:26] <xivulon> davmor2 did you test CD ISO davmor2 extraction by the way? 207137
[17:27] <davmor2> xivulon: I need to test that on my xp box I'll do that after it's running something at the moment
[17:32] <xivulon> thx, please edit the bug as usual
[17:32] <davmor2> xivulon: np's
[17:33] <xivulon> evand it might be good to have a second build with at least one entry in the isolist.ini pointing at the beta, that is the only way to test bittorrent
[18:01] <cjwatson> evand: mind if I do a quick ubiquity upload? I'm getting fed up of questions about ext4
[19:19] <festor> Hi!
[19:19] <festor> there is a problem with ubuntu installer and ext4?
[19:20] <festor> I use ubuntu beta 9.04 and I dont see ext4 option in installer
[19:25] <lool> 2 minutes!
[19:32] <cjwatson> and has left IRC now too
[19:32] <cjwatson> also, he's lying; he's not using the beta. I introduced that bug on 1 April.
[19:33] <cjwatson> (or, more charitably, is unaware that post-beta daily builds != beta)
[19:40] <CIA-28> ubiquity: cjwatson * r3186 ubiquity/debian/ (80 files in 2 dirs): Update translations from Launchpad.
[19:40] <cjwatson> ogra: ^- r3186 fixes the German translation bugs you noticed
[19:44] <CIA-28> ubiquity: cjwatson * r3187 ubiquity/ (d-i/manifest debian/changelog): Automatic update of included source packages: partman-ext3 56ubuntu2.
[19:48] <CIA-28> ubiquity: cjwatson * r3188 ubiquity/debian/changelog: releasing version 1.12.4
[19:57] <lool> cjwatson: parted / jaunty has a -m ("machine parseable") flag; does it make sense to request backport of parted for this feature, or should I adapter the script to not use this feature?
[20:38] <lool> cjwatson: So I implemented a non-"-m" version and keep the other one around
[20:58] <cjwatson> lool: we don't use parted on antimony for anything else, so I don't mind *too* much either way
[20:58] <cjwatson> lool: though in general I'd prefer not to have too many new dependencies
[20:58] <lool> cjwatson: Sent you an email
[20:59] <lool> cjwatson: I can live with either; the advantage of a new parted would be robustness; however it's unlikely to break if we don't touch it, so I don't know
[20:59] <lool> cjwatson: I'm going afk now; happy to read you opinion on ~lool/cdimage as usual
[21:07] <xivulon> evand hi
[21:10] <xivulon> evand, could you please create a build with following patch: http://paste.ubuntu.com/145693/
[21:10] <xivulon> there is no need to commit that, it is to address 207137
[21:12] <xivulon> so that davmor2 can test that, the patch will not let the read opearation fail, we would need to check the md5 of the extracted ISO afterwards
[21:12] <cjwatson> lool: I don't suppose that sfdisk -f would avoid the CHS issues you mentioned and allow you to set an arbitrary partition type?
[21:14] <cjwatson> lool: otherwise, a 0x83 partition might not be all *that* bad? you can't infer filesystem from 0x83 anyway
[21:14] <cjwatson> at least not the particular filesystem involved
[21:14] <cjwatson> lool: some more quoting would be nice in arguments to [(1) for robustness
[21:15] <xivulon> on IOerror, the patch will try to read smaller and smaller blocks, and then skip the read operation completely. I am not sure whether A) reading smaller chunks will fix it, B) whether the error is irrelevant
[21:15] <cjwatson> lool: and indeed variable expansions in general unless there's a specific reason not to double-quote them
[21:15] <xivulon> of course if the file seek point is moved by a previous read attempt, then reading smaller chunks is not goin to help
[21:16] <cjwatson> lool: but I'm OK with merging this in the meantime, modulo the need for extra packages on antimony
[21:16] <cjwatson> lool: you can commit fconfig.bin as a binary in data/jaunty/ in the meantime if you want
[21:22] <CIA-28> oem-config: cjwatson * r649 trunk/debian/ (64 files in 2 dirs): Update translations from Launchpad.
[21:28] <xivulon> evand ^
[21:37] <CIA-28> oem-config: cjwatson * r650 trunk/ (d-i/manifest debian/changelog): Automatic update of included source packages: user-setup 1.23ubuntu16.
[21:39] <CIA-28> oem-config: cjwatson * r651 trunk/debian/changelog: releasing version 1.54.11
[21:45] <CIA-28> cdrom-detect: cjwatson * r437 ubuntu/debian/ (7 files in 2 dirs): Update Ubuntu-specific strings from Launchpad.
[21:49] <CIA-28> console-setup: cjwatson * r100 ubuntu/debian/ (12 files in 2 dirs): Update Ubuntu-specific strings from Launchpad.
[21:52] <CIA-28> partman-auto: cjwatson * r285 ubuntu/debian/ (7 files in 2 dirs): Update Ubuntu-specific strings from Launchpad.
[21:54] <CIA-28> partman-base: cjwatson * r152 ubuntu/debian/ (25 files in 2 dirs): Update Ubuntu-specific strings from Launchpad.
[21:57] <lool> cjwatson: Ok, I usually always double-quote, but I copied the style of not quoting from other scripts; will change it
[21:57] <cjwatson> oh, I don't mind so much if it's following other scripts
[21:57] <lool> cjwatson: I thought sfdisk was deprecated, I'll look into it when I go for a partition for the FIS; thanks
[21:57] <cjwatson> but probably better to fix anyway
[21:58] <lool> I agree, I hate test failing in weird ways when some vars are unset
[21:58] <cjwatson> I hadn't heard of sfdisk being particularly deprecated, aside from the comment at the top of the manual page suggesting that you use parted instead; but in cases where parted doesn't suffice I think you're justified in ignoring that :)
[21:58] <cjwatson> lool: oh, the alternative would be to dd in the partition type you want. It's only one byte at a known position, after all!
[21:59] <lool> cjwatson: Eh I was tempted to do this, but I didn't want to be accused of not using standard tools :)
[21:59] <cjwatson> *shrug* as long as there's a comment :)
[21:59] <lool> cjwatson: Do you know the ticket # for the redboot-tools stuff?  Is it filed?
[21:59] <lool> I think ogra did it, but I'm not 100% sure
[22:00] <CIA-28> partman-crypto: cjwatson * r677 ubuntu/debian/ (8 files in 2 dirs): Update Ubuntu-specific strings from Launchpad.
[22:00] <cjwatson> lool: not as far as I can see
[22:00] <lool> Argh
[22:00] <cjwatson> lool: please go ahead and file it
[22:00] <lool> cjwatson: I think I'll first merge fconfig.bin in redboot-tools in some way and then ask for it
[22:00] <cjwatson> ok
[22:00] <lool> Otherwise I'll have to ask another time
[22:01] <lool> cjwatson: thanks for the review
[22:08] <CIA-28> partman-target: cjwatson * r757 ubuntu/debian/ (62 files in 2 dirs): Update Ubuntu-specific strings from Launchpad.
[22:49] <CIA-28> pkgsel: cjwatson * r136 ubuntu/debian/ (34 files in 2 dirs): Update Ubuntu-specific strings from Launchpad.