/srv/irclogs.ubuntu.com/2013/10/29/#ubuntu-unity.txt

MirvSaviq: it seems to work great here! too bad you got a Disapprove in the MR as well :P00:05
larsuSaviq: how do I get unity8 to load the notification plugin?01:10
* larsu has patches but no way to test them01:10
Saviqlarsu, export QML2_IMPORT_PATH to wherever unity-notifications installed the plugin01:10
Saviqlarsu, easiest probably to just build the package01:10
larsuSaviq: that's what I thought, but it doesn't seem to get called01:11
Saviqlarsu, is the qmldir file there with the .so?01:12
Saviqlarsu, and does the path include Unity/Notifications/ ?01:12
larsuSaviq: I'm running ot off the package01:13
Saviqlarsu, ah, and why do you feel it's not working?01:13
larsuSaviq: it's neither taking the name nor printing the message that it wasn't able to01:13
Saviqlarsu, it must be calling it, TBH, try adding some debugging around it, or just gdb it?01:15
=== _salem is now known as salem_
larsuSaviq: meh, I must have messed up. It worked after rebuilding and reinstalling. Sorry 'bout that01:16
=== salem_ is now known as _salem
=== duflu_ is now known as duflu
=== Trevinho_ is now known as Trevinho
=== iahmad is now known as iahmad|afk
=== iahmad|afk is now known as iahmad
tsdgeosmzanetti: can you have a look at https://code.launchpad.net/~aacid/unity8/tabbar_dash/+merge/192505 ?08:48
mzanettitsdgeos: yes, on it08:48
tsdgeosgood08:48
mzanettitsdgeos: this will conflict badly with the switching_previews08:53
mzanettiI thought you were basing this on that one08:53
tsdgeosi wasn't i said i would rebase after "getting it to work"08:53
tsdgeosso i guess i should do this now :D08:54
mzanettiah right... well, gimme a bit first to understand it better08:54
tsdgeos[OFFTOPIC] I just bought tickets for a Kavinsky concert in London one of the days of the sprint, going with two friends, if anyone wants to join head to ticketmaster :D08:58
tsdgeosmzanetti: commented in the wrong MR?09:03
mzanettitsdgeos: not sure if this is in the patch you did in the SDK or if it's in unity: https://code.launchpad.net/~aacid/ubuntu-ui-toolkit/tabbar_external_use/+merge/192307/comments/44432009:03
mzanettiright... I wanted to comment it on the other09:03
tsdgeosi mean, we didn't use a tabbar before09:04
tsdgeosit may as well be an existing bug :D09:04
mzanettitsdgeos: right. but at least in apps the tabbar's text adjust to the background color09:05
mzanettitsdgeos: no clue where the logic for that is located tho09:05
tsdgeosmzanetti: what do you mean "in apps"?09:05
mzanettitsdgeos: well, apps09:05
tsdgeosisn't just that your background is darker at the bottom + the border/shadow ?09:06
mzanettitsdgeos: no... check out some app with white background, the tabs text will be dark grey09:07
mzanettitsdgeos: while e.g. in the Authenticator app which has black background, the text is white09:07
tsdgeosmzanetti: yes, but that's not a tabbar09:08
tsdgeosis it?09:08
mzanettitsdgeos: yes it is. but looking at the rest of unity's text, I think in our case want to keep it white, but just add the shadow etc like you said09:09
tsdgeosstill09:09
tsdgeosthat's the tabbar09:09
tsdgeosnot related at all to the MR you commented at all that only makes the tabbar usable externally09:10
mzanettitsdgeos: yeah... I added another comment and clarified that09:10
mzanettitsdgeos: I think exactly this is the issue tho. the SDK components are created to change (invert) their text colors depending on the background, while unity's text always stays white but has shadows which make it readable on bright background09:11
tsdgeoshmmmm09:13
mzanettibrb09:14
tsdgeosand that's why we should not ignore the SDK :D09:14
tsdgeosmzanetti: pushed something that should help, can you update both branches and try again?09:24
tsdgeosmhr3: can you still create a "search bar broken position" as you reported in https://bugs.launchpad.net/unity8/+bug/1240118 ?09:26
ubot5Ubuntu bug 1240118 in Unity 8 "Scrolling in dash breaks" [High,Triaged]09:26
mhr3tsdgeos, can you ping me about it in an hour? my phone's battery died over the night :)09:30
tsdgeosmhr3: sure09:30
mzanettitsdgeos: yes, this does. The progression arrow is still "invisible" tho. I guess we need a chat with design on what to do with that09:31
tsdgeosmzanetti: that progression arrow is the same as any white background app09:33
tsdgeosi.e. the SDK doesn't do anything smart to fix stuff if the background is white for it09:33
tsdgeosbasically because it's an image09:33
mzanettitsdgeos: no. in apps it uses a black arrow. But I see why we can't use that one09:33
tsdgeoswell that is if you changed the theme09:34
tsdgeosto a dark theme09:34
mzanettitsdgeos: yeah... I think MainView changes the theme depending on the background color09:34
tsdgeosok09:35
tsdgeostbh the only thing i can see happening is getting an "image with border"09:35
tsdgeosand use that all the time09:35
mzanettitsdgeos: yes, I think that's what we'd want in unity09:36
tsdgeosmzanetti: can you take another screenshot with your white background and attach it to the MR? and then we add someone from design to the MR and hope they read their emails?09:37
mzanettitsdgeos: ack09:40
mzanettitsdgeos: http://i.imgur.com/LH6xe0D.png09:41
tsdgeosmzanetti: what you are seeing with the scrolling is fine09:41
tsdgeosit's just that the dash home is some pixels highers than the screen09:41
tsdgeosso it doesn't come back09:41
=== shiznix_ is now known as shiznix
tsdgeosthat happens with non patched version too09:41
mzanettitsdgeos: really? but it's empty09:41
tsdgeosi.e. lp:unity809:41
tsdgeoswell it's "bottom margin"09:42
tsdgeosthat or we're speaking about two different things :D09:42
mzanettitsdgeos: right. confirming it in trunk too09:42
tsdgeosit feels weird09:43
tsdgeosbecause it's just something like 1 gu09:43
tsdgeosbut it's how it's supposed to behave if oyu think about it09:43
mzanettireally? don't see why yet09:44
tsdgeosbecause the header is "special"09:44
tsdgeosit's not a hard body09:45
tsdgeosit comes out from the bottom09:45
tsdgeosso that's what's happening here09:45
mzanettitsdgeos: it really feels like one last update() call of something is missing09:46
tsdgeoswhy?09:47
mzanettitsdgeos: for example this: http://i.imgur.com/27z12c4.png09:47
mzanettiwould you still say that the content is higher that the listview?09:47
tsdgeosnope09:48
mzanettibut it still happens09:48
mzanettiand I can't slowly drag it to that place. only when overshooting it not always comes back09:48
mzanettihere's good one: http://i.imgur.com/qRGh9zV.png09:49
tsdgeosi can't do that here09:50
mzanettiflick it upwards as quickly as you can09:50
tsdgeossure09:51
mzanettibut yes. it's the same in trunk09:51
tsdgeosalways comes to the correct position09:51
tsdgeosmzanetti: that's on your phone or in the desktop too?09:52
tsdgeosi'm guessing the desktop because of the screenies09:52
mzanettitsdgeos: yep. desktop09:52
tsdgeosmzanetti: can you repro if you resize the window?09:53
tsdgeosjust random asking :D09:53
mzanettitsdgeos: just got it reproduced with the video scope on the phone09:53
mzanettitsdgeos: to which size do you want me to resize it?09:53
tsdgeosto any09:53
tsdgeossince i don't know how do you to do only have one category there09:54
tsdgeosi'm just making it taller09:54
tsdgeosso that all fits on screen09:54
tsdgeosand then doing the overshoot09:54
tsdgeosi can't make it behave wrong09:54
mzanettitsdgeos: yeah. it happens with the standard size and also with that larger one as in the screenshot09:54
tsdgeos:/09:54
mzanettitsdgeos: well, one difference is that I use GRID_UNIT_PX=18. Don't think that's the reason, but who knows09:55
mhr3tsdgeos, checked and yea, the header is still moving weirdly09:55
mhr3no scrolling issues though09:56
tsdgeosmzanetti: can't repro with that either09:57
tsdgeos:(09:57
mzanettitsdgeos: now I can reproduce it 100%09:58
tsdgeoslol09:58
tsdgeosreally?09:58
mzanettitsdgeos: grab the dash, move it quickly up and down a couple of times09:58
mzanettitsdgeos: and at some point release it after the upwards movement09:58
tsdgeosok09:58
tsdgeosthat helped09:58
mzanetti \o/09:59
tsdgeosbut seems to be a regular LVWPH bug09:59
mzanettiyes09:59
tsdgeosnothing to do with this one09:59
mzanettisame in trunk, as I said09:59
tsdgeoscan you report it and assign it to me?09:59
mzanettiyes10:00
Cimitsdgeos, why don't we colour the image depending on the background colour?10:02
Cimitsdgeos, http://qt-project.org/doc/qt-5.0/qtgraphicaleffects/qml-qtgraphicaleffects1-colorize.html10:02
tsdgeosbecause i don't do graphical decisions10:03
tsdgeosas i don't expect the designers to tell me how to code10:03
tsdgeosmhr3: you have that problem only on the device or also on the desktop?10:05
mzanettitsdgeos: https://bugs.launchpad.net/unity8/+bug/124582410:05
ubot5Ubuntu bug 1245824 in Unity 8 "[LVWPH] Header position can be messed up by flicking up the content" [Undecided,New]10:05
mhr3tsdgeos, both10:06
tsdgeosmzanetti: tz10:06
tsdgeostx10:06
mzanettitsdgeos: is it wanted by design that we removed the DashBar?10:13
tsdgeosmzanetti: yes10:13
mzanettiok10:13
mzanettitsdgeos: this looks a bit odd: http://i.imgur.com/n2OWijK.jpg  Do you think we could make it fade out towards the right edge in that case?10:19
tsdgeosdo we do any such fading anywhere?10:20
mzanettitsdgeos: hmm... probably not. or something else you can think of?10:21
tsdgeosbesides doing the fading wouldn't be easy either10:22
mzanettiI guess so, yes10:22
tsdgeosmzanetti: not really, personally it doesn't look bad to me, but i can understand it does for some people10:23
mzanettiwell, not blocking the merge because of this. but I think it's worth to have a chat with jouni10:23
Cimitsdgeos, mzanetti http://qt-project.org/doc/qt-5.0/qtgraphicaleffects/qml-qtgraphicaleffects1-opacitymask.html10:35
Cimijust anchor it left to the search bar10:35
mzanetti:D10:35
mzanettithat docs page is badly broken10:36
tsdgeosCimi: but you need a mask source for that10:38
tsdgeoswhich if we stretch10:38
tsdgeosis going to look weird, no?10:38
tsdgeosotoh we can have it fixed size 20 and not stretch10:38
tsdgeosignore me10:38
tsdgeosCimi: are you in the office?10:38
Cimitsdgeos, nope10:38
Cimitsdgeos, all designers are in oakland apart rosie10:39
Cimimight go later10:39
tsdgeosah10:39
tsdgeosok10:39
Cimilike after lunch so I can eat good stuff :)10:39
tsdgeoshe he10:43
mzanettitsdgeos: some smaller comments: https://code.launchpad.net/~aacid/unity8/tabbar_dash/+merge/192505/comments/44438110:45
mzanettitsdgeos: it will also conflict a lot with nic-doffay's filter-selector branch.10:45
tsdgeos\o/10:46
tsdgeosnot10:46
mzanettitsdgeos: that would be /o\10:47
tsdgeoshow do you put your arms on top of your head?10:47
tsdgeos:D10:47
mzanettiit's more like this: http://i1.wp.com/theminorityreport.co/stixblog/files/2012/07/demotivational-posters-quadruple-facepalm.jpg10:48
mzanetti:P10:48
MacSlowtsdgeos, there's only so much acrobatics one can do in ascii :)10:48
=== iahmad is now known as iahmad|afk
=== _salem is now known as salem_
=== iahmad|afk is now known as iahmad
mzanettitsdgeos: hey, when you have some time, mind taking over this one while Saviq is away? https://code.launchpad.net/~unity-team/unity8/switching-previews/+merge/18955611:43
tsdgeosmzanetti: sure11:43
tsdgeoswill have a look later11:43
tsdgeosworking on mhr3's LVWPH bug now11:44
mzanettiok.11:44
tsdgeosmy scopes are on strike11:44
tsdgeosWARN  2013-10-29 12:44:10 unity.dash.scopeproxy ScopeProxy.cpp:516 Could not search '' on applications.scope => Timed out waiting for scope proxy connection11:44
tsdgeosWARN  2013-10-29 12:44:10 unity.dash.scopeproxy ScopeProxy.cpp:516 Could not search '' on home.scope => Timed out waiting for scope proxy connection11:44
tsdgeosWARN  2013-10-29 12:44:10 unity.dash.scopeproxy ScopeProxy.cpp:516 Could not search '' on photos.scope => Timed out waiting for scope proxy connection11:44
tsdgeosWARN  2013-10-29 12:44:10 unity.dash.scopeproxy ScopeProxy.cpp:516 Could not search '' on music.scope => Timed out waiting for scope proxy connection11:44
tsdgeosWARN  2013-10-29 12:44:10 unity.dash.scopeproxy ScopeProxy.cpp:516 Could not search '' on video.scope => Timed out waiting for scope proxy connection11:44
tsdgeosWARN  2013-10-29 12:44:10 unity.dash.scopeproxy ScopeProxy.cpp:516 Could not search '' on files.scope => Timed out waiting for scope proxy connection11:44
tsdgeos:-S11:44
mhr3tsdgeos, are you able to reproduce it now?11:44
tsdgeosmhr3: yes11:44
tsdgeosmhr3: btw, any idea about ↑↑↑↑11:45
mhr3tsdgeos, hm, just a timeout... should fix itself11:45
davidcalletsdgeos, you have activated the french Dash mode. Please deal with the unions first.11:45
mhr3lol, what?11:45
tsdgeosmhr3: nope11:46
tsdgeosjust nothing in the dash11:46
tsdgeosand restarting unity doesn't help either11:46
mhr3tsdgeos, odd... pkill -f unity-scope-home11:46
tsdgeosit's eating 60% of my cpu11:46
mhr3what is?11:46
tsdgeos unity-scope-home11:47
tsdgeosand init was eating 100%11:47
tsdgeoswhatever init is11:47
mhr3init is upstart11:47
mhr3tsdgeos, so, yea, kill the home scope and see if it helps11:47
tsdgeosit did11:47
hyperairinit taking 100% is really weird.11:47
hyperairif it's taking 100%, that would probably be something dying nad getting restarted11:48
mhr3tsdgeos, interesting, if you ever see that again ping me, sounds like something very badly broken11:48
tsdgeosmhr3: ok, will11:48
mhr3tsdgeos, and don't kill it, we'll try to do some debugging ;)11:49
tsdgeossure11:49
tsdgeosit's not the first time i've had those timeouts11:49
tsdgeosnever realized if the scope was eating the cpu though11:49
=== MacSlow is now known as MacSlow|lunch
mhr3well fwiw scopes aren't using upstart, so the init cpu usage is weird11:50
mzanettireally? in a world where 110% of your life is subject to surveillance we are the ones that get the big brother award for a feature that can be turned off?11:54
mzanettireally Austria, really?11:54
mhr3mzanetti, link?11:59
mzanettimhr3: this is german tho: http://www.golem.de/news/big-brother-award-oesterreich-nominierung-fuer-mark-shuttleworth-1310-102415.html12:00
mzanettiah wait. this is only the nomination. But we actually got it... let me find more12:00
sergiusensmzanetti, it's on omg ubuntu12:01
sergiusensmzanetti, http://www.omgubuntu.co.uk/2013/10/ubuntu-wins-big-brother-austria-privacy-award12:01
ricotzcharles, hi12:16
ricotzcharles, is it possible that indicator-session makes a more sophisticated check for unity rather than just looking for an available "com.canonical.Unity" dbus12:17
ricotzwhile the libunity support can only be activated for other docks while providing this dbus, indicator-session gets fooled and will leave you without an option to shutdown/logout12:20
=== MacSlow|lunch is now known as MacSlow
Cimitsdgeos, we needed to stars landing stuff, the queue is impressive!12:59
Cimialmost 20 branches = BB party (big break)13:00
mzanettiCimi: https://code.launchpad.net/~mzanetti/unity8/dont-tease-while-moving/+merge/192366/comments/44443213:14
mzanetti+1 on getting stuff merged somehow. we're building up conflicts like crazy13:14
dandradermzanetti, speaking about it, any news on when stuff will start landing on trunk again?13:17
mzanettidandrader: I have no idea13:17
mzanettifginther: any updates?13:18
mzanettimhr3: what do I need to do to make the music scope pick up album artwork? I tried with folder.jpg or embedding it into the id3 tag.13:25
mhr3mzanetti, only fetching from lastfm works atm13:25
Cimimzanetti, Greeter moved even tho its locked13:26
Cimimzanetti, lil typo in the test13:26
mzanettimhr3: does that mean it should appear automatically?13:26
mhr3mzanetti, if it has proper album and artist tags, yes13:26
fginthermzanetti, sorry, I haven't been keeping up with unity8 specifically, but I see that nothing is passing, does anyone know why?13:27
mzanettifginther: Last time I checked it was some setup issues. let me check latest logs13:27
mzanettimhr3: works, thanks13:27
mzanettiCimi: can't spot it13:28
Cimimzanetti, tho is like slang imho :D13:28
Cimiand it's instead its13:28
mzanettiah ok13:29
mzanettiyeah, can fix it13:29
Cimimzanetti, also13:29
Cimimzanetti, it's not anymore only when it's locked13:29
Cimiso I'd change the text13:29
mzanettiCimi: ok13:30
mzanettifginther: hmm... so the issue seems still to be that we require some post installation script which fails on the read only fs13:33
mzanettifginther: not sure if you're indeed the right one to fix this.13:33
mzanettifginther: hmm... actually I don't know. This is an example failure: https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2491/console13:35
mzanettifginther: happens only on the phones13:35
mzanettiunfortunately I'm quite out of date nowadays on how those jobs work13:35
fginthermzanetti, that job is ancient13:37
fginthermzanetti, :-) anything newer?13:37
mzanettisure. one sec13:38
mzanettifginther: https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2713/console13:38
fginthermzanetti, the test runner has changed quite a bit, but this one is recent enough13:40
fginthermzanetti, ahhh, I remember a bug now related to this: "unable to make backup link of `./usr/bin/unity8' before installing new version: Invalid cross-device link"13:40
fginthermzanetti, let me dig it out of irc logs13:40
fginthermzanetti, https://bugs.launchpad.net/ubuntu/+source/lxc-android-config/+bug/124343213:46
ubot5Ubuntu bug 1243432 in lxc-android-config (Ubuntu) "Packaging issue makes unity un-installable" [Undecided,Fix committed]13:46
fginthermzanetti, I fix was committed, but can't yet tell if it's in the image13:47
fginthers/I fix/a fix/13:48
mzanettifginther: where was this fixed? in unity?13:48
fginthermzanetti, lxc-android-config13:48
fginthermzanetti, looks like 0.117 is on my phone which was flashed several days ago, so should be fixed...13:50
mzanettifginther: ok. thanks. I'll run some build to see if it passes13:50
fginthermzanetti, so in the runs from the last few days. I see nothing but failed unlock attempts on the touch devices13:53
tsdgeosback13:55
mzanettifginther: hmm, this one? http://10.97.0.26:8080/job/autopilot-testrunner-otto-trusty/134/console13:55
fginthermzanetti, while I have your attention... Do you recall the origins of the A10checklicenseheaders hook script? It doesn't allow GPL and LGPL, was there a specific reason for this?13:56
mzanettifginther: huh? I don't think so... are you referring to renato's branch?13:56
fginthermzanetti, yes, that started it13:56
mzanettifginther: well, it does allow GPL and LGPL, but not if the copyright holder is something else than Canonical (or Digia or Google - which are our exceptions)13:57
mzanettifginther: the solution for renato is to move the copied stuff into a subdir called 3rd_party which then will be excluded by the check13:57
mzanettifginther: well, I guess the proper long term solution would be to make the hook more customizable and include allowed copyright holders on a per project basis13:58
mzanettifginther: in renato's case "Philip Withnall"13:59
fginthermzanetti, ahh. thanks13:59
fginthermzanetti, I'm attempting to rebuild http://10.97.0.26:8080/job/autopilot-testrunner-otto-trusty/134/console with the ubuntu-daily ppa, to see if that resolves the package issue14:04
mzanettifginther: cool, thanks14:04
fginthermzanetti, it passed - http://10.97.0.26:8080/job/autopilot-testrunner-otto-trusty/147/console14:11
mzanettifginther: nice!14:12
fginthermzanetti, should that be made into a configuration change? We had to remove that PPA in the past due to building with unreleased package issues14:12
fgintherbut maybe this needs to be changed, at least temporarily?14:13
mzanettifginther: hmm... thining about it... I don't think that would be a good idea to make persistent14:13
mzanettifginther: afaics the month without CI introduced a lot of testing failures...14:15
mzanettifginther: I'm going to check our tests and contact you later on how to proceed14:16
mzanettithanks a bunch so far14:16
fginthermzanetti, please let me know if I can help14:16
mzanettifginther: I will14:16
mzanettifginther: but I need to figure what failures are our tests and what failures are in the CI system first14:16
mzanettiCimi: fixed btw14:20
SaviqMacSlow, hey, could you please look at https://code.launchpad.net/~larsu/notify-osd/allow-being-replaced/+merge/192990 when you have a minute14:27
SaviqMacSlow, we want notify-osd to give up the service name when unity8 starts14:27
MacSlowSaviq, yeah... saw Lars' eMail... on my radar today.14:28
SaviqMacSlow, ok great, thanks14:28
larsuMacSlow: let me know when you have any questions14:28
MacSlowlarsu, ok14:28
Saviqnic-doffay, hey, I didn't get a chance to find you anything major to do - but there's plenty of bugs waiting to be fixed - couldn't find nothing interesting?14:30
MacSlowSaviq, btw... solved all the issues/regression due to the fullscreen-support for the sim-unlock ext. snap-decision14:30
tsdgeosdandrader: standup?14:31
SaviqMacSlow, cool14:40
* Saviq → breakfast, ttyl14:40
nic-doffaySaviq, I haven't look yet, it was more of a pre-emptive ask for an hour or two from now!14:44
mzanettiMacSlow: do you know why Notification AP tests fail in jenkins?14:47
MacSlowmzanetti, last time I saw them fail was due to timeouts14:47
mzanettiMacSlow: http://10.97.0.26:8080/job/autopilot-testrunner-otto-trusty/144/14:48
MacSlowmzanetti, afaik they've started failing since the move to t14:48
CimiSaviq, how is there?14:49
MacSlowmzanetti, since the AP-test didn't change I expect some underlying (maybe autopilot itself) parts to have introduced this regression14:49
MacSlowmzanetti, I can try diving into that after I did the MRs for the fullscreen-support and worked on larsu's notify-osd branch he and Saviq asked for14:51
=== jhodapp is now known as jhodapp|brb
MacSlowmzanetti, will that work for you?14:56
mzanettiMacSlow: sure. I'll try collect some more input for you in the meantime14:57
MacSlowmzanetti, only autopilot-related bit I remember is a switch from ap 1.3 to 1.4 when going from s to t... but not sure what that actually comes with in terms of changes14:58
=== dandrader_ is now known as dandrader|lunch
=== alan_g is now known as alan_g|tea
=== jhodapp|brb is now known as jhodapp
=== alan_g|tea is now known as alan_g
SaviqMacSlow, mzanetti not due to timeouts15:35
MacSlowSaviq, what's the cause then?15:35
SaviqMacSlow, mzanetti, due to notify-osd stealing notifications from unity815:35
Saviqat least that's my suspicion15:35
MacSlowSaviq, ah... so my unity-notification branch will come in handy then :)15:35
SaviqMacSlow, mzanetti and that's exactly what the notify-osd and unity-notifications MPs from larsu are about15:36
MacSlowSaviq, larsu: you have the lp:unity-notification part of the service-request problem already?15:36
SaviqMacSlow, https://code.launchpad.net/~larsu/unity-notifications/replace-existing-service/+merge/19298915:37
SaviqMacSlow, only thing missing is notify-osd exiting gracefully when having been replaced15:37
MacSlowSaviq, from larsu comment I took I needed to do the unity-notification parts still...15:37
Saviqotherwise we'll be leaving ghoul notify-osd things15:37
SaviqMacSlow, no, he made it non-replaceable is all15:38
SaviqMacSlow, http://bazaar.launchpad.net/~larsu/unity-notifications/replace-existing-service/revision/18715:38
MacSlowSaviq, sure... then let me test that15:38
SaviqMacSlow, so the thing should be working, but notify-osd won't exit15:38
Saviqwhich we should probably fix15:39
MacSlowSaviq, yeah... I'm looking into that now15:39
SaviqMacSlow, if you have anything we can/should take over at your EOD, please let us know15:39
MacSlowSaviq, I just now jumped on the Dbus issue... hope I can resolve it before my EOD15:40
SaviqMacSlow, cheers15:40
MacSlowlarsu, Saviq: the taking over (unity8 replacing notify-osd) works using your two branches15:45
MacSlowlarsu, Saviq: what's not working on your end? how did you test it?15:45
SaviqMacSlow, yeah, notify-osd is still running, though, right?15:45
SaviqMacSlow, which means the next time notify-osd is triggered, there will be two, then three, then four etc...15:46
MacSlowSaviq, ah ok... yeah it is...15:46
Saviqonly the last of which will actually handle notifications15:46
mzanettianyone knows why our autopilot tests take a minute to start?15:46
Saviqmzanetti, hud15:47
Saviq?15:47
mzanettiSaviq: ./run works fine15:47
Saviqmzanetti, that trunk on desktop?15:47
mzanettiyes15:47
Saviqmzanetti, started fine here15:49
mzanettiok... they don't even seem to start... but wait for like 20 secs on each test to fail15:50
mzanettiProcessSearchError: Search criteria returned no results15:50
Saviqmzanetti, look into ~/.cache/upstart/unity8.log15:50
Saviqmzanetti, to see why it didn't start15:50
Saviqmzanetti, you can also manually start with15:50
Saviqinitctl start unity8 BINARY=builddir/unity815:51
Saviqthis way it starts with upstart (you'll then need initctl stop unity8 to stop it respawning)15:51
mzanettiSaviq: ah, thanks.15:51
Saviqwe should transition ./run to upstart ,btw15:51
mzanettifood is ready. will try in a sec15:51
=== salem_ is now known as _salem
=== dandrader|lunch is now known as dandrader
Cimidoes stop() reset timers?16:33
Cimilike, is stop(); start(); the same or restart() ?16:33
mzanettiCimi: yes, it does16:37
Cimithx16:37
tsdgeosmhr3: you there? happened again16:40
tsdgeosinit 100% and home-scope 60% cpu16:40
mhr3tsdgeos, uuuh :)16:40
mhr3tsdgeos, so, let's get some logs first16:41
mhr3`bustle-pcap dbus-traffic.bustle`16:41
mhr3ctrl+c after a few seconds16:41
mhr3tsdgeos, perhaps also install upstart-monitor to see if that is doing something interesting16:42
tsdgeosmhr3: did that, got nothing?¿16:43
mhr3tsdgeos, as in the file is empty?16:43
tsdgeosah saves to a file :D16:43
mhr3yea, the second param is filename :)16:43
tsdgeosok, there's stuff in there16:44
mhr3pls open a bug and attach it there (+ make it private you never know what's in dbus messages)16:44
mhr3tsdgeos, anything interesting showing up in upstart-monitor?16:45
tsdgeosinstalling16:45
tsdgeosmhr3: bug against what?16:46
tsdgeoshome-scope?16:46
mhr3tsdgeos, also, the init using 100% cpu is the one owned by root or your user?16:46
mhr3tsdgeos, yea, home-scope is fine16:46
mhr3we'll move it if necessary16:46
tsdgeosme16:47
mhr3ok16:47
tsdgeosmhr3: ran upstart-monitor16:48
tsdgeosseeing nothing16:48
mhr3tsdgeos, and last thing, clean your /var/crash if you have any unity-scope-home crash files there, and get a crash file from the home scope16:48
tsdgeosempty16:48
mhr3tsdgeos, hmm, i guess it's busy processing some events that are just being discarded then16:49
=== _salem is now known as salem_
mhr3tsdgeos, anyway, `ulimit -c unlimited` and pkill -11 -f unity-scope-home16:49
mhr3that will produce a crash file ^16:50
tsdgeosdidn't :-S16:50
mhr3tsdgeos, did you do -11?16:51
tsdgeosyep16:51
tsdgeoscopied from here16:51
mhr3wth16:51
tsdgeosmhr3: in /var/crash you mean the file should end up, right?16:52
mhr3yea16:52
mhr3tsdgeos, it might take a while for apport to process the core?16:52
mhr3although it shouldn't be too long16:52
tsdgeosdon't think so16:52
tsdgeostop and iotop say nothing is going on16:52
mhr3tsdgeos, you didn't disable apport or anything like that?16:53
tsdgeosnot that i remember16:53
tsdgeosbut it may have happened16:53
tsdgeosthis is a very old install16:53
mhr3hmm, clean /var/crash would suggest that... there's always something crashing :)16:54
tsdgeosyeah16:54
mhr3tsdgeos, grep enabled /etc/default/apport ?16:55
tsdgeosenabled=116:56
MacSlowSaviq, sadly no news yet on the notification-service takeover front... still trying to figure out why I'm not receiving any "NameLost" or "NameOwnerChanged" signals to which I installed callback-handlers in larsu's notify-osd branch...17:23
Saviqlarsu, pointers ↑?17:23
kgunnman mterry quits a lot17:30
larsuMacSlow: you should be receiving a NameLost17:33
larsuas soon as somebody else tries to grab the name17:33
larsu(if you allow being replaced)17:34
MacSlowlarsu, not happening17:34
MacSlowlarsu, using your branch with that "G_TYPE_UINT, DBUS_NAME_FLAG_ALLOW_REPLACEMENT," change... maybe that's the wrong way to advertise this? but then... unity8 would not be able to step in...17:35
MacSlowlarsu, lots of unknowns to figure out still17:35
larsuMacSlow: no, that's exactly right. Do you have a branch somewhere for me to look at?17:37
MacSlowlarsu, one sec... let me push my stuff to a +junk one...17:37
larsuMacSlow: take you're time. Gotta run for ~15 mins anyway17:38
MacSlowlarsu, I'm past my EOD too... and just said screw it...17:39
MacSlowlarsu, lp:~macslow/+junk/notify-osd-allow-being-replaced (still pushing)17:41
MacSlowlarsu, finished17:41
MacSlowlarsu, just touched src/dbus.c there17:44
larsuMacSlow: thanks, looking into it now17:54
MacSlowlarsu, oh wait...18:01
MacSlowlarsu, I fixed it... just pushing18:02
MacSlowlarsu, I can continue on that if you need to run18:03
larsuMacSlow: I already did but I also have tons of other stuff to do. Would be nice if you continued if you found the bug18:04
MacSlowlarsu, sure... I'll push a proper branch (non-junk) against yours once I've it all running cleanly...18:05
MacSlowlarsu, watch you inbox by tomorrow18:05
larsuMacSlow: awesome. Have a nice evening!18:05
ricotzlarsu, hi :)18:33
ricotzlarsu, i am hoping you received my mail18:34
larsuricotz: ah sorry I did and meant to answer18:35
* larsu is on a sprint right now18:35
ricotzlarsu, ok, don't worry, just wanted to make sure it reached you18:35
larsuricotz: it did. It looks like I'll have time to port that patch early this cycle.18:36
ricotzlarsu, having it soon would be great :) while it is pretty much integrated and its absence breaks things18:38
MacSlowSaviq, all done... MR for notify-osd is up18:39
SaviqMacSlow, awesome, thanks18:40
MacSlowSaviq, see you tomorrow... emails also sent with a quick summary.18:40
=== dandrader is now known as dandrader|bbl
Saviqlarsu, that looks good in you G-spectacles https://code.launchpad.net/~macslow/notify-osd/exit-if-being-replaced/+merge/193121 ?18:56
larsuSaviq: yes, this looks like it's working. Connecting gtk_main_quit directly to the dbus signal is a bit ugly, but won't be a problem18:59
larsuSaviq: do you want me to comment on the MR?18:59
Saviqlarsu, yeah, that was what I was hoping for input on18:59
larsuSaviq: ok. I'll approve19:00
Saviqlarsu, you'd rather have a "quit()" that would do cleanup and call gtk_main_quit at the end?19:00
larsuSaviq: yes, that would be cleaner. The signal puts arguments on the stack. Usually we connect to funtions that expect these arguments. It will work in this case because gtk_main_quit() doesn't take any arguments19:01
larsuso it just ignores the stack19:01
larsuI say it's fine, that code base is a bit of a mess at this point anyway19:02
Saviqlarsu, k19:02
=== jono is now known as Guest23572
=== salem_ is now known as _salem
=== jhodapp is now known as jhodapp|afk
=== ChrisTownsend1 is now known as ChrisTownsend
=== dandrader|bbl is now known as dandrader

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