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

=== _salem is now known as salem_
=== salem_ is now known as _salem
=== SAM113101 is now known as sam113101_afk
=== sam113101_afk is now known as sam113101
=== jalcine is now known as jalcine_
=== jalcine_ is now known as jalcine
=== jalcine is now known as jalcine_
didrocksTrevinho: hey! can you please take a look (once you are around) to unity FTBFS in trusty? https://bugs.launchpad.net/unity/+bug/124352906:39
ubot5Ubuntu bug 1243529 in Unity "unity FTBFS on trusty" [Critical,New]06:39
didrocksjamesh__: your turn now ;) we have a FTBFS in trusty on mediascanner (seems one test timeout on all archs): https://bugs.launchpad.net/mediascanner/+bug/124353606:39
ubot5Ubuntu bug 1243536 in mediascanner (Ubuntu) "mediascanner FTBFS in trusty" [Undecided,New]06:39
didrocksjamesh__: can you take a look asap?06:40
=== iahmad_ is now known as iahmad
=== jamesh__ is now known as jamesh
jameshlooking06:42
didrocksthanks06:46
jameshdidrocks: is there a new grilo or grilo-plugins in trusty?06:51
didrocksjamesh: no, both are the same than in saucy from what I see06:51
jameshdidrocks: I've just put up https://code.launchpad.net/~jamesh/mediascanner/bug-1243536/+merge/192292 for review.  Is the CI likely to catch this problem, or would it still be testing against saucy?07:14
didrocksjamesh: I don't really know about upstream-merger, seems they have issues to switch to trusty07:14
didrocksjamesh: but I'll keep you posted once this lands, I have a ppa build then :)07:14
jameshdidrocks: looking at the error, I think the changes in that branch should address it though.07:15
didrocksjamesh: excellent! is someone going to review it?07:17
didrockslike mhr3?07:17
jameshI'll get someone on my team to07:17
didrocksthanks a lot jamesh ;)07:17
didrockshey mardy!07:26
didrocksmardy: it seems that libaccounts-glib is FTBFS on saucy (https://launchpadlibrarian.net/154768960/buildlog_ubuntu-saucy-i386.libaccounts-glib_1.14%2B13.10.20131023-0ubuntu1_FAILEDTOBUILD.txt.gz)07:26
didrocksmardy: but I guess first that the 13.10 just rather just be a bzr push from trunk to that branch07:27
didrocksMirv: why didn't you just bzr push trunk to https://code.launchpad.net/~online-accounts/libaccounts-glib/13.10?07:27
didrockssil2100: any opinion?07:35
didrockssil2100: otherwise, I guess we have a diff and that's why it tries to rerelease it with no change07:35
tsdgeoslarsu: sometimes the sound indicator on the desktop does not go to the "muted" state when i middle click, even if it indeed mutes the sound coming out from the speakers, is that something known?07:38
sil2100didrocks: how does libaccounts-glib trunk look like? Is that all that we released into saucy already?07:42
didrockssil2100: can you check it, please? it seems so to me07:42
=== jalcine_ is now known as jalcine
sil2100didrocks: it looks like so to me too - you think I can do a bzr push --overwrite to 13.10?07:46
sil2100Or merging it in with conflict resolving?07:47
didrockssil2100: no, better to bzr push --overwrite IMHO07:48
mardydidrocks: hi! I'm not sure I understand what the problem is. That m4/gcov.m4 file is in trunk07:49
didrocksmardy: yeah, same here :/07:49
didrocksmardy: anyway, let's fix the 13.10 branch, and we'll see07:50
didrocks(we have enough other issues ;))07:50
sil2100didrocks: pushed the trunk, let's maybe re-spin it and see if it's the same?07:50
sil2100Or actually07:51
mardydidrocks: what is wrong with the 13.10 branch? it looks fine to me07:51
didrockssil2100: to trunk?07:51
didrocksmardy: now it's fine, it was a manual merge where a pushed should have been enough07:52
sil2100No, pushed the trunk to 13.10 ;)07:52
mardydidrocks, sil2100: weird, I think that I pushed trunk to 13.10; not sure why it appeared as a merge07:53
didrockssil2100: looks good :)07:54
didrocksmardy: it was… anyway, all fixed now :)07:54
didrocksthanks sil2100, mardy07:54
Mirvdidrocks: ok seems now resolved with the branches identical?07:54
Mirvbut yes pushing the trunk is a lot better in that case07:55
didrocksyeah, ok ;)07:57
sil2100Well, who would have thought that those two can be pushed directly ;)08:05
pstolowskinic-doffay, ping08:13
SaviqCimi, in carousel-music-video, why do we need the XXX comments, can't we just test it? ;)08:20
nic-doffaypstolowski, pong08:25
jameshdidrocks: fyi, https://code.launchpad.net/~jamesh/mediascanner/bug-1243536/+merge/192292 is now merged08:29
didrocksjamesh: excellent! will keep you posted at next build :)08:29
jameshit passed Jenkins' version of Trusty, so hopefully it will pass Soyuz's one08:29
pstolowskinic-doffay, I looked at the filters; it seems to me that UI always marks the 1st filter option as selected (be it 'All' or a real option)08:29
didrocksjamesh: that should do it, indeed :)08:30
pstolowskinic-doffay, perhaps it has something to do with the index of selected option to you manage yourself?08:30
=== jalcine is now known as jalcine_
pstolowskinic-doffay, and btw, home scope doesn't have 'all' option by design, in that case it means no filter option should be selected by default. and 'All' is not currenntly pre-selected by the bindings which is something I'll need to fix. but most important, UI shouldn't assume 1st item is selected by default08:32
nic-doffaypstolowski, ah I see.08:42
nic-doffaypstolowski, if you add that then I'll remove the selection on my side.08:42
pstolowskinic-doffay, I think you can remove it right now, because now it makes 'Applications' filter option pre-selected in Home, and this needs to be fixed anyway08:57
dandradermzanetti, what's qCDebug?08:58
nic-doffaypstolowski, yeah I was planning to.08:59
dandradermzanetti, I mean, what's the difference between it and qDebug08:59
mzanettidandrader: that's the new qDebug which allows you to define categories and turn on/off categories at runtime08:59
mzanettidandrader: let me find the docs08:59
dandradermzanetti, does it append the category name as well to the print message? So it would replace the [DDA] prefix?08:59
dandraders/append/prepend09:00
mzanettidandrader: its fully configurable. you can make it append function names, classes, line numbers and what not09:00
mzanetti(at least the preview I've been working with in Nokia was)09:01
dandradermzanetti, but we would still keep the "#if DIRECTIONALDRAGAREA_DEBUG" clauses, right?09:01
mzanettiI'd say no09:01
mzanettiQt devs claim it's very fast if the category is disabled09:01
dandradermzanetti, hmmm, but won't it affect runtime performance as you would have at least to do a check on whether you should proceed with the debug09:01
mzanettidandrader: according to their makers they say it won't affect performance really. I don't have any numbers though09:02
dandradermzanetti, and the code that produces the debug message costs something09:02
mzanettidandrader: yeah true. if you do heavy calculations just for the debug message, then yes09:03
mzanettidandrader: http://doc-snapshot.qt-project.org/qdoc/qloggingcategory.html09:04
CimiSaviq, sorry missed the notification - I don't have a nexus 10, when I tried oren one was crashing and was not able to test it09:08
SaviqCimi, no worries - I put more comments on the MP09:08
mhr3sil2100, ping?09:11
sil2100mhr3: pong!09:11
Cimimzanetti, what's the lock screen?09:11
mzanettiCimi: do a "echo password=pin > ~/.unity8-greeter-demo" and restart unity809:12
Cimimzanetti, thx09:12
mhr3sil2100, hey, got a question - should we branch S branches for our projects and continue pushing new stuff to trunk? or is everything setup to land in T now, and therefore enough to create S branches only if we need to sru something?09:12
sil2100mhr3: which projects do you have in mind?09:13
sil2100mhr3: since we basically did a split for all the projects that needed splitting, and trunks are now basically targetting T right now09:13
sil2100mhr3: touch projects didn't get a split as we're not having maintenance branches for saucy there - but those that are shared with desktop have now a trunk branch (for T) and a saucy branch for saucy (usually /saucy or /13.10)09:14
mhr3sil2100, online guys wanted us to push a small change to home-scope and it should be sru-ed to S09:14
=== dandrader is now known as dandrader|afk
sil2100mhr3: ok, then push it to09:15
mhr3ah, yea, i see the /saucy branch now09:15
sil2100lp:unity-scope-home09:15
sil2100and then backport to lp:unity-scope-home/saucy09:15
mhr3ok, cool, thx09:15
sil2100We'll then pick it up for SRUing when the time comes09:16
mhr3sil2100, when will that time come btw?09:16
mhr3roughly09:16
sil2100mhr3: ...I have no idea right now09:16
sil2100Sadly09:16
mhr3well... let me know once you do :)09:17
sil2100Sure ;p09:21
=== dandrader|afk is now known as dandrader
mhr3why does messaging app run some odd indicator app on 101?10:10
mzanettidandrader: but you don't reject the gesture now when a second touch point comes in, right?10:16
mzanettinot a problem for me... just noticed10:17
tsdgeosQt 5.2 Beta Available10:21
dandradermzanetti, not if that second touch point comes after the composition time10:23
mzanettiah...10:23
dandraderrephrasing: if a second touch point comes within the composition time window in the recognition phase (undecide status) the gesture gets rejected. otherwise it's ignored10:24
Saviq/food10:25
nic-doffaypstolowski, we have an issue with no filter option.10:28
nic-doffayJust had a look at the branch again (was on something else)10:29
nic-doffaySince an option always has to be selected if it's a drop down selector.10:29
nic-doffayWe'd need a None option in the backend.10:29
nic-doffayFor Home.10:29
mzanettidandrader: I see traces of a hintingAnimation. where is that used?10:29
dandradermzanetti, you mean from DragHandle.qml?10:31
dandradermzanetti, in the panel10:31
dandradermzanetti, press on the indicators bar. it will come up a bit10:31
mzanettidandrader: ah, that's in draghandle already. I assumed that's in the panel10:31
dandradermzanetti, same when you press on the bottom of an open indicators panel10:32
mzanettidandrader: ah ok.10:32
mzanettidandrader: I managed to break the right edge once. Not able to reproduce yet10:32
mzanettiagain10:33
mzanettiok... happens with trunk too. I finally have a way to reproduce the stuck greeter \o/10:33
dandradermzanetti, what do you mean by "breaking the right edge"?10:34
mzanettidandrader: it's not in the DDA. but try this: drag the greeter away but release it in a way that it would return back to locked state. while it returns, tap on it10:35
pstolowskinic-doffay, oh, that's not good. we would need to add 'All' button in home scope backend for phone only (based on form factor hint). But there is no nice way of giving it a 'None' label just in home10:36
dandradermzanetti, another funny thing is to slide in the launcher and in the middle of it start sliding down the panel with another finger :)10:37
mzanettiyeah. you can operate all edges simultaneously :) but it doesn't break, does it for you?10:38
dandradermzanetti, there's logic to hide the launcher if the panel is open, thus there's some fighting taking place in this scenario10:39
mzanettidandrader: yeah, its worse the other way round. first the panel and then the launcher10:40
nic-doffaypstolowski, yeah it's unavoidable. It's part of the components design. The only way to get around this would be to use an expanded selector, but that wouldn't be per the filter design.10:40
Cimihow do I populate unity with my music/video on the desktop?10:41
Cimiunity810:42
pstolowskinic-doffay, I see10:42
SaviqCimi, banshee or rhythmbox for music10:42
SaviqCimi, videos - just put stuff in your Videos folder10:42
Saviqnic-doffay, don't say it's impossible, and instead try and think of how to solve that conundrum - maybe a property on OptionSelector to allow selecting none or something10:42
Saviqnic-doffay, although I agree the design doesn't really fit with either none-selected or multiple-selected10:43
Saviqnic-doffay, it's a classic single-selection drop down10:43
* Saviq really food now10:43
CimiSaviq, still empty here, maybe I have to wait10:48
mhr3Cimi, what did you do? added stuff in ~/Videos?10:51
Cimiseems to work now10:55
davmor2Cimi: just do a search for a video you know is in there same for music10:56
nic-doffaySaviq, the only way this can be solved is by going against the design which I think is a bad idea. Adding another option in the backend seems like the best solution currently.10:56
pstolowskinic-doffay, for the record, adding new option in the backend will be against design as well ;), because they specifically requested 'all' and alike not to be shown in Home ;)11:00
nic-doffaypstolowski, cool, I'll bring it up with them.11:08
CimiSaviq, which subtitle do we need to support in carousel video?11:08
mzanettidandrader: why is TimeSource abstract? do we need different ones for different devices?11:10
pstolowskinic-doffay, awesome, thanks. keep in mind 'All' in  Home would be misleading (just in case this idea is discussed), because it never shows all results11:10
dandradermzanetti, for the tests. they will implement a fake timer and pass it to the DirectionalDragArea under test11:11
mzanettidandrader: ah, I see11:11
dandraders/fake timer/fake time source11:12
nic-doffaypstolowski, will do11:13
SaviqCimi, comment, probably - same as in music11:15
CimiSaviq, but the video grid does not have them11:15
SaviqCimi, the video carousel does11:15
CimiSaviq, I can add one line for the comment11:16
SaviqCimi, yeah, I believe that's the idea https://lh3.googleusercontent.com/Y9l0nJDyaCA7fZ92jNb6dA5SjlPnvkDy6aFHsZxBwVd9tWVODvgVfvlQ0VwBHurq3fZn7ELxctwF-uh4WvZlnEVZv9eflCTSgCNPC3WgMRDMQ6THrjmRh1Vnqg11:16
CimiSaviq, that's cheesy title, who did this mockup? :D11:17
SaviqCimi, Rosie11:17
Cimihah11:17
Cimicute rosie <3 :D11:17
Cimievery time I drop by her desk she wants gossip on my girls :D11:18
mzanettidandrader: your branch looks really good. left some small comments11:20
* dandrader reads11:20
Cimimzanetti, you have the 13 or 15 retina?11:21
mzanettiCimi: 1511:21
Cimimzanetti, I fancy the 13, but it has dual core11:21
* Cimi upset at apple11:21
Cimiwhy??? why?????11:21
* mzanetti likes looking at htop displaying 8 cores :)11:21
Cimimzanetti, not sure 15 is portable11:22
mzanetti4 + ht that is11:22
Cimimzanetti, if I have to travel to the office11:22
mzanettiCimi: it is. it's so thin and light...11:22
mzanettiCimi:  I always use backpacks tho11:22
CimiI like messenger bags11:22
mzanettiCimi: well, the weight is definitely not the issue11:23
Cimimzanetti, how about hand luggage on flights?11:23
Cimiis it quite big inside?11:23
mzanettiI always keep it in my backpack11:23
Cimiyes but when I fly I have a trolley11:23
mzanettiand it's really thing (ethernet doesn't even fit into it)11:23
Cimiwell, might not fly with the 1511:23
mzanetti-g11:24
CimiiPad air seems pretty cool11:24
CimiI like a device to browse the bed on sofa/bed11:24
mzanettican't install ubuntu on that... not for me11:24
Cimiand one productivety monster11:24
Cimimzanetti, you can have two11:25
Cimimzanetti, one ubuntu one not11:25
* mzanetti doesn't own a tablet...11:25
Cimimzanetti, it's good to have a look at competitors11:25
=== dandrader is now known as dandrader|lunch
mzanettiCimi: sure. but I won't spend 500€ for a thing that can't do anything except displaying browser bookmarks and call it apps11:26
Cimimzanetti, it is expensive11:26
Cimimzanetti, I have a nexus 7 2012 and all browsers are shit11:26
mzanettiI don't mind expensive if it is of real use11:27
mzanettibut I don't feel like the iPad is11:27
mzanettifor me11:27
Cimican't wait to have my hands dirty on our tablet11:27
Cimiso I can push to have what I want ;)11:27
sil2100bregma: hi! I see updates on bug #1243529 - can you assign anyone from your team to modify unity accordingly to the new ABI?11:28
ubot5bug 1243529 in Unity "unity FTBFS on trusty" [Critical,Triaged] https://launchpad.net/bugs/124352911:28
Cimiin the meanwhile, 45 mins to finish downloading mavericks11:28
sil2100bregma: thanks!11:28
* Cimi really likes the ubuntu touch sounds, I put them on android11:29
=== MacSlow is now known as MacSlow|lunch
nic-doffaySaviq, your last comment on the filters mo11:37
nic-doffay*mp11:37
nic-doffayWhat do you mean by the crop line?11:37
Saviqnic-doffay, there's 2gu between the point where the overlay ends and where the content gets clipped11:49
Saviqnic-doffay, and it's transparent to input - you can activate the stuff in the dash through it11:49
SaviqMirv, hey, any plans on building Qt 5.2 somewhere? :D11:51
SaviqMirv, we need to kick the crap out of the new js engine ;)11:52
MirvSaviq: I just heard the beta is out. I'm currently out of steam, but yes makes sense. earlier this week I did manage to push the first trusty 5.1 builds into qt5-beta-proper - it seems it's built for arm https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta-proper/+packages?field.series_filter=trusty11:53
SaviqMirv, cool, no worries - get some rest :)11:53
Saviqyou've been busy the last few weeks - we had it easier the last days at least11:53
nic-doffaySaviq, cool, gotcha]11:54
MirvSaviq: no rest, but other stuff like the first trusty landings of ~everything :)11:54
MirvSaviq: oh right, those 5.1.1 trusty builds are now debug builds11:55
SaviqMirv, meaning with CONFIG+=debug?11:55
=== alan_g is now known as alan_g|lunch
Saviqtsdgeos, you'll be pleased ↑ ;)11:55
MirvSaviq: so the qtbase with -debug and the rest with CONFIG+=debug11:55
SaviqMirv, cool beanz11:56
tsdgeosMirv: so qtbase is still not debug?11:57
MirvI'll update the recipes to build for trusty too11:57
Mirvtsdgeos: it has a configure switch -debug instead11:57
Mirvtsdgeos: I'd trust it's the same thing as qtbase doesn't have a qmake to give the parameter to before it has built the qmake in the first place?11:57
greybacktsdgeos: did you get anywhere with running unity8 with 5.2?11:58
greybackI've not tried since early last week tbh11:58
CimiSaviq, I updated the branch, but still waiting jouni for a better asset for music12:14
SaviqCimi, k12:14
CimiSaviq, I have no idea where he is12:15
CimiSaviq, first day I stay home for working he is not online :)12:15
Cimidammit12:15
=== dandrader|lunch is now known as dandrader
nic-doffayCimi, have you gotten hold of him?12:18
Ciminic-doffay, nope12:19
* Cimi install mavericks, see you soon (hopefully)12:19
=== jhodapp|afk is now known as jhodapp
Cimibest use of lunch break I hope12:19
Saviqgreyback, hey, this should be relatively easy to track down https://bugs.launchpad.net/unity-mir/+bug/1243444/comments/512:37
ubot5Ubuntu bug 1243444 in unity-mir "Unity (and sometimes maliit) crashes after maliit-server is restarted and is used." [Medium,New]12:37
greybackSaviq: ta12:38
larsutsdgeos: yes, that's known and a fix is in -proposed (libindicator)12:51
tsdgeoslarsu: awesome12:51
larsutsdgeos: thanks for the report!12:51
tsdgeosMirv: ah sure, i read "-debug" as "wihtout" debug, not as -debug passed to configure :D12:52
tsdgeosgreyback: didn't try because it wanted more stuff to be linked against 5.212:52
greybacktsdgeos: ah yes you mentioned that. Ok so12:53
Saviqtsdgeos, here's one for you to look at in your KDE hat: bug #124040813:03
ubot5bug 1240408 in Unity 8 "unity8 crashed with SIGSEGV in QCoreApplication::postEvent()" [High,Triaged] https://launchpad.net/bugs/124040813:03
tsdgeosSaviq: ?13:03
Saviqtsdgeos, it's all Qt in the stack trace ;)13:04
tsdgeosah :D13:04
Saviqtsdgeos, I suspect either the album art provider fails, or the qpixmapcache13:04
tsdgeosok, will have a look13:04
tsdgeoshow urgent is it?13:04
Saviqtsdgeos, and since the album art provider deals with QImages13:04
Saviqtsdgeos, relatively urgent - it reliably crashes the shell if you expand Music in the home scope13:05
Saviqtsdgeos, but not something that should throw you off of whatever you're doing now13:05
tsdgeosok13:05
Saviqespecially since we can't merge the fix anyway ;)13:05
* tsdgeos never had any crash in the music scope .S13:05
tsdgeosSaviq: desktop too? or phone only?13:06
Saviqtsdgeos, not music scope - music in *home* scope13:06
Saviqtsdgeos, or when opening a music preview, apparently13:06
Saviqtsdgeos, didn't try on desktop, let me13:06
dednicklarsu: ping13:07
Saviqtsdgeos, doesn't seem to crash on desktop, no - but we don't get music in Home on desktop13:07
larsudednick: hey13:08
Saviqtsdgeos, no crash after searching for music in home and expanding, either13:08
Saviq+on desktop13:08
tsdgeosi see13:08
tsdgeosso only crash is13:08
Saviqtsdgeos, expand music in Home - that was reliable for me13:09
tsdgeosexpand music in home in phone before searching13:09
tsdgeosor13:09
Saviqtsdgeos, yeah13:09
tsdgeosok13:09
tsdgeosSaviq: shall i get trusty on the phone? or the "old" images ?13:10
Saviqtsdgeos, I had trusty13:10
dednicklarsu: hi. I've been working on the datetime indicator and it gives me time + time-format (in strftime format). I'm going to need your TimeFormatter to parse it, but it's taking Qt time. Any way we can Separate the TZ part from the Formatter so we can use it like a plugin?13:10
tsdgeosook, so let's do trusty then13:10
Saviqtsdgeos, so yeah, trusty-proposed to get the latest image13:10
dednicklarsu: although i guess this could be a later mod13:10
chelgrianMay I ask a question about Icons in the unity dock?13:11
dednicklarsu: *taking Qt time format13:11
dandradermzanetti, updated13:11
mzanettidandrader: ack13:11
larsudednick: if you don't need the timezone changing functionality, can't you just call strftime?13:11
dednicklarsu: it gives a time in us from epoc. so will need the tz change.13:12
dednicki'm guessing13:12
* chelgrian has an QT executable which has no .desktop file and doesn't oviously call a setIcon or other method but never the less seems to show an icon in the dock rather than a ?. I was looking for pointers on how to work out where it is getting this image from.13:13
larsudednick: strftime does this for you...13:16
larsudednick: or does the service give you a timezone as well?13:16
* larsu rereads your question13:16
dednicklarsu: we dont get the change notification. :)13:16
dednicklarsu: nevermind. I'll sort it out once the formatter is in.13:17
dandradermzanetti, is CI working at the moment or did it get confused by the saucy->trusty transition?13:17
larsudednick: the service should notify you about tz changes, no?13:17
mzanettidandrader: ci is still on saucy afaik. but there's an issue that we have a very dirty hack when installing unity8 on the phone which doesn't work on the read only image. so ci can't install the test packages there and fails13:18
larsudednick: because I could have the clock set to a different tz than my system is running on13:18
mzanettidandrader: that's why autolanding is disabled right now and we have 15 or so approved branches waiting to land13:18
dednicklarsu: This is for the alarms/event menu items in the indicator. They are given in us since epoc, which wont change when the tz changes, so it wont update the UI. I'm saying i need to use your TimeFormatter so that we get the dbus notification that the timezone has changed so we can update the UI.13:20
dednicklarsu: as far as i know, same as you did for message indicator13:21
dednicklarsu: but the datetime indicator also provides the format which we want to show the alarm in.13:21
=== MacSlow|lunch is now known as MacSlow
larsudednick: I know, my point was that the service has a timezone property as well. If you have a timezone and a time, you don't need and 3rd party notifications. But it just occured to me that the service doesn't need to export a tz, because it changes the system's tz as well. So your way would work as well.13:24
larsudednick: if we're going that route, I'm totally fine with extending TimeFormatter13:25
dednicklarsu: ok.13:26
dednicklarsu: is g_bus_get and g_dbus_connection_signal_subscribe lightweight calls?13:27
dednicklarsu: still not sure about doing it for every TimeFormatter :)13:27
larsudednick: signal_subscribe adds an entry to a hash table.13:28
larsudednick: g_bus_get adds an event to the end of the message queue in the usual case (which is that we already have a connection to the system bus)13:28
dednicklarsu: yeah, it's the queue that i'm concerned with.13:29
larsudednick: it's a mainloop, we have tons of events on there already. I wouldn't worry about that tbh13:30
dednicklarsu: meh. we can singleton'ize it later i guess.13:30
larsudednick: ya, if there's a problem. Also, having a singleton means doing atomic ref counting...13:30
larsuI just didn't think it was worth it13:30
dednicklarsu: ok. i'll review and test the branch now.13:31
tsdgeoseh?13:31
tsdgeoshow did i quit the hangout13:32
=== _salem is now known as salem_
larsudednick: thanks13:45
=== alan_g|lunch is now known as alan_g
=== dandrader is now known as dandrader|afk
=== dandrader_ is now known as dandrader
tsdgeosMirv: i'd say you apply only the patch i gave you (the fix) instead of fix+test14:37
tsdgeosthe test is upstream and proves it works14:37
tsdgeosnot sure we need it in our packages14:37
tsdgeosMirv: if you really want the test in the patch too i can try to port it against 5.1.114:38
mzanettiSaviq: just had a chat with jhodapp about our edges. and he has a valid point that we're triggering them too easily. and I think shell.edgeSize should be rather 1 gu instead of 2 (or even less). Would you agree? I could create a branch trying to tune it a bit14:49
tsdgeosmzanetti: was it you that did the narrowMode thing for the PageHeader/Search ?14:58
mzanettitsdgeos: nic-doffay15:02
nic-doffaymzanetti, yep15:02
tsdgeosnic-doffay: i may have to change that code a bit with the upcoming "tabbar as header", what's the logic in there? the idea is that on the tablet you get the search in a different place? right?15:05
nic-doffaytsdgeos, it's still in the same place.15:07
tsdgeosok15:07
nic-doffaytsdgeos, it's all governed by internal states in the component.15:07
tsdgeosyes15:08
tsdgeosi have to change the component15:08
tsdgeosthat's why i am asking15:08
nic-doffaytsdgeos, what sort of changes? Chances that it will create big conflicts?15:08
tsdgeosprobably15:09
tsdgeosas said15:09
tsdgeosthe label goes away15:09
tsdgeoswe have a tabbar tere15:09
tsdgeostere -> there15:09
tsdgeoslike the one in indicators15:09
tsdgeoswonder how the searchbar has to play with it15:09
tsdgeosi need a designer :D15:09
Saviqmzanetti, problem with 1gu is that the touch hardware is crap15:39
mzanettiSaviq: oh really? *that* bad?15:40
Saviqmzanetti, and if you swipe too fast it will just go unnoticed, 'cause the first touch you get is already beyond the 1gu threshold15:40
Saviqmzanetti, yes, dandrader knows all about it15:40
Saviqmzanetti, just run the qml edge test under surfaceflinger15:40
Saviqmzanetti, and you can experiment there15:41
mzanettiSaviq: ah ok. but we do have some improvements on our roadmap, right?15:41
mzanettior is the general opinion that this is good enough?15:41
Saviqmzanetti, not in the strictness with which we detect whether something's an edge drag or not15:41
Saviqmzanetti, we've loosened the rules after the first iteration15:41
Saviqmzanetti, 'cause there were too many false negatives15:41
mzanettiSaviq: but right now we a lot of false positives15:42
Saviqmzanetti, what we are looking at is allowing apps to get the events on the edges if they're *not* edge swipes15:42
Saviqmzanetti, that's not really for us to make that determination15:42
nic-doffaytsdgeos, were you chatting about this new bar that's obscuring the current search field earlier?15:43
tsdgeosnic-doffay: which new bar?15:43
Saviqmzanetti, the situation we have now was an improvement according to people that have complained about too many false positives15:45
Saviqmzanetti, so it's really a he-said/she-said - maybe we didn't catch the right balance yet15:45
Saviqmzanetti, but still, none of us are the right judges I'm afraid15:45
mzanettiSaviq: yes. I agree that at some point we were too strict15:45
Saviqmzanetti, feel free to file an ubuntu-ux bug, try and explain in what situations you're getting the false positives15:46
mzanettiSaviq: ok, I will15:47
Saviqmzanetti, I talked to jhodapp yesterday or so and IIUC his beef was mostly with the bottom edge15:47
mzanettiSaviq: jhodapp made a good example. try tapping the time in the video player15:47
Saviqmzanetti, which does not have the fancy edge detection that the shell has15:47
Saviqmzanetti, that's different15:47
mzanettiSaviq: no15:47
mzanettiSaviq: when holding the player in landscape15:47
Saviqmzanetti, nothing should happen on tap15:47
mzanettithe right edge (which at that time is the bottom one) kicks in15:47
mzanettiSaviq: exactly15:48
mzanettiSaviq: but it does15:48
Saviqmzanetti, so you're not tapping ;)15:48
mzanettiSaviq: well... we could argue on that :D15:48
Saviqmzanetti, not sure what the minimal movement is - maybe that's just too small15:48
mzanettiSaviq: I think our rules are too loose15:48
Saviqmzanetti, tbh whatever I do, when I tap, the right edge does not kick in...15:49
mzanettiSaviq: I guess our first iteration had the angle way too strict. but beause of that we also dropped minimum speed etc15:49
jhodappSaviq, you're trying that from the mediaplayer in landscape?15:49
Saviqjhodapp, yes15:49
Saviqmzanetti, it wasn't random, though - people sat down and tried to find a balance - and that's what we have now15:50
jhodappSaviq, using your thumb?15:50
Saviqjhodapp, no, you can't tap with a thumb :D15:50
Saviqjhodapp, still15:50
jhodapplol15:50
Saviqjhodapp, I can make a video ;D15:50
mzanettiI can tap on the N9 with a thumb15:50
mzanetti:P15:50
Saviqjhodapp, mzanetti, tapping with a thumb does not bring the right edge for me, no15:50
mzanettidoes it toggle the time display?15:51
jhodappinteresting...maybe you have an improved image that I don't?15:51
Saviqjhodapp, it wasn't touched15:51
Saviqmzanetti, that's a different thing15:51
Saviqmzanetti, the fact that it doesn't toggle is indeed a problem - and one that we're solving15:51
Saviqmzanetti, but that's not the same as the edge being triggered15:52
jhodappSaviq, yes, I see what you're saying now and the same happens for me15:52
* mzanetti tries again15:52
SaviqI need to do an explicit edge gesture to get the right edge15:52
jhodappI don't trigger the gesture most of the time, but it is very hard to get the time to toggle15:52
Saviqjhodapp, ok then - that's two different things :)15:52
Saviqjhodapp, and yes - that we're working on (dandrader is)15:52
jhodappSaviq, but I can tap to get the edge gesture just below the time15:52
Saviqjhodapp, that doesn't happen here15:53
jhodappinteresting...no matter how close to the edge you get?15:53
Saviqjhodapp, no - it's not about how close to the edge you get15:54
jhodappSaviq, seems to be for me15:54
Saviqjhodapp, you need to be doing a 1.5GU movement ~perpendicular to the edge15:54
Saviqjhodapp, for it to be triggered15:54
mzanettijhodapp: just tried it again. I think you're confusing the edge drag gesture with the panel movement15:55
jhodappSaviq, I'll have to take a video of what I'm doing, because I am not sliding my finger at all15:55
Saviqjhodapp, the edge drag gesture would pull the panel in as well - does it?15:55
Saviqjhodapp, I mean the indicator sliding in - that's the first indication that the edge gesture was triggered15:56
Saviq*indicators15:56
jhodappSaviq, let me show you my video, you'll see what I'm saying then15:56
jhodappone min15:56
Saviqjhodapp, there is a bug in media player though, that causes the app panel to move when you tap it for some reason15:56
jhodappthis is different15:57
mzanettiSaviq: is all the hardware that bad or is it just some of them?15:57
Saviqmzanetti, right - yes we were thinking of doing it per-device15:58
mzanettijust tried the n9. their edge drag area seems to be really really thin... Its better hardware for sure than the google stuff we use.15:58
Saviqmzanetti, or even increasing the threshold with time - when the user starts learning the gesture15:58
mzanettibut I think it might be worth getting the max per-device yes15:58
jhodappSaviq, mzanetti, just forwarded the video to your emails15:58
mzanettijhodapp: right... I noticed that too before16:00
mzanettiSaviq: this is really triggering the edge gesture16:00
mzanettiand there's no way he does the 1.5 gu movement on that video16:00
jhodappexactly16:00
Saviqjhodapp, that's a galaxy, btw?16:00
jhodappyes16:00
jhodappmaguro16:01
mzanettiI've seen this too before. but I just can't reproduce it any more now16:01
CimiSaviq, the carousel branch should be ready16:01
SaviqCimi, k16:01
Saviqjhodapp, so yeah, that's obviously *not* how it's supposed to be, but I'm not seeing that - and haven't seen for a while now16:02
Saviqjhodapp, can you try in portrait and, say, gallery16:02
jhodappSaviq, weird...sure16:02
jhodappSaviq, doesn't happen in gallery16:03
jhodappnor in portrait16:03
mzanettijhodapp: I rebootet the phone and it's gone. could you please try to find a way to reproduce getting into this state?16:04
mzanettias you're saying it happens often for you16:04
jhodappmzanetti, weird...let me reboot16:04
* mzanetti has an idea16:05
Saviqjhodapp, I'm feeling this has to do with the media player's panel, and it's not actually the edge drag gesture, but maybe the panel's interaction with HUD instead16:05
mzanettiyep... so it seems to only happen if the media player is the only opened app16:06
jhodappSaviq, I think you might be correct, I just rebooted and it's still there16:06
jhodappmzanetti, oh interesting16:06
jhodappSaviq, so do you think filing a bug with mediaplayer-app is appropriate, or the SDK?16:07
Saviqjhodapp, start with mediaplayer-app, unless you can confirm with another app - say webbrowser in landscape?16:07
mzanettijhodapp: well, besides the media player's panel behaving weird, I think the fact that you can see unity8's panel popping in is unity8's fault16:07
jhodappmzanetti, good point16:07
greybackSaviq: who good to review: https://code.launchpad.net/~gerboland/unity-mir/crash-fix-on-IFA-removal/+merge/19235216:08
jhodappSaviq, mzanetti: alright, I'll file a bug with mediaplayer-app and unity816:09
mzanettiSaviq: happening with gallery too. but only if there's only one running app16:09
jhodappmzanetti, do you know the URL of where unity8 bugs get filed?16:09
Saviqmzanetti, so that looks like something with fullscreen apps16:09
Saviqgreyback, -DLOG +LOG, that on purpose?16:09
mzanettiSaviq: yeah, it's probably the same with others but you don't notice it16:10
mzanettiSaviq: with fullscreen you see the panel coming in16:10
Saviqmzanetti, yeah maybe16:10
Saviqgreyback, wow man, you've been playing with mir too long ;P16:10
Saviq[=]()16:11
greybackSaviq: heh16:11
Saviqthat a chimpanzee?16:11
greybackSaviq: read your C++11 book :)16:11
Saviqone that has its body temp measured?16:11
Saviqgreyback, only thing - we never disconnect - can it happen that an input area is moved between two surfaces?16:12
mzanettijhodapp: lp:unity816:12
jhodappthanks mzanetti16:13
greybackSaviq: I don't see why not, the API let's that happen anyway.16:13
Saviqgreyback, yeah, so should we not disconnect from the old surface?16:13
greybackSaviq: am open to discussing that. These APIs could do with a review16:13
greybackSaviq: fair point16:13
greybackSaviq: ah yeah, I forgot: I expected that disconnectFromAscendantsChanges would do the disconnect for me16:15
greybackSaviq: though that may not be 100% correct16:15
Saviqgreyback, ok, will look into it tomorrow16:18
greybackSaviq: np, lemme chew over it a bit16:18
sil2100ChrisTownsend: hi!16:23
ChrisTownsendsil2100: Hey!16:23
sil2100ChrisTownsend: any progress with #1243529 ? :)16:23
sil2100https://bugs.launchpad.net/unity/+bug/124352916:24
ubot5Ubuntu bug 1243529 in Unity "unity FTBFS on trusty" [Critical,Triaged]16:24
ChrisTownsendsil2100: Working on it.:)  Only having xpathselect1.4 in the daily-build PPA has been a challenge to get a local build up and running, but I've overcome that and now I'm working on the code.16:25
ChrisTownsendsil2100: Trevinho has a branch hanging around that fixes this, but the branch does a little too much for my taste, so I'm stripping out only what is needed to get this to work.16:26
Saviqcrap, I just noticed I can't switch to a remote session without logging out of my local one...16:36
SaviqIIRC there was a "switch users" entry in the session menu before, but it's nowhere to be found now :/16:37
Saviqah ok, have to switch to the guest session and log out... <facepalm> :|16:39
mdeslaurSaviq: click lock, and then click the switch users button in the screensaver dialog16:39
mdeslaurSaviq: or press ctrl-alt-f116:39
mdeslauroh wait, no16:39
sil2100ChrisTownsend: awesome! Thanks for the update :)16:40
Saviqmdeslaur, right, that's where the "switch users" is ;)16:40
mhr3odd, i still have it in the indicator16:40
sil2100ChrisTownsend: I would also prefer just to fix the issue at hand, and do additional fixes/refactoring later in some other merge16:40
Saviqshame when I try to configure the remote account the session dies anyway :/16:40
mhr3right above Guest16:40
Saviqmhr3, I only have Lock16:40
ChrisTownsendsil2100: Yep, that's my aim.16:40
mhr3it says "Lock/Switch account" here16:40
mhr3Saviq, ^16:41
mdeslaurmhr3: that's odd, it should only say lock16:41
Saviqlol16:41
mhr3mdeslaur, and if i boot guest session it says *only* "Switch account" :)16:42
mdeslaurmhr3: yes, the lock screen is disabled in the guest account as it doesn't have a password16:42
mdeslaurso that part makes sense16:42
mhr3but anyway, i'd find it, so my ubuntu is better :P16:43
Saviqmdeslaur, do you know who to talk to about being unable to get to the "configure" part of the remote sessions?16:43
mdeslaurSaviq: hrm...good question...what happens when you try it?16:43
Saviqmdeslaur, it just goes back to the greeter16:44
Saviqmdeslaur, nothing obvious in the lightdm logs, too16:44
mdeslaurSaviq: I'd file a bug against lightdm16:44
Saviqmdeslaur, but guest session works16:44
mdeslaurSaviq: it may be a side-effect of bug 124333916:44
ubot5bug 1243339 in lightdm (Ubuntu Trusty) "lightdm no longer runs guest session through wrapper" [High,Confirmed] https://launchpad.net/bugs/124333916:44
Saviqmdeslaur, could be16:45
mdeslaurSaviq: let me know the bug number when you file it16:46
=== jalcine_ is now known as jalcine
Saviqmdeslaur, bug #124380116:52
ubot5bug 1243801 in lightdm (Ubuntu) "Can't access remote login "Configure now" session" [Undecided,New] https://launchpad.net/bugs/124380116:52
mdeslaurthanks Saviq16:53
Saviqmdeslaur, the greeter log mentions a lack of /usr/share/lightdm/sessions - I do have remote-sessions, wonder if that's related16:54
mdeslaurSaviq: any apparmor logs in dmesg?16:55
Saviqmdeslaur, yeah16:55
mdeslaurSaviq: can you add them to the bug? it may be an issue with the remote sessions apparmor profile16:55
Saviqmdeslaur, will do16:56
Saviqmdeslaur, https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/1243801/comments/416:56
ubot5Ubuntu bug 1243801 in lightdm (Ubuntu) "Can't access remote login "Configure now" session" [Undecided,New]16:56
=== alan_g is now known as alan_g|EOD
CimiSaviq, is there anything I can run while testing unity on the desktop to have an app17:08
Cimilike the old cameramockup17:08
SaviqCimi, yeah, the old cameramockup ;)17:10
CimiSaviq, doesn't run here17:10
CimiSaviq, looks for camera desktop17:10
SaviqCimi, you need to export XDG_DATA_DIRS17:11
SaviqCimi, should be added to run17:11
CimiSaviq, which value for that?17:11
SaviqXDG_DATA_DIRS=tests/mocks/data/:$XDG_DATA_DIRS ./run17:11
Saviqmdeslaur, yeah, having added the dbus abstractions to the apparmor profile it works again17:25
Saviqnow I can't log in... wonder if lightdm supports dual-factor auth17:26
Saviqmterry, do you know ↑?17:26
mterrySaviq, should...  it's about what PAM supports and if the greeter happens to correctly handle the PAM prompts17:27
mterrySaviq, which greeter?17:27
Saviqmterry, current unity-greeter17:27
mterrySaviq, sure, it should17:27
mterrySaviq, does it work on a VT?17:27
Saviqmterry, yes17:27
mterrySaviq, and in greeter, what happens?17:28
Saviqmterry, so I was trying to log in to use the remote login features17:28
Saviqmterry, typing in my U1 mail/pass - and it just comes back with "wrong password" :/17:28
mterrySaviq, sorry, got pulled away17:33
mterrySaviq, oh interesting...  remote login doesn't use PAM17:33
mterrySaviq, and got developed by tsgeos mostly I think.  I don't know about its support for U1 dual-factor17:33
Saviqmterry, tsdgeos? you sure?17:35
Saviqmterry, or tseliot?17:35
mterrySaviq, I probably meant tseliot17:36
mterrySaviq, there was definitely a ts17:36
mterrySaviq, but I could totally believe it got left out.  That feature was written close to FF17:37
Saviqmterry, yup, thought so17:37
* Saviq tries disabling SSO17:37
=== boiko_ is now known as boiko
=== salem_ is now known as _salem

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