/srv/irclogs.ubuntu.com/2014/07/10/#ubuntu-devel.txt

hallyndesrt: hey - do you know of any other sample code that would show how to deconstruct a dbus msg as gvariant which has a(sv)?  I don't seem able to pull it off successfully, and most tutorials seem to stick to 'as'00:08
hallyn(i'm trying with http://paste.ubuntu.com/7772700/ )00:08
* hallyn suspects desrt is in a currently-asleep tz :)00:10
desrthallyn: a(sv) or a{sv}?02:35
hallyndesrt: a(sv)02:58
desrtweird!02:58
desrtwhat is this?02:58
desrtlike, what data is in there?02:58
desrtone does not typically see the a(sv) type....02:58
hallyndesrt: it's the StartTransientUnit from systemd.  Each element of the array has s for the description, then v is just an s with the value02:59
hallynseemed rather silly to me, suppose it's meant to be future-proof02:59
desrttypically you use a{sv} unless you really care about the order02:59
desrtwhich is difficult to imagine in this situation...03:00
desrtanyway... did you figure it out?03:00
hallyn     "<arg name='properties' type='a(sv)' direction='in'/>"03:00
hallynnope03:00
hallyni mean i get a valid list, but the n_elements is always 003:00
desrtso what do you have there?03:00
desrtyou're implementing this inside the shim?03:01
hallynyeah03:01
hallynhttp://paste.ubuntu.com/7773201/03:01
desrtso you have name mode properties aux03:01
hallynthat's the chunk i'm using right now03:01
desrtso typically you whack out all the parameters at once03:02
desrtsomething like so:03:02
desrtconst gchar *name, *mode;  GVariantIter *properties, *aux;03:02
desrtg_variant_get (parameters, "(&s&sa(sv)a(s(a(sv)))", &name, &mode, &properties, &aux);03:02
hallynthen properties will be an iter?03:03
desrtyes03:03
desrt  while (g_variant_iter_loop(iter, "(s@v)", &str1, &v)) {03:03
desrtthis is weird03:03
hallyni dumped aux actually bc systemd in 208 wasn't yet sendin git03:03
desrtif you do that then the "v" will contain a value of type 'v'03:03
desrtyou want only to have "(sv)" or even better "(&sv)" with a const gchar *str103:03
hallynok i'll try that, thx.  (stuff going on so will take a few mins)03:04
desrtg_variant_iter_n_children() yuck03:06
desrti can't believe i let chpe convince me into adding that API :p03:06
desrti guess it helps in your case, at least...03:06
hallynintrospection ftw :)03:07
hallynwill i still need to use g_variant_iter_free?  i assume so03:08
desrtyes03:09
desrtit's not possible to use stack-allocated iter here because something needs to keep alive the value that you are iterating over03:10
desrtwhich was itself only extracted from the parent03:10
hallyndesrt: yay, i get contents, thanks!03:10
hallyndesrt: and, yay, it has the info i need to connect the TransientUnit to the owner, so i should be able to do the proper cgroup config (tomorrow :)03:16
desrthallyn: i'll probably take a look over the dbus-specific parts of this patch and may clean them up a bit03:38
desrtjust make sure the cgroup stuff is solid... i don't know a thing about it, and i don't want to start learning :)03:38
hallyndesrt: I get to the point where my nonexistant cgroup code says: cgmanager_create_all: called for /user/1000.user/c11.session03:41
hallyn\o/03:41
desrtbtw: i'm going on vacation pretty much immediately after work on friday... i'll be gone for a week03:42
hallyndesrt: so yeah, i'll hook in cgmanager code (which uses nih-dbus, boy thsi will be a beautiful frankenstein :), then toss you a git tree so you can sob at waht i've done and fix up the glib dbus parts :)03:42
hallyndesrt: oh, hm.  well i'll try to get this done tomorrowmorning - at least enough for you to look at the glib-dbus part03:42
desrtsure thing... otherwise, i guess no big deal if it takes an extra week?03:43
hallyndesrt: well debian-devel is crying already, so it may be a big deeal to them :)03:43
desrtokay03:43
desrtwhat's your TZ, btw?03:43
hallyndesrt: so yeah i'll toss you a git tree in the morning (in about 14 hours)03:43
hallynUS central03:44
hallynits' 22:45 here03:44
desrtcool.  toronto time here.03:44
hallynoh!  i thought you were in nl for some reason03:44
desrtnope03:44
desrtalthough toronto and amsterdam have a bit in common :)03:44
hallynoh?03:45
hallynnot something i'd ever heard03:45
desrtya... just similar feel03:45
desrtultra-progressive, etc03:45
hallynnice - i only ever went into nl as a kid when we went shopping there (from be).  i need to go spend some time there03:45
hallynanyway!  tty i nthe morning, thx again for your help03:46
desrtsleep well03:46
hallyndesrt: well, i went ahead and pushed it to github.com/hallyn/systemd-shim #cgm.104:11
* hallyn out04:11
pittiGood morning05:03
=== NCommand` is now known as NCommander
=== NCommander is now known as Guest2749
=== Guest2749 is now known as NCommander
pittiinfinity: did you happen to see the glibc autopkgtest failure? It fails to build on "../include/stap-probe.h:24:22: fatal error: sys/sdt.h: No such file or directory"05:26
pittiinfinity: obviously that didn't happen on the real buildds, do you have an idea why that could be?05:26
slangasekpitti: Debian bug #748694?05:33
ubottuDebian bug 748694 in systemtap "sys/sdt.h is architecture specific, and causing issues on unsupported architectures" [Serious,Fixed] http://bugs.debian.org/74869405:33
pittislangasek: ah, thanks; we don't seem to have that in Ubuntu yet (at least not in the changelog)05:36
pittihttps://launchpad.net/ubuntu/+source/systemtap/2.3-2ubuntu1 sounds related though05:36
pittibut isn't, glibc just failed again05:37
infinitypitti: It's entirely possible the 'fixes' broke glibc's build.  I can look tomorrow.05:42
pittioh, that way around05:42
pittiinfinity: thanks; not urgent, I was just wondering why it fails on this build but not the other05:42
pittiinfinity: it counts as "always failed" now, so doesn't block anything05:42
pitti(due to the package rename)05:43
infinitypitti: Oh, ugh.  Yeah, we should get it fixed at least once so it stops behaving that way.05:43
infinitypitti: I'll look tomorrow after I knock off a few more urgent items.05:43
infinitypitti: Test build started now, so I have something to play with in the morning.05:54
* infinity says, as it fails 5 seconds in...05:55
infinitypitti: Oh, failure is obvious.05:57
infinitypitti: Fix testbuilding, will upload in the morning if it's all good here.06:05
dholbachgood morning07:01
=== Mirv_ is now known as Mirv
mvo_@pilot in08:42
=== udevbot changed the topic of #ubuntu-devel to: Archive: open | Devel of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of lucid -> trusty | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots: mvo_
=== Eisbrecher_xnox is now known as xnox
=== doko_ is now known as doko
Noskcajmlankhorst, Has libxi had it's breaks long enough that we can sync again?09:49
mlankhorstNoskcaj: sure, it was just needed until the lts09:52
=== Zic is now known as Guest96198
=== psivaa-off is now known as psivaa
=== MacSlow is now known as MacSlow|lunch
=== Guest96198 is now known as Zic
=== ara is now known as Guest56630
=== apachelogger is now known as octopus
=== octopus is now known as apachelogger
=== MacSlow|lunch is now known as MacSlow
=== pete-woods is now known as pete-woods_lunch
=== _salem is now known as salem_
=== pete-woods_lunch is now known as ptet-woods
mvo_pitti: could you please apply http://paste.ubuntu.com/7775018/ to the pm-utils git repo its a trivial typo? reference is bug 129997512:51
ubottubug 1299975 in pm-utils (Ubuntu) "wireless script checks wrong sys file" [Undecided,Confirmed] https://launchpad.net/bugs/129997512:51
pittimvo_: will apply in Debian; I don't have upstream commit12:53
mvo_pitti: ok, there is a debdiff in the bug if thats easier12:53
pittiright, that's the one I'm looking at12:53
mvo_pitti: ok, I will do a trusty SRU for that then and we get utopic via the next debian sync :)12:53
pittimvo_: thanks!12:54
mvo_pitti: thank you! I assume forwarding this to upstream via their bts make sense (I assume they are responsive)?12:55
pittimvo_: no, it's been dead upstream for ages, so don't bother12:55
mvo_pitti: ok, good to know12:55
pittimvo_: I think mbiebl_ has commit rights, but he can also pick them from the debian package12:55
pittimvo_: I'll add a DEP-3 header12:55
mvo_pitti: \o/ thanks!12:56
* mvo_ writes the test case etc stuff in the meantime12:56
pittimvo_: I'll do a Debian upload now, so that it'll be fixed in utopic soon12:58
desrthallyn: why the conditionals?12:59
desrt(in configure)12:59
pittimvo_: (done)13:01
mvo_thanks pitti13:01
LocutusOfBorg1wow! MoM seems broken again13:16
LocutusOfBorg1cjwatson, ;)13:17
LocutusOfBorg1sil2100, did you see upstream for lucene? they fixed the gtest stuff13:17
LocutusOfBorg1I asked for a new release13:17
cjwatsonLocutusOfBorg1: sigh.  hopefully fixed13:23
hallyndesrt: yeah i was undecided on that.  at first i was going to not have them and make it depend on cgmanager13:26
hallyni suppose i should do it that way13:26
hallynbc also supporting cgfs would be very complicating and not nearly as safe13:27
LocutusOfBorg1cjwatson, thanks as usual :)13:28
LocutusOfBorg1I would like to do some merge, can I send to somebody the debdiffs instead of opening a bug, wait for somebody and so on?13:28
LocutusOfBorg1like gccxml, gambas3, wxwidg13:29
LocutusOfBorg1they are all trivial to sync13:29
LocutusOfBorg1s/sync/merge13:29
LocutusOfBorg1curl13:29
cjwatsonwxwidgets3.0 can't be synced so I'm not sure I believe you :)13:31
cjwatsonNor can gccxml13:31
cjwatsonOh, merges, right, don't have time for that right now sorry ...13:32
cjwatsonReviewing merges is generally harder work than just doing them13:32
cjwatsonLocutusOfBorg1: You should always talk to the person who uploaded the package last in the first instance, anyway; please don't just steal other people's merges13:33
LocutusOfBorg1cjwatson, I mean wx2.813:36
LocutusOfBorg1not 3.013:36
LocutusOfBorg1and s/sync/merge13:37
LocutusOfBorg1cjwatson, ok13:37
LocutusOfBorg1just sometimes is bad to see ubuntu so much behind debian13:37
LocutusOfBorg1I hope to see merges done when freeze approaches :)13:37
LocutusOfBorg1I will offer my help again at that time :D13:38
cjwatsonLocutusOfBorg1: Like I say, help may be welcome but please talk to the people who touched it last rather than just firing diffs at people13:38
cjwatsonLocutusOfBorg1: Sometimes they're working on it, or sometimes there's a reason not to move forward that you're not aware of13:38
LocutusOfBorg1yes of course13:39
cjwatsonOr sometimes it's just not important :)13:39
LocutusOfBorg1I'm wondering about some programs like gambas13:39
LocutusOfBorg1the rebuild against llvm 3.2 in debian failed on all archs because of a cmake problem13:39
LocutusOfBorg1why the ubuntu rebuild did not fail? llvm 3.4 is in ubuntu since trusty13:40
LocutusOfBorg1s/3.2/3.413:40
LocutusOfBorg1anyway not a problem :)13:40
mvo_xnox: what is your opinion about lp:~zctgbhu/plymouth/bug-1339954 ? it seems for Chinese we need a better font than the current dejavu and the suggested fonts-droid would add 4.4mb uncompressed into the initrd13:53
popeyxnox: is lp:~ubuntu-mate-dev/ubiquity/ubiquity-marco sane enough for merging to our ubiquity?13:56
hallynslangasek: stgraber: infinity: do i recall correctly y'all said the tp x240 kbd sucks?14:15
=== timrc is now known as timrc-afk
xnoxpopey: need to test and merge. Can't remember if it's safe to set keys which are not universal in all flavours.14:17
stgraberhallyn: I'm not so concerned by the keyboard as it's mostly similar to the x230 except for the lack of an insert key. The one with the terrible keyboard is the X1 carbon.14:17
stgraberhallyn: the main problem I've got with the x240 is the very low amount of RAM and no possible upgrade there. You either get 4GB or 8GB, that's it.14:17
hallynstgraber: ah, ok.  x240 has much longer batt life and is slightly cheaper, so it's catching my eye right now14:17
xnoxmvo_: i'd rather include it conditionally. We are including ubuntu font i think, is that pretty in chinese or not?14:17
hallyn(though i do like thinness.  i'm vain)14:17
hallyni'm used to netbooks, so 8G will suffice if i' not spending too much14:18
stgraberhallyn: so far, the least sucky haswell range thinkpad I'm aware of is the t440s which is a bit bigger (14" instead of 12.6") but comes with a keyboard with an insert key and has one so-dimm slot so you can upgrade it to 12GB if you want to.14:18
hallynbut that display resolution!14:18
hallynhm, yeah.  i *do* need an insert key14:19
hallynt440.  will look, thx14:19
stgraberhallyn: also, the x240 isn't so cheap once you switch to a reasonable screen, IIRC the low price is with the 1366x768 non-IPS display. It's an extra $150 for 1366x768 IPS or an extra $300 for 1920x1080 IPS display.14:19
hallynwaht is that like 12lb?14:19
hallynt440 also has the low-res though.  sad14:20
stgrabert440s is around 3.5lbs I believe, surprisingly light when I was checking out wgrant's at the last sprint14:20
stgraberhallyn: note, there's a big difference in quality between the t440 and t440s, so make sure you look at the latter14:21
hallynwgrant: any experience to tell me whether t440 screen is reasonable outdoors?14:21
hallynstgraber: ok14:21
stgraberhallyn: also, when offered, take the IPS screen option, otherwise you get a TN display and those usually look bad. If the IPS is similar to that on my x230, you can read it fine outside, it's pretty bright.14:22
hallyn3 cell battery?  is that a joke?14:22
pittiev_, bdmurray: do you still remember why ubuntu-bug needs to check /etc/apport/autoreport and exec whoopsie-upload-all if it exists?14:22
stgraberhallyn: no, there are two 3 cell batteries14:22
hallynstgraber: wrote that down, thx14:22
hallynoh, ok :)14:22
pittiev_, bdmurray: cause that doesn't make sense at all when you try and actually use it (bug 1339663)14:22
ubottubug 1339663 in apport (Ubuntu) "ubuntu-bug fails with "whoopsie-upload-all: error: unrecognized arguments" when /var/lib/apport/autoreport exists" [Medium,Triaged] https://launchpad.net/bugs/133966314:22
stgraberhallyn: so you can swap one while you are on battery if needed14:22
hallynso hopefully at least 5 hrs batt lfie14:22
pittiev_, bdmurray: this completely breaks reporting bugs, and crashes should be uploaded automatically anyway; at most, ubuntu-bug could check if /etc/apport/autoreport is on and then point that out instead of processing a .crash file14:23
pitti(that's a bit hackish though)14:23
stgraberhallyn: it's got a front 3cell battery and a back battery that can be 3 or 6 cell. Haswell should be better at power management than Ivy Bridge and my x230 (ivy bridge) with a 9-cell battery lasts 7-8 hours on wifi (like at sprints) and up to 12 without wifi (when traveling)14:24
pittiev_, bdmurray: oh, that's for adding the extra apport info to .crash files and creating the whoopsie stamp14:25
stgraberhallyn: oh, and not sure if that matters to you, but all the new thinkpads no longer have physical mouse buttons, so if you were used to the trackpoint + physical mouse buttons with the touchpad disabled, it's going to be a bit trickier...14:25
pittiev_, bdmurray: but that's a totally different operation than reporting a bug or uploading a single .crash file, so I think if you want to do that you shoudl just call whopsie-upload-all?14:25
pitti(or we even cron that)14:26
hallynstgraber: it'll annoy me, but every laptop is a compromise these days14:26
hallynstgraber: so with 9 cell batt and IPS display i'm around $1300.  compared to $300 for chromebook.  but much nicer, and hopefully useful otuside.  will think a bit.  thanks :)14:26
hallynmaybe is hould buy a used x230 :)14:26
stgraberhallyn: yeah... I need to change laptop by October and so far I'm pretty disappointed, they all suck in different ways...14:27
stgraberhallyn: that's what slangasek ended up doing earlier this year :)14:27
stgraberhallyn: well, not used, but an x230 rather than current gen thinkpad14:27
hallyn"need to change by october" ?14:27
hallynheh, amazon still sells x23014:28
stgraberhallyn: yeah, a family member already bought my current laptop and we agreed I'd give it to her in October ;)14:28
mvo_xnox: apparently not, eog https://bugs.launchpad.net/ubuntu/+source/plymouth/+bug/1339954/+attachment/4149207/+files/UK14.04 show some missing chars14:28
ubottuUbuntu bug 1339954 in plymouth (Ubuntu) "Unable to display Chinese in Plymouth-lable" [Undecided,New]14:28
hallynstgraber: lol14:28
stgraberhallyn: I've not ruled out buying another x230 though, I'd like to have an haswell laptop, but I also would really like to keep trackpoint with physical mouse buttons...14:29
hallyncourse i don't use mouse all that much, and could get a mini-mouse for when i need it, i guess14:29
hallynstgraber: would the default x230 screen compare to the IPS screen on t440s i wonder14:30
stgraberhallyn: the only downside of the x230 is that you can only find it with the 1366x768 IPS display, the variant with the 1920x1080 IPS display is pretty much impossible to find. So it's a bit lowres (like most laptops...) but the display quality is very good.14:30
hallynagain, especially for outdoors14:30
hallynok - i care more about visibility outdoors than i do resolution14:30
mvo_@pilot out14:31
=== udevbot changed the topic of #ubuntu-devel to: Archive: open | Devel of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of lucid -> trusty | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots:
stgraberright, so I believe the x230 was only sold with IPS, so then that's the same I've got here which is pretty bright and fine outdoors14:32
hallynhm, nice14:32
stgraberI once had some problem reading it in direct sunlight but that day was very very bright, it was the early afternoon, no shade whatsoever, the laptop got pretty hot and I got sunburnt, so that may have been a sign that I shouldn't have been staying there :)14:33
hallynbut being stubborn you stayed out an extra 30 mins just to tell the sun off :)14:34
stgraberwell, I ended up staying there for maybe 2 hours, my dark terminals were a problem to read but firefox (or anything with a bright background) was fine14:34
barrymdeslaur: do you have any particular interest in merging file 5.19 from debian to fix LP: #1340212 ?  i ask since you did the last merge.14:35
ubottuLaunchpad bug 1340212 in file (Ubuntu) "file 5.18 incorrectly identifies mime type of zip files" [Undecided,New] https://launchpad.net/bugs/134021214:35
mdeslaurbarry: I was just looking at file security updates. Are you volunteering?14:35
mdeslaur:)14:36
mdeslaurbarry: else, yes, I will do 5.1914:36
barrymdeslaur: well, it's moderately blocking me, but i think i can work around it :)14:36
barrymdeslaur: but hey, that sounds like you volunteer :)14:36
mdeslaurbarry: hehe, I'll do it this afternoon14:38
barrymdeslaur: awesome, thanks14:41
=== roadmr is now known as roadmr_afk
slangasekhallyn: yes, the x240 keyboard sucks :)14:55
stgraberslangasek: not nearly as much as the new X1 carbon's (not sure if you ever saw that one, it's the weird one with split keys and where they shuffled stuff around quite a bit too just for fun)14:59
ev_pitti: that sounds reasonable to me, but I think bdmurray wrote that code, so let's see what he thinks.14:59
ev_all I care about is that the phones automatically send crash reports14:59
ev_not that ubuntu-bug does anything fancy :)14:59
pittiev_: you wrote the initial one in apport-bug with the apport-noui backend, but that behaved much more like the others; whoopsie-upload-all doesn't do at all15:00
pittiev_: I'm not sure what /etc/apport/autoreport should mean -- i. e. automatically upload all crashes unreviewed, or something else15:00
pittiif so, it should have something like a cron job which calls whoopsie-upload-all if that flag exists15:01
ev_oops :)15:01
ev_autoreport should mean automatically send all crashes15:01
ev_there's no reviewing on the phone15:01
ev_you either send them or you don't15:01
pittiif it has another semantics which involves ack'ing crashes by the user, then we need some interactivity and ubuntu-bug integration15:01
pittiok, thanksk15:01
pittiso that means that it doesn't really belong into ubuntu-bug, which you have to call manually (and then for different reasons)15:01
ev_yeah, indeed15:02
ev_I don't know why I did it, but I admit it was wrong15:02
ev_and apologies to bdmurray for thinking it was his code15:02
pittiev_: well, it was quite right back then with apport-noui15:02
pittiit just got mashed up with moving to whoopsie-upload-all in r268615:03
ev_oh15:03
ev_yay, not a complete muppet15:03
pittiev_: ok, so ISTM that we should kick that out of ubuntu-bug, and we instead need a cron job/upstart hook/whoopsie integration/etc. to call w-u-all if that flag exists?15:04
pitticron runs on the phone, so in theory we could just use that15:04
ev_isn't that what the apport-noui job does?15:04
pittioh!15:04
pittiev_: yes, that very15:04
ev_:)15:05
pittiev_: so that fix is simple then: remove 4 lines of shell, done :)15:05
ev_high five!15:05
pittiev_: thanks for the heads-up, wanted to make sure I understand how these bits are working15:05
ev_if only our jobs were always so easy15:05
pittidelete code, upload, drink beer15:05
ev_I'm just going to sit around and delete code all day15:05
pitti(and watch soccer)15:05
ev_lol15:05
ev_you forget that there are ou's in my words now. You can safely call it football around me. ;)15:06
pittibut after next Sunday, that madness is over at least15:06
* pitti hugs ev_15:06
ev_:D15:07
bdmurraypitti: I don't think cron runs on the phone15:09
pittibdmurray: it does (I just checked), but that file-triggered apport-noui.conf upstart job is better indeed15:09
pittibdmurray: so it's all good now15:09
bdmurraypitti: okay, while you are here I noticed a call to apt.apt_pkg.SourceRecords() in backends/packaging-apt-dpkg.py and it seems to me that isn't in the sandbox cache15:11
bdmurraypitti: so if source package contents change between releases there will be an issue15:11
pittibdmurray: oh, you mean that doesn't use the rootdir= ?15:12
pittiouch, yes15:12
bdmurrayit looks to me like SourceRecords doesn't take any arguments15:13
bdmurrayso we can't pass the cache15:13
pittimvo_: is there some way of getting the SourceRecords from an apt.Cache(rootdir=) object, isntead of apt.apt_pkg.SourceRecords()?15:14
hallynstgraber: heh, i think both gaughen and rharper have the x1, it *looks* sleek from a distance15:14
gaughenhallyn, yes I have the x1 carbon. I love it.15:15
mvo_pitti: it should setup all the config so that happens automatically after you create the binary cache, is that not the case for you?15:15
gaughenhallyn, and we know people who could get us corporate discounts on said laptop15:15
pittimvo_: I haven't tried myself, bdmurray just pointed it out15:15
pittimvo_: ah, so the intent is that once you instantiate a Cache(rootdir=), everything else uses that config?15:16
hallyngaughen: oh, they could do same on a t440s?15:16
hallynwho are these mythical creatures?  (msot ppl i knew have fled :)15:17
gaughenhallyn,  I still have people. let me know and I shall get you a serial #15:17
gaughenhallyn, Nivedita for one15:17
mvo_pitti: yeah15:17
pittimvo_: ok, thanks15:17
pittibdmurray: did you actually see that  break?15:18
mvo_pitti: it should be this way, if not, please let me know15:18
pittimvo_: danke15:18
mvo_yw15:18
hallyngaughen: can i also use that for hotel discount?  :-)15:18
hallyngaughen: ok i may have to take you up on that!  i really need a new laptop15:19
* hallyn hopes smoser will keep his mouth shut15:19
gaughenhallyn, I have just gone around with confidence and pretended I deserve the corporate rate and it has worked15:19
bdmurraypitti: I thought it might be the cause of bug 133606215:19
ubottubug 1336062 in apport (Ubuntu) "apport-retrace on precise confused by source package name changing" [Undecided,New] https://launchpad.net/bugs/133606215:19
pittibdmurray: ok, thanks; I opened that in a tab, will look at it15:20
pittijust need to run out for a bit now15:20
mvo_bdmurray: in a meeting right now, I check once its finished15:21
mdeslaurbarry: merged and uploaded15:28
barrymdeslaur: you rock, thanks15:29
=== roadmr_afk is now known as roadmr
=== timrc-afk is now known as timrc
=== mhall119_ is now known as mhall119
hallynstgraber: for systemd-shim.  i want to depend on apiversion >= 5, so the shim can just use controller "all" instead of enumerating the controllers.  Trusty doesn't yet have that version.  Preference?15:57
hallynshould I SRU apiversion 5, or have extra code ins ystems-shim to list all the controlelrs?15:57
stgraberhallyn: we don't need systemd-shim for trusty do we?15:58
hallynstgraber: good point15:58
=== kentb is now known as kentb-afk
hallynfeh.  wishing i had done remove_on_empty for group controllers16:43
hallynstgraber: yay, shim is putting me into cgroups16:48
stgrabernice!16:49
hallynwould be nice if they sometimes got removed (i'16:49
hallynm on c27.session now :)16:49
hallynif i could only remember why i decided not to do autremove for "all" controllers16:51
hallynreally can't think of a good rason.  'get_value' and 'set_value' make sense16:51
* hallyn goes to think about it16:51
=== Dr_Who is now known as Guest91519
hallynall right i'll just implement it then <shrug>17:24
hallynand, yay, we have autoremove.17:29
hallyndesrt: let me wrap this up into a clean commit and push it to git so you can yell at m^W^W^Wreview17:29
desrt:)17:31
hallyndesrt: github.com/hallyn/systemd-shim cgm.1   (to work it requires an update to cgmanager which i'll push shortly)17:34
hallyndesrt: and apologies for dumping this so late when i know you're leaving in a few hours17:35
desrthallyn: i'm still here until tomorrow17:35
hallynoh!  i thought you were out tomorrow :)  ok then i can relax a bit :)17:36
hallynall right, going out for a walk, bbl17:36
desrthallyn: i notice you still take a conditional depend on the cgroups stuff...17:37
desrt+AM_CONDITIONAL([ENABLE_CGMANAGER], [test "x$enable_cgmanager" = "xyes"])17:38
desrt...even if you never actually use the conditional17:38
desrti guess i'd be happier if you just took the pkg-config depend like usual17:38
hallyndesrt: do i ?  i thought i removed that17:38
hallynoh huh17:39
desrtalso: your inclusion of nih/nih-dbus/dbus is not required17:39
hallynwhy?17:39
desrtthe pkg-config on libcgmanager will pull them in automatically17:40
desrtoh.  you copy-paste a big chunk of code here that actually uses this stuff.....17:40
desrti thought you were only going to make calls to libcgmanager...17:40
hallyndesrt: ok, i pushed the trivial drop of that AM_CONDITIONAL;17:41
desrti'd definitely prefer that you rewrite this rather small use of nih dbus into gdbus17:41
hallynno, i'm using the autogenerated cgmanager-client library code17:41
desrtbut i guess i can do that for you17:41
hallyndesrt: that would *rock*17:41
hallynpretty sure that woudl take me days17:42
desrtokay17:42
desrti'll look into that this afternoon17:42
desrtjust send me the cgmanager update....17:42
hallyndesrt: but let me say in favor of keepign it as is, that it then uses the same client library code that upstart, lxc, and the older logind are using17:42
hallyndesrt: cgmanager update is building in utopic17:42
hallyn(and is applied in git://github.com/cgmangaer/cgmanager)17:42
desrtah.  nice.17:42
desrtso the issue is that cgmanager_get_api_version_sync() wants to take a nih dbus connection?17:43
desrta proxy, i guess, rather...17:43
desrtthis is some very bad API :(17:43
hallyndesrt: well you should be able to just get the API version using dbus without a proxy - you can do it with dbus-send17:44
hallyndesrt: the API is designed to do things that simple dbus cannot :)17:44
desrtso this entire client library is just codegen'd dbus client calls?17:44
hallyndesrt: in particular, if you want to weep, look at how the *_scm work17:44
desrti notice "scm" does not appear in your patch17:45
hallyndesrt: yes.  which uses half of the acutal dbus calls17:45
hallynright17:45
desrtso will you depend on the non-dbus-call parts of the library?17:45
desrtor could i just rewrite it all to straight dbus calls?17:45
hallynnot from systemd-shim, no.17:45
hallyngo for it17:45
desrtk.  i prefer to do that, then17:45
hallynwait, hm.17:45
hallyntrying to think if you need to pass any fds...  but i guess not, no. so yeah that should be fine17:46
desrtgdbus does fd passing just fine...17:46
hallyn(yea but not SCM passing :)17:46
hallynyeah but the issue i had was that if you didn't FORCE fd passing early enough, the call woudl fail17:47
hallynbut i don't think that is an issue here17:47
desrtso are we on the same page?  i rewrite everything in straight-up dbus and drop the nih/libcgmanager deps?17:47
hallyndesrt: yup17:47
desrtcool.17:47
hallyndesrt: thanks!17:47
desrthopefully i can have that for you by tomorrow morning17:47
hallynperhaps in the meantime i should be looking at cgmanager for debian,17:48
hallynslangasek: ^  we're just about there17:48
hallynstgraber: I'm going for that walk now, but may have some sysv questions for you when i get back17:48
stgraberhallyn: do we care about scm from systemd-shim? I mean we should only talk to either the host cgmanager or to a proxy, never to host cgmanager directly from a container17:49
stgraberhallyn: so in theory it's all standard dbus, none of our weird fd + creds fancy thing17:49
hallynstgraber: right,17:50
hallynstgraber: i was just defending the "that's bad API" :)17:50
hallynbut we dont' need that at all for systemd-shim17:50
slangasekhallyn: just about where? :)17:51
hallynslangasek: i've got systemd-shim doing cgroup setup for me17:51
slangaseksweet17:51
hallyn(well, cgroup creation and putting me in it;  not configuration of cpusets and such)17:51
hallynslangasek: do we really only care about this with upstart, or do we care about sysv people?17:52
slangasekhallyn: well, as I figured out yesterday only after you ask, we do care in Debian about it for sysv because of the wheezy->jessie upgrade case17:52
slangasekhallyn: but it's also ok to say "here's the code, Debian, please wire up the sysvinit scripts"17:53
hallynslangasek: ok thanks.  (the stuff's not un-subtle wrt starting up proxy and manager, so may be better to provide the scripts)17:54
* hallyn out17:55
hallyn(well, back later, of course)17:55
=== kentb-afk is now known as kentb
rharperstgraber: I really enjoy my x1, but it's the older model with dedicated mouse buttons.  the new keyboard is rather strange; not sure if I could adapt to that one.18:18
stgraberright, the older x1 was pretty nice, it's the new one that's just weird :)18:19
mhall119stgraber: I send an email to the DMB that's stuck in moderation, can you let that through?19:07
stgrabermhall119: sure19:09
mhall119thanks19:10
bdmurraypitti: I've noticed that libmirclientplatform-android-dbgsym is missing from Packages for main and universe on ddebs.ubuntu.com but the package is there19:11
bdmurraypitti: Could you look into that?19:11
mhall119stgraber: got another one in moderation19:24
stgrabermhall119: processed19:25
mhall119thanks again19:32
nxvldoko: is there a roadmap for what's missing for replacing python2 with python3?19:38
nxvldoko: or, who can i talk to to help on this?19:38
nxvlbarry: ^^19:40
barrynxvl: that's a pretty general question :)  can you narrow that down a bit?19:45
nxvlwhat's the work that needs to be done to get rid of python2, and get python3 by default19:46
nxvli want to help getting that done19:46
nxvland see python3 in 14.1019:46
nxvlso, basically, what can i do to help?19:46
jtaylorwhat do you mean with by default?19:47
jtaylorit already is default on the iso or?19:47
=== roadmr is now known as roadmr_afk
barryjtaylor: not for desktop19:47
jtayloroh, whats missing?19:47
nxvljtaylor: according to https://wiki.ubuntu.com/Python/3 it's not19:47
slangasekit is "the default" :)19:48
barryone thing to start with is to review https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AiT4gOXSkmapdGdFejk0MjFydUlNMDVoMXNRdGdkbFE&pli=1#gid=019:49
barryto see if that's still accurate19:50
barryanother thing is to work on getting python3 as default in debian, which is blocked on https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=73264419:51
ubottuDebian bug 732644 in python-debianbts "python-debianbts: Port to Python 3" [Normal,Open]19:51
barryit's been a few months since i reviewed status, but it should be true today that python2 is banished from touch, removed from default server iso or very nearly so, and still has a ways to go on desktop (hopefully not too far, but there may be big blockers)19:52
nxvlwho might know better?19:53
barrynxvl: nobody ;)19:53
nxvlohhh, it's good to be back :D19:53
barrynxvl: it's just a matter of continually reviewing status and updating the relevant docs, etc.  things change quickly!  e.g. i have been uploading a *ton* of stuff to debian and syncing to ubuntu to improve py3 library support where upstream supports it19:54
barrynxvl: good to have you back, and really glad you want to help!19:54
nxvlok, i'll review the doc and update it19:55
barrynxvl: cool, thanks.  e.g. it would be good to know what the current state of upstream twisted is.  i haven't seen any announcements, but that doesn't mean stuff isn't available19:56
barrynow19:56
barrynxvl: e.g.x2: i'm hoping that once zope.untrustedpython clears new and lands, that will unblock a bunch of the ztk stack from proposed19:56
barry(dep8 tests are all passing for me locally, but depend on python-zope.untrustedpython which was split out from the zope.security source package & upstream)19:57
barryhttp://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html19:57
nxvljust reading "update excuses" on the url makes me not want to open the link19:58
nxvl:P19:58
barry:)19:58
nxvl(did i already mention how i missed this open source chaos?)19:58
barrynxvl: how's the real world? :)20:00
nxvlbarry: booooooooooring20:00
bdmurraywgrant: Would you have any idea about Packages files being incomplete on ddebs.ubuntu.com?20:02
jtayloris it just my system or does valgrind not have a apport exception for segfaults?20:30
jtayloreverytime my app crashes as expected in valgrind the ubuntu bug reporter pops up :(20:30
hallynstgraber: thanks for the help with sysv scripts.  Not yet tested, but I' pushed what I have to github.com/hallyn/cgmanager initscripts.1 .  bbl21:00
bdmurraypitti: I've submitted a merge proposal for bug 133606221:01
ubottubug 1336062 in apport (Ubuntu) "apport-retrace uses system package lists which may return a different source package for a binary" [High,In progress] https://launchpad.net/bugs/133606221:01
=== roadmr_afk is now known as roadmr
=== salem_ is now known as _salem
wgrantbdmurray: Afraid not, that's all pitti AFAIK.22:05
bdmurraywgrant: okay, slangasek thought you might have an idea22:06
wgranthallyn: My T440s has the 1080p touch screen because it was cheaper than the non-touch version, and it's a bit glossy but still very usable outside. AIUI the non-touch version is matte. It's by no means a classic ThinkPad, but it's very good and still the best option AFAIK.22:09
hallynwgrant: matte should be easier to read outside ?22:25
wgranthallyn: Right.22:26
hallynk :)22:26
wgrantThe screen is lovely, the keyboard is great except for the lack of dedicated media buttons and the weird Print Screen location, and the touchpad/trackpoint aren't as bad as I initially thought, very usable once correctly configured.22:28
hallynwgrant: which batteries do you have?22:29
hallyni think i'm sold, tbh22:30
wgranthallyn: I have the external 3-cell and 6-cell. I normally run with the 3-cell, as the 6-cell is fairly hefty.22:30
hallynwill probably get an ssd and memory on amazon as stgraber had suggested22:30
wgrantBut nothing compared to the old 9-cell that sticks out the back.22:30
wgrantYeah, I stuck an 8GiB SODIMM in mine as soon as I got it.22:30
hallynwgrant: so you bought both? i may get3 cell in back and both 3 and 6 cell for back22:31
hallynwgrant: for the lazy and disaster-prone, do you have the model # somewhere for the simm you got? :)22:31
wgranthalfie: Afraid not, but I think it was Crucial.22:32
wgrantOpening the case can be a bit challenging. Make sure you watch the videos and have some plastic phone opening tools around.22:32
hallyncan't be worse than the vostro i recently did surgery on :)22:33
hallynthanks wgrant.22:33
desrthallyn: hey... what's the purpose of all this work?22:33
desrtyou said it's so debian can land the new systemd version, right?22:34
hallyndesrt: right22:34
desrtbut aren't they also landing systemd-as-pid-1?22:34
desrtor do they keep some sysv backcompat option for the time being?22:34
hallyndesrt: slangasek says we promised to provide what is needed for running upstart22:34
hallynyes, i think so22:34
desrtso i'm helping people to run upstart on debian?22:35
desrthow perverse :)22:35
hallyndesrt: perverse?  what is your preference i wonder? :)22:35
desrti can state it in two non-words: systemd asap22:36
hallynbut also sysv22:36
desrti wonder how we ended up holding this particular bag...22:36
desrtseems like the systemd advocates should have had to do the work there......22:36
hallynside effect of arguments and promises made during the debian vote22:37
desrt...which we lost :/22:37
hallynyup22:37
desrti won't pretend to understand :)22:38
hallynbut you prefer systemd anyway22:39
desrtya.... i'm just wondering how we lose the right _and_ have to do the extra work :p22:39
hallyndesrt: bc we're good folks and we want to support people who still want choice22:40
hallynhm, my debian sid system doesn't want to mount a memory cgroup.  what on earth...22:40
desrtthat strikes me as a good enough reason22:40
desrthallyn: sure it's not mounted anywhere else?22:40
hallynyup22:41
hallyncustom kernel courtesy of rackspace, maybe it's wonky22:41
desrthallyn: so far you only use create_all and remove_all... i guess you will expand this soonish?22:48
hallyndesrt: create_all also moves the tasks in there.  But yes, the intent is to also respond to requests to set memory and cpu limits22:49
hallynif that's what you mean22:49
desrti don't really grok how all of this works... but do you have a race here?22:50
desrtspecifically, what if some process fork() after you collect unit->pid but before you cgm_enter() the list of pids?22:50
hallyndesrt: logind won't do that22:53
hallynwhat you describe is a race in how libcgroup does it (bc it detects 'firefox is running', grabs the pid, and reclassifies it - which can't be done in non-racy wya.  which is why libcgroup is dead)22:53
hallynbut logind will take the pids, ask us to setup, and only after that proceed to let the user login22:54
hallyni'm processing the 'array of pids' but it is always only one pid,22:54
hallynfor ssh it's the root-owned sshd which becomes parent of the user's sshd and shell22:54
desrtthe use of a(sv) here is really deeply annoying.... prevents use of all of the nice convenient APIs we have :(22:54
hallynin the StartTransientUnit you mean?22:55
desrtbtw: for types like "au" we have g_variant_get_fixed_array() that returns a guint*22:55
desrtya...22:55
desrtthis is a very strange dbus api...22:55
hallynwell it only gets worse :)22:55
hallynin later systemd we'll have to add the processing of the22:55
hallyn'aux'22:56
desrtanother btw: g_object_new() doesn't fail22:56
hallynoh, good :)22:56
desrtanother minor btw: doing initialisation of the object after g_object_new() returns is considered bad form22:56
hallynstgraber: have you ever seen /proc/cgroups have 'num_cgroups 0' ?22:56
hallynhttp://paste.ubuntu.com/7777549/22:56
hallyni think i'll just have to try elsewhere bc ihave no idea what's going on22:57
desrtanother: g_free() on a GObject is a no-no :)22:57
Unit193hallyn: "with 204-14 being the first version where libpam-systemd will pull in systemd-sysv , I figured a few more days to wait to see how the dust settles wouldn't be so bad"  So will this by chance be completed soon enough for the Debian changes?22:57
hallynUnit193: i'm hoping so22:58
Unit193hallyn: Great, if you happen to be on OFTC, heads up in -systemd might be handy, no?22:59
* Unit193 goes back to lurking now. :)22:59
hallynUnit193: debian-systemd?22:59
Unit193Mhm.22:59
hallynor just #systemd?22:59
Unit19310:OFTC/#debian-systemd23:00
Unit193hallyn: Thanks and sorry for the bother. :)23:01
hallynUnit193: thx for the pointer23:02
Unit193Sure.23:02
Unit193hallyn: I have 213 in a utopic VM, I presume there's no testing I can help with?23:10
hallynUnit193: I suspect 213 will need  a 1-line patch to the systemd-shim patch actually23:11
* desrt is mid-rewrite :)23:11
hallyndesrt: look at systemd.git commit 86b8d289717bad2800342efca0a5023aa8374e9c23:17
hallynadds 3 lines which do23:17
hallyn+        r = sd_bus_message_append(m, "a(sa(sv))", 0);23:17
hallyn+        if (r < 0)23:17
hallyn+                return r;23:17
hallynadding something which the listener didnt' want anyway23:17
=== timrc is now known as timrc-afk
hallynok this is a general debian thing.  any sid image, i can't mount memory cgroup.  'mount -t cgroup cgroup /mnt' mounts everythign but memory.  what on earth?23:35
hallynhuh, GRUB_CMDLINE_LINUX="cgroup_enable=memory swapaccount=1"23:37
hallyn helped23:37
=== juliank0 is now known as juliank
desrtcgroups!  what do they do?  why are they here?  have i lost my mind?!23:55
desrtnobody knows...23:55

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