/srv/irclogs.ubuntu.com/2014/02/24/#ubuntu-desktop.txt

=== duflu_ is now known as duflu
=== Zachary_DuBois|A is now known as Zachary_DuBois
=== Zachary_DuBois is now known as Zachary_DuBois|A
=== thumper is now known as thumper-afk
pittiGood morning07:01
larsuhi pitti!07:06
larsuhappy Monday07:06
pittihey larsu, wie gehts? Dir auch!07:06
* pitti tries to readjust to CET07:06
larsusehr gut danke :)07:07
larsuhow was your sprint?07:07
pittilarsu: long and intense, but I really enjoyed it07:07
larsuthe way sprints are supposed to be ;)07:07
pittiworking with jibel to solve a lot of autopkgtest issues, and with thomi to figure out the py3 test migration, and others07:08
pittilarsu: jibel and I were actually sprinting in the morning, to Lake Merritt :)07:10
larsupitti: nice! I did that last time I was there as well. It's quite nice there07:11
pittilarsu: yes, I still remember your "running barefeet" attempt :)07:11
larsuthat was before07:11
larsulast time I had shoes ;)07:11
larsuimagine that!07:12
larsu(still only 3mm soles though)07:12
pittiyeah, me too, my Vibram five-fingers07:12
pittibut with them it's excellent07:12
pittiI didn't see any glass splinters or similar anywhere, it's fairly clean07:12
larsudid you go around the lake?07:13
pittino, we didn't have enough time for that, we started at 8 already07:13
larsucrazy!07:14
pittilarsu: did you ever go around? it looks like a total of 8 to 10 km07:19
pittiwe usually ran about 30 to 40 mins, up to ~ 6 km07:19
larsupitti: I did, but didn't plan to (it was just very nice weather that morning and I had time)07:21
larsubut yeah, 10k is a bit much for a morning run :)07:22
=== e11bits_ is now known as e11bits
seb128good morning desktopers, happy monday!09:00
pittibonjour seb128, hereux lundi !09:01
seb128pitti, salut, ça va ? bien rentré ?09:01
pittiseb128: oui, le voyage était bien09:01
pittiseb128: still feeling a bit zombie-like today, though :)09:02
seb128jetlag for the loose09:02
Laneymorning!09:02
seb128how was the QA week otherwise?09:02
seb128Laney, hey, how are you?*09:02
pittiseb128: well, not that bad this time; I didn't get a cold, that makes it much easier to bear09:02
pittihey Laney, how are you?09:02
seb128good09:03
pittiseb128: intense, but productive; I really enjoyed being able to work hand in hand with jibel to fix a lot of britney/autopkgtest things, and with thomi to further the py3 test story09:03
Laneyhey seb128 & pitti09:03
Laneyi'm good thanks, fun weekend beer festivalling09:04
Laneythey were selling off cheap pies at the end which we ended up sharing with drunk people on the train back09:05
Laneywas funny09:05
seb128nice09:05
pittihah09:05
seb128did you have some nice weather for the festival?09:06
seb128hum, u-c-c is blocked in trusty-proposed because indicator-sound is not available on all archs09:09
* seb128 demotes that depends to a recommends09:09
Laneyyeah, but it was inside anyway (mostly)09:09
seb128I start thinking we should put that gsettings key in the shared schemas rather than indicator-sound09:09
larsuseb128: I agree09:11
larsurobert_ancell seems to dislike putting the key in indicatr-sound as well09:11
larsu(judging by his comment on yesterday's MR)09:11
seb128what mr?09:12
* seb128 didn't see that one09:12
seb128oh, your fix scaling09:12
seb128no :p09:12
seb128nice done on that one btw, thanks for fixing it ;-)09:13
seb128did you figure out how you ended up on a different version of the code?09:13
larsuseb128: I meant the one for u-s-d09:14
larsuseb128: no, I didn't figure that out - but fixing it this way was easier than rebasing (which I tried before and was a mess)09:14
seb128larsu, I just saw that, stop working during W.E! ;-)09:15
larsuhttps://code.launchpad.net/~larsu/unity-settings-daemon/support-allow-amplified-volume/+merge/207838/comments/48778309:15
seb128yeah, I just saw that09:16
larsuseb128: I had kind of a late start on Friday and even though I worked later, too, I was missing some hours09:16
larsuso yesterday I thought I might as well do it now :)09:16
seb128larsu, ok, your call, just saying ;-)09:16
seb128(I did some hacking as well to be honest, I looked at the users panel history issue yesterday evening :p)09:17
larsuts ts09:17
seb128;-)09:17
seb128larsu, I found new bugs for you btw!09:17
larsu:'(09:17
seb128but I'm not going to distract your from nautilus yet09:17
seb128btw how reasonable does nautilus/menu look like?09:17
seb128did you have time to look properly at it yet?09:17
larsuyes, and I have half a patch already09:18
larsuit seems very doable, but ask me again in a couple of hours09:18
seb128ok09:18
larsubuilding gnome apps is a huge warning-fest these days09:19
seb128larsu, just for the record the other bugs I have for you are file-roller only starting one instance (e.g you can't open 2 archives from nautilus), seems like GTKish09:19
larsuthanks to all the deprecations09:19
seb128larsu, the other one is evince missing its context menu, which I guess has to do with your menu work09:19
seb128yeah :/09:19
larsuseb128: noted.09:21
larsuhm, context menus work on links and such09:21
larsubut are empty everywhere else09:21
seb128same here09:21
larsuare you sure they worked before?09:21
larsuhm, probably... I'll investigate later09:21
seb128larsu, yeah, just downgraded to the version before the gmenu patch, the context menu has preview/next/refresh/... items09:29
seb128larsu, but yeah, no hurry, nautilus first, then bugfixes ;-)09:29
seb128Laney, can you try the update on lp:~seb128/unity-control-center/users-backport-commits and tell me if you get a proper history?09:36
Laneyyep, give me a minute to send this email09:37
seb128sure, thanks09:38
seb128Laney, shrug I had a typo, if you already pulled please do it again, I've added an extra revision09:47
Laneydidn't yet09:47
seb128good09:47
Laneyseb128: yay, looks good now10:18
seb128Laney, \o/10:19
Laneybe nice to send that upstream10:19
seb128right, I'm going to do that10:20
seb128though https://bugzilla.gnome.org/show_bug.cgi?id=692253 suggest they would prefer to have lightdm changed10:20
ubot2`Gnome bug 692253 in User Accounts "user-accounts: empty login history window" [Normal,Resolved: fixed]10:20
seb128I emailed robert_ancell about that, he replied10:20
seb128"I have no idea about wtmp. I couldn't find any real spec for it."10:20
Laneyheh10:21
seb128let's see if they are any details on why they don't like "pts" if they don't10:21
seb128Laney doesn't like hacks/workarounds :p11:02
seb128Laney, would you prefer if I build-depends on indicator-sound to make it depwait on those archs?11:03
Laneythat wouldn't fix the migration problem11:07
Laneythen you'd get out-of-date11:07
LaneyI would /maybe/ be okay if you arch-qualify the dependency, if it's for a realllllly short amount of time11:08
Laneyoh, just saw the mail - if you're fixing it today anyway then why not just do it like that the first time?11:09
=== alan_g_ is now known as alan_g
seb128shrug, wifi stopped transferring again and I didn't notice before IRC timeout11:20
seb128Laney, not sure if my reply went through, I basically didn't want to mix landing of the fixes and of the schemas transition, but I'm going to reverse the order and start with the schemas11:21
Laneynope, didn't get any message from you11:21
seb128Laney, https://bugzilla.gnome.org/show_bug.cgi?id=725053 btw11:22
ubot2`Gnome bug 725053 in User Accounts "History login empty on Ubuntu (doesn't list "pts" type sessions)" [Normal,New]11:22
=== davmor2_ is now known as davmor2
Laneyseb128: nice, thanks11:22
=== Sweetsha1k is now known as Sweetshark
seb128Laney, do you feel like audio is part of "user-interface", or do we need a new schemas?11:28
seb128let's do a "com.ubuntu.sound"11:29
LaneyI don't have a very good intuition for schema layouts11:30
seb128GNOME has a "sound", doing the same11:30
Laneywe'll have the 'touch' versions too11:30
Laneygrr11:30
seb128those are going to be deprecated though?11:30
Laneythat'd be nice11:31
Laneyoh, you mean the sound ones specifically11:31
Laneyyes11:31
seb128right11:31
=== MacSlow is now known as MacSlow|lunch
seb128Laney, larsu: https://code.launchpad.net/~seb128/gsettings-ubuntu-touch-schemas/shared-sound-setting/+merge/20790012:02
seb128https://code.launchpad.net/~seb128/unity-control-center/rename-sound-key/+merge/20790512:02
seb128https://code.launchpad.net/~seb128/indicator-sound/use-shared-key/+merge/20790912:02
seb128Laney, I didn't add packaging transition for the key move since u-c-c had a version using the key out of trusty-proposed12:03
seb128u-c-c *never* had*12:03
Laneywhy would you need that?12:04
seb128some transition?12:04
seb128because I dropped the key for indicator-sound12:04
larsuseb128: awesome, thanks!12:04
seb128so if u-c-c had been using it and you updated the indicator u-c-c wouldn't be happy12:04
seb128larsu, please update your u-s-d branche to depends on "gsettings-ubuntu-schemas (>= 0.0.1+14.04.20140224),"12:05
Laneyoh like a Breaks12:05
seb128rather than indicator-sound12:05
Laneyseems okay to me12:05
seb128Laney, right12:05
Laneyyou added a blank line in POTFILES.skip :P12:05
seb128Laney, shrug, blame my editor12:05
seb128:p12:06
larsuwill do12:06
seb128danke12:06
larsushould I wait to top-approve i-sound until gsettings-ubuntu-touch lands?12:07
larsu*-schemas12:07
Laneyit should be silo'd in together12:07
Laneyso doesn't matter12:08
seb128what Laney says12:08
seb128I'm going to put those 3 in a silo (once you acked them)12:08
tjaaltonchrisccoulson: hey, are there plans to merge enigmail from debian at some point?12:09
seb128what's the best way to see what keeps something in main?12:10
seb128sorry, was meant for -devel12:10
seb128(re-asking there)12:10
larsuseb128: hm, u-s-d doesn't have auto-landing?12:12
larsuI guess I can just reuse the MR from yesterday12:12
seb128larsu, no, it's under CI train, the lander just need to list vcses12:12
seb128larsu, yeah, just do another commit in the branch robert_ancell approved12:13
seb128I'm going to land it later12:13
* larsu is totally confused by the new process12:13
larsumostly because I didn't really look into it yet12:13
seb128larsu, basically things stopped being merged to trunk automatically12:14
seb128larsu, the "lander" needs to list the merge requests that he/she wants to land, the system give you a ppa with that set (to test), then you press buttons for distro upload and once it's in distro it's merged back to trunk12:14
seb128it just puts the distro/product upfront12:15
larsuseb128: thanks :)12:19
larsuthat sounds like a good system actually12:19
seb128it's quite nice indeed12:20
seb128especially getting a ppa with all your changes12:20
larsuya12:20
seb128like in that case I get all 3 sources, so I can test a real dist-upgrade12:20
larsuMR is updated12:21
seb128danke12:22
seb128Laney, btw, I looked at doing proper commit backports for u-c-c, turns out bzr doesn't do real cherry-picks, it basically commit the diff of the change you merge12:23
Laneyyep12:23
seb128so I can as well keep doing it the way I was doing it, copying diffs from the webui, it's more efficient :p12:24
Laneyyou could use git-bzr to do nice cherry-picks :-)12:24
seb128haha12:25
larsuLaney: we're getting the new gtk sooner than you convince seb128 to use git :P12:25
seb128I guess I could, or at least I should look at that when things are more quiet12:25
seb128for now I'm sticking to my efficient workflow :p12:25
seb128larsu, lol12:25
=== alan_g is now known as alan_g|lunch
=== Trevinho__ is now known as Trevinho
=== MacSlow|lunch is now known as MacSlow
xnoxwhat do i need to do to make print-screen button work with unity-*-* ?13:04
xnoxwell, it works as an audio bell at the moment =)13:04
Laneyare you *running* the latest unity-settings-daemon?13:06
xnoxLaney: you sound like apport "Unreportable error due to out of date packages"13:16
xnoxLaney: let's say i did reboot my machine in a week =)13:16
Laneywe uploaded a new one which fixed print screen being broken on friday (I think)13:17
seb128(correct)13:17
Laneyxnox: unity-settings-daemon --replace13:19
xnoxLaney: hm, restart unity-settings-deamon would not work?13:20
seb128that's what the command he gave is doing?13:20
seb128(note that some apps don't pick up the correct theme again when you do that)13:20
xnoxseb128: yeah, xchat went all window95 on me =)13:20
seb128right, xchat-gnome does the same13:21
xnoxhowever printscreen did work =)13:21
seb128;-)13:21
seb128note that next time the key doesn't work, you can run gnome-screenshot from the dash13:21
seb128it has an UI that let you capture the active win/screen/selection/after timeout13:21
Laneymmm, I've not restarted u-s-d like that before13:22
Laneytry it13:22
seb128it's nothing with the way it restarts13:22
seb128it's just that some apps don't seem to pick up again theme, same thing happen in g-s-d segfaults and respawn13:22
Laneywasn't talking about that issue13:24
seb128oh ok13:24
=== mhall119_ is now known as mhall119
xnoxLaney: restart unity-settings-daemon, mostly worked.13:29
LaneyI was scared it would restart some other things, but I haven't actually checked if that is true13:29
xnoxLaney: but since "stopped" is not emitted nothing should be torn-down, and some things might see "started" again, but that will not launch/relauch things that are already running anyway.13:35
didrockslarsu: but it means you know how to launch a session upstart job with an env variable, right? :)13:37
seb128didrocks, larsu and upstart? you must be confused :p13:37
larsuhahahahah13:37
larsuno, I don't13:37
seb128it's like me and git13:37
seb128:p13:37
larsudidrocks: but aren't they already doing that with current unity8?13:37
didrocksseb128: "launch a session upstart job with an env variable", I didn't see any larsu there :p13:37
didrockslarsu: no, there is no env variable for that13:37
Laneyseb128: do you have a phone which isn't fully updated?13:38
seb128Laney, I probably do, I updated on friday morning and follow the proposed channel13:38
seb128Laney, why?13:39
Laneywilling to test the updates panel fix?13:39
LaneyI want to add https://code.launchpad.net/~diegosarmentero/ubuntu-system-settings/concurrent-check/+merge/207525 to it13:39
Laneybut I just updated13:39
seb128sure13:39
Laneyfor me it downloaded and prompted me to install13:39
Laneybut in the updates panel it span "Checking for updates" the whole time13:39
Laneyinstead of showing the progress13:39
LaneyI'm wondering if this branch would fix that13:40
seb128weird13:40
seb128what should I test exactly?13:40
seb128stock image + what?13:40
Laneywait until I get this approved / reconfigured / built13:40
=== BigW is now known as BigWhale
Laneythen add the PPA for the landing, upgrade to it, go into u-s-s, wait for "updates available" to show, click it and see if you see the download's progress and/or get prompted to install the update13:41
Laneyyou need to be on automatically download on wifi13:41
seb128ok13:42
seb128just give me a ping with what you need to be tested, I'm keeping outdated until then13:42
Laneyif not, well, it didn't give me a corrupt download so it's way less broken than it was already13:43
Laneywould just be nice for the ui to work properly too13:43
seb128k13:44
ochosiLaney: sorry for the (probably silly) question, but you mentioned that light-locker-settings had been uploaded, still i can't find it in the trusty repos (https://bugs.launchpad.net/light-locker-settings/+bug/1281536)13:47
ubot2Launchpad bug 1281536 in light-locker-settings "[needs-packaging] light-locker-settings" [Wishlist,Fix committed]13:47
Laneyochosi: https://launchpad.net/ubuntu/trusty/+queue13:47
Laneyit needs to be reviewed by an archive administrator13:47
ochosioh13:47
ochosithanks!13:47
* ochosi bookmarks the page13:47
=== e11bits_ is now known as e11bits
=== attente_ is now known as attente
=== alan_g_ is now known as alan_g
seb128mpt, hey, could you review/comment on https://bugs.launchpad.net/ubuntu/+source/unity-control-center/+bug/1283150? (it's a string suggestion)14:02
ubot2Launchpad bug 1283150 in unity-control-center (Ubuntu) "Missing article in the new LIM option dialog" [Low,Triaged]14:02
seb128bregma, since when do we have a "no commit without bug reference" for unity?14:05
bregmaseb128, since I have to personally manually approve every single commit that goes in14:06
Laneyseb128: http://162.213.34.102/job/landing-010-1-build/18/console when you get the new u-s-s, please try ^^^ steps14:07
* Laney lunches14:07
seb128bregma, hum, k, I've the feeling that's not a discussion I wanted to have ... happy monday anyway then ;-)14:07
seb128Laney, what should I try exactly? install that ppa and then try to update through system settings with wifi autodownload on?14:08
bregmaseb128, there's a bug that needs fixing, there's a MP that fixes it, what's wrong with requiring we be able to track work and make sure effort is not duplicated?14:08
seb128bregma, nothing, I just don't like strict rules like that because it goes in the way of easy "just send a patch". If we have a bug about the issue we should link to the commit indeed ;-)14:11
seb128bregma, btw what is blocking landing/enabling the hi-dpi work?14:13
* seb128 has been watching the silos/CI train for an unity entry since friday14:13
=== mjohnson15_2 is now known as mjohnson15
om26erLaney, ^ same question here :)14:19
om26er(when you get back) :)14:20
seb128same question being14:20
seb128 Laney, i compile system settings and run it on my desktop but its not showing any panels in it. what am i doing wrong ?14:20
seb128om26er, how did you build it?14:20
=== psivaa is now known as psivaa-afk-bbl
om26erseb128, mkdir 'build' outside of u-s-s source, and then in that build dir, cmake ../ubuntu-system-settings;make -j414:21
seb128om26er, that doesn't work14:23
om26erseb128, do i need to specify some panel path ?14:24
seb128let me check, I usually bzr bd14:24
seb128om26er, it's looking into ${CMAKE_INSTALL_PREFIX}/share/ubuntu/settings/system for the .settings14:27
seb128om26er, you need to define CMAKE_INSTALL_PREFIX I guess then14:27
seb128e.g "cmake .. -DCMAKE_INSTALL_PREFIX=/usr"14:27
om26erseb128, and I would have to point that to the local dir, i would assume14:28
seb128om26er, if you plan to add panels, otherwise getting the list from the system is probably good enoguh14:29
seb128but we don't really support loading local panels (yeah, that's suboptimal), we usually test by copying over the system .so14:29
om26erseb128, i am actually changing the autopilot tests to load everything from local dir if i am running tests locally and only use anything from the system if tests are running from outside of the source14:30
om26erthere can be cases where ubuntu-system-settings is not installed on the system14:30
seb128good luck, we would welcome patches to load local plugins for sure14:31
seb128but we don't have support in our cmake/plugin loader for that atm14:31
om26erseb128, i would like to learn cmake better, so that's my chance there ;)14:31
seb128seems so ;-)14:32
=== alan_g is now known as alan_g|vt
GunnarHjseb128: Bon jour!14:35
seb128GunnarHj, hey, how are you?14:35
GunnarHjseb128: I'm fine, thanks. Hope you are as well.14:36
GunnarHjseb128: We are talking on bug #1164252 about seeding a few IM engines. The question is what would be possible considering the ISO size restrictions.14:36
ubot2Launchpad bug 1164252 in ubuntu-meta (Ubuntu) "Improve default selection of IBus related packages" [Low,In progress] https://launchpad.net/bugs/116425214:36
seb128GunnarHj, I'm good thanks14:41
seb128GunnarHj, can't we just make language-selector install those where it makes sense?14:41
GunnarHjseb128: That's basically how it's done already. The point with seeding a few engines for a few big languages would be to make it possible to input 'funny characters' without installing respective language. A matter of convenience.14:42
seb128you are also going to miss e.g fonts for that no?14:44
seb128I think that if you want to type into e.g Japanese, you should install the language support for it14:44
GunnarHjseb128: Fons are already seeded.14:44
GunnarHjfonts14:44
seb128k14:44
seb128well, same issue as always14:45
GunnarHjseb128: Guess so. ;-)14:45
seb128we could pre-install everything in the archive in case somebody needs it14:45
seb128but then you have big images and that has a cost on infra, download time, install time, etc14:45
seb128not easy to make the cut on what is really useful or not14:46
seb128you should maybe email ubuntu-devel@ about thaty14:46
GunnarHjseb128: Ok, I can do that, so we get input from a few others.14:46
seb128right14:46
seb128but things like14:47
seb128"ibus-sunpinyin14:47
seb128- pulls: libglade2-0 libsunpinyin3 python-glade2 sunpinyin-data14:47
seb128- would need 25.5 MB"14:47
seb128that seems a no go to me14:47
seb128we don't want old unmaintained libs back on the iso14:47
seb12825MB is too much overhead as well14:47
seb128the other ones seems more reasonable14:47
GunnarHjseb128: OTOH, ibus-sunpinyin seems to be one of the most desired (according to happyaron and damage3025).14:48
Sweetsharkseb128: FWIW, I digged out openoffice.org-dictionaries (which are still at the state of go-oo 3.3) and updated them to libreoffice-dictionaries on libreoffice 4.2.1. Also killed dpatch, bumped dephelper and standards-version and made lintian happy. Would you review that? Its currently in a git repo as I hope Rene will take that to debian (after removing the slight ubuntufication).14:49
seb128Sweetshark, I can review that, probably tomorrow rather than today though14:49
seb128Sweetshark, can you drop me an email as reminder?14:49
Sweetsharkseb128: sure, that was kinda the timeline I was looking for ;)14:50
seb128Sweetshark, I still have nlpsolver on my list as well (just letting you know I didn't forget about it;-)14:50
=== vrruiz_ is now known as rvr
chrisccoulsonseb128, any idea what's causing this? seems like a recent trusty change (bug 1278062)15:01
ubot2Launchpad bug 1278062 in firefox (Ubuntu) "firefox crashes on startup (but apparently layunched properly to enable me to post this)" [Medium,Confirmed] https://launchpad.net/bugs/127806215:01
seb128chrisccoulson, hey15:02
seb128chrisccoulson, no, is that a one user thing? we didn't change gtk+2 recently...15:02
chrisccoulsonseb128, no, there's quite a few15:03
seb128weird15:03
seb128buggy theme?15:03
Laneywhat should I try exactly>15:05
Laney24/02 13:40:32 <Laney> wait until I get this approved / reconfigured / built15:05
Laney24/02 13:41:10 <Laney> then add the PPA for the landing, upgrade to it, go into u-s-s, wait for "updates available" to show, click it and  see if you see the download's progress and/or get prompted to install the update15:05
Laney24/02 13:41:21 <Laney> you need to be on automatically download on wifi15:05
seb128Laney, right, testing that in a bit15:07
Laneymerci15:07
seb128de rien15:07
hikikohello :)15:10
hikiko1 question: I'd like to be sure that I have deleted all the cached gsettings (to have the default values)15:10
hikikowhich directories do I have to check?15:11
attente.config/dconf/user i think15:11
hikikoI ve tried this mmm yes but maybe I re-set some accidentally :p15:13
hikikothank you attente :)15:13
seb128hikiko, just "gsettings reset <key>"?15:13
hikiko:D15:13
hikikothanks seb12815:14
seb128yw15:14
attenteor what seb128 said :P15:14
=== alan_g|vt is now known as alan_g|tea
seb128Laney, I get a "checking for update" that never stops spinning (or it's very slow to check)15:33
Laneysame15:33
Laneyso it didn't fix that15:33
Laneywait a bit and see if you get the popover to actually install the update15:33
Laneydon't let the screen blank in case that bug still exists15:33
=== alan_g|tea is now known as alan_g
seb128it seems to be download, seeing the latency increase on my IRC15:35
seb128downloading even15:35
Laneyya15:37
Laneynot sure if we should have this fixed before landing stuff?15:37
seb128yeah, trying to :p15:37
Laneydoes Diego irc?15:37
seb128shrug, I wonder what it's doing, the update shouldn't be that much data!?15:39
seb128is that a regression?15:39
Laneymine was 230M or so15:40
seb128he's "gatox"15:40
seb128Laney, ok, got the popover "install&restart" screen15:41
seb128what do you want me to do from there?15:41
seb128decline so I keep in a state where I can test new changes?15:41
seb128though I guess they are going to keep generating images, we should have a newer one with small delta soon15:41
seb128(wth with the lag?)15:41
Laneyyeah15:42
Laneygatox: hey ;-)15:42
seb128Laney, I think I should be over the lag15:42
gatoxsecond..... meeting15:42
LaneyCan you read the previous few minutes ↑?15:42
LaneyThere's a problem with the update panel in system-settings if you enter it while a download is already in progress15:42
Laneyit spins 'checking for updates' and doesn't clear that15:43
Laneyseb128: you need some QoS man15:43
seb128it's never an issue on my laptop with dist-upgrade, I don't know how the phone download manage to DoS15:43
Laneyone big thing vs many smaller ones I guess15:43
seb128I wonder if they do uploads as well, that's usually what creates latency for me15:43
Laneyable to saturate your connection more15:43
seb128likely15:44
seb128so, should I apply the update?15:44
seb128ogra, didrocks: when are you guys rolling a new image? ;-)15:44
Laneylemme see if I can get this with --testing=update-manual-success15:45
ograseb128, latest at 3am UTC15:45
ograseb128, we have issues that we need to identify first15:45
seb128ok15:45
ogra(android 4.4 switch was on the weekend)15:45
seb128let me decly that upgrade then15:45
seb128decline even15:45
seb128I need to be outdated to be able to test the upgrade panel15:46
seb128ogra, thanks15:46
hikikoseb128, I didn't manage to reproduce this bug: #1283297 but from what I see in the backlog, his key (monitor_name) was empty, so I've fixed the race condition where we have an empty monitor_name, so, if you have a moment, could you verify that you don't get a seg fault before I do the MP?15:46
ubot2Launchpad bug 1283297 in unity-control-center (Ubuntu) "unity-control-center crashed with SIGSEGV in g_variant_lookup_value()" [High,New] https://launchpad.net/bugs/128329715:46
ogra:)15:46
seb128hikiko, ok, testing that in a bit15:48
=== psivaa-afk-bbl is now known as psivaa
hikikothanks a lot seb12815:50
=== charles_ is now known as charles
gatoxLaney, seb128 i'm back.... which seems to be the problem with updates?? i didn't follow16:00
seb128gatox, if you have "download auto on wifi" and do "system-settings -> wait to see the banner with update available -> click on it", the updates panel has the "check fo update with spinner" displayed for the whole download time16:02
gatoxseb128, can you file a bug and assign that to me? I'll try to take a look at that today or tomorrow16:03
seb128Laney, ^ do we have one already?16:03
LaneyNo, because it's caught up with all the stuff being tested for the other updates problem16:05
GunnarHjseb128: Hopefully some CJK users add their views on the list. I'm just a 'middle man'. ;-)16:12
seb128GunnarHj, yeah, let's see16:13
seb128GunnarHj, thanks for stepping up for those users ;-)16:13
GunnarHjseb128: No problem. I tend to jump at everything i18n...16:14
Laneygatox: I'm thinking we should hold back on releasing this batch of updates (re-enabling the main screen check) until this problem is fixed16:19
seb128Laney, the "displays spinner rather than the download bar" is a regression from the silo? in which case, yes please16:22
Laneyyeah16:22
Laneylike it's better than it was when we had the first bug16:22
Laneybut worse than just not having the two checks16:22
seb128hikiko, the commit looks fine, why did you move the adj line though? also please use g_warning() rather than fprintf(stderr...)16:38
hikikook :)16:39
hikikowhich adj line?16:39
hikikoI guess accidentally16:39
* desrt raises an eyebrow16:39
hikikoI'll put it back16:39
seb128hikiko, http://bazaar.launchpad.net/~hikiko/unity-control-center/unity-control-center.race-cond-empty-monitor-name/revision/1272816:39
seb128it just seems and unecessary change16:39
seb128thanks16:39
hikikoI wanted to move it16:39
seb128desrt, good morning, how are you?16:39
hikikobecause it's C and you have to put the decl on top16:40
desrtseb128: good.  just wondering what gvariant abuse is going on today :)16:40
hikikobut forgot to paste the line :s16:40
hikikofail :///16:40
hikikooh no, it's not a fail I moved it16:40
seb128desrt, haha16:40
hikikoseb128, it's just 2 lines above16:40
seb128hikiko, right, I was just wondering "why" you moved it ... the declaration makes sense ;-)16:41
seb128hikiko, so please change the fprintf to a g_warning (no need of the "\n" in the string)16:41
seb128hikiko, looks fine to submit then ;-)16:41
hikikoyes I ll push that and do an mp in a moment thanks a lot for the review :)16:42
seb128thanks for the work16:42
desrthikiko: did you know of https://developer.gnome.org/glib/unstable/glib-GVariant.html#g-variant-dict-init ?16:42
seb128it seems a bit weird that we get a null monitor in some cases, but that's not your fault16:42
desrtthis is in the distro at this point, so usable for this release, i think...16:43
seb128indeed16:43
desrthttp://bazaar.launchpad.net/~unity-control-center-team/unity-control-center/trunk/view/head:/panels/display/cc-display-panel.c#L566 is a bit scary :)16:47
desrtanother fprintf(stderr, ...) there as well -- but that one should just be removed entirely.  it's not possible for str to be NULL there.16:48
ritzSweetshark, hi, https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/120027716:49
ubot2Launchpad bug 1200277 in libreoffice (Ubuntu) "[LibreOffice] - libreoffice-writer.desktop when drag/drop to desktop, 100% broken. " [Low,Confirmed]16:49
ritzwhat is the probability of including this fix in precise16:49
desrtthis function also calls malloc() instead of g_new() and then leaks the result16:55
hikikoseb128,16:56
desrthikiko: did you write the add_dict_entry() stuff?17:04
desrthttp://paste.fedoraproject.org/79876/61464139/ should do the same, a lot easier17:04
desrtalso: it's better to use g_settings_get_value() and g_settings_set_value() instead of the varargs API when dealing with a GVariant*17:06
Laneygatox: https://bugs.launchpad.net/ubuntu/+source/ubuntu-system-settings/+bug/1284217 --- will you have time to work on this quite soon?17:07
ubot2Launchpad bug 1284217 in ubuntu-system-settings (Ubuntu) "[updates] Displays a spinner indefinitely when an update download was started externally" [Critical,Triaged]17:07
gatoxLaney, could you add a screenshot, so i'm sure i understood that correctly?17:09
LaneyNot really, but imagine a spinner that says "Checking for updates"17:09
LaneyTell me what you don't think is clear and I will see if I can improve it17:10
hikikoyes17:11
hikikoyes desrt17:11
desrthikiko: it takes a bit of time to 'get used to' gvariant :)17:12
hikikotrue :) I didn't know there's such a tool (GVariantBuilder)17:12
desrtin general i recommend using the iter/builder interfaces where possible -- they make things a lot easier17:12
hikikoI was not familiar with gvariants desrt and since the iterator was very well documented I used that17:12
hikikoI'll have these in mind17:12
desrthikiko: the code i pasted should work as a drop-in replacement for the add_dict_entry() you had before but i didn't test it or anything17:13
seb128_hikiko, let's maybe land the fix and default position on the slider in that mp and do another one tomorrow for cleaning the code a bit/addressing desrt's comments?17:13
hikikoshould I change it in the unity-control-center desrt ?17:13
chrisccoulsonwho killed freenode?17:13
desrthikiko: i think seb has the right idea...17:13
gatoxLaney, ah, i understand the problem now.... to be honest, i think i can start working on that maybe on thursday, but probably not before17:14
desrthikiko: there are memory leaks and unnecessary checks in the old code (gvariant and gsettings are both quite strict about things never being NULL, so these checks are not necessary)... but this can wait until tomorrow17:14
seb128_desrt, thanks for the review btw ;-)17:15
seb128_desrt, did you see my ping for the one from attente btw?17:15
=== seb128_ is now known as seb128
seb128seems like there was a split17:15
seb128not sure what went through17:15
hikikosure desrt and seb128_17:15
seb128hikiko, thanks17:16
hikikothanks for the reviews/help :)17:16
desrtseb128: i think i reviewed the attente one already?17:16
desrtunless there is a new one17:16
desrthikiko: np :)17:16
seb128desrt, that one is the mnemonic for u-g-m, I don't see a comment from you on https://code.launchpad.net/~attente/unity-gtk-module/gtk-enable-mnemonics/+merge/20775217:17
desrtseb128: indeed i didn't see this one17:17
seb128k17:17
seb128it would be nice if you could review it17:17
seb128no hurry, that doesn't need to be today17:18
desrti already don't like it :p17:18
seb128haha17:18
seb128attente, hide from desrt :p17:18
desrt+ gulong settings_notify_handler_id;17:18
desrt'oh my....'17:18
attentelol17:18
desrtthis is a double whammy17:18
desrtfirst of all... handler IDs are not cool... second of all... you're trying to dynamically deal with this being changed at runtime.... that should be fun :)17:18
* desrt digs in for a fun review17:18
hikikohave a good evening everyone :)17:19
Laneybah, I can't reproduce that bug on the desktop17:31
seb128Laney, the system-update one?17:32
Laneyya17:32
desrtattente: commented.17:32
seb128:-(17:32
Laneytesting=update-auto-success actually does a long running download17:33
attentedesrt, thanks17:33
seb128ok, time to get ready for sport, have a good evening everyone!17:36
desrtseb128: did you join a curling club?17:37
Laney3GRR17:37
seb128desrt, lol, no, I'm not sure we have any of those ;-)17:37
seb128"just" tennis so far, that's enough for me17:37
Laneyit works on the phone with --testing too17:37
desrtseb128: oh.  what sport then? ;)17:37
desrtindoors, i hope :)17:37
seb128desrt, ^17:37
seb128yeah17:37
seb128they have indoor and outdoor17:37
seb128we play indoor during winter17:38
desrt(even when used for actual sports, this sentence sounds weird) :)17:38
seb128on that note, time to pack my stuff and go,see you tomorrow17:38
desrtenjoy :)17:39
=== alan_g is now known as alan_g|EOD
=== gatox is now known as gatox_lunch
=== gatox_lunch is now known as gatox
=== CardinalFang is now known as qengho
mterryLaney, I finally got around to looking at your as-ringtone branch20:19
Laneyold skool20:19
mterryLaney, after a merge from trunk, it works fine.  Was about to approve, but what is the story with the SecurityPrivacy changes in that branch?20:19
mterryJust some old cleanup?20:19
Laneymterry: the AS schemas all move to a separate package20:19
mterryLaney, and the names have changed, so we don't have a conflict currently?  i.e. these are just old files sitting around?20:20
Laneyactually it's already in distro20:22
LaneyI should have set a prereq branch / merged trunk into that one20:22
mterryLaney, ah.  Can you fill out the new checklist stuff for that branch?20:23
Laneywoah20:25
Laneywhat checklist?20:25
mterryLaney, https://wiki.ubuntu.com/Process/Merges/Checklists/ubuntu-system-settings20:35
Laneyhaha20:37
robert_ancelldesrt, in the lightdm tests, sometimes lightdm blocks on shutdown in g_bus_unown_name. Is there any reason that you can think of why that might happen?20:38
desrtrobert_ancell: blocks for how long?  forever?20:38
robert_ancelldesrt, it seems to be so20:38
desrtany fork() in that process?20:38
robert_ancellthe usuals, yes20:39
desrtas in, fork() only if immediately followed by exec()?20:39
robert_ancellthey should be, but that's where I'm going to look next20:39
desrtcould be a weird bug in the worker or something... does your testing involve a virtual dbus environment?20:40
desrtwe have some issues with test busses disappearing....20:40
robert_ancelldesrt, it has it's own test bus, but it's run in a parent process so it shouldn't disappear20:40
robert_ancelldesrt, yeah, I think it might be some dodgy forking, I'll work on that20:40
desrtrobert_ancell: does the backtrace show blocking on a mutex, a cond, a poll() or what?20:41
desrtah... well, let me know if you fail to find fork issues20:41
robert_ancelldesrt, I haven't got a backtrace because the harness makes it a bit hard to get one20:41
robert_ancellwill do20:41
robert_ancelldesrt, where is that list of fork safe functions?20:43
desrtrobert_ancell: it'20:45
desrtrobert_ancell: there is none because it's highly contextual, but in general the list in signal(7) should all be safe20:45
robert_ancellthat's the one I was thinking of20:45
mterryLaney, did you have a branch prepared to use these settings in telephony-service or shall I go and do that?20:47
mterryLaney, oh also, I recall there was a trivial conflict with trunk for your as-branch.  So it'll need an update before merging20:49
Laneymterry: There is one (how did you test it without that?)20:49
mterryAh, I found https://code.launchpad.net/~laney/telephony-service/accountsservice/+merge/20163020:49
mterryLaney, I tested it manually by poking AS20:49
Laneynod20:50
LaneyI'll merge trunk now20:50
mterryLaney, those gross gdbus lines are second nature to me now  :)20:51
LaneyI'm getting pretty good at them too20:51
Laneythe variant syntax confused me at first20:51
mterryyeah20:52
Laneythere we go, merged trunk20:53
Laneydid you check that the migration worked?20:53
mterryLaney, yup20:54
mterryLaney, like a charm20:54
Laneygreat20:54
mterryLaney, also...  Is volume a shared setting yet?21:13
* mterry looks at code21:14
mterryOK, it isn't.  Hm21:17
* mterry looks into making that happen21:17
LaneyIsn't that pulse?21:50
Laneyit's not handled by system-settings, and it would feel weird to me to put it into either gsettings or AS22:01
mterryLaney, what about having indicator-sound sync from pulse to AS?  Pulse doesn't have a native way for the greeter to see each user's volume, I assume22:23
brainwashdoesn't pulseaudio stop playing sound after switching to the greeter?22:25
mterryLaney, it wouldn't be the authoritative storage place for it, just a synced value22:26
mterrybrainwash, this is more about the greeter being able to play some sounds like incoming text or call ringtone22:26
mterry(at the right volume)22:26
brainwashah, ok22:26
* robert_ancell -> lunch23:15

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