/srv/irclogs.ubuntu.com/2013/12/13/#ubuntu-desktop.txt

desrtrobert_ancell: gnome-calculator seems not to be building in jhbuild....00:10
desrtrobert_ancell: presumably because it's taking the stable branch of vala but master of gnome-calculator00:11
robert_ancelldesrt, I'm not maintaining that anymore00:33
robert_ancelldesrt, but yeah, I think they're using bleeding edge vala00:33
=== duflu_ is now known as duflu
pittiGood morning04:55
pittiLaney: I think the properties need to be added to the introspection XML for that, which doesn't currently happen (it's not trivial to do, at least when I tried last)04:58
pittiLaney: but it's a real issue indeed04:58
pittiLaney: some nitpicks in https://code.launchpad.net/~laney/ubuntu-system-settings/tests-mock-upower/+merge/19862505:43
didrockshey robru, still awake?07:04
=== Ursinha-afk is now known as Ursinha
ricotzLaney, hello :), please take a look at accountsservice which is currently broken while the typelib is installed in the wrong place due the multiarching http://paste.debian.net/plain/7063908:58
Laneyhey09:02
Laneyricotz: can you tell xnox?09:02
Laneypitti: I haxed the xml in Introspect() and it works correctly for properties with python types, but breaks when you add them over the dbus interface09:02
Laney& will look/address the issues you found, thanks09:03
pittiLaney: good morning09:03
pittiLaney: thanks09:03
Laney'breaks' → they get 'v'09:03
pittiLaney: your upower template update is in sid, waiting for autosync09:03
Laneyyay09:03
Laneyoh, assert raises is nice!09:04
Laneythe 'v' thing is probably due to the signature in AddMethod on the bus09:05
Laneyguess_method(dbus.String('foo')) gives you 's', which is correct09:05
pittiLaney: I don't much like the testtools assertions, they are too verbose for my taste09:10
pittibut it seems they are autopilot style09:10
pittiLaney: standard unittest is self.assertRaises(FooError, function, arg1, arg2, ...)09:10
pitti(just FYI)09:10
Laneyaha09:10
LaneyI did look in the autopilot docs for something like that09:10
pittiit's testtools09:11
Laneyyeah09:11
Laneygot it09:11
Laneyso, for this 'v' problem, I bet it would go away if we could unpack the variant by one level09:11
pittiI don't remember the details any more what was hard about adding dynamic properties to introspection09:11
Laneyit might have been that dbus-python just doesn't do it?09:12
seb128good morning desktops, happy friday!09:12
Laneythere's some patches that never got committed09:12
Laneyhey seb128, happy friday to you09:12
pittiLaney: it also doesn't do it for methods AFAIR, dbusmock has to do that by itself09:15
LaneyYeah, there's something there that I didn't fully understand09:15
pittiLaney: but dbusmock knows the signature of methods09:15
Laneynod09:15
pittibonjour seb12809:15
seb128pitti, salut, ça va ?09:17
pittiseb128: ça va bien, merci ! j'attends comment utiliser TRIM avec LVM :)09:19
pittierr, "j'apprends"09:19
pittiseb128: as-tu vu mon nouveau radio ?09:19
seb128pitti, oui, j'ai vu ça, original comme calendrier ;-)09:20
pittissssh -- "ma nouvelle radio"09:20
pitti(quoi est feminine dans une radio ???)09:20
seb128elle chante bien ?09:21
seb128;-)09:21
pittihéhé09:23
pittiseb128: c'est un bon moyen à souvenier09:25
pittisouvenir09:25
=== jhernand1z is now known as jhernandez
Mirvdidrocks: if you happen to time today, preNEW reviewing lp:properties-cpp would be welcome. no worries if not, I'll ask someone else.09:58
didrocksMirv: I've already other NEWing, but will see if I have some time09:59
didrocksas you understood in the hangouts, it's really not the day ;)09:59
Mirvdidrocks: no problem09:59
seb128didrocks, Mirv: I can review properties-cpp10:07
didrocksseb128: that will be a big help, thanks!10:07
seb128didrocks, yw ;-)10:07
Mirvseb128: thanks, great!10:10
Mirvit's pretty identical in packaging to process-cpp10:10
Mirvmeanwhile, back to qt 5.210:11
Laneyphew10:11
Laneyfound a weird workaround10:11
Laneycan haz mocked properties in introspection now10:11
seb128nice10:12
seb128Mirv, didrocks: I'm not going to be picky about that, but why do we put simple CMakeLists.txt we write under GPL when all the sources in the project are LGPL? would it make sense to use the license (or no license, those are simple build file)10:12
didrocks+1 on using the same license (so nothing written preferably)10:13
Mirvtvoss: can you answer that ^?10:14
MirvI know the loaned cmake files elsewhere are GPL10:15
seb128Mirv, it's weird to have a "library" package not providing an actual lib10:15
* seb128 wonders wth there ;-)10:15
tvossMirv, seb128 happy to remove licenses from the build files10:16
tvossseb128, it's header-only, not sure how to name the package10:17
seb128tvoss, I guess lib is fine, from the descriptionI was expecting an actual lib/.so ;-)10:17
tvossseb128, sure :) I might adjust that to header-only library10:17
seb128tvoss, thanks for the licences, it makes thing easier (if we have GPL sources otherwise we need to update copyright/ship the license, etc)10:17
seb128Mirv, tvoss: otherwise package looks fine for NEW10:18
tvossseb128, ack, gimme a few and I will have an MP up10:18
Mirvseb128: ok, thanks for these I'll wait for tvoss's MP and continue then10:18
Mirvseb128: was it now that you could be also asked to refresh the whitelist of allowed packages to be copied from cu2d to archives? properties-cpp is already in the cu2d-config10:19
Mirvwe need this again while did_rocks is away10:19
seb128Mirv, sure, can do that10:19
Mirvgreat!10:19
seb128Mirv, done10:20
seb128Mirv, did you try to build the package in a pbuilder/ppa yet?10:20
Mirvseb128: yes, built yesterdat https://launchpad.net/~ubuntu-unity/+archive/daily-build/+sourcepub/3722961/+listing-archive-extra10:22
seb128Mirv, great, so I don't need to test that ;-)10:22
seb128thanks10:22
=== sabdfl_ is now known as sabdfl
=== mhr3_ is now known as mhr3
pittiLaney: ah, so I even had a disabled test case for that already11:21
Laneyyup11:21
pittiLaney: many thanks for working on this! /me reviews11:21
Laneynp, sorry for the weird workarounds11:21
Laneyadding support for explicitly specifying the type would probably be decent to be able to avoid the guessing11:24
pittiLaney: yeah, but would break the API11:25
pittior we need an AddPropertyType11:25
pitti(i. e. new method)11:25
Laneyyep11:25
LaneyI suppose one way around the type guessing thing would be to write our own function to do it that works with both dbus.foo and the python types11:26
pittiwrt. the branch, this reminds me that we don't have a test case for introspection of properties added in python11:26
pitti(this should already work, as you said)11:26
Laneywith the XML adding thing, yeah11:27
pittiLaney: btw, https://launchpad.net/ubuntu/+source/python-dbusmock/0.9.2-111:30
pittithat should unblock your MP?11:30
pittiLaney: you should bump the dbusmock dep there11:30
Laneyoh yes, nice11:30
Laneywill do11:30
pittiLaney: ah right, it can't guess signatures from empty dicts/arrays11:34
Laneyyeah, something like that, I didn't fully analyse it11:34
pittiLaney: why the read-only? in the mock you can change all properties at any time11:36
LaneyI semi-arbitrarily chose it because it's what your test-case was going to look for. ;-)11:36
Laneycan fix both to be readwrite instead11:36
pittiLaney: can do during merging11:37
Laneyok11:37
pittiLaney: I change ET → ElementTree already anyway11:37
Laneynod11:37
pitti(hope it doesn't hurt your feelings :) )11:37
Laneynope, that's just a habit of mine11:37
LaneyI think I picked it up from the docs11:37
Laneyit amuses me to see autopilot's cursor get stuck on the pointer barrier for a bit11:40
pittiLaney: et voilà, d-feet now shows mock properties \o/11:44
seb128I wish autopilot tests were not that slow11:44
seb128the ~30 basic tests from u-s-s take already like 3 minutes11:44
LaneyI'm sure it's not necessary to close and open the settings each time11:45
seb128which would be alright if running those was not blocking your computer11:45
pitti<jedi wave>use autopilot-sandbox-run --xephyr11:45
seb128learning every day ;-)11:46
seb128pitti, thanks for the trick, I didn't know that exists11:46
* seb128 tries11:46
pittiby default it uses xvfb, which is nice for automatic tests during build etc.11:46
Laneyoh neat, it does work11:46
pittibut with Xephyr you can see what's going on, but it's more isolated11:46
Laneyah, just needed to merge trunk for the hotplugging test11:47
Laneynice when things work11:47
pittiLaney: merged; how soon do you need this in trusty?11:47
seb128pitti, thanks for the trick, that's very useful ;-)11:47
Laneypitti: Well, I'm writing some tests which use it right now, but I don't necessarily have to get them in right away11:48
Laneythanks for merging :-)11:48
Laneyit means that you can write tests which use QDbusInterface::property against dbusmocked interfaces11:49
pittiLaney: ok, let me know when you need  this uploaded; I'm working on some ofono template ATM11:52
Laneyoh, yeah, I forgot - timedated templates coming up :P11:52
Laneyso whenever you have a suitable batch of stuff11:53
pittiLaney: I just don't want to over-do the Debian uploads, and I like landing them in testing11:54
pittiurgency=medium helps a lot with that, though11:54
Laneysure, 5 days is fine11:54
Laneyindeed, medium-by-default is quite nice11:55
LaneyI actually forgot until I checked the pts11:55
=== Sweetsha1k is now known as list
=== list is now known as Sweetshark
pittiLaney: approved; can't top-approve, though12:00
Laneywoot, thanks12:00
Laneyseb128 can do that12:00
seb128Laney, pitti: done12:18
Laneymerci12:19
seb128de rien ;-)12:20
=== alan_g is now known as alan_g|lunch
=== psivaa_ is now known as psivaa
=== alan_g|lunch is now known as alan_g
seb128dpm, pitti: do you know how to debug/figure out why launchpad didn't import an updated template from a build?14:05
pittiI'm sorry, no14:08
seb128pitti, is the pot supposed to be collected on the buildds in the translations tarball?14:11
seb128same as the .po for langpacks ?14:11
seb128or is that a different process?14:11
pittiseb128: yes, it is; if the .pot is not built during packge build, it can't be imported14:12
seb128pitti, it's gtk+3.0 in that case and the pot is shipped with the source14:12
seb128yet https://translations.launchpad.net/ubuntu/trusty/+source/gtk+3.0/+imports doesn't have the upload from this week14:12
seb128https://launchpadlibrarian.net/159463876/buildlog_ubuntu-trusty-i386.gtk%2B3.0_3.10.6-0ubuntu2_UPLOADING.txt.gz14:13
seb128pkgstriptranslations: no translation files, not creating tarball14:13
seb128ok14:13
seb128hum, no, wrong line14:14
seb128pkgstriptranslations: preparing translation tarball gtk+3.0_3.10.6-0ubuntu2_i386_translations.tar.gz...dpkg-distaddfile: warning: File::FcntlLock not available; using flock which is not NFS-safe14:15
seb128done14:15
seb128Publishing chroot-autobuild/build/buildd/gtk+3.0_3.10.6-0ubuntu2_i386_translations.tar.gz for rosetta.14:15
seb128so it seems that worked14:15
seb128but it doesn't tell us the content14:16
seb128pitti, do you know if I can download that file somewhere?14:16
pittioh, it's not linked from soyuz?14:17
pittihang on14:17
seb128oh, maybe, let me check14:17
pittiah, seems not14:17
seb128https://launchpad.net/ubuntu/+source/gtk+3.0/3.10.6-0ubuntu2/+build/5323322 doesn't have it at least14:17
pitti>>> trusty = lp.distributions['ubuntu'].getSeries(name_or_version='trusty')14:21
pitti>>> list(trusty.getPackageUploads(custom_type='raw-translations-static', name='gtk+3.0'))14:21
pitti[]14:21
pittihmm14:21
pittiseb128: ah sorry, -static is for doc builds14:21
seb128that would explain it14:21
Laneyhmm14:21
Laneydbusmock> can I use AddProperty to add properties whose values change?14:22
Laneye.g. timedated's Timezone property being updated by SetTimezone14:22
pittiLaney: yes, you can change any property at any time14:22
Laneypitti: http://paste.ubuntu.com/6566928/ what am I missing?14:23
pittiseb128: http://paste.ubuntu.com/6566929/14:23
pittiLaney: looks fine to me, what is wrong with that?14:24
seb128pitti, that has the pot :/14:25
LaneyI just get the default back14:26
=== mjohnson15_2 is now known as mjohnson15
seb128pitti, thanks for the help, I pinged wgrant on #ubuntu-devel, seems like it could be a lp bug...14:27
pittiLaney: oh, of course14:27
pittiLaney: yes, mock.timezone isn't automagically considered the property value14:28
pittiLaney: i. e. AddProperty() doesn't do some kind of reference to it,it just takes the current value14:28
pittiLaney: so what you want to do in e. g. SetTimezone is self.Set()14:29
Laneyah, yeah, that's what I was after14:29
Laneyyep, that works - thanks!14:30
Laneymuhahaha, timedatectl believes my lies14:31
seb128;-)14:31
pittiLaney: nice :)14:31
seb128pitti's tools are made of awesome ;-)14:31
pittiLaney: that's in fact the very tool you should use for the tests for that14:31
Laneyyeah I was copying the logind tests14:32
pittiLaney: similarly, I use loginctl, upower, nm-cli etc.14:32
pittiright14:32
Laneynext question will be if there's a way to mock changing the time14:32
Laneyguess this one will be a bit more tricky14:32
pittiLaney: with timedatectl?14:32
pittiisn't that the very same?14:33
pittiLaney: you mean because tools don't usually ask timedatectl for the time, they just use it to set it?14:33
LaneyRight, they use whatever time.h function or equivalent14:34
pittiLD_PRELOAD FTW :)14:34
Laney:-)14:34
pittiLaney: actually, apt-get install faketime14:35
dpmseb128, I'm not sure I can add more to that. When we used to have problems with imports, we generally checked first if the translations tarball contained the .pot locally https://wiki.ubuntu.com/UbuntuDevelopment/Internationalisation/RecipeVerifyingTranslationUploads and then looked at the imports queue for the package if it did. If it's an LP bug, then yes, wgrant is the one who could help :/14:35
Laneyooh14:35
desrthappy troll^Wfriday!14:35
LaneyI've not heard of faketime, exciting14:35
seb128dpm, thanks, see #ubuntu-devel14:35
Laneyguess that will be useful14:35
seb128desrt, hey, happy friday!14:35
pittiLaney: similar hack as fake{root,chroot}14:35
Laneyhey desrt14:35
pittiLaney: I've seen it being used in some tests14:35
dpmseb128, yeah, looking at the imports queue now14:35
pittiyo desrt, are you a paraskavedekatriaphobia ?14:36
desrtLaney: you think we're going to have packages in debian that allow you to pretend to be root and allow you to pretend to be fsync()ing but not allow you to lie to your programs about the time? :)14:36
LaneyI hope it supports setting too14:36
seb128dpm, https://translations.launchpad.net/ubuntu/trusty/+source/gtk+3.0/+imports you mean?14:36
Laneyhaha14:36
seb128dpm, it's also weird that this url doesn't like any of the po files for those uploads14:36
Laneyhow deep do the lies go?14:36
desrtpitti: you'll have to excuse me... i'm having difficulty parsing that on account of not being german14:36
pittiLaney: I think so far umockdev is the culmination :)14:36
desrtoh!  today is friday the 13th!14:37
pittidesrt: it's supposedly latinscienglish14:37
Laneylaney@iota> fakeconsciousness am_i_alive && echo $?14:37
Laney014:37
pittidesrt: got it from http://owad.de/check.php4?id=3429&choice=5&sid=1371626 :)14:37
desrtpitti: but it's massively long and compound-wordy14:37
* desrt was having difficulty determining the word boundries14:37
pittilike everyone else in the world14:38
seb128nice15:26
seb128gvfs upstream merged our/Debian changes to use a private library for backends rather than static linking in each binary15:27
seb128one less patch there ;-)15:27
Laneynice15:27
pittiniice!15:28
seb128seems there is a new maintainer as well15:28
pittithat always has been an icky one15:28
seb128quite some good work/bug fixes getting in15:28
=== alan_g is now known as alan_g|tea
seb128good for the LTS15:28
seb128he also improve the sftp copy performance by copying what openssh is doing15:28
seb128they have performance close from the command line with the new version15:28
seb128https://git.gnome.org/browse/gvfs/commit/?id=7890d2801a7f3998b336452fadc8ad0d01d06e6015:29
pittisweeet15:30
attenteis there an issue with fonts in plymouth recently? i only get rectangles for glyphs15:34
seb128attente, hey, somebody on #ubuntu-unity is pointed out what looks like a bug in u-g-m, http://sourceforge.net/p/geany/bugs/1013/15:35
seb128attente, is that known?15:35
seb128attente, basically eating "_" in menus (you can reproduce by opening an image in inkscape which has a "_" in its name)15:35
seb128attente, @plymouth: not that I know/have noticed, maybe a font issue? did you change you local/install-uninstall fonts?15:36
attenteseb128, didn't notice that problem, sounds easy to fix15:37
attentehaven't done anything special with the fonts, but plymouth changed dependencies from ttf-dejavu-core to fonts-dejavu-core15:37
seb128attente, I'm trying to get him to file a bug15:37
seb128attente, you might want to ask on #ubuntu-devel, slangasek&co might know better15:38
seb128https://launchpad.net/ubuntu/+source/plymouth/+bugs?orderby=-id&start=0 doesn't have any similar reports15:38
=== gatox is now known as gatox_lunch
seb128shrug15:58
seb128new glib made libdbusmenu gtk2 tests unhappy15:59
seb128oh, another g_source_remove() fun16:00
Laneythat change!16:00
seb128tedg, we need somebody to take https://bugs.launchpad.net/ubuntu/+source/libdbusmenu/+bug/1260779 I guess, would that be you?16:03
ubot2Launchpad bug 1260779 in libdbusmenu (Ubuntu) "Test issues in trusty (g_source_remove invalid use warnings), fails to build" [Undecided,New]16:03
tedgUhm, sure.16:04
seb128tedg, thanks16:04
tedgWe need to stop upgrading glib :-)16:04
seb128haha16:04
seb128attente, btw, did you get the osk to work on new versions? any news about enabling back those options Bill listed in his email?16:14
attenteseb128, sorry, i'll get to it today16:15
seb128attente, no worry, I was mostly wondering if you tried again the osk/if your issues were resolved16:15
=== m_conley_away is now known as m_conley
=== alan_g|tea is now known as alan_g
seb128attente, I've tried to make sense of the modifier-tweaks compiz merge request, but I don't have the motivation to read all the comments ... is MCReturn blocking stuff because he wants to fix the gsettings integration mess and ccsm and settings all together?16:53
seb128attente, or is there a practical issue with the changes?16:54
attenteseb128, it's a lot of yak shaving, but there is one concern which i think is legitable16:55
attente*legitimate16:55
westfale7016:56
seb128attente, I just commented asked him to not block fixes on "we need to fix the world or we don't land anything"16:56
seb128attente, which one?16:56
attentethat we can't currently do something like "right control and left shift"16:56
seb128why not?16:57
attentebecause the merge only considers something like left shift with a control modifier pressed16:57
attentewe need to consider extra state if we want to handle that case16:58
attenteand we need to converge on a format for writing out such a shortcut16:58
attentebecause shortcuts currently only allow at most one keysym16:59
seb128ok17:00
attentei only wrote that MP with the intention of fixing the <Modifier>Mod_KeySym case17:00
seb128well, it seems like extra cases to handle later17:00
attenteyeah17:00
seb128but the current changes should be an improvement and not create any new issue17:00
seb128so I think we should still get that in ;-)17:00
seb128anyway I commented on there saying that17:00
seb128let's see how it goes17:00
attenteseb128, thanks :)17:00
seb128don't worry too much about it17:00
seb128if he's unreasonable, just put that on the side after holidays17:01
attenteseb128, i'm trying to figure out a way to get it to work without that MP17:01
seb128ok...17:01
seb128well, don't spend too much efforts on finding workaround17:01
seb128if you think that mp is the right think, let's just convince him17:01
attenteok17:02
attentei don't want to antagonize him though because we will need more changes soon17:02
seb128right17:03
seb128let's be civilized but try to have him to agree than fixing one issue is better than fixing none17:03
seb128even if the fix is not perfect, we can iterate ;-)17:03
attentebtw, did you see his comment about integrating ccsm into g-c-c/u-c-c?17:03
seb128yeah, that doesn't seem something we want to do17:04
seb128but I don't want an argument over it17:05
seb128just tell him to open a wishlist bug about it and that we would need to get design input17:05
=== gatox_lunch is now known as gatox
=== marrusl_ is now known as marrusl
=== m_conley is now known as m_conley_away
=== desrt_ is now known as desrt
Laneyok, off now18:02
Laneyhappy weekend and holidays if you're starting them now!18:02
seb128Laney, thanks, have a good w.e, good luck for next week! you can always drop me emails if there are issue, I might read a few next week (still not to book travel for the desktop week at least)18:18
=== infinity_ is now known as infinity
kenvandineseb128, i'm heading out to lunch, in case i miss you... have a great holiday!18:44
seb128kenvandine, thanks, you too!18:44
kenvandinethe sad part of this time of year... people go offline for too long :)18:45
kenvandineseb128, you'll be missed :)18:45
seb128kenvandine, I'm going to miss you guys as well18:45
seb128it's good that there is a week off for everyone in the middle18:45
kenvandineLaney, give me a shout if i don't respond quick enough to your merge proposals :)18:46
seb128I would really feel like out of the loop otherwise18:46
kenvandineseb128, indeed18:46
* kenvandine waves, bbiab18:46
seb128bye!18:46
Laneyoh crap, I forgot to review attente's one21:03
LaneySORRY!21:03
attente?21:04
Laneywas going to look at the language mp21:12
attenteoh, no worries Laney21:15
attenteit can wait till next week21:15

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