/srv/irclogs.ubuntu.com/2010/12/13/#ubuntu-devel.txt

=== dendrobates is now known as dendro-afk
ebroderbdrung: Have you released a version of ubuntu-dev-tools with SPONSOR_PATCH_BUILDER yet? Because I want to factor that code into a library that sponsor-patch and backportpackage share, at which point it should probably be UBUNTU_TOOLS_BUILDER or something02:37
ebroder(If you've released it already, I can just leave some backwards compatibility code lying around)02:37
=== ssj6akshat|bed is now known as ssj6akshat
=== Amaranth_ is now known as Amaranth
=== ssj6akshat is now known as ssj6akshat|schoo
=== _LibertyZero is now known as LibertyZero
hychen_freeflying, I added https://launchpad.net/~ubuntu-sru to the bug I reported, is this right?06:27
freeflyinghychen_: subscribed?06:29
hychen_freeflying, yes06:29
freeflyinghychen_: is the package in universe or main?06:33
hychen_freeflying, universe06:33
hychen_freeflying, it is from debian06:33
micahgin most cases subscribing ubuntu-sru isn't correct anymore as the SRU team reviews in teh upload queue06:33
hychen_freeflying, I also have a question, when pkg in proposed goes to update archive?06:34
freeflyinghychen_: after ftp-master check and approve it06:35
freeflyingmicahg: so, in hychen_'s case, he gonna seek for a sponsor to upload it into maverick-proposed?06:35
hychen_freeflying, so the issue also need to be fix in next release?06:35
micahgfreeflying: so, ubuntu-sponsors should be subscribed, not ubuntu-sru06:36
micahgalso, Ubuntu doesn't have ftpmasters, the archive admins will move an update from -proposed to -updates after the test case has been verified and it's been in -proposed for a week06:37
krishna_m28how can i contribute ? I am a experienced C,c++ programmer06:37
hychen_micahg, is any wiki page mentioned this?06:38
freeflyingmicahg: same job, different titles06:38
micahghychen_: which thing?06:38
freeflyinghychen_: you can send me debdiff, I will upload it for you :)06:39
micahghychen_: should all be here: https://wiki.ubuntu.com/StableReleaseUpdates06:39
hychen_micahg, thanks , I think I miss some important sections, I'll reviewed it again06:40
hychen_freeflying, the debdiff is http://tinyurl.com/28ylvl7, but I don't know how to version the package correctly06:42
hychen_freeflying, should I change the series to maverick-proposed?06:42
freeflyinghychen_: in which version did you find it?06:43
hychen_freeflying, 1.3.7.20100910-1 in debian06:43
hychen_freeflying, and is already sync to Natty06:43
micahgactually, 1.3.9.2-1 is in sid and natty06:44
hychen_micahg, it is ibus-chewing not ibus06:45
freeflyinghychen_: the one in maverick is ibus-chewing-1.3.6.20100730?06:45
hychen_freeflying, yes06:45
micahgibus is at 1.3.8-106:45
hychen_micahg, you are right, my information is old, sorry06:46
freeflyinghychen_: A minimal patch applicable to the stable version of the package. If preparing a patch is likely to be time-consuming, it may be preferable to get a general approval from the SRU team first.06:46
freeflyinghychen_: maybe you shall consider of backport it from natty06:47
hychen_freeflying, follow https://help.ubuntu.com/community/UbuntuBackports . right?06:49
freeflyinghychen_: I think so06:51
xnoxThere isn't a lp:ubuntu/libtool06:51
xnoxlibtool doesn't show up on merges.ubuntu.com (although it should?)06:51
freeflyinghychen_: btw, ibus-chewing is in main06:51
xnoxand it's outdated06:51
* xnox libtool is wonderful06:51
micahgxnox: no, it's only newer in experimental (that doesn't mean it can't be merged though)07:00
xnoxmicahg, will be a bit of pain without lp:ubuntu/libtool and lp:debian/libtool07:00
micahgxnox: you should have bzr branches available07:01
xnoxmicahg, http://package-import.ubuntu.com/status/libtool.html07:03
xnoxAssertionError: Can't find the needed upstream part for version 2.2.6a-407:03
micahgxnox: oh well, I guess not07:04
micahgxnox: just diff natty against sid and apply to experimental07:05
xnoxthat's my plan =)07:05
xnoxAnyone ever used pull-debian-debdiff?07:07
micahghmm, fascinating07:07
xnoxmicahg, I'll use import-dsc to import the three packages - base, mine, other, just to use merge-package07:09
xnox=)07:09
micahgxnox: ok, but that seems like a lot harder07:10
xnoxmicahg, but I will be able to diff without pain =) push to lp and use a recipe for test builds :-P07:10
xnoxor $ bzr bd-p07:11
xnoxbzr alias bd-p='bd --builder="pdebuild"'07:11
micahgxnox: there's no Ubuntu branch from what I can tell07:11
xnoxSo? =) I can still push to lp:~/libtool/udd-fail07:12
micahgxnox: if you like :)07:12
micahgxnox: you can also check for a bug on teh udd project07:12
xnoxmicahg,07:17
xnox$ bzr merge-package ../experimental/Text conflict in debian/patches/series07:17
xnox1 conflicts encountered.07:17
xnoxThe merge resulted in 1 conflicts. Please resolve these and commit the changes with "bzr commit".07:17
* xnox loves bzr-bd =)))))07:17
didrocksgood morning07:20
dholbachgood morning!07:37
pittiGood morning07:38
micahgpitti: good morning, can you take a quick look at a response for me to a bug before I send it?  http://pastebin.ubuntu.com/542951/ for bug 68878107:39
ubottuLaunchpad bug 688781 in sqlite3 (Ubuntu) "Please update to 3.7.3 or 3.7.4 in Maverick" [Undecided,Confirmed] https://launchpad.net/bugs/68878107:39
pittimicahg: it's certainly an answer I can agree to :) we did put new upstream microversions into stables in the past, and sqlite has tons and tons of tests, but if people want to go ahead with that, we need to review the upstream changelog07:43
pittimicahg: if .2 -> .3 is just three bug fixes, we might as well take it as it is07:43
pittiif it has 50 changes, then backporting is better07:43
pitti(IMHO)07:43
micahgpitti: I can only find one instance of an update of sqlite3 and it was for a single patch in jaunty07:44
pittimicahg: I mean for other packages, not for sqlite07:44
micahgpitti: ah, ok07:44
micahgpitti: .3 adds 2 interfaces, not just bug fixes07:44
micahghttp://www.sqlite.org/releaselog/3_7_3.html07:45
pittimicahg: I guess it's then worth looking at a diff, and checking if that just adds new code (that should be rather harmless, as existing sw doesn't use it)07:45
pittimicahg: anyway, this kind of research should be done by the requestor of the SRU07:46
tumbleweedanyone know what's happening in bug 689345? (ancient amd64 deb still published in natty, arch:any package only seems to get built on i386 these days)07:47
ubottuLaunchpad bug 689345 in python-kinterbasdb (Ubuntu) "3.2-3ubuntu3 still in natty/amd64" [Undecided,New] https://launchpad.net/bugs/68934507:47
micahgpitti: ok, I watch the bugs because xulrunner depends on it, also, this started because of a OMGBuntu post, so I'm guessing the users on the bug are less technical07:49
pittimicahg: I guess in this case your response is just fine :)07:49
micahgpitti: if you think it's worthwhile, I can review the upstream changes07:51
pittimicahg: it's a lot of testing and risk involved (sqlite is used by a lot of software), so I'd still like the requestor to at least do a more through analysis of the changes07:54
micahgpitti: I definitely agree with that, I'll post my comment, thanks07:54
pittimicahg: "I" wouldn't go ahead with it on my own personally07:54
micahgpitti: I'm also adding a line about a lot of apps using it and the burden of testing07:55
micahgpitti: thanks, I have trouble sometimes deciding how much work the reporter should do08:03
pittimicahg: of course, if you would like to do that, please do08:04
micahgpitti: not really, I just wanted to make sure I wasn't asking too much08:05
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
=== almaisan-away is now known as al-maisan
=== smb` is now known as smb
=== doko_ is now known as doko
=== nenolod_ is now known as nenolod
smb@pilot in09:10
=== udevbot changed the topic of #ubuntu-devel to: Archive: Open | Development of Ubuntu (not support, not app development) | #ubuntu for support and general discussion for dapper -> maverick | #ubuntu-app-devel for application development on Ubuntu | http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://wiki.ubuntu.com/HelpingWithBugs | Current Friendly Patch Pilots: smb
* smb has little clue yet how to go on...09:10
=== axp2_ is now known as axp2
dholbachhey seb128!09:16
seb128dholbach, hey! how are you?09:18
dholbachgreat - thanks - how are you?09:18
seb128I'm fine thanks09:18
seb128we was nice09:18
seb128I just have to catch up on 3 days backlog now ;-)09:19
=== Tm_T_ is now known as Tm_Tr
cjwatsondoko: is the pastewebkit sync in cocoplum:~lp_archive/syncs/ yours?  perhaps you meant to flush it?09:33
dokocjwatson: yes, forgot to sync09:35
cjwatsondoko: shall I flush it now?09:46
dokocjwatson: sorry, otp. done now09:48
cjwatsonok, thanks09:49
Keybukjhunt: maw-neeng10:00
smbdoko, moin, should I be able to dist-upgrade a maverick chroot to natty by now or is that still known to be broken (it seems one I started last week still does not get ahead, but maybe I should start over)?10:14
dokosmb: which package version of python is currently on your system>10:16
doko?10:16
smbdoko, Well it is python2.7-minimal that fails with pycompile:240: requested versions not installed. But python2.6 seems still installed10:17
dokosmb: that was not my question. what does dpkg -l python show10:18
doko?10:18
smbpython 2.6.6-2ubuntu110:19
smb(ii)10:19
dokoand dpkg -l python-dev python-dbg ? (start the lines with ii?)10:20
smbno10:22
smbboth un10:22
smbAs noted, I started that upgrade last week. Probably some upgrade path was not right then. And now it won't get out. But I can easily just start over10:23
dokosmb: please get the packages python-minimal and python from https://launchpad.net/ubuntu/+source/python-defaults/2.7.1-0ubuntu3/+build/209439110:24
dokothen do install these two deb's with sudo dpkg --unpack python*deb10:25
dokoand run sudo dpkg --configure --pending10:25
dokothen run an apt-get update && apt-get dist-upgrade10:25
smbdoko, python depends on python2.7 which is not installed10:30
dokosmb: ok, then get this too from the archive, and dpkg --unpack that too10:32
janimopitti: hello. Did my previous upload to libgphoto2 introduce a langpack related regression or was that always there (the one you just fixed) ?10:33
pittijanimo: langpack?10:33
pittijanimo: the one I fixed came from Debian10:33
smbdoko on the lp page you pointed me to, is that a special name? Cause I do not see a python2.7_...10:33
pittiI sent the fix there, too10:33
janimopitti: ok,, I got some mail related to Chinese .po10:34
janimoa few days after uploading10:34
pittijanimo: ah, ignore those10:34
pittiwell, they are bugs in the .po files10:34
pittibut we don't touch them in Ubuntu10:34
janimopitti: ok, thanks10:34
pittifrankly these are mostly spamn10:34
dokosmb: dpkg -l python2.7-minimal python2.7 ?10:35
smbii  python2.7-mini 2.7.1-1ubuntu1 A minimal subset of the Python language (ver10:36
smbNo packages found matching python-2.7.10:36
smbBut I guess I need to follow https://launchpad.net/ubuntu/+source/python2.710:36
smbnot python-default10:36
smbdoko, Ok, with manually installing python2.7, python (2.7) and python-minimal (2.7), the installation seems to sort itself out10:42
dokosmb: ok, thanks for confirming10:42
smbdoko, sure. thanks for sorting it out.10:43
janimoLaney: hello10:47
janimoLaney: ghc6 ftbfs on armel again, but this time it looks like a hung build after 18h+ and not package specific problems.10:49
Keybukthe funny thing about that exim security notice is that I'd given up with exim and purged it from my mail server about one minute before kees' mail arrived10:57
bdrungebroder: i released it already, but we can change it10:57
ebroderbdrung: I went ahead and wrote up the compatibility code, but whatever you prefer11:00
bdrungebroder: it's up to you to drop the compatibility code11:01
ebroderbdrung: Fine. If you're going to make me make the call, I'll leave it since it's already written11:02
bdrungok11:02
=== diwic is now known as diwic_afk
smb@pilot out11:09
=== udevbot changed the topic of #ubuntu-devel to: Archive: Open | Development of Ubuntu (not support, not app development) | #ubuntu for support and general discussion for dapper -> maverick | #ubuntu-app-devel for application development on Ubuntu | http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://wiki.ubuntu.com/HelpingWithBugs | Current Friendly Patch Pilots:
* smb -> lunch11:09
ebroderKeybuk: OOC, are you planning to have Upstart create implicit jobs from .service files for D-Bus activation, or make people that want to use it write proper Upstart jobs?11:10
Keybukimplicit jobs?11:10
ebroderI'm not sure what exactly I'd call them. "Use /usr/share/dbus-1 as a configuration source"11:11
Keybukyou mean jobs that are based on different configuration parsers?11:11
ebroderWhy yes, that's exactly what I was fumbling to say11:12
Keybukthe main use case for that is parsing /etc/init.d - which is already done by someone else11:12
Keybukso es11:12
Keybukyes11:12
KeybukI'm not sure there'd be a benefit to parsing /usr/share/dbus-1 as long as there's a dbus-daemon11:12
Keybuksince that's parsing it anyway11:12
Keybukyou'd need to somehow tell dbus-daemon to ignore its own configuration and send upstart events instead11:12
Laneyjanimo: really? looks built to me11:12
ebroderKeybuk: I had sort of figured that's what you would do. Otherwise what's the point of D-Bus activation?11:13
janimoLaney: I'll check again, I looked on Saturday and it said it waskilled fter a timeout of 15 minutes11:13
Laneyjanimo: don't know if it works though11:13
janimoLaney: I'll check, thanks11:13
Keybukebroder: the end goal of having init do is that it means dbus-daemon doesn't have to any longer11:14
Keybukand means we could get rid of dbus daemon in the future11:14
Keybukplus you can combine it with other things only init knows about, e.g. init can know that it can't activate that job right now, because its other requirements aren't met11:14
ebroderKeybuk: Sure. It seems like a useful intermediate state would be to do exactly what you described, and have D-Bus use Upstart for service supervision11:15
ebroderKeybuk: It doesn't seem like replacing the service starter with an Upstart event emitter should be that bad. Presumably dbus-daemon already has to separate service activation from actually delivering the message that spawned the service, since the message doesn't get delivered until the service finishes initializing itself11:17
Keybukright indeed11:21
KeybukI have that bit on my work items for this release11:21
ebroderKeybuk: I know, I've been following. I was just seeing if I understood the plan correctly11:21
Keybuknot expecting it to take more than a week to get working11:21
Keybukyeah initially the plan is to just have dbus emit the event though11:21
Keybuknot for upstart to also parse dbus's config for it11:21
bdrungebroder: the builder class needs an "update" function11:21
Keybukif that turns out to be necessary11:22
Keybukthen it's not hard11:22
ebroderbdrung: "update" as in "run apt-get update; apt-get dist-upgrade in this chroot"?11:23
ebroderKeybuk: Got it. Thanks11:23
bdrungebroder: yes. for pbuilder "sudo pbuilder update"11:23
bdrungebroder: and then a command line parameter for updating the builder before building11:24
ebroderbdrung: I'm happy to do that, but can I do it as a separate branch? The backportpackage branch is already starting to become sort of massive11:25
ebroderbdrung: I can open a ticket about it and assign it to myself11:25
bdrungebroder: yes11:26
Laneyjanimo: looks broken: see bug 689496, trying the patch there now11:51
ubottuLaunchpad bug 689496 in ghc6 (Ubuntu) "ghci can’t load unix package (/usr/lib/libpthread.so: invalid ELF header)" [Undecided,New] https://launchpad.net/bugs/68949611:51
janimoLaney: hmm so it builds packages but does not make them correct?11:51
Laneyright11:52
janimook11:52
janimoLaney: at least Setup.hs built, since some of the packages progressed on retry12:06
Laneyjanimo: It's in the RTS, so I hope the next version will fix it12:08
janimogreat12:08
pitticjwatson: would you mind if I test/apply the patch in bug 415038 before you consider it for Debian?12:16
ubottuLaunchpad bug 415038 in debconf (Ubuntu) "port GNOME frontend to GtkAssistant" [Medium,Triaged] https://launchpad.net/bugs/41503812:16
cjwatsonpitti: it's on my queue to review today-ish12:27
cjwatsonpitti: I'd prefer if I reviewed it before we applied it12:27
cjwatsonthere are all sorts of amusing things that could go wrong with a broken frontend12:27
pitticjwatson: ok; it needs some porting to the current version, so I'll do that and attach the patch12:27
cjwatsonnon-trivial porting?12:27
pittiyes12:28
cjwatsonhm, ok12:28
pittithe current version disables some window functions (close button, etc); should be "easy", but not just simple fuzz12:28
pittiand I generally want to test it with synaptic and s-c before we roll this out12:28
cjwatsonI want to test some of the more obscure features of debconf12:29
pitticjwatson: right; so it sounds we'll work on different things then12:30
cjwatsonI'll wait 'til you attach your current version before starting12:30
bdrungebroder: the "# TODO: Add "retry" and "update" option" was for adding a option to ask_for_manual_fixing() to update the builder from there12:32
pitticjwatson: is there a quick method to use "test_debconf.pl" or similar with the gnome frontend? (quicker than rebuild/install/synaptic install xdm)12:32
pittiaah12:34
pittiDEBIAN_FRONTEND=gnome samples/demo12:34
cjwatsonyes12:36
asacpitti: whats the status of the kernel SRU for bug 651370 ?13:08
ubottuLaunchpad bug 651370 in linux (Ubuntu Maverick) "ec2 kernel crash invalid opcode 0000 [#1]" [Medium,Fix committed] https://launchpad.net/bugs/65137013:08
pittiasac: it awaits regression testing from our QA team13:10
asacpitti: thanks. is that happening automatically? or should we send a reminder?13:11
pittiasac: reminder already happened13:11
asackk13:12
pitticjwatson: I ported janimo's patch, but it apparently doesn't return correct values now; I'll investigate further after lunch13:18
smb@pilot in13:22
=== udevbot changed the topic of #ubuntu-devel to: Archive: Open | Development of Ubuntu (not support, not app development) | #ubuntu for support and general discussion for dapper -> maverick | #ubuntu-app-devel for application development on Ubuntu | http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://wiki.ubuntu.com/HelpingWithBugs | Current Friendly Patch Pilots: smb
=== alecu_ is now known as alecu
=== dendro-afk is now known as dendrobates
=== jtechidna is now known as JontheEchidna
smbHm, for requests that ask for merging newer debian versions into ubuntu and there is a debdiff. Is there a simple way to get the new orig.tar.gz or do I have to search?14:00
cjwatsonyou can set up chdist to have a configuration pointing to Debian unstable, and then use 'chdist apt-get unstable -d source <packagename>'14:01
cjwatsonman chdist14:01
pittismb: I usually download it from the PTS (http://packages.qa.debian.org/pkgname) or my unstable chroot14:01
smbpitti, Ah thanks. Dumb follow up: what does PTS stand for?14:02
pittismb: Package Tracking System14:02
KeybukPackage Tracking System14:02
smbHeh, ok. Thanks14:03
Keybukcjwatson: I have a C/POSIX-ish query for you14:04
cjwatsonI always love those14:04
Keybukneed a competant lawyer to validate my idae14:04
Keybuk(and my spelling)14:04
Keybukwould this be wrong?14:04
Keybukunion {14:04
Keybuk    struct tm tm;14:04
Keybuk  int bits[7];14:05
Keybuk};14:05
cjwatsonwhat were you then planning to do with that union?14:05
Keybukwell, the idea is that bits[0] is then directly equivalent to tm.tm_sec14:06
Keybukbits[3] is tm.tm_mday14:06
elmoin the library with the revolver14:06
Keybukbits[6] is tm.tm_wday14:06
Keybukand use the array as a way of indexing the components of the struct14:06
cjwatsonmy gut feel is that that's the sort of thing where strict-aliasing trips you up14:06
cjwatsonwhich would be likely to cause optimisation difficulties aside from anything else14:08
cjwatsonthough are you asking if the alignment is guaranteed to be identical?14:08
Keybukah, but it's a union14:08
Keybukstrict-aliasing doesn't apply to unions14:09
Keybuk(that's kinda the point of them)14:09
cjwatsonum14:09
cjwatsonsort of14:09
penguin42is the ordering of that even vaguely guaranteed to be portable?14:09
Keybukpenguin42: that's kinda my question really14:10
KeybukI'm pretty sure that a struct of 7 ints is directly equivalent to an array of 7 ints14:10
Keybukand have put the book down14:10
cjwatsonhttp://cellperformance.beyond3d.com/articles/2006/06/understanding-strict-aliasing.html points out that using unions for that is technically undefined unless one of the punned elements is a char *14:10
cjwatson(although widely supported anyway)14:10
cjwatson(best explanation I've found of strict-aliasing and related issues)14:11
Keybukyeah I was reading that14:13
Keybukthe idea of my hack above is that by putting the tm and int array into the union14:14
Keybukyou're declaring to the compiler that they *do* share the same memory location14:14
Keybukit's not a pointer to the tm, it's the tm itself that's in the union14:15
cjwatsonOK, so.  structs are memory-ordered in declaration order (6.2.7.1(5,13)).  There may be unnamed padding within a struct (6.2.7.1(13)).  Arrays are "contiguously allocated" (6.2.5(20)), while structs are merely "sequentially allocated".14:16
cjwatsonI think you're in an area that in practice will always work but is not technically defined thus14:16
cjwatson(because for it not to work, you'd have to need alignment for ints, which would be a strange implementation choice)14:16
Keybukeven if you had alignment for ints, both the array and struct would have to align on that boundary, no?14:17
Keybukit'd only not work if the struct as a whole was aligned to something far wider than the int alignment14:17
cjwatsonthat's not how I read "contiguously allocated"14:17
Keybuke.g. each member of the struct was on an 8-byte boundary14:17
Keybukon a platform with a 2-byte boundary14:17
Keybukwhich I guess the compiler is allowing for14:17
cjwatsonarrays don't get to have internal alignment, even if that's what the platform would normally want14:17
Keybukcjwatson: sure they do, otherwise &(a[2]) wouldn't be valid14:18
Keybukyou can't take the address of an unaligned object14:18
Keybuks/compiling is allowing for/standard is allowing for/14:18
cjwatsonerr.  char a[16]; &(a[3]) is permitted on any platform, AIUI.14:18
Keybuksure because the alignment rules of char * are 114:18
smoserpitti, is there any thing blocking moving cloud-init into lucid-updates.  (0.5.10-0ubuntu1.5  bug 683890)14:19
ubottuLaunchpad bug 683890 in cloud-init (Ubuntu Lucid) "config-grub does not run" [High,Fix committed] https://launchpad.net/bugs/68389014:19
cjwatsonnone of this means that arrays ever have internal alignment14:19
cjwatsonshow me an example where they ever do14:19
Keybukok, hypothetical platform14:19
cjwatsonmy understanding is that that's explicitly forbidden by "contiguous allocation"14:19
Keybukthis platform has 8-byte alignment for int14:19
Keybukbut 4-byte ints14:19
cjwatsonotherwise, what does the difference between "contiguous" and "sequential" mean?14:20
Keybukif you had an array of two ints that was 4-byte aligned rather than 8-byte aligned14:20
Keybukthen:14:20
Keybuk  int *a = &(a[1]);14:20
Keybukwould be useless14:20
Keybukyou couldn't retrieve *a or assign to *a without breaking the platform's alignment rules14:20
smbpitti, Another of my stupid questions: in the maintainers section is a change from "Ubuntu Core Developers" to "Ubuntu Developers" correct?14:20
KeybukI was suggesting that the standard is allowing the struct to use a wider alignment if it needs to14:21
Keybuke.g. even if the int was 4-byte aligned, it could choose to 8-byte align the struct members14:21
Keybuk(perhaps it aligns them to the largest alignment of any member)14:21
cjwatsonthat works as a reductio ad absurdum, but only if the alignment of ints is the only reason why the platform might insert internal padding into a struct14:21
cjwatsonright, and *that* would break your model.  but in practice I doubt it will happen ...14:21
Keybukyeah, it'd be an odd thing to do :p14:21
dholbachsmb, we changed to "Ubuntu Developers" (https://wiki.ubuntu.com/DebianMaintainerField)14:21
Keybukdeliberately inserting space into structs would be curious14:22
Keybukbut maybe the standard isn't disallowing it14:22
Keybukthus the difference in terminology14:22
dholbachsmb, so we don't distinguish between packages that are in main or in universe14:22
cjwatsonthe standard doesn't forbid it, that's all.  and since you were asking for standards lawyering :)14:22
smbdholbach, Ok, thanks. Then it is correct as it is14:22
Keybuk(likewise in practice, no platform I've ever heard of uses anything other than word or int-sized alignment for int types)14:22
cjwatsonright14:22
Keybukso I guess it's a decision of whether to err on the side of "this works, and gcc doesn't even complain" or not14:23
cjwatsonwait, one other question14:23
cjwatsonPOSIX doesn't, AFAICS, guarantee the *ordering* of struct tm members14:23
cjwatsonit says "The <time.h> header shall declare the tm structure, which shall include at least the following members:"14:23
cjwatsonbut nothing about what order they come in, whether extra things can be inserted in between, ...14:23
Keybukah14:24
Keybukgood catch14:24
cjwatsonI imagine most new platforms copy and paste the sequence from POSIX, but I'm not sure about legacy platforms14:24
Keybukso glibc could randomly move that around between releases14:24
dokoLaney: how did you find out about the missing -lpthread in ghc6?14:24
Keybukwell, I don't really care about legacy platforms14:24
Keybukbut I do care about Drepper having a brain-wave and breaking code14:24
cjwatsonwhere "legacy" means "anything that was initially created before POSIX"14:25
Keybukthough that'd be an ABI break I guess14:25
cjwatson(er, yeah, slightly eccentric meaning of "legacy" there)14:25
smbdholbach, Hm, reading the link you posted. Should then, instead of XSBC-Original-Maintainer", the "Debian-Maintainer" be used?14:25
KeybukLinux's documentation explicitly declares the order of the fields, of course14:26
dholbachsmb, no XSBC-Original-Maintainer should be fine (update-maintainer from ubuntu-dev-tools usually does that for me :-))14:26
cjwatsonsmb: the poll said Debian-Maintainer, but we never used that in practice14:26
smbAh, ok.14:27
* smb is just looking at the changes from a debdiff14:27
cjwatsonKeybuk: right, and it would indeed be pretty hard (impossible?) to change without breaking existing binaries14:27
Keybukimpossible, certainly14:27
cjwatsonKeybuk: again, not sanctioned by standards but probably OK in practice14:27
Keybukyaeh, hmm14:28
Keybuklikewise you couldn't actually change the alignment of the struct :p14:29
Keybukso if it works now, it'll work forever14:29
Keybukand break when we roll out a new architecture14:29
cjwatsonyeah, nothing to stop a new arch having whatever field ordering it feels like - in theory14:29
Keybuk   * `A member of a union object is accessed using a member of a14:37
Keybuk     different type (C90 6.3.2.3).'14:37
Keybuk     The relevant bytes of the representation of the object are treated14:37
Keybuk     as an object of the type used for the access.  *Note14:37
Keybuk     Type-punning::.  This may be a trap representation.14:37
Keybuk..14:37
KeybukIt's a Trap!14:37
pittismoser: bug 683379 hasn't been verified yet, but otherwise it looks ok14:39
ubottuLaunchpad bug 683379 in grub2 (Ubuntu Natty) "user prompted twice on ec2 grub-pc upgrade from 1.98-1ubuntu7 to 1.98-1ubuntu8" [High,Confirmed] https://launchpad.net/bugs/68337914:39
Keybukcjwatson: hmm, reading a bit more14:40
Keybukapparently it's *recommended practice* for a compiler to align struct members by the most restrictive member14:41
Keybukrather than the individual member's own alignment14:41
Keybuk(c-faq)14:41
Keybukok14:41
KeybukI think I've persuaded myself out of doing that now14:41
smoserpitti, i had verified that last week, just forgot to mark the tag on that bug (since really the 3 bugs fixed were the same bug)14:45
smoseri'll go ahead and put a quick verify in though.14:45
kirklandstock natty + unity, dist-upgraded this morning;  everything across the top bar disappeared (ie, no netwok manager, no sound indicator, no messaging menu)14:49
kirklandsuggestions?14:49
kirkland(besides "reboot into classic desktop")14:50
penguin42kirkland: Do you see a seg of unity-panel-ser in your dmesg?14:50
kirklandpenguin42: yeah, there is a video related segfault14:52
penguin42kirkland: I'm trying to file abug against unity-panel-ser but apport is refusing most annyoyingly14:52
kirklandpenguin42: cool, thanks, subscribe me (kirkland) when you do14:53
didrockskirkland: uninstall indicator-datetime14:53
didrockskirkland: and restart compiz14:53
smbOk, so that debdiff looks mostly ok. What would be the usual mode to go on. Ask the person that has provided the debdiff to bzr pull request? Do it ourselves?15:00
kirklanddidrocks: okay, thanks, will do in a few minutes15:01
dokocjwatson: ping about bug #68883915:04
ubottuLaunchpad bug 688839 in dpkg (Ubuntu Natty) "proposal to turn dh_shlibdeps warning into an error" [High,New] https://launchpad.net/bugs/68883915:04
cjwatsondoko: I'm not confident with that - please bring it up with the Debian dpkg folks15:08
cjwatsondoko: I suspect that the proposed change would break plugins15:08
cjwatsonbut buxy will know more15:10
dokocjwatson: yes, it would. but making this an error for anything in /usr/lib would be a good thing15:10
dokohttp://bugs.debian.org/cgi-bin/pkgreport.cgi?tag=unresolved-symbols-so;users=peter.fritzsche@gmx.de15:11
seb128doko, it seems it would bring extra work for no real reason15:12
seb128can't we just set it as a soft goal for natty?15:13
seb128rather than forcing people to have to deal with builds breaking now15:13
dokoseb128: I don't consider unresolvable symbols at runtime a soft reason15:14
cjwatsondoko: I'm not going to take responsibility for making this change - please discuss it with buxy et al, they have more facts than I do15:15
dokocjwatson: I'll do15:15
cjwatsondoko: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=558872 is listed there and I happen to know that that library would break if you didn't allow that unresolved symbol15:15
cjwatsonit's a slightly weird library that requires the application to supply that symbol15:16
Laneydoko: misbuilds on armel15:16
Laneygeser reported it to me15:16
dokocjwatson: however I know buxy's reply. debian doesn't build with --no-add-needed, so why do you report it here ...15:16
Laneysorry, ftbfs not misbuilds15:16
cjwatsondoko: why not let buxy reply rather than putting words in his mouth?15:16
cjwatsonanyway, for me, Debian bug 558872 is an example of why this shouldn't be an error across the board - there has to be a way to override it15:17
ubottuDebian bug 558872 in src:libcgic "Resolve unresolved symbols in shared libraries" [Wishlist,Open] http://bugs.debian.org/55887215:17
dokowell, I got similiar replies from him on dpkg-gensymbols ...15:17
Laneydoko: Anyway, I think I can revert the explicit linking on -lpthread as it was only needed for the bootstrap afaics15:18
Laneythe real fix was a patch I grabbed from the upstream tracker together with the one in ubuntu315:19
dokoLaney: ahh, ok. just wanted to understand why it was needed15:19
Laneyhttp://hackage.haskell.org/trac/ghc/ticket/452315:20
Laneythat ticket15:20
RoAkSoAxdoko: I have a question that you might help me with :). I have a package that FTBFS because of the --no-add-needed thingy... So I modify Makefile.am, but the changes don't apply in build time. So, do I also have to manually modify the Makefile.in or should I run autoreconf to recreate the .in file?15:21
sconklin@pilot in15:22
=== udevbot changed the topic of #ubuntu-devel to: Archive: Open | Development of Ubuntu (not support, not app development) | #ubuntu for support and general discussion for dapper -> maverick | #ubuntu-app-devel for application development on Ubuntu | http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://wiki.ubuntu.com/HelpingWithBugs | Current Friendly Patch Pilots: smb, sconklin
dokocjwatson: could you outline why libcgic needs it?15:23
* JFo waves at the friendly neighborhood patch pilots15:24
cjwatsondoko: I followed up to the Debian bug15:24
cjwatsondoko: it's just how the library is designed, it expects the application to provide that symbol.  I wouldn't have done it that way myself, but it is how it is15:24
cjwatson(just one I happened to recognise off the list since I maintained the Debian package for it at some point in the past)15:25
dokook, thanks for the feedback15:25
KeybukUgh, WHY is this necessary?!  http://paste.ubuntu.com/543086/15:25
Keybuk*cry*15:26
cjwatsonI think the way it works is that libcgic provides main() itself and you provide cgiMain() which it calls after initialisation15:26
cjwatsonbit weird15:26
geserRoAkSoAx: I prefer to also patch the Makefile.in if the change is simple and the package doesn't already use autoreconf as I don't know exactly how to autoreconf it properly and not break it15:26
dokoRoAkSoAx: depends on the build system ...15:27
RoAkSoAxgeser: ok thanks ;)15:27
RoAkSoAxdoko: debhelper, package is openhpi15:28
=== dendrobates is now known as dendro-afk
hallyn_mvo: looking to do a vmbuilder upload.  soren points out you recently did an upload.  What was your process? Did you just make the changes to lp:vmbuilder, drop in a copy of debian/ from the package, and then dput the results?15:31
mvohallyn_: hello! I used the "packaging" brnach of vmbuilder for the build15:33
mvohallyn_: and in there just "bzr-buildpackage --merge"15:33
mvohallyn_: well "bzr-buildpackage --merge -S" to be exact :)15:33
kenvandinekirkland, did you get a chance to test unity with removing indicator-datetime?15:34
hallyn_mvo: so you didn't touch lp:vmbuilder at all?15:34
mvohallyn_: if you do another bzr snapshot, make sure to use the updated bzr revno in the change, bzr-buildpackage will checkout the right one based on this number (I love bzr-buildpackage)15:34
mvohallyn_: initially I didn't, I applied some patches into trunk now though, let me quickly check15:35
hallyn_mvo: i don't know what that means :)  if i do 'bzr co lp:ubuntu/natty/vm-builder', am  'doing a bzr snapshot'?15:35
mvohallyn_: oh, sorry. hold on a sec, I will try to express it better15:36
mvohallyn_: I did "bzr co lp:~vmbuilder-dev/vmbuilder/packaging" and then in there "bzr-buildpackage -S" to get the source upload (and without -S to get the deb of course)15:37
hallyn_interesting, trying15:38
kirklanddidrocks: that worked, thanks15:38
kirklandkenvandine: worked, thanks!15:38
mvohallyn_: the version number is current "0.12.4+bzr459-0ubuntu1" in packaging/changelog, if you want to create a upoad from revno480 just add a new version with revno480 in it15:38
mvohallyn_: its magic :)15:38
kenvandinehumm15:38
kenvandinekirkland, thx15:38
mvohallyn_: but note that I'm not really attached to this workflow, you are the maintainer, pick the one that you like best15:38
mvohallyn_: it was just what soren used (or at least it looked likt it, the branch was later a bit out-of-sync)15:39
hallyn_mvo: well on the one hand i don't want to decide until i try this way :)  OTOH i'm afraid i'll totally mess up the archive like i did with kvm15:40
hallyn_wait, now *why* is version # 0.12.4+bzr459-0ubuntu1 ?15:40
mvohallyn_: I'm happy to help and check before uploading etc15:42
hallyn_mvo: so the rationale is that you have multiple small packaging branches (one per distro), with one shared real source branch?15:43
mvohallyn_: I used this version because I did not want to make a new upstream release, afterall I'm not maintianer of the upstream branch15:43
highvoltageRiddell: hey, as an archive admin you probably noticed a bunch of zope and schooltool related packages uploaded to NEW the last week15:43
mvohallyn_: yeah, its frowned upon by some people (hey james_w ;) but its nice in the sense that its tiny15:43
hallyn_so is debian/watch ignored?15:44
james_whi mvo :-)15:44
hallyn_i'm trying to find a reference to lp:vmbuilder for the main source15:44
highvoltageRiddell: they are a bit of an exception to the usual process since they do not have needs-packaging bugs15:44
Riddellhighvoltage: I don't really care about needs-packaging bugs15:45
RiddellI'll process new tomorrow15:45
sorenhallyn_: What do you mean?15:46
mvohallyn_: no, its used if there is a upstream release15:46
highvoltageRiddell: ok, great! if there's anything you need with them, feel free to poke me or dholbach15:46
mvohallyn_: again, magic, if you put 1.2.23 there and that is a released version, it will automatically download the right file for you15:46
mvohallyn_: I love it15:46
mvohallyn_: aha, sorry. that one if in .bzr-builddeb/default.conf15:47
hallyn_oh, i see, so we do have to create a new release tarball of the lp:vmbuilder source in order to get a new release?15:47
mvohallyn_: there is a ref to the right branch15:47
hallyn_oh15:47
mvohallyn_: both works15:47
hallyn_ok15:47
mvohallyn_: for releases, just put a release version number and run bzr-builddeb and it will fetch the tarball from LP15:47
mvohallyn_: for bzr versions, it will checkout the right branch15:47
hallyn_yeah,'put a release number' into the packaging changelog?15:48
* hallyn_ likes magic, but onlymagic he can control15:48
* smb wonders how just some additional upload done by pitti to e2fsprogs helps to reduce the cd size. There did not seem to be a change to the package15:50
pittismb: I set debian/build_pressure to 10 kPa :)15:51
mvohallyn_: ok, so if there is 0.12.5-0ubuntu1 in the changelog, it will look at the debian/watch location15:51
smbpitti, archive magic...15:51
pittismb: seriously, natty's pkgbinarymangler removes upstream changelogs, trims debian changelogs, and optimizes PNG files now15:51
mvohallyn_: for a upstream version like this, I think the bzrXXX triggers the magic15:51
pittismb: so I made a list of all packages which we have on the CD, haven't been rebuilt in natty, and carry large changelogs or PNG files15:52
smoserpitti, so apologies for both nagging and being unfamiliar with process, but after I've verified a bug in -proposed , and its sat in -proposed for a week, what then happens ?15:52
pittismoser: it can go to -updates15:52
smoserso you (or someone else on SRU team) then pushes a button ?15:53
smbsmoser, is that in the kernel?15:53
pittismb: rightg15:53
pittismb: no, cloud-init15:53
smosersmb, i'm that one, no.15:53
smoserhttps://launchpad.net/bugs/651370 would be nice to see pop into -updates sometime this week though15:53
pittioh, smoser is a kernel :)15:53
ubottuUbuntu bug 651370 in linux (Ubuntu Maverick) "ec2 kernel crash invalid opcode 0000 [#1]" [Medium,Fix committed]15:53
smosersmb, that should have said "not that one, no".15:54
dokomterry: cluster MIR ping15:55
smoseranyone with more familiar wth recent changes in the python stack able to suggest what might have caused bug 68877315:55
ubottuLaunchpad bug 688773 in euca2ools (Ubuntu) "euca2ools give 'SignatureDoesNotMatch' error" [Undecided,New] https://launchpad.net/bugs/68877315:55
mterrydoko, hi15:55
smoser(euca2ools and boto -- the relevant packages have not recently changed)15:55
pitticjwatson: do you want me to commit http://launchpadlibrarian.net/60009278/debconf_1.5.36ubuntu1_1.5.36ubuntu2.diff.gz to the packaging branch? or did you abandon this branch now and use lp:ubuntu/debconf?15:57
pitti(this just caused some confusion here)15:57
dokomterry: what do you want me to look at?15:57
mterrydoko, oh, nothing anymore.  I had a question about a library correctness thing, but it got answered by others.  thanks15:58
cjwatsonpitti: I want to use something based on the bzr import of upstream svn, but that's currently failing so I can't15:59
cjwatsonpitti: right now I'm just not using revision control for debconf at all15:59
ari-tczewcjwatson: could you sponsor lilo merge?15:59
cjwatsonpitti: I'd rather you just attached it to the bug for now15:59
cjwatsonari-tczew: yes, let me drop everything and do that15:59
pitticjwatson: ok15:59
ari-tczewcjwatson: you've my ACK16:00
dokomterry: so ok to promote pacemaker, cluster-glue, cluster-agents, heartbeat, libasyncns, libesmtp, libnet, openhpi?16:01
mterrydoko, no, actually.16:02
mterrydoko, there was a licensing issue with pacemaker at least.  waiting on upstream.  I don't know about all those bugs16:02
mterrydoko, but bug status is up to date for my bugs anyway16:03
dokomterry: so we need to promote these all at once, and it's needed to build for python2.7 ...16:12
mterrydoko, sorry, I don't follow.  You're saying the cluster stack is needed for python2.7 or that the cluster stack hasn't yet been rebuilt for python2.7?16:12
dokomterry: the latter. ocfs2-tools needs to be rebuilt16:13
mterrydoko, OK.  So what's the issue?16:13
dokomterry: compoment mismatches16:13
RoAkSoAxmterry: im working on the licensing thing alreayd. patches have been accepted upstream16:14
mterryRoAkSoAx, awesome16:14
=== beuno is now known as beuno-lunch
mterrydoko, I'm just not understanding how the rebuild and the MIR are connected is all16:14
dokoseb128, didrocks: any chance to look at the libgda4 build failure?16:16
dokomterry: $ apt-cache show ocfs2console|grep Depends16:16
dokoDepends: libblkid1 (>= 2.15~rc2-1ubuntu1), libc6 (>= 2.7), libcomerr2 (>= 1.01), libglib2.0-0 (>= 2.12.0), libuuid1 (>= 2.16), python (<< 2.7), python (>= 2.6), python-support (>= 0.90.0), ocfs2-tools (= 1.4.3-2), python-gtk216:16
dokomterry: uninstallable and cannot be rebuilt: https://launchpad.net/ubuntu/+source/ocfs2-tools/1.4.4-3build116:16
seb128kenvandine, mterry: ^ how busy are you? could you look at the gda build issue?16:17
seb128it's gir related and you know those issues the best atm I think16:17
mterryseb128, I'm doing unity stuff right now, but nothing important16:17
mterryseb128, can look16:17
kenvandineseb128, i really want to get to the bottom of this unity/indicator-datetime problem16:17
seb128kenvandine, mterry: ok thanks, let's say mterry can do it, thanks!16:18
seb128doko, ^16:18
mterrydoko, ah, I see.  So someone promoted ocfs2-tools too soon?16:18
dokosaw it, thanks16:18
kenvandinethx mterry16:18
dokomterry: no, it always was in main, just gained new dependencies16:19
mterrydoko, well we can either depromote it, promote the cluster stack, or wait.  Sounds like you don't want to wait.  I can't speak for how ready the rest of the cluster stack is.  I'd have to look at other MIR bugs16:19
mterrydoko, fine, then gained new dependencies too soon.  :)16:19
seb128doko, can we promote geoclue? it's blocking an indicator build which is breaking unity...16:20
dokoseb128: mterry did the review, I did ask kees / security at the daemons too16:20
dokoon Friday he said to do it on Monday (today)16:21
seb128doko, right, what I'm asking is if we really need to wait on security16:21
seb128unity is broken until we sort that situation16:21
seb128so we either need to roll back to old versions not using geoclue in some way16:21
seb128ot to get geoclue promoted16:21
dokoplease can we wait until tonight? I think I should write something about pre-promotions ...16:22
seb128doko, well we can, unity is just broken for everybody running natty while we wait16:23
seb128no indicator is loading at all16:23
pittiI thought we could rollback to the previous version for now?16:23
seb128kenvandine, can you upload a new.is.old version then?16:23
kenvandineyup16:24
seb128pitti, well geoclue and ofono got mir review acks from mterry16:24
seb128pitti, so promoting seemed the easier way forward16:24
seb128but seems we need a security team review as well before16:24
pittiseb128: ah, ok; didn't know the full story16:24
kenvandineseb128, how do i handle that in the bzr branch considering the branch is also merged from the upstream source?16:24
seb128kenvandine, don't use the vcs16:24
kenvandineok16:25
seb128just take the old version from launchpad, rename it and upload16:25
kenvandinefigured16:25
kenvandineand with the next upload, i'll just include the changelog entry?16:25
seb128or not as you want16:25
seb128but yeah you can16:25
kenvandineok16:25
kenvandinewill do16:25
mterrydoko, at least cluster-agents seems unreviewed16:28
mterrydoko, and libasyncns doesn't have a MIR filed?  Is it part of the cluster stack?  So pacemaker has an easily-fixable license oddity change coming, I can review cluster-agents today.  That would leave libasyncns16:33
dokomterry: heartbeat, libnet and openhpi were already in main, do you want the dh_makeshlibs -V thing happen before the promotion?16:33
dokomterry: ok, preparing a MIR for libasyncns. can you review it later?16:34
mterrydoko, for cluster-glue?  That change already happened16:34
mterrydoko, sure16:34
dokoohh, right16:35
=== ssj6akshat|schoo is now known as ssj6akshat
kenvandinekirkland, could you please get me a full package list?16:42
kirklandkenvandine: on my system?16:42
kenvandinekirkland, i am trying to narrow down the root cause of the unity panel not loading16:42
kirklandkenvandine: k16:42
kenvandinethe one with the indicator-datetime problem16:42
kenvandinethx16:42
kenvandinekirkland, today's live image works16:43
kenvandineso something installed is causing the problem16:43
kirklandkenvandine: https://pastebin.canonical.com/40874/16:43
kenvandinethx16:43
kirklandnp16:43
=== ssj6akshat is now known as akshatj
=== dendro-afk is now known as dendrobates
smbpitti, cjwatson Question about style and process:16:55
smbbug 68141816:55
ubottuLaunchpad bug 681418 in e2fsprogs (Ubuntu) "Please merge e2fsprogs 1.41.12-2 (main) from debian unstable (main)" [Wishlist,New] https://launchpad.net/bugs/68141816:55
pittilooking16:55
pittismb: if you mean my extra changelog, feel free to ignore it; it was just a no-change rebuild16:56
smbThe debdif was mostly ok. The guide says sort of "ask for a merge request"16:56
smbFor training I created a branch (with your null change)16:56
smbbut Ted says he has the next version ready soonish16:57
ari-tczewsmb: could you have a look on bug 669363 ?16:57
ubottuLaunchpad bug 669363 in lilo (Ubuntu) "Please merge lilo 1:22.8-8.3 (main) from Debian unstable (main)" [Wishlist,Confirmed] https://launchpad.net/bugs/66936316:57
smbSo would we want to upload the current version anyway? Or wait? And if taking the current version, offer my bzr branch although it is just reflecting the other guys merge work.16:58
smbari-tczew, I can, but bear with that I am not too familiar with the process yet and thus slow. :)16:59
pittismb: if your branch is not more than just the attached debdiff applied, I don't think it buys much17:00
ari-tczewsmb: not good17:00
benstehi, no one in #ubunut and #ubuntu-de could yet answer me how to mount smb shares in 10.10 unity17:00
pittismb: I don't think there's anything wrong with uploading it now, if it works and was tested17:00
benstei did it back in 10.4 via gvfs and nautilus cklicking the places menu17:01
pittismb: there was no definitive "tomorrow" for the new releasey17:01
smbpitti, The only small change would be to drop the vcs-git from the control.in as well17:01
benstesomeone here who knwos more about how unity works ?17:02
BlackZsmb: I was about to update the debdiff but I seen you have been more fast than me :) thanks17:02
=== tkamppeter_ is now known as tkamppeter
smbBlackZ, Hi, well actually I have not done a separate debdiff file (thought that would be quick). I was interested in learning how the bzr merge request would work17:04
smbAs I was neither sure whether the right way is just a debdiff or have that merge request way17:05
BlackZsmb: yeah, I noted that you haven't update the debdiff but I seen you linked a bzr branch to the bug report (if it contains the corrections it is fine)17:06
BlackZsmb: either way are fine17:06
smbpitti, Which happens if you let kernel devs be patch pilots without mandatory training sessions before. ;)17:06
smbBlackZ, I did?17:06
smbMust be something magic going on...17:06
cjwatsonari-tczew: stoppit, I'm doing it.17:06
cjwatsonsmb: no need to deal with lilo17:07
smbcjwatson, Ok, cheers17:07
BlackZsmb: maybe it's because you close that bug in your bzr revision?17:07
smbBlackZ, Oh I see. Just having done the debcommit to the bzr branch and then pushing it, created the link. :)17:07
smbBlackZ, Actually I wanted to ask whether that is a bit rude before doing the link, but automatic things decided that for me.17:08
BlackZheh17:08
=== beuno-lunch is now known as beuno
cjwatsonari-tczew: in future, please don't bother to ask me to do something if you're just going to ask somebody else about an hour afterwards even when I've said yes.17:10
cjwatsonit's a waste of everyone's time ...17:10
smbpitti, So the only difference between the attached debdiff and my branch is your changelog entry and the removed vcs-git in control.in17:10
cjwatson(it's done now)17:10
ari-tczewcjwatson: I got you response as sarcasm17:10
ari-tczew(and I re-reply as the same)17:11
cjwatsonwell, I *was* doing other things, but nevertheless I did it17:11
=== dendrobates is now known as dendro-afk
tumbleweedcan an archive-admin please reject the googleearth-package maverick-proposed upload?17:20
smbBlackZ, Ok, as it is actually the same code with just minor changes I just made a merge request for it to get the process ahead17:22
BlackZthanks for you work, smb ! :)17:22
smbBlackZ, NP. Actually you did most of it.17:22
dokomterry: hmm, libasyncns is unrelated17:24
mterrydoko, k, cool17:25
dokomterry: well, noticed too late, already updated ...17:25
=== dendro-afk is now known as dendrobates
* smb believes to have done enough chaos for one day17:28
smb@pilot out17:28
=== udevbot changed the topic of #ubuntu-devel to: Archive: Open | Development of Ubuntu (not support, not app development) | #ubuntu for support and general discussion for dapper -> maverick | #ubuntu-app-devel for application development on Ubuntu | http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://wiki.ubuntu.com/HelpingWithBugs | Current Friendly Patch Pilots: sconklin
keesdoko, seb128: lost to backscroll... which thing did you want me to look at?17:28
seb128kees, geoclue17:29
seb128or ofono17:29
seb128kees, I think it's ofono (whichis a dep of geoclue)17:29
keesokay, give me a bit, and I'll look it over.17:30
dokokees: the daemons in ofono and gypsy, ohh and maybe a security for each unity upstream version? ;p17:30
dokos review even17:30
SpamapSKeybuk: around? I'm wondering about abstraction of events in upstart .. seems like 'stop on runlevel [016]' isn't exactly doing what we want (see bug 688541)17:34
ubottuLaunchpad bug 688541 in mysql-5.1 (Ubuntu) "race condition on shutdown (leads to corrupted fs)" [High,Triaged] https://launchpad.net/bugs/68854117:34
Keybukhey17:34
Keybukabstraction of events?17:34
SpamapSYes, like, instead of having people put 'stop on runleve [016]' they'd put 'stop on stopping-network-services'17:35
RoAkSoAxmterry: would you like to review pacemaker's diff before I upload?17:36
=== zyga is now known as zyga-gone
SpamapSKeybuk: I'm just concerned that by being so explicit in the events of regular services, we are making a lot of work for ourselves later on.17:37
KeybukI don't understand, sorry17:38
Keybukcould you expand a little?17:38
dokomterry: #68976617:39
=== zyga is now known as zyga-vaio
SpamapSKeybuk: I'm suggesting hiding common stop on / start on scenarios behind other events or other jobs... so an example would be mysql .. which does 'start on (local-filesystems and net-device-up IFACE!=lo)' and 'stop on runlevel [016]' .. squid also works this way. But it may be that we need to actually have them stop on something else that commonly happens.. like have them stop *before* unmounting all filesystems .. it would be easier to make th17:44
Keybukthat's what you're supposed to do17:44
Keybukyou're supposed to make jobs that encapsulate a comon start on/stop on combination17:45
Keybukand then use other jobs off that17:45
Keybukrather than copy/pasting lines between jobs17:45
SpamapSI'm so glad you said that. :) I thought maybe I was crazy thinking this way. ;)17:45
Keybukone solution to the rc0/6 race is to have the umountfs job start with17:48
Keybuk  initctl emit unmounting-filesystems17:48
Keybukthen if you had any job (or state) wiht stop on unmounting-filesystems17:48
Keybukit'd wiat17:48
SpamapSheh.. GMTA https://bugs.launchpad.net/ubuntu/+source/sysvinit/+bug/688541/comments/517:48
ubottuUbuntu bug 688541 in mysql-5.1 (Ubuntu) "race condition on shutdown (leads to corrupted fs)" [High,Triaged]17:48
SpamapSKeybuk: basically, local-filesystems needs a mirror event.17:49
Keybukyes17:50
mterryRoAkSoAx, uh, if it fixes the license issue, I can just review after the fact17:55
mterryRoAkSoAx, should be fine after that is fixed17:55
mterrydoko, so do you want me to review libasyncns after all?  I thought you said we didn't need it?17:56
dokomterry: no, not now17:57
RoAkSoAxmterry: there's one file that hasn't been changed given that upstream wasn't unsure to do that because they didn't write that code in particular18:00
RoAkSoAxmterry: other than that, everything was changed18:00
RoAkSoAxs/changed/fixed18:00
seb128who is doing syncs?18:04
seb128whoever that is it would be nice to -S experimental pyclutter18:05
seb128-S experimental telepathy-glib18:05
dholbachok my friends18:08
dholbachI call it a day18:08
dholbachsee you18:08
=== sam---___---- is now known as sam-_-
pittimdke: here by chance?18:20
brycehheya pitti18:32
pittihi bryceh18:33
brycehpitti, question on bug #680811, which is in maverick-proposed for sruing...18:33
ubottuLaunchpad bug 680811 in xorg-server (Ubuntu) "incorrect maximise behaviour with twinview" [Undecided,Incomplete] https://launchpad.net/bugs/68081118:33
brycehpitti, cnd has another xserver patch in the hopper, and we're wondering if it should be numbered 7.2, or if the above sru is not going to go through soon in which case it should be 7.1 ?18:34
ScottKbryceh: If it's in -proposed already you'll have to bump the version number since soyuz already has marked 7.1 used.18:35
cndI'd also like to know why it's 7.1 and not 8?18:35
brycehScottK, ok18:35
ScottKcnd: We ~always do that for post-release updates18:35
brycehcnd, the 8 would presumably be in natty18:35
pittibryceh: you always have to bump the version number18:35
cndahh18:36
cndok18:36
pittibryceh: the only difference is that you'd build with -v<version_in_updates> if you stack multiple -proposed uploads on top of each other18:36
brycehcnd, so if someone upgraded, they can safely upgrade 7.1 -> 8, whereas if both natty and maverick had 8 it wouldn't upgrade them18:36
=== al-maisan is now known as almaisan-away
brycehpitti, ok thanks18:37
Davieyslangasek: Hi... Are you doing Archive Admin stuff today?18:37
=== drspeedo is now known as drspeedo|away
c_kornwhen do the nvidia drivers in the repository get updated? there is a race condition bug which has been fixed in 260.19.21. it causes some games to crash: http://www.nvidia.com/object/linux-display-amd64-260.19.21-driver.html19:27
=== drspeedo|away is now known as drspeedo
SpamapSc_korn: lots of bug reports here, but I'd guess it would need to be reported and triaged if it hasn't already:  https://bugs.launchpad.net/ubuntu/+source/nvidia-graphics-drivers19:44
ari-tczewc_korn: ask tseliot19:59
c_kornhm, just filed bug 68983220:01
ubottuLaunchpad bug 689832 in nvidia-graphics-drivers (Ubuntu) "[package request] Version 260.19.21 fixes race condition" [Undecided,New] https://launchpad.net/bugs/68983220:01
RoAkSoAxmterry: uploaded!20:08
RoAkSoAx(pacemaker)20:08
mterryRoAkSoAx, oh nice, I'll recheck it20:08
RoAkSoAxmterry: awesome! Thanks... ;) Btw.. there's only one single file under lib/ without the change given that upstream was not confortable changing that file license header due to that it was not written by them20:10
mterryRoAkSoAx, but they are comfortable shipping it without knowing the license?20:11
sabdflRoAkSoAx: congrats on the graduation20:11
RoAkSoAxsabdfl: thank you! :)20:11
slangasekDaviey: I can do some, but haven't been diligent about doing so routinely; anything in particular that you need?20:11
Davieyslangasek: I'm just wondering why the libvirt lucid SRU has been baking for some time now..   I guessed that perhaps the AA's were uneasy, or had questions about it.20:12
RoAkSoAxmterry: upstream is. I mean it is not a license change per se but typo on the header. That file used to be in heartbeat afaik (before the split).20:13
Davieyslangasek: to -proposed, currently in Unapproved20:13
mterryRoAkSoAx, but do they know what license it should be?20:13
slangasekDaviey: that's a separate question from AA duty20:13
Davieyslangasek: I thought these days it was largely one and the same.20:14
slangasekDaviey: the SRU team decides on SRU acceptance, that's not at all the same as archive admin20:14
RoAkSoAxmterry: yes, it is GPL 2.1+, when should be LGPL 2.1+, but upstream doesn't feel confortable doing the change given that the code wasn't written by them20:14
mterryRoAkSoAx, !?  That doesn't make any sense, but OK.  As long as we note that in debian/copyright20:14
slangasekDaviey: some of us are on both teams, but SRU accepts are definitely not part of the regularly scheduled AA duties20:15
RoAkSoAxmterry: it is this file: lib/plugins/lrm/raexecstonith.c20:16
RoAkSoAxmterry: and it has the same error as other files. It says it is GPL2.1+ when should say LGPL2.1+20:16
RoAkSoAxand as I said, upstream is aware but he said "i don"20:16
Davieyslangasek: Oh aye... based on what i've been seeing, the divide has become murky.20:17
RoAkSoAxand as I said, upstream is aware but he said "i didn't write that file so I don't feel confortable doing the license change"20:17
RoAkSoAxanyway, bbl20:17
slangasekDaviey: ehm, there's really no reason that should be the case.  Only a minority of the AAs are on the SRU team, and other members of the archive team aren't *authorized* to make decisions about accepting packages into -proposed...20:18
slangasekanyway, looking at libvirt now with my ubuntu-sru hat on ;)20:19
Daviey\o/20:19
Davieyslangasek: you rock :)20:19
slangasekDaviey: yeesh, someone take the parentheses gun away from that upstream20:23
slangasekDaviey: accepted20:23
Davieyslangasek: heh, great - thanks!20:23
keesslangasek: can I borrow you for upstart debugging for a moment? does a "task" go through "starting" like for "services" ?20:25
keesslangasek: and how I can get a dump of the emitted events during startup?20:27
slangasekkees: yes, there's a 'starting' phase, whether it's a task or a service20:32
slangasekkees: dump of the emitted events - boot with --verbose, grab the log and postprocess20:32
keesslangasek: really? because nothing seems to be triggering my job20:32
keesokay, will try20:32
keeswait, boot with "verbose" not --verbose? kernel cmdline?20:32
slangasekno, --verbose20:33
slangasekon the kernel commandline20:33
slangasekwhat's the job look like?20:33
kees/etc/init/networking.conf (the task) and /etc/init/network-interface-security.conf (the prereq)20:34
keesthe latter isn't actually running :(20:34
kees(testing for apparmor in lucid -proposed uncovered this existing race)20:34
slangasekodd20:34
keesyeah, building log...20:35
keeswhich log does it spew to?20:35
slangasek/var/log/syslog, IIRC20:36
keesoh yes20:36
keesslangasek: utterly no mentio of /etc/init/network-interface-security.conf20:41
kees:(20:41
slangasekkees: but you do see 'network-interface' or 'networking' or something in there?20:42
keesslangasek: yeah, I watch both network-manager and networking make their way through their entire lifecycles20:43
bigoncjwatson: https://bugs.edge.launchpad.net/ubuntu/+source/gdk-pixbuf/+bug/665768 do you have a list of these pkg?20:43
ubottuUbuntu bug 665768 in gdk-pixbuf (Ubuntu) "libgdk_pixbuf-2.0.la missing from packages" [Undecided,Confirmed]20:43
keesand n-i-s is shown as "start/running" O_o20:43
keesslangasek: wait, I liked. I think I was accidentally searching for networkING-inter...20:44
keesone moment...20:44
=== drspeedo is now known as drspeedo|away
keesslangasek: I'm totally baffled. http://paste.ubuntu.com/543279/ is upstart just not logging events in early boot?20:55
keesslangasek: it seems like jobs aren't waiting for pre-start stanzas to finish before unblocking.20:57
keesslangasek: i.e. n-i-s runs when I expect it to, but the jobs it has listed aren't waiting for its pre-start to finish20:58
slangasekkees: IIRC, upstart is supposed to have a ring buffer internally for logging, which it dumps once syslog comes up; that, or Keybuk and I *talked* about having such a thing and it never got implemented, not sure21:04
keesslangasek: looks like it was never implemented :(21:05
slangasekkees: since I don't see any jobs on my system that trigger a dump on rsyslog start, yeah, I guess so21:05
keesslangasek: if jobB starts on starting jobA, and jobB has a pre-start script, isn't jobA supposed to wait until jobB's pre-start has finished before jobA can move to its own pre-start?21:06
slangaseker, I'm not sure21:07
keesthis design was predicated on that, based on what Keybuk told me. :(21:07
slangasekI'd have to check the code; it might be jobB prestart, jobA prestart, job A script, job B script21:07
keesOh! nevermind. I think I see the problem.21:08
KeybukThis is one of the many things that scares me about systemd21:09
Keybukhttp://cgit.freedesktop.org/systemd/tree/src/modules-load.c21:09
keesI have    start jobX on (starting job1 or starting job2 or starting job3).  job1 -> starting, jobX -> starting, job2 -> starting & runs. jobX -> finishes, job1 -> finishes21:09
keesKeybuk: just the person I was looking for!21:09
Keybukkees: hah21:09
KeybukI felt a disturbance in the force21:10
keesKeybuk: heh. I'm trying to debug a race. it's /etc/init/network-interface-security.conf21:10
keesKeybuk: it seems like if any 1 of those 3 jobs it waits for start, it will start, but if it takes a while, the other two might start and get to running before n-i-s finishes.21:11
Keybukyeah21:11
Keybukonly the first of the jobs in the "or" to start gets held up21:11
keeswhat are my options for making them all wait for it to finish?21:12
Keybukhuh, doesn't look like there's a bug for that one21:12
Keybukdon't use starting, instead use started network-interface-security in the named jobs?21:13
Keybukhttps://bugs.launchpad.net/upstart/+bug/56886021:13
Keybukah there it is21:13
ubottuUbuntu bug 568860 in upstart "init: "before" functionality should block all services mentioned, not just the first" [Wishlist,Triaged]21:13
KeybukI knew it was filed somewhere21:13
keesKeybuk: so I need to modify each of the jobs I want to block?21:13
Keybukyes :-/21:13
keesand they will trigger the spawning?21:14
Keybuk"the spawning" ?21:14
keesah, nevermind. n-i-s would just become "start on virtual-filesystems"21:14
Keybukis n-i-s indemponent and atomic?21:15
Keybukie. could you cope with it being run *again* if the second event happened?21:15
keesit can, but it's a waste of time21:15
Keybukbecause in that case you could stick instance $JOB in21:15
Keybukyou'd get a copy for each of the start on clauses21:15
keesI just want the one.21:15
Keybukyeah21:16
Keybukjust thinking of solutions for you for the time being21:16
slangasekkees, Keybuk: or since network-interface-security is idempotent, you could 'instance' it?21:17
Keybukslangasek: ah, we appear to have reached the middle of this conversation21:17
slangasekah21:17
KeybukI just suggested that21:17
* slangasek whistles21:17
keeshow does instances help this, exactly?21:17
Keybukbut I feel really quiet warm, comfortable and fluffy knowing that a signficant portion of my brain has been backed up into yours21:17
slangasekkees: you get a separate n-i-s job for each of the triggering jobs21:18
Keybukjust be caseful, I may need my katra back one day21:18
slangasekheh21:18
Keybukcaseful? careful!21:18
keesslangasek: hm. that's the least intrusive change, but it's a small waste of time.21:18
Keybukwaste of time > bad security21:19
keescould skip later runs by examining something in /var/run ...21:20
Keybukyeah21:20
ebroderkees: By the way, I think Upstart's output from before rsyslog is up will go to /var/log/boot.log21:20
Keybukkees: but this bug is definitely being fixed21:21
Keybukthe trouble is it isn't a bug so much as a missing feature21:21
Keybukstart on .. or .. was never designed to block both :p21:21
* kees nods21:21
keesthis race only happens rarely, but I still want to fix it.21:22
keesso... just add the line "instance $JOB" ?21:22
slangaseknot quite21:22
slangasekbecause network-interface itself can have multiple instances21:22
Keybukthat has $INTERFACE right?21:23
Keybukyou could do instance $JOB$INTERFACE21:23
Keybuk:p21:23
=== dendrobates is now known as dendro-afk
slangasekright21:23
slangasekI was just looking up the variable :)21:23
ebroderWill instance $JOB/$INTERFACE work? Because that looks slightly less dumb21:23
Keybukif you prefer21:23
Keybukit's just an arbitrary string21:24
keesbut it's only to use $INTERFACE when some jobs don't use it? (i.e. network-manager.conf)21:24
=== dendro-afk is now known as dendrobates
Keybukit's "" if not21:25
gesercjwatson: can you please also add "lapack" to Sylvestre Ledru's (sylvestre) PPU (see his recent mail to devel-permission).21:29
keesebroder: ah-ha! yes, it is in boot.log. that'll make this easier to debug. :)21:29
ebroderkees: Though I think there might be a race between when plymouth dumps its log and when rsyslog starts, so there might be a window where you don't get logs21:30
keesinit: Failed to obtain network-interface-security instance: Unknown parameter: INTERFACE21:31
keesKeybuk, slangasek: adding "instance $JOB/$INTERFACE" failed as above, and reducing it to "instance $JOB" had no change in behavior.21:36
keesI take it back, it does work, just not with the "network-interface" job due to the lack of $INTERFACE21:38
keesis there some way to access it?21:39
keesKeybuk: so, how do I access $INTERFACE? it seems like it's isolated to inside the job, and I can't see it from $JOB.21:44
Keybukkees: ?21:54
Keybukoh, unknown parameter21:54
Keybukwhat's that shell thing21:54
Keybukinstance ${JOB:}${INTERFACE:}21:55
Keybukor something21:55
Keybukuse other value if unset21:55
Keybuk${JOB:-}${INTERFACE:-}21:55
Keybukthat one21:55
* kees tries it21:57
keesKeybuk: it still doesn't see it.21:59
keesinit: network-interface-security (network-interface/) goal changed from stop to start21:59
Keybukright21:59
Keybukhmm21:59
Keybukwhat was the event again?21:59
keesinit: network-interface (eth0) state changed from waiting to starting21:59
keesI need $JOB_WITH_INSTANCE21:59
Keybukno, no21:59
kees?22:00
Keybukyou're starting on what terms?22:00
Keybukcan you paste the start on line22:00
Keybukoh22:00
Keybukwait22:00
Keybukit's not using net-device-added is it?22:00
keesstart on (starting network-interface22:00
kees          or starting network-manager22:00
kees          or starting networking)22:00
keesnope22:00
Keybukyeah22:00
Keybukd'oh22:00
Keybukok22:00
Keybukyou need to change /etc/init/network-interface.conf22:00
Keybukadd22:00
Keybukexport INSTANCE22:00
keesah-ha22:01
Keybuksorry, had it in my head that your job worked directly off net-device-added22:01
Keybukso yeah, network-interface needs to export the INSTANCE thingy22:01
Keybukwhich really begs the question why export isn't the default22:01
Keybukthat might be useful, or something22:01
Keybuk(I know why it isn't, but meh)22:01
* kees tries again22:01
kees\o/22:02
keesinit: network-interface-security (network-interface/eth0) state changed from waiting to starting22:02
Keybukok, now try starting something like network manager22:03
Keybukyou should see22:03
Keybuk(network-manager/)22:03
keesDec 13 22:02:01 sec-lucid-amd64 init: network-interface-security (network-manager/) goal changed from stop to start22:03
keesyup22:03
Keybuksweet22:03
keesexcellent!22:03
keesKeybuk, slangasek: thank you! :)22:04
sconklin@pilot out22:24
=== udevbot changed the topic of #ubuntu-devel to: Archive: Open | Development of Ubuntu (not support, not app development) | #ubuntu for support and general discussion for dapper -> maverick | #ubuntu-app-devel for application development on Ubuntu | http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://wiki.ubuntu.com/HelpingWithBugs | Current Friendly Patch Pilots:
RoAkSoAxkirkland: just made an upload for TestDrive. Apparently just needed a rebuild for python 2.7. Please give a try if you can and let me know!22:30
=== dendrobates is now known as dendro-afk
kirklandRoAkSoAx: cool, will do22:47
SpamapSanyone know why lp:ubuntu/sysvinit is so far behind the actual package?22:50
micahgSpamapS: http://package-import.ubuntu.com/status/sysvinit.html#2010-06-16%2012:21:03.21248322:52
SpamapSmicahg: weird, who can resolve this?23:06
micahgSpamapS: you can file a bug against the udd project I think (lp.net/udd)23:07
SpamapSmicahg: thanks, bug 513277 seems to be an older one. ;)23:15
ubottuLaunchpad bug 513277 in Ubuntu Distributed Development "bzrlib.errors.DivergedBranches: These branches have diverged when importing" [High,Triaged] https://launchpad.net/bugs/51327723:15
=== bjf is now known as bjf[afk]
=== dendro-afk is now known as dendrobates
hallyn_anyone deal a lot with debirf?23:34
hallyn_i'm curious about the fact that it uses debootstrap, which must be run as root, but complains when run as root...23:35
=== dendrobates is now known as dendro-afk
=== dendro-afk is now known as dendrobates

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