/srv/irclogs.ubuntu.com/2014/11/28/#ubuntu-unity.txt

Mirvtsdgeos: FYI I'm running AP:s on the qtbase ubuntu8, and it's looking good/better now08:08
tsdgeos:)08:08
tsdgeosmeanwhile it's been approved upstream08:08
tsdgeosstill not merged because their CI basically sucks08:08
tsdgeoswell not their CI08:09
tsdgeosit's just that their tests are unstable as hell08:09
Mirvyeah, enabling the unit tests has been troublesome for our builds too, but at least we nowadays have them mostly enabled08:10
Mirvtsdgeos: so do I already have +1 from you on that or shall you test it today?08:10
tsdgeosi tested it with my self built thing, i can give it a try with the proper pacakges08:11
tsdgeossame silo?08:11
Mirvtsdgeos: yes please try the PPA, 027, check that you get ubuntu808:11
tsdgeosoki08:11
Mirvtsdgeos: not necessarily all perfect, I at least need reruns and a moment ago I had security code being asked which sounds like unity8 restart08:32
MirvUITK passed everything though, while before it barely get through a couple of tests08:32
tsdgeosi see08:36
tsdgeoswonder if that restart/reruns is something we already had without this patchset though08:36
tsdgeosMirv: i had never run uitk tests before, what's your opinion?08:37
Mirvtsdgeos: not during UITK tests, and like I said I didn't get them there but later (I think during calculator AP). the way I'm running all AP suites in one pass causes all kinds of random trouble, so this might be it.08:45
tsdgeosi see08:45
MirvI'm mostly worried about a lot of ubuntu-system-settings AP failures + that security code screen appearing (I think that's uncommon), but I'll see when I get to do reruns of the failed suites in about 2h when this pass finishes08:45
Mirvusually the fact that whole UITK suite passes without any problems is a good indication, so let's hope there's nothing reproducible.08:46
tsdgeosMirv: one good thing to do08:53
tsdgeosis phablet-hsell08:53
tsdgeosand then08:53
tsdgeossudo gdb attach `pidof unity8`08:53
tsdgeosthis way if unity8 crashes it'll get stuck there08:54
tsdgeosand you can later on bt and all that08:54
Mirvtsdgeos: there was a crash, and I remembered the whoopsie log, so here's the crash https://errors.ubuntu.com/oops/faf1d51c-76d8-11e4-966f-fa163e75317b09:36
tsdgeosMirv: not much of a backtrace there :/ doesn't seem dbus related, but you can never tell if it's because of the patches or not09:38
Mirvtsdgeos: well there's _g_dbus_auth_run_client in the trace, but not really anything Qt09:40
Mirvanyhow, since I had the test run stuck in mir_screencast (which is common...) I'll start the rerun now omitting those that had full pass.09:41
tsdgeosyeah here it gets stuck there too09:41
tsdgeosi was wondering if it was common or not09:42
=== vrruiz_ is now known as rvr
=== karni is now known as karni-otp
=== karni-otp is now known as karni
tsdgeosMirv: been testing it, could not find any anomally, so +1 for the patchset from my side11:02
=== _salem is now known as salem_
Mirvtsdgeos: thanks. I'll need to do this AP testing for some while still... and probably need to compare with/without PPA until there's some sort of consistency.11:42
dandraderSaviq, is ./run.sh still being used nowadays?12:17
Saviqdandrader, sure, although it's not been fixed for split dash12:18
=== alan_g is now known as alan_g|lunch
=== MacSlow is now known as MacSlow|lunch
greybackSaviq: have chance for second pass? https://code.launchpad.net/~mir-team/qtmir/cmake/+merge/23523912:32
Saviqgreyback, yeah, although I hate cmake today (been refactoring QmlTest.cmake for a long time now) ;P12:37
greybackSaviq: I don't have a lot of love for it either12:38
greybackif you'd heard my cursing when you poked your head in and started commenting... :D12:38
Saviqgreyback, it's not like I was picky (yet) :P12:39
* greyback thinks qmake ain't so bad now12:40
=== karni is now known as karni-lunch
=== alan_g|lunch is now known as alan_g
Saviqgreyback, none of the inline comments are addressed in there13:19
greybackSaviq: damn sorry, I pushed to wrong branch13:23
greybackupdates are there now13:23
Saviqgreyback, nw13:23
Saviqgreyback, I'll move it to Needs Review to get ci feedback13:24
greybackok13:24
Saviqgreyback, hmm, shouldn't Unity.Application remain in the shell-specific QML import dir?13:26
greybackSaviq: it never was there13:26
Saviqgreyback, ah indeed, we should probably move it there then13:27
greybackSaviq: it's not necessarily unity specific13:27
greybackI'm considering renaming it13:27
Saviqgreyback, hmm ok, maybe we'd need another custom import path, not sure it's good in the default one13:28
Saviqgreyback, ah, and there's a running effort to rename qtdeclarative5-foo13:29
greybackyep that too13:30
=== karni-lunch is now known as karni
=== MacSlow|lunch is now known as MacSlow
=== dandrader is now known as dandrader|afk
mterryCimi, what's the story with the wizard branches?14:07
Cimimterry, thought you were on holiday :D14:08
mterryCimi, was yesterday and was sick the day before14:08
mterryCimi, but I'm back!14:08
Cimiah ok, let me test... thought I could do that later in the day for monday14:08
tsdgeosCimi: there?14:20
Cimitsdgeos, yes14:20
tsdgeosCimi: have you shown https://code.launchpad.net/~cimi/unity8/elastic-not-close-spread/+merge/242949 to someone in design?14:20
Cimitsdgeos, yes, approved14:20
tsdgeosit seems very similar to what we have now14:20
Cimitsdgeos, what do we have "now" ?14:21
tsdgeoswell i never drag that much when closing an app14:21
tsdgeosi just flick14:21
Cimitsdgeos, well, probably people who tested did...14:21
tsdgeosand the flick back we have now and this one are the same14:22
Cimitsdgeos, yes but this limits the movement14:22
tsdgeosbut ok, i guess it's still and improvement14:22
Cimitsdgeos, vesa tried the branch and approved14:22
Cimitsdgeos, he even asked me to tweak a value :/14:22
tsdgeoshe could have written so in the thing, saved us this trouble :D14:22
=== dandrader_ is now known as dandrader
Saviqgreyback, any idea what caused bug #1396244 btw? it's fine on rtm still...14:24
ubot5bug 1396244 in unity8 (Ubuntu) "Unlocking the phone over dialer is very slow" [High,Triaged] https://launchpad.net/bugs/139624414:24
Saviqgreyback, we should try and make sure to not land what caused this14:25
greybackSaviq: can get it with camera too. Seems fullscreen + lock causes the slowdown.14:29
Saviqgreyback, yeah, any idea of a change that could've introduced this?14:29
greybackno idea14:30
Saviqok, I'll have to remember to look out for this when landing in RTM14:31
greybackwould be worth bisecting14:32
greyback*could* be that when screen locked, panel is brought in, and app is told to resize its surface to non-fullscreen - which would be wrong. And then on unlock, it gets a flood of resize events14:33
greybackjust a theory tho14:33
* greyback runs to post office14:34
tsdgeosCimi: any chance you'll review https://code.launchpad.net/~aacid/unity8/photoscopeimprovements/+merge/239834 ?14:44
Cimitsdgeos, I think is fine to go14:45
Cimitsdgeos, as said, only bugs I had we agreed they were unrelated14:45
Saviqtsdgeos, I'll put it in a silo for vivid today, Monday latest14:45
tsdgeosSaviq: oki14:48
tsdgeosdednick: you have conflicts in https://code.launchpad.net/~nick-dedekind/unity8/lp1385331.led/+merge/24141714:50
Cimimterry, wizard page after wifi seems broken here, where is upstart log?15:03
Cimimterry, wizard page after wifi seems broken here, where is upstart log?15:08
mterryCimi, hmm that would be the location page -- upstart is ~/.cache/upstart/unity8.log15:09
mterryCimi, it doesn't show up?  Just looks blank?15:09
Cimimterry, I see just wallapper15:09
mterryCimi, yeah hmm.  On which device?  I'll try to reproduce/fix15:09
Cimimterry, mako15:09
mterryCimi, ok -- it just means the wizard is waiting for an answer to the question "is HERE installed" before showing the location page or not15:10
Cimimterry, thought about here15:10
Cimimterry, maybe I can install it and see if it goes after15:10
mterryCimi, well we need to work in the mako/no-here case though :)15:11
Cimimterry, sure, was just to confirm...15:11
mterryCimi, you have the latest debs?  I changed how we look for HERE to avoid needing the special " " value -- might have broken something15:11
Cimimterry, the last on -test15:12
mterryCimi, hm yeah I see same thing, will let you know when I've fixed15:19
Cimimterry, thanks15:19
greybackSaviq: re those cmake errors, I've no idea what's causing them. Have you seen them before? Was suspecting a qt5 module but have zero evidence for it15:31
alesageSaviq, reasonable suggestion re: cmake-extras, will look into today15:51
Saviqalesage, there's actually a branch already, but it's a bit naïve, just s/2.8/3.0/15:55
karniHi folks. Who can I talk to about the badge implementation?15:55
Saviqkarni, define "the badge"15:55
Saviqgreyback, no, have not seen them yet15:55
karniSaviq: the number that shows up on the app icon in the launcher at the left edge15:56
alesageSaviq, ack, I possibly need Satoris' advice on this15:56
Saviqkarni, ah, that one, you can talk to me since mzanetti's only pretending to be around15:57
Saviqkarni, "count emblem" is what we call it I think ;)15:57
karnihaha, all right.15:57
karniSaviq: let us file a bug, I'll contact you shortly. basically, it seems to be adding the value to the badge, instead of setting the value of it. if we go it wrong and it's documented, then I'll blame myself.15:58
karniSaviq: we pass the "unread messages count" in there, but it seems it's adding them app, is this expected?15:59
Saviqkarni, you're talking about push notifications here are you?15:59
karniSaviq: yessir15:59
Saviqkarni, in that case you'd have to talk to Chipaca15:59
karniSaviq: okay :) WFM15:59
Saviqkarni, we're setting the value as-is15:59
karnigotcha, good news for me16:00
karnithanks Michał :)16:00
Saviqgreyback, yeah, looks like Qt5Core is the culprit16:04
Saviqgreyback, also, just found...16:05
Saviq""The qt5_use_modules macro is obsolete. Use target_link_libraries with IMPORTED targets instead."16:05
greybackoh16:05
greybackok fixing16:06
Saviqgreyback, sorries16:06
greybacknp16:06
Saviqgreyback, but it looks like you still don't need explicit INCLUDE_DIRS then16:06
greybackSaviq: you do for private header dirs16:07
Saviqgreyback, yeah, that you do of course16:07
tsdgeosshouldn't http://paste.ubuntu.com/9284813/ make going from one scope to the other slower?¿16:13
tsdgeosbecause it doesn't16:14
=== dandrader is now known as dandrader|lunch
Saviqtsdgeos, no16:47
Saviqtsdgeos, we don't have a highlight16:47
Saviqgreyback, cmake done, was a bit more picky this time16:48
greybackSaviq: oh gee great thanks16:48
tsdgeosSaviq: well, hmmm, right, damned code that does nothing and confuses me :D16:49
Saviqgreyback, I thought you'll like it on a Fri afternoon16:49
Saviqtsdgeos, trueth16:49
Saviq*you'd16:50
tsdgeosSaviq: though we kind of have highligh16:50
tsdgeoshighlightRangeMode: ListView.StrictlyEnforceRange is what makes the current index chane16:50
tsdgeoschange16:50
Saviqtsdgeos, right, but we don't have a highlight object, really16:51
tsdgeosright tight16:51
Saviqtsdgeos, but I get what you mean16:51
Saviqtsdgeos, we're abusing the ListView there a bit... and we have no control over the swipe threshold, which I was asked about recently16:51
Saviqwe might need to stop abusing ListViews for PageLists :P16:52
tsdgeosSaviq: nor about the snap speed for what i can see16:52
Saviqtsdgeos, nope16:52
tsdgeosmeh :/16:52
tsdgeosi *think* the snapping animation is borked going back/front in some cases16:52
tsdgeosbut i can't "see" it, thus i wanted to make it go slower16:53
greybackQUnifiedTimer::instance()->setSlowMode or something like that16:53
Saviqok I think I've enough cmake for one night and one day...16:55
tsdgeoshe he16:58
Saviq*almost* there16:59
Saviqeverything was fine, and then someone added dbus-launch to their test, *really*, I ask you!?17:00
Saviqand then there's dbus-test-runner, even worse17:00
tsdgeosyep17:01
Saviqhave a good weekend, y'all! o/17:03
tsdgeosyeah i'm leaving too17:04
tsdgeosoff to a cabin17:04
tsdgeosthe weekend of the year with most predicted rain17:04
tsdgeos\o/17:04
tsdgeos:D17:04
tsdgeos:/17:04
Cimiahah17:04
Cimitsdgeos, where u heading?17:04
tsdgeosBesalú17:04
tsdgeosnice romanic vilalge17:05
tsdgeoshttp://upload.wikimedia.org/wikipedia/commons/a/a5/Pont_Medieval_%28Besal%C3%BA%29_-_7.jpg17:05
Cimitsdgeos, looks lovely on google (with sun)17:05
tsdgeosyeah ^_^17:05
tsdgeosanyway, tty on monday!17:05
Cimialoha17:06
SaviqI was there!17:10
Saviqooh funky new search dropdown in Firefox, maybe useful17:11
=== dandrader|lunch is now known as dandrader
=== alan_g is now known as alan_g|EOW
dandraderOh no... seems I lost all my search engines in Firefox?19:12
dandraderhmm... they're are "hidden"19:12
=== salem_ is now known as _salem

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