/srv/irclogs.ubuntu.com/2013/05/15/#ubuntu-unity.txt

=== cyphase_ is now known as cyphase
didrocksthomi: hey, around? it seems there are a lot of QA tests failing (autopilot tests on autopilot)06:04
didrocksthomi: https://jenkins.qa.ubuntu.com/job/ps-generic-autopilot-release-testing/370/06:04
=== jussi01 is now known as jussi
thomididrocks: already fixed in a MP :)07:04
* thomi is EOD now07:04
sil2100!07:04
didrocksthomi: oh, everything \o/ ;)07:04
sil2100\o/07:04
thomisorry guys - I've been up since 2AM07:04
didrockshey sil2100! so it seems we can rerun the QA stack!07:04
thomitime to sleep07:04
sil2100thomi, didrocks: yaay!07:05
didrocksthomi: no, perfect, we'll rerun the stack now, rocking! :-)07:05
* sil2100 was worried he'll miss thomi 07:05
didrocksthomi: enjoy, see you tomorrow :)07:05
didrockssil2100: quick quick quick! can't wait to see the QA stack passing :)07:05
sil2100didrocks: all is merged already ;)?07:07
didrocksah, not sure what thomi meant in fact :)07:08
didrockssil2100: mind checking trunk and MPs if not there? ;)07:08
sil2100I'm checking that07:08
sil2100I see two MR's filled in approx an hour ago07:09
didrockssil2100: do you have enough knowledge to review them?07:11
sil2100didrocks: those already got reviewed and approved by Martin it seems, just waiting for the merger to finish merging07:13
sil2100But I wonder if all the commits will fix our problems, hmmm07:13
didrockssil2100: let's it give it a try once merged :)07:13
didrockssil2100: how are we on the other apps front? did you see that UTAH was behaving quite well tonight?07:14
sil2100didrocks: didn't look at the stacks fully yet, but I was pushing upstreams about fixes to their respective projects yesterday - some fixes have been filled in, some have still been worked on07:17
sil2100didrocks: will look at trunks to see how this worked out in the end07:17
sil2100didrocks: the webbrowser app should be fixed, as I verified the fix on jenkins by myself07:18
sil2100didrocks: also, I had a chat with Omer, and we'll be re-enabling mediumtests for all the apps once those issues get patched up07:18
didrockssil2100: excellent! :-) once you have looked over jenkins, please keep me posted (on webapps for now to finish the bootstrapping)07:18
didrockssil2100: any ETA for having everything fixed?07:19
didrockssil2100: as this is our almost last blocker to have -next enabled :)07:19
sil2100didrocks: I was hoping yesterday evening ;p But let me check if there is progress, as I helped with finding those issues as much as I could - Omer also was poking people07:20
didrockssil2100: ok, good :)07:20
sil2100didrocks: btw. how's the new test framework going?07:24
didrockssil2100: jean-baptiste has a prototype, we'll work on it together today, just finishing the webapps stuff first07:25
sil2100didrocks: excellent ;)07:28
sil2100didrocks: regarding the apps AP's, it seems that fixes have been filled for all the failing projects besides notes-app07:29
sil2100didrocks: I'll poke nerochiaro again about that, and if anything I'll create a patch myself07:29
sil2100As I know the problem and a possible solution07:29
didrockssil2100: great news! :)07:30
didrockssil2100: even the media-apps?07:30
sil2100didrocks: yes, both camera-app and mediaplayer-app had some fix landed07:30
sil2100So I hope those would cover the failures we got07:31
didrockssil2100: so to sum up, apart from QA that we don't know the status, SDK that Mirv is working on, and note-apps that you are looking, eveyrything's fine?07:31
sil2100didrocks: we should be fine, yes, I see the branches got merged so I'll re run the QA stack with rebuilding07:33
didrockssil2100: good, just rebuild the components that changed please :)07:38
tsdgeosnotepad-qml - Demos for Qt Components for Ubuntu08:17
tsdgeosbad naming of the package?08:17
tsdgeosanyone knows who do i ping? Ugo?08:17
greybacktsdgeos: yep I think Ugo.08:22
greybackwhat's wrong with the name?08:22
tsdgeos" Demos for Qt Components for Ubuntu" ?08:22
tsdgeos:D08:22
tsdgeosseems it's fixed already though08:23
tsdgeosat least can find that text in the repo08:23
greybackah I see08:23
tsdgeosyep, somehow i probably have something old in my desktop08:23
tsdgeosphone is fine08:23
mzanettiCimi: ping08:26
mzanettiMacSlow: hey. what was the issue with the failing test then?08:30
MacSlowmzanetti, waitForRendering() was the call needed08:33
MacSlowmzanetti, thanks to that I also was able to get rid of the wait(0)08:34
mzanettiMacSlow: ah right... yeah. that seems to be new since we use the proper mesa driver.08:34
mzanettiMacSlow: yeah. makes sense08:34
MacSlowmzanetti, test looks a much nicer now without the odd wait(0)08:34
mzanettiMacSlow: agreed08:35
MacSlowmzanetti, until you and/or Saviq looked over my branch again I'm back on frontend-backend work again helping Jussi08:35
mzanettiMacSlow: ok08:36
MacSlowbasically checking was missing for things like "append", "replace" etc to work correctly08:36
MacSlowcool... with the working NotifyOSD backend-frontend running on my system, all regular notifications show up via the new renderer :)08:37
MacSlowSaviq, mzanetti: btw... what's the deal with GIcon support in Qt/QML? What's the best starting point to read up on it?08:38
mzanettiMacSlow: hmm... I don't know much about GIcon. But I would start looking if registering a QQmlImageProvider and converting the GIcons into a QPixmap somehow is an option08:39
Saviqmzanetti, already done08:39
SaviqMacSlow, you can use "image://gicon/" as source08:39
MacSlowSaviq, ah... interesting... thx08:40
SaviqMacSlow, or the Icon component from the SDK08:40
SaviqMacSlow, which has a "name" property08:40
mzanettiah. does that work on GIcon?08:40
mzanettithe Icon {} item08:40
MacSlowSaviq, I'll opt for the SDK-component then if possible08:40
mzanettiMacSlow: definitely08:40
SaviqMacSlow, there's one caveat: you can't use Icon with a UbuntuShape08:41
* mzanetti fixes his code in the ubuntu port of xbmcremote :D08:41
SaviqMacSlow, http://developer.ubuntu.com/api/ubuntu-12.10/qml/mobile/qml-ubuntu-components0-icon.html btw08:42
MacSlowSaviq, hm...08:42
SaviqMacSlow, but the thing is so small anyway - http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/view/head:/modules/Ubuntu/Components/Icon.qml08:42
SaviqMacSlow, but anyway, we accept both GIcon names and full paths for the notifications, no?08:43
SaviqMacSlow, Icon only support GIcon names, so it's of not much use for us in that sense08:44
SaviqMacSlow, what we did before is just checked if the given source has / at the beginning08:45
SaviqMacSlow, and if not, prefix it with image://gicon/"08:45
MacSlowSaviq, yes that's what NotifyOSD currently does and what we want to keep with the new implementation08:47
SaviqMacSlow, so yeah, Icon is of no use to us08:48
* greyback moving to office, bbiab08:49
=== gusch is now known as gusch|away
davidcalledidrocks, hey :)09:04
Cimimzanetti, pong09:05
mzanettiCimi: hey. wanted to ask whats the state/plans for your branch regarding the Launcher using Panel09:05
Cimimzanetti, I have other tasks pending, but I am happy to work on that09:06
Cimimzanetti, unfortunately the Panel branch that fixes the Panel is not review yet09:06
Cimimzanetti, we need this branch before the launcher work09:07
mzanettiCimi: yeah... also dandrader changed the Launcher in a way it uses his dragging area09:07
mzanettiso I'm a bit lost on what I should wait or not09:07
Cimimzanetti, it should just use the panel09:08
Cimimzanetti, I guess we can remove dragging area from shell, what is it needed for?09:08
mzanettiCimi: no, the new one Daniel is creating09:09
mzanettiCimi: the one where it can find out if the drag should be used by the shell or by the running app09:09
Cimimmm I need to see this code09:11
Cimithe goal was to use the SDK for those elements09:11
CimiSaviq, are you aware of daniel's work?09:11
SaviqCimi, yes I am, it will end up in the SDK ultimately, too09:12
CimiSaviq, what's its purpose?09:12
SaviqCimi, it's an directional drag detector09:13
CimiSaviq, isn't it colliding with the request of writing the Launcher using the Panel?09:13
CimiPanel or this smart dragging area?09:13
SaviqCimi, Panel will ultimately use this dragging area09:13
CimiI see09:14
CimiSaviq, but I was writing the Launcher using the Panel09:14
SaviqCimi, that's fine09:14
CimiSaviq, now he is writing the Launcher using the dragging area09:14
SaviqCimi, we won't be able to do everything in one go09:14
Cimiwhich one shall we merge?09:14
SaviqCimi, even if we merge his for now09:14
SaviqCimi, once Panel uses Daniel's draggingArea09:15
SaviqCimi, we'll merge yours09:15
CimiSaviq, I would say it will be much better if he was simply working into integrating the new dragging area in SDK09:15
MacSlowDoes Qt have an equivalent to gtk+'s gtk_icon_theme_append_search_path()?09:15
SaviqMacSlow, I don't think it does09:16
CimiSaviq, Panel useg for launcher removes all references to dragging area09:16
CimiSaviq, it's work wasted09:16
Cimi*usage09:16
SaviqCimi, no it's not, because once Panel uses the new DraggingArea (which can't be integrated into SDK yet)09:16
SaviqCimi, we'll just switch to what you did for the Launcher with panel09:17
CimiSaviq, which is basically removing bunch of code including dragging area09:17
CimiSaviq, which will mean removing his branch09:17
MacSlowSaviq, hm... then we either have to use that call in the new NotifyOSD or relocate the special icons we have for NotifyOSD (see https://wiki.ubuntu.com/NotifyOSD#Icon)09:17
SaviqCimi, yes, when it's integrated into the SDK, it will mean removing his branch09:18
SaviqCimi, but it will not happen for some time still09:18
MacSlowSaviq, the special icons currently live in /usr/share/notify-osd/icons and that path gets added to the icon-theme search-path in NotifyOSD09:18
CimiSaviq, mumble? I am not understanding :)09:19
SaviqCimi, and until then we can use what you did to integrate with the current SDK Panel09:19
MacSlowSaviq, btw... I've already added the support for "image://gicon" in the backend-frontend merge and it works nicely... just not yet for the special notification icon-set09:20
MacSlowSaviq, we can discuss that in the after the stand-up today perhaps09:20
Cimimzanetti, had a chat with Saviq … essentially my work on integrating the Panel with the Launcher is on hold until daniel's dragging area will be proposed and merged into the SDK09:28
Cimimzanetti, his dragging area is supposed to be much better than the current state of the Panel, so it does worth the wait09:28
Cimimzanetti, we don't think the new dragging area will be integrated in the SDK before 2-3 weeks09:29
mzanettiCimi: ok. thanks09:29
Cimimzanetti, so I'll probably propose a branch in 3-4 weeks time09:29
SaviqMacSlow, it would probably be best to extend the GIconProvider to support adding custom paths09:29
Saviqhttp://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/view/head:/modules/Ubuntu/Components/plugin/giconprovider.cpp09:29
SaviqMacSlow, can you talk to the SDK guys on how to go about that?09:30
MacSlowSaviq, I can write a patch for it and mr it09:30
MacSlowSaviq, should not be too much work.09:30
MacSlowSaviq, who would be the first contact-person for discussion on the SDK-team? Tim?09:31
SaviqMacSlow, sure, you could also talk to renato as he wrote it09:32
MacSlowSaviq, ok09:32
sil2100didrocks: the autopilot tests from after the fixes are in, there are less failures but still not enough ;/09:50
sil2100didrocks: 13 failures on every platform09:50
sil2100didrocks: I mean, per-platform09:50
sil2100I see some import failures, maybe some missing dependencies this means - but that's just 3 of the 13 failures09:51
sil2100I *might* be able to fix some of those then09:52
didrockssil2100: keep poking! :)09:52
sil2100But Thomi and the other devs would be sweet to have right now ;p09:52
didrocksdavidcalle: hey, in a hangout, will be available afterwards :)09:52
didrockssil2100: right ;)09:52
davidcalledidrocks, ok09:52
=== alan_g is now known as alan_g|cold
sil2100didrocks: https://code.launchpad.net/~sil2100/autopilot/autopilot_test_fixes/+merge/163886 <- some fixes related to the autopilot failures10:29
sil2100fginther: ^10:29
sil2100Not all I guess, but that's always something10:29
didrockssweet, let's wait for some QA people for reviewing10:29
sil2100didrocks: ah, wait, I'll also add a changelog entry for those10:29
didrockssil2100: as you wish, otherwise, it will take the commit log :)10:30
=== matthijs is now known as htenkalsdfjsdf
smspillazfyi people - https://code.launchpad.net/~compiz-team/compiz/compiz.fix_1170013/+merge/161183 <- should be easy to make AP autopkgtests out of that one10:30
=== htenkalsdfjsdf is now known as matthijs
=== dednick_ is now known as dednick
dednickmardy: ping10:36
mardydednick: pong :-)10:36
mardydednick: I'm trying the QMenuModel example file (render-menumodel.qml), but I always get an empty view10:37
mardydednick: I changed the busName and path to "com.canonical.indicator.messages" and /com/canonical/indicator/messages/menu10:38
mardydednick: and I added a few qDebugs here and there. I see that the model is connected, and that QMenuModel::onItemsChanged() is called with "added" > 010:39
mardydednick: but the view still stays empty10:39
nic-doffaymzanetti, got a moment?10:39
mardydednick: any idea of what's wrong? Does it work for you?10:39
mardydednick: (and, are you the right person to be bothered about this? :-) )10:40
dednickmardy: i havent actually used it myself. :)10:40
mardydednick: OK, I'll ask Renato10:42
dednickmardy: yeah, i was just going to suggest that10:42
mzanettinic-doffay: hey10:44
mzanettinic-doffay: yes10:44
nic-doffaymzanetti, regarding the infographics QML tests. Would the best approach be to create a standalone Infographics and create whatever tests need to be done for it?10:45
nic-doffayAt the moment it's created by the greeter.10:45
mzanettinic-doffay: I don't really understand the question10:47
mzanettinic-doffay: Isn't the Infographic already separated in Infographics.qml?10:49
nic-doffaymzanetti, it is.10:49
mzanettinic-doffay: what exactly is the problem then?10:49
nic-doffaymzanetti, basically should the tests be a part of the greeter or for the component specifically.10:50
mzanettinic-doffay: ah. for the component specifically10:50
nic-doffaymzanetti, right on, cheers!10:50
=== gusch|away is now known as gusch
=== MacSlow is now known as MacSlow|lunch
nic-doffaymzanetti, just reading through your comments, thanks for the thorough review of the branch.11:10
mardycyphermox_: hi :-)11:11
mardycyphermox_: see the backlog, about 20 lines ago :-)11:11
mzanettinic-doffay: :)11:11
cyphermox_urgh11:12
cyphermox_mardy: not sure I can help you with how QMenuModel works quite yet ;)11:13
mardycyphermox_: np, I'll wait for Renato then :-)11:14
didrocksdavidcalle: available now :)11:22
mzanettipete-woods: hey11:22
pete-woodsmzanetti: hi!11:22
mzanettipete-woods: still don't really understand the reason for the QVariantListModel11:22
pete-woodsperhaps it's a result of my lack of Qt understanding11:23
pete-woodsbasically I want my API to provide two QAbstractItemModels11:23
davidcalledidrocks, super small mp https://code.launchpad.net/~davidc3/cupstream2distro-config/100scopes-remove-imdb/+merge/16386411:23
pete-woodsand I wanted the quickest implementation I could do11:24
didrocksdavidcalle: ok, doing it :)11:24
pete-woodsso I based it on QStringListModel11:24
didrocksdavidcalle: it's removed from the scope json file?11:24
mzanettipete-woods: you want the backend API providing 2 models and aggregate them in the frontend into 1 model?11:24
davidcalledidrocks, hmm. Of course not^^ Will mp it in a moment, need to go :)11:24
nic-doffaypete-woods, are you going to handle this: https://pastebin.canonical.com/91011/11:25
pete-woodsI just want to provide two lists11:25
pete-woodsthat can provide change notification - i.e. element changed, row inserted, etc11:25
pete-woodsnic-doffay: I've already done that one :)11:25
pete-woodsnic-doffay: I11:26
davidcalledidrocks, by the way, not sure you know about that website: http://framli.eu/scopes_browser/index.html?scope=unity-scope-yahoostock11:26
pete-woodsnic-doffay: I've basically left you the animation and more QML-y ones11:26
nic-doffaypete-woods, ok wicked11:26
didrocksdavidcalle: oh, nice, the name rings a bell to me, but I didn't remember what it was :)11:27
davidcalledidrocks, don't trust all the data in it, though, still working on the parser :)11:28
pete-woodsmzanetti: I thought what I was doing was pretty simple? Qt should really have a QVariantListModel built in, so that you don't have to extend QAbstractItemList all the time :)11:28
didrocksheh ;)11:28
nic-doffaymzanetti, not to worry, I was holding off on the tests in case I had butchered a lot of the QML and needed to redo it ;P11:32
nic-doffayBut it looks like that wasn't the case.11:33
MCR_JohnLea, ping !11:40
MCR_JohnLea, your document says: Ctrl + Super + Cursor Left or RightSemi-maximises current window11:40
MCR_which is actually something else than what Grid does as Grid not only semi-maximizes, but also moves the window11:41
MCR_we can of course have shortcuts to just semi-maximize as well11:41
MCR_regarding the other problem:11:42
MCR_it would be (easily) possible to have 2 different shortcuts doing the same thing, so I can implement that for Grid without any problems...11:42
=== greyback is now known as greyback|lunch
=== _salem is now known as salem_
JohnLeaMCR_; hyia!11:55
JohnLeaMCR_; yes, ideally it would be good to do both.  The "Ctrl + Super + Cursor Left or Right" is the more important of the 2 though11:55
JohnLeaMCR_; scroll down to the end of that doc ;-)11:56
JohnLeaMCR_; if this is something you could take a look at, could you revert the status of that bug back to triaged?11:56
JohnLeaMCR_; Thanks ;-)11:56
MCR_JohnLea, yep11:57
Saviqmzanetti, pete-woods re: mock vs. real implementation, we already have a mechanism in place that will "favour" system-wide installations if available and fall back to those from tests/mocks - I missed that in the lightdm review, but we should do the same - just put the LightDM plugin under tests/mocks now11:58
Saviqand if there is a system-wide plugin installed, it will be used11:58
pete-woodsSaviq: the LightDM mock is already there before this branch11:58
Saviqpete-woods, yeah, I know11:59
pete-woodsshouldn't it be moved in a separate MR?11:59
=== mmrazik is now known as mmrazik|afk
Saviqpete-woods, that's what I meant by "I missed that in the lightdm review"11:59
pete-woodsoh, okay :)11:59
mzanettiSaviq: we are fine with having Digia LGPL code included, are we?11:59
Saviqpete-woods, sure, we could have a separate MR for that11:59
MCR_JohnLea, consider this bug fixed11:59
JohnLeaMCR_; awesome!12:00
Saviqmzanetti, depends where, I'd say12:00
mzanettiSaviq: real code12:00
mzanettiSaviq: not just tests as the other from albert12:00
Saviqmzanetti, well, yeah should be fine12:00
mzanettiSaviq: talking about the QVariantListModel12:00
Saviqmzanetti, yeah I gathered12:00
=== cyphermox_ is now known as cyphermox
MCR_JohnLea, I cannot set to "Triaged" -> I have no rights for that...12:06
JohnLeaMCR_; no worries, I have already done that ;-)12:06
MCR_JohnLea, I've updated the report with a HOWTO12:07
Saviqmzanetti, huh, is it normal that jenkins/pbuilder works in non-UTF-8 locale (LANG=C, probably?)12:07
Saviqmzanetti, https://jenkins.qa.ubuntu.com/job/unity-api-raring-i386-ci/5/console12:07
mzanettiSaviq: don't know. I don't think that has changed lately tho12:08
Saviqmzanetti, no it probably didn't12:08
Saviqmzanetti, was just surprised (and it only showed up because my name is non-ascii ;))12:09
mzanettiI have seen this exact error in autopilot _because_ we were using UTF12:09
mzanettiSaviq: ^12:09
mzanettiSaviq: exporting LANG=C before running autopilot tests works around it12:09
Saviqmzanetti, it probably wasn't this exact error ;)12:10
Saviqmzanetti, otherwise setting LANG=C wouldn't help :D12:10
Saviqmzanetti, as setting LANG=C here actually exposes this issue (it's fine in my default locale)12:11
mzanettiSaviq: https://bugs.launchpad.net/autopilot/+bug/115410712:11
ubot5`Launchpad bug 1154107 in Autopilot "autopilot crashes on failed test" [Critical,Incomplete]12:11
pete-woodsSaviq: LANG=C is what it's been set to for a while I think - I had this problem when adding tests for some gettext related stuff in HUD12:15
Saviqmzanetti, pete-woods well, the LANG=C is for sure not fixing anything - just hacking around an autopilot bug12:16
pete-woodsfair enough :)12:17
Saviqautopilot | testtools12:17
mzanettiyep12:17
Saviqbut OTOH having LANG=C in there makes sure that we're explicit about some things12:17
Saviqinstead of assuming everything is UTF-8, so it has some advantages12:18
=== MacSlow|lunch is now known as MacSlow
sil2100fginther: ping12:44
=== greyback|lunch is now known as greyback
GuidoPallemanshow can I import unity next in qt creator? http://unity.ubuntu.com/getinvolved/development/unitynext/ lp:unity/phablet13:03
vesarMirv, or anyone else, I'm trying to run my shell version on device with run_on_device script but as a result I'm getting: "Cannot mix incompatible Qt library (version 0x50001) with this library (version 0x50002)13:03
vesar" Any idea?13:03
GuidoPallemansvesar can you start qt creator?13:03
vesarMirv, I saw a bug report relating on this: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/117814713:03
ubot5`Launchpad bug 1178147 in qtdeclarative-opensource-src (Ubuntu) "qmlscene not pulling all required dependencies" [Undecided,Confirmed]13:03
vesarGuidoPallemans, yes13:04
vesarQt Creator works just fine13:04
GuidoPallemanshmm, I had the same error earlier when launching qt creator13:04
GuidoPallemansfixed it by just reinstalling everything qt*13:04
GuidoPallemansbut you may not want to do that...13:04
vesarGuidoPallemans, yeah sounds a bit excessive. I would expect that someone would have had the same issue when trying to run on device,13:06
vesarI guess the reason is somehow related to different Qt versions btw my machine and device13:06
GuidoPallemanshmm13:07
mzanettimterry: https://code.launchpad.net/~mzanetti/unity/phablet-tease-launcher/+merge/16390613:09
GuidoPallemanshow can I import unity next in qt creator? http://unity.ubuntu.com/getinvolved/development/unitynext/13:09
SaviqGuidoPallemans, just open the CMakeLists.txt file and the .qmlproject file13:15
SaviqGuidoPallemans, unfortunately QtCreator can't have a CMake project that also has QML files currently13:15
Saviqvesar, apt-get upgrade13:16
Saviqvesar, Qt on the image uses 5.0.1 still, but when you install new packages they are already 5.0.213:17
kenvandinedidrocks, in test_parameters, what is "packages" for?13:19
kenvandineis that the packages that the test provides coverage for?13:19
kenvandineessentially what it tests integration for13:19
didrockskenvandine: yeah, it's the list of all packages we need to install from the ppa to get the new set of packages tested13:20
kenvandineok13:20
didrockskenvandine: normally it's only the binaries from the stack that we want to install13:20
kenvandinethought so13:20
didrocks(all the rest being in the iso)13:20
didrocksbut in this case, we need qt and so on, as it's not in the iso :)13:20
kenvandinei got webcred autopilot tests now :)13:20
kenvandineabout to submit the MP13:20
didrocks\o/13:21
kenvandinealthough 1 test is failing in jenkins... that isn't failing for me13:21
* didrocks publishes webapps13:22
didrocksrobru: FYI ^13:22
=== mmrazik|afk is now known as mmrazik
Mirvvesar: try to run apt-get dist-upgrade?13:25
vesarMirv, Saviq : both give me just 0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.13:26
Mirvvesar: on the device that is. but I don't have a good insight on how the ubuntu plugin works when you order it to run on device13:26
vesarMirv, oh ok will try that.13:27
Saviqvesar, apt-cache policy libqt5declarative5 ?13:27
Saviqvesar, a potential "apt-get update" might be needed on the device, but you should get 5.0.2 for sure13:27
vesarSaviq, Mirv: ok running apt-get update + upgrade helped. Thanks!13:30
greybackSaviq: nic-doffay: standup?13:37
Saviqgreyback, good idea13:37
greybackCimi: ^^13:37
Mirvvesar: ok, cool13:37
nic-doffaygreyback, sorry in another hangout13:39
nic-doffaytotally forgot13:39
greybackMacSlow: thanks for fixing my summary, it's hard to listen & type properly ;)13:46
MacSlowgreyback, sure np yw... it's always hard to listen (and understand) and write everything down13:46
greybackmterry: ok, what do you need? If you update your phone, it should be possible to get Mir running now. You just need to do a few things13:47
Cimigreyback, I'm having lunch… thought we were skipping stand ups this week with the uds13:47
greybackCimi: first I heard of that :) No worries anyway13:47
Cimigreyback, we did skip last vUDS13:48
mterrygreyback, ooh, OK.  Can you describe them to me?  I could try to test the lightdm setup that way13:48
Saviqmzanetti, can you vote on https://code.launchpad.net/~macslow/unity/phablet-notification-renderer/+merge/155512 please?13:49
mzanettiSaviq: sure13:49
greybackmterry: sure. Where do you want to test, on your PC, or on a phone/tablet?13:49
mterrygreyback, I guess on my nexus713:50
greybackmterry: ok cool, that's supported13:51
mzanettiSaviq: done13:51
greybackmterry: first, update it, as recent changes landed in mir that you need13:51
kgunnmterry: half watching this...what are you going to run on nexus7?13:51
kgunnif mir...you'll need a special package from kdub13:51
mterrykgunn, mir, then testing lightdm workflow13:51
greybackmterry: kgunn: oh that's right, I forgot that13:52
tedgbregma, Are you on saucy?  I'm getting a panel-service crash everytime I close a window.13:57
bregmatedg, I do have a saucy test machine, would you like me to verify?13:58
tedgbregma, Uhm, sure.  It'd help to know if I'm crazy or not :-)13:58
tedgTrying to see if I have any locally installed packages that'd matter here.13:59
bregmaI don't get that right now, let me upgrade to the latest packages13:59
* bregma hums a random tune while his computers fight for internet bandwidth14:01
* tedg has his money on the little one14:03
Saviqdandrader, btw, some interestning read Mr. Voß pointed me at today: http://yangl.org/pdf/gstudio.pdf14:08
dandraderhmm14:08
nic-doffaymzanetti, in what cases should verify() and tryCompare() used in the QML tests?14:10
mzanettinic-doffay: use compare() and tryCompare() whenever possible14:11
mzanettinic-doffay: there are some cases where they won't work and you have to use verify()14:11
MacSlowWhen changing/updating an item in a model, does the delegate displaying that item get notified automatically ? Or is there something else, which needs to be triggerd?14:13
nic-doffayOk  cheers. What's the difference between tryCompare and compare though mzanetti ?14:13
mzanettinic-doffay: compare() does it NOW while tryCompare() waits a certain amount of time for the condition to become true14:14
bregmaah, holy crud, my trackpad has gone wild after I upgraded Saucy14:15
dandraderSaviq, did you read it?14:17
Saviqdandrader, not yet14:17
Saviqdandrader, just skimmed14:17
=== olli_ is now known as olli
thomididrocks: sil2100, any luck with the autopilot run?14:23
didrocksthomi: there are still some issues. sil2100 fixed more, but I guess there are some failures14:23
thomisorry I didn't stick around any longer yesterday, I was about ready to fall asleep standing up14:23
thomididrocks: got a link? I can fix them up today14:23
didrocksthomi: no worry, it was already way too late :)14:24
* didrocks looks for a link14:24
didrocksthomi: http://10.97.0.1:8080/job/ps-generic-autopilot-release-testing/385/14:24
* thomi looks14:25
didrocksthomi: sil2100 has a MP to fix some I guess, but not all14:25
thomiahh ok14:25
thomiwill look there first then14:25
didrockshttps://code.launchpad.net/~sil2100/autopilot/autopilot_test_fixes/+merge/16388614:25
didrocksthomi: ^14:25
thomicheers14:25
bregmatedg, yes, I get the crash in unity-panel-service14:27
tedg\o/14:27
tedgOh, wait.14:27
* bregma speculates on perhaps there are other explanations for tedg's madness14:28
bregmait seems apport experiences problems reporting bugs and wants to report a problem about reporting problems14:29
tedgbregma, Do you have libunity-gtk3-parser0 installed?14:29
bregmatedg, nope, according to dpkg...  this is a clean 13.04 install upgraded to current Saucy with no added extras14:33
thomisil2100: can you ping me when you get a moment please?14:33
tedgbregma, Okay, that kills one theory.14:34
bregmaI also see a crash in unity-greeter related to libido314:34
seb128bregma, tedg: unity-panel-service segfaulting is a known issue/being fixed14:35
seb128I didn't follow the start of the discussion14:35
seb128just pointing that14:35
tedgOh, cool!14:35
tedgseb128, Also, do you know when unity-gtk-parser is going into saucy?14:36
bregmaunity-panel-service has many many outstanding segfault bugs, this crash is new as of the last week or so and is consistently reproducable14:36
seb128tedg, http://bazaar.launchpad.net/~dbusmenu-team/libdbusmenu/trunk.13.10/revision/44214:36
seb128bregma, it should be new since yesterday (side effect of the new glib exposing a libdbusmenu bug)14:37
seb128bregma, ^ that's the fix14:37
seb128it happens when you close a win14:37
seb128tedg, no, cyphermox/sil2100 might know, I think the autopilot issues got resolved, not sure what else is missing14:38
bregmaexcellent, not my team's fault (this time)14:39
cyphermoxtedg: unity-gtk-module was ready afaik, I enabled it last week, perhaps it's just failing tests atm14:39
cyphermoxsil2100: ?14:39
seb128cyphermox, do you plan to land libdbusmenu from trunk to saucy now that the fix is in for that?14:39
cyphermoxI do, just waiting for autopilot to finish so I can override the results.14:42
seb128great14:43
* tsdgeos hits greyback and mzanetti with a trout14:56
mzanettitsdgeos: huh?14:56
tsdgeosnow you smell fishy :D14:56
tsdgeostst_ListViewWithPageHeader.qml is full of broken tryCompare14:57
mzanettiis it?14:57
greybackwhy?14:57
tsdgeostryCompare(listView.view.contentY, -1);14:57
tsdgeosbad14:57
mzanettihehe14:57
tsdgeostryCompare(listView.view, "contentY", -1);14:57
tsdgeosgood14:57
mzanettiit sucks quite hard that this evaluates to true14:57
tsdgeosyep14:57
tsdgeosit does14:57
greybackah that's my bad14:57
robrudidrocks: how'd the webapps publishing go? everything in saucy yet?14:57
mzanettiand mine :/ did the review14:57
tsdgeoswonder if it would be patcheable to fail14:58
* tsdgeos is fixing14:58
mzanettiyeah, that'd be awesome14:58
greybackit could check to make sure 3 args were passed14:58
didrocksrobru: look at saucy-changes :)14:59
didrocksrobru: however, some components are yellow as you can see on jenkins14:59
didrocksrobru: so they need to be reconciled I guess14:59
mterrygreyback, I got distracted, but am coming back.  So what's this package from kdub?14:59
greybackmterry: nexus7 not supported in mir trunk. Need kdub to supply you with a patch, so you'd best ask him (I don't have it, sorry)15:00
mterryk15:01
robrudidrocks: I don't know how to fix this... yellow means there's a newer version in the PPA? how did that happen?15:08
didrocksrobru: hum, let me look15:08
didrocksrobru: which ppa? it's checking the dest15:08
didrocksrobru: A version (2.2ubuntu1) is available at the destination for that component but is not in trunk which is still at 2.2ubuntu1daily13.05.14.1ubuntu.unity.next-0ubuntu1.15:09
didrockslet me look distro15:10
robrudidrocks: that's a weird version number15:10
didrocksrobru: see saucy?15:10
didrocksrobru: 2.2ubuntu1 release (universe) 2013-04-2515:10
kenvandinedidrocks, i have a MP for the friends scope, for libunity7 compatibility, is that landing today?15:11
didrockskenvandine: not sure, sil2100?15:11
robrudidrocks: I don't know how that release exists or what to do about it; it's not in the changelog or anything15:11
kenvandinei don't want to approve it and get broken builds :)15:11
didrocksrobru: I don't see 2.2ubuntu1 in debian/changelog15:11
didrocksrobru: so someone uploaded that to distro and it needs to be backported to trunk :)15:12
robrudidrocks: well is there a debdiff of what's in the distro that isn't in trunk? I expect it's trivial15:12
didrocksrobru: it's a security to ensure we don't overwrite stuff that were manually upload to distro15:12
didrocksrobru: I would say download the source package and create the debdigg15:13
didrocksdebdiff15:13
didrocksor you can check what was uploaded to 2.2ubuntu115:13
didrocksrobru: http://launchpadlibrarian.net/125703135/unity-webapps-facebookapps_2.2_2.2ubuntu1.diff.gz15:13
didrocksrobru: I think it's part of the manual SRU I had to do some months ago15:13
didrocksrobru: check the other, but I guess it's the same15:14
robrudidrocks: so let's say I get the debdiff and discover that it's a trivial change and they just forgot to update the changelog or something. How would I go about forcing this through jenkins? just add the version number to debian/changelog?15:14
didrocksrobru: right, just add it to debian/changelog (prefer ordering it properly)15:14
didrocksif the change is really in15:14
didrocks(I think it is, but better to check)15:14
robrudidrocks: ooooh, it actually isn't. tricksy!15:15
robrudidrocks: https://code.launchpad.net/~robru/unity-webapps-facebookapps/backport-sru-to-trunk/+merge/16396815:20
didrocksrobru: do you mind fixing that with ken? (same for the other components?). I'm working on the utah replacement ;)15:21
didrocksrobru: btw, see that this check is useful? we would have overwritten a security fix otherwise ;)15:21
didrocks(it looks good to me FYI)15:21
robrukenvandine: ^^15:22
robrudidrocks: yes, it's good we didn't regress a security fix, but i don't know how comfortable I am that our security team is pushing fixes directly to distro without getting them landed in trunk first...15:22
kenvandineoh joy... these are the reasons we need daily releases :)15:22
robru(or at all)15:22
kenvandinethis prevents us from reverting those fixes15:23
kenvandineso all good15:23
tsdgeosgreyback: yep, that seems ot work, i'll submit a MR for 5.115:23
kenvandinewe'll have to catch these and make sure the fixes land15:23
robrukenvandine: didrocks: oh, it seems amazoncloudreader has the security fix in trunk already, but facebookapps didn't15:24
greybacktsdgeos: kewl. You could also check the second argument is a string, but perhaps that's too much15:24
tsdgeosgreyback: well, i guess that'll fail if not a string, no15:24
tsdgeos?15:24
greybacktsdgeos: probably yeah15:24
tsdgeosyep it does15:25
tsdgeosFAIL!  : qmltestrunner::ListViewWithPageHeader::test_upSwipeCausingBounceKeepsHeaderHidden() Uncaught exception: atYEnd is not defined15:25
tsdgeoswhen using15:25
tsdgeos            tryCompare(listView.view, atYEnd, true);15:25
tsdgeosgreyback: can you handle https://code.launchpad.net/~aacid/unity/fix_tryCompare/+merge/163970  ?15:26
greybacktsdgeos: ok15:26
robrukenvandine: https://code.launchpad.net/~robru/unity-webapps-amazoncloudreader/fix-changelog/+merge/16397215:28
tsdgeosgreyback: mzanetti: https://codereview.qt-project.org/#change,5622215:31
tsdgeosi actually suggest you manually edit /usr/lib/x86_64-linux-gnu/qt5/qml/QtTest/TestCase.qml and add those two lines15:32
tsdgeosSaviq: ↑↑↑15:32
greybacktsdgeos: agreed15:32
mzanettitsdgeos: better :D15:32
mzanettitsdgeos: will that cause all our tests to fail? :D15:33
tsdgeosmzanetti: i already fixed the other one that failed15:33
tsdgeostst_PageHeader15:33
=== jhodapp is now known as jhodapp|lunch
Saviqtsdgeos, could you add msg to it? ;d15:33
tsdgeosSaviq: ?15:34
Saviqtsdgeos, there's no way to set a failure message with tryCompare15:34
tsdgeosah15:35
mzanettidamn javascript... undefined === undefined15:35
mzanettieven though it sometimes is only undefined == undefined15:35
tsdgeosSaviq: could do that, but would mean having to pass the timeout all the time15:36
Saviqtsdgeos, or undefined15:36
Saviqfor default15:36
tsdgeosSaviq: sure, i mean the parameter15:36
Saviqtsdgeos, yeah, I know15:36
Saviqtsdgeos, still, better than nothing...15:36
tsdgeosSaviq: sure, can do, should be 5 min, but let me have this one merged first since the other touches the same line and it'll conlficted and don't want to go having to update requests given how hard is sometimes to get hold of people :D15:38
Saviqtsdgeos, sure, I was just complaining ;)15:38
robrukenvandine: last one: https://code.launchpad.net/~robru/unity-webapps-twitter/fix-changelog-manifest/+merge/16398015:40
robrukenvandine: if we get these merged, webapps can finally land ;-)15:41
kenvandinerobru, you need commit messages on the first 215:44
robrubah15:44
robrukenvandine: ok, can you rerun the stack for me once these land?15:49
kenvandinesure15:49
robrukenvandine: thanks15:50
nic-doffaySaviq, any idea what I need for the Unity.Test module?16:23
nic-doffayI already have qtdeclarative5-test-plugin16:24
Saviqnic-doffay, it's in tests/utils/modules16:24
=== larsu_ is now known as larsu
nic-doffaySaviq, cool cheers.16:24
Saviqnic-doffay, if you try to run with qmlscene just go `qmlscene -I builddir/tests/utils/modules ...`16:24
Saviqnic-doffay, if you add a test in tests/qmltests via the CMakeLists.txt - that should Just Work™16:25
Saviqthanks to the qmltest_DEFAULT_IMPORT_PATHS setting16:25
nic-doffaySaviq, I've been trying to run qmltestrunner -input <filehere>16:26
Saviqnic-doffay, in that case16:26
Saviqnic-doffay, you need -import builddir/tests/utils/modules/16:26
nic-doffaySaviq, cool thanks!16:26
nic-doffaySaviq, still not getting any joy. This is what I'm doing from test/qmltests/Greeter - qmltestrunner -input tst_Infographics.qml -import ../../utils/modules/16:33
Saviqnic-doffay, you need to go to builddir16:33
Saviqnic-doffay, also, you can "make testInfographics" in the builddir16:34
Saviqassuming Infographics is added as a test in tests/qmltests/CMakeLists.txt16:34
Saviqnic-doffay, I mean you can't import from the source tree, because Unity.Test is a module that needs to be compiled16:35
Saviqnic-doffay, so you need to import builddir/test/utils/modules16:35
=== dandrader is now known as dandrader|lunch
nic-doffaySaviq, I see.16:38
=== dandrader|lunch is now known as dandrader|bbl
=== mmrazik is now known as mmrazik|afk
mterrySaviq, mzanetti: After using ./run_on_device, it seems that my device doesn't boot normally.  Do I have to re-flash it to get it back to a good state?17:23
Saviqmterry, you mean you get a black screen?17:24
mterrySaviq, yeah17:24
mterryIt runs fine when I do the first run_on_device17:24
Saviqmterry, your /etc/tablet-services got broken probably17:25
mterryI'm just talking about, say, the next day when I boot the device up17:25
Saviqmterry, http://pastebin.ubuntu.com/5668288/ here's the right content17:25
mterrySaviq, will try17:25
Saviqmterry, it happens when you use run_on_device17:25
Saviqmterry, and you disconnect the device without killing the shell first17:25
Saviqmterry, unfortunately that also happens when you lock the Nexus 1017:25
Saviqmterry, as it disconnects adb :/17:25
mterrySaviq, yup, thanks.  The qml-phone-shell line is missing and with it back, everything works17:32
=== jhodapp|lunch is now known as jhodapp
kgunnmterry: Saviq so is this this http://pastebin.ubuntu.com/5668288/ something you have to do from the android shell ?18:12
Saviqkgunn, it's just the contents of /etc/{tablet,phone}-services18:13
mterrykgunn, yeah18:13
Saviqkgunn, which we need to mangle when doing ./run_on_device18:13
mterrykgunn, from either the ubuntu_chroot shell bit or modifying /data/ubuntu/etc/tablet-services from the android shell18:13
Saviqor ssh into the device and do it straight in ubuntu18:14
kgunnSaviq: mterry thanks....just thinking instructions on wiki18:14
Saviqkgunn, yeah, might make sense to add there18:14
Saviqand/or we should maybe try and make sure that we don't break that file...18:15
kgunnSaviq: made a note of it... :)18:15
=== mmrazik|afk is now known as mmrazik
* greyback eod19:12
=== racarr is now known as racarr|lunch
=== dandrader|bbl is now known as dandrader
sil2100thomi: ping!19:52
thomisil2100: hey, I commented on your autopilot MP19:52
sil2100thomi: I saw, thanks! I replied ;)19:53
thomioh ... I guess I'll go read it19:53
=== dandrader is now known as dandrader|afk
thomisil2100: ahhh, of coure. hmmmm20:00
thomiideally we'd not run that code unless -r was specified. but OK, I'm happy with the MP20:01
sil2100thomi: me and fginther were anyway wondering whether to include the additional python-evdev dependency for python-autopilot-tests or not20:03
sil2100thomi: or maybe we should just skip those tests20:03
bschaefersil2100, hey, i've got a question for you when you get moment :)20:03
bschaeferget a*20:03
sil2100bschaefer: hi! What's up? ;)20:06
bschaefersil2100, hey! So whats the policy on SRUs that break ABI?20:06
bschaefer(it would be in Nux, and I would also be SRUing a fix to unity as wel...)20:07
sil2100Uuuuuuuuuu20:09
sil2100uuuuu20:09
sil2100We no like!20:09
bschaeferi know :(, the XIM fix in 13.04 is wanted in 12.04 annnd it breaks the ABI, but getting around that could be dangerous20:10
bschaefer(getting around the ABI breakage)20:10
sil2100bschaefer: is it a very big bug that's being fixed?20:10
bschaefersil2100, yes :)20:10
* bschaefer digs up the bug report20:11
bschaefersil2100, https://bugs.launchpad.net/unity/5.0/+bug/98325420:11
ubot5`Launchpad bug 5 in Launchpad itself "Plone Placeless Translation Service metadata missing from po files" [Low,Fix released]20:11
bschaeferopp?20:11
bschaeferhttps://bugs.launchpad.net/unity/5.0/+bug/98325420:11
bschaefero haha, bot parsed the milestone haha20:11
bschaefersil2100, the problem, is I need to fix this as well in the backport: https://bugs.launchpad.net/nux/+bug/111204420:12
ubot5`Launchpad bug 1112044 in nux (Ubuntu) "Fcitx can't input word group more than 6 Chinese characters in Dash. Causes a crash if entering to many characters." [Critical,In progress]20:12
bschaeferwhich I need to change the event header to handle dynamic allocation of text :(20:12
=== dandrader|afk is now known as dandrader
sil2100hmmm20:17
sil2100Troublesome indeed20:17
sil2100Normally we do not accept any ABI-breakage changes to SRU releases already released, but hm20:18
bschaeferhmm well I can keep trying to find a way around it, nux just doesn't handle events that can generate more then char text[16] size....20:19
bschaeferall the work arounds are at someone a stopping point, causing memory leaking, mixing up static/dynamic memory...which can lead to some bad problems20:20
sil2100I'll have a chat with the SRU team about that, let me bookmark it20:20
bschaeferare at somewhat* of a20:20
bschaefersil2100, thanks!20:20
* bschaefer will try to come up with a way around it..20:20
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== racarr|lunch is now known as racarr

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