/srv/irclogs.ubuntu.com/2013/04/17/#ubuntu-unity.txt

=== _salem is now known as salem_
=== salem_ is now known as _salem
SaviqCimi, ping07:04
tsdgeosmzanetti: any idea of why the difference between https://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/564/console and https://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/563/consoleFull ?07:34
mzanettitsdgeos: the only thing I could imagine right now is that the main -ci job was started, the i386 started immediately while the arm one was waiting in the queue. in the meantime you pushed something that changed the codebase07:39
mzanettior the other way round07:39
mzanettiif you say no, you didn't change the code, I'm lost too07:39
tsdgeosah, that might make sense07:40
tsdgeosthe modeltest was pushed a bit later07:40
mzanettihuh... luckily... those kind of things really scare me07:40
tsdgeosso yeah07:40
tsdgeoswhat do we do here?07:41
tsdgeosSaviq: ping?07:41
Saviqtsdgeos, here07:41
Saviqlooking07:41
tsdgeosSaviq: for the qmlsortfilterproxymodel thing, i've added a class named modeltest that comes from Qt itself, and does lots of "nasty" things to the model, to make sure it still works07:41
tsdgeosSaviq: the copyright checked is complaining07:41
tsdgeoschecked/checker07:42
Saviqtsdgeos, that's because you didn't include the actual license07:42
tsdgeoshere the MR https://code.launchpad.net/~aacid/unity/sortfilterproxymodelqml_split/+merge/15917407:42
Saviqtsdgeos, $QT_BEGIN_LICENSE:LGPL$07:42
tsdgeosSaviq: well I copied the file verbatim from the repos07:42
Saviqtsdgeos, exactly07:42
Saviqtsdgeos, you need to take it from a tarball07:42
Saviqwhere that var is replaced with the actual license07:43
Saviqtsdgeos, but then...07:43
Saviqtsdgeos, can we actually put it with our code?07:43
tsdgeosit's LGPL07:44
tsdgeosi don't see why not07:44
tsdgeoswhat's your concern?07:45
Saviqtsdgeos, well, it's tests, so we're not really going to distribute it in binary to anyone07:48
Saviqtsdgeos, but IANAL...07:50
tsdgeosstill07:50
Saviqtsdgeos, at the very least, update debian/copyright07:50
tsdgeosLGPL is just compatible with everything we use07:50
Saviqtsdgeos, not if we later want to ship it under a commercial license07:50
Saviq+or need07:51
tsdgeosit is still commercial "friendly"07:51
tsdgeosjust requires you to release the changes to that file07:51
tsdgeoswhich are 007:51
tsdgeosand as you said, it's a test, we don't even need to distribute it binarily to people07:51
tsdgeosSaviq: i downloaded the tarball, the file is exactly the same, still hsa that  $QT_BEGIN_LICENSE:LGPL$07:52
Saviqtsdgeos, huh07:52
tsdgeoswell, what did you expect?07:52
tsdgeosthe license is already there07:52
tsdgeoslines 17 and down07:53
Saviqah ok07:53
Saviqso $...$ are just markers07:53
Saviqnot vars07:53
tsdgeosyep07:54
Saviqtsdgeos, then we need to add a licensecheck-compatible line in the comment there07:54
tsdgeosthat'd be modifying the file and meaning we have to distribute the changes :D07:55
tsdgeosoh actually no07:55
tsdgeoswe can assume gpl307:55
tsdgeosand since it's a test who cares07:55
Saviqand the header is not under copyright07:55
Saviqtsdgeos, apt-get install devscripts07:57
Saviqtsdgeos, and make the copyright header pass07:57
Saviq+test07:57
Saviqtsdgeos, and run tests locally before submitting :P07:57
Saviqat least the test target07:57
tsdgeossure07:57
tsdgeosi'd tell you i had done that07:58
tsdgeosbut i didn't07:58
tsdgeosbut still, it passes07:58
tsdgeoshttp://paste.kde.org/~tsdgeos/726206/07:58
tsdgeosotoh don't see any copytight check in there07:58
Saviqtsdgeos, right, because it's not merged yet07:58
Saviqtsdgeos, the copyright check is part of jenkins jobs for now07:59
tsdgeosSaviq: totally my fault for not running the tests07:59
tsdgeosi see07:59
Saviqtsdgeos, gotcha ;)08:00
Saviqtsdgeos, but you can use licensecheck from devscripts08:00
tsdgeosyep08:00
Saviqtsdgeos, something like `licensecheck -r . | grep -v -F ./builddir | grep "No copyright"`08:00
Saviqthe .sci files are ignored later08:00
tsdgeosor i can do08:01
tsdgeoslicensecheck tests/unittests/plugins/Utils/modeltest.cpp08:01
tsdgeosand that's it :D08:01
tsdgeosi just added a few files08:01
dandradermzanetti, hi. have you had time to check this autopilot issue I e-mailed you about?08:02
mzanettidandrader: I've seen the mail, but still working on the mail queue. I know the solution to your issue and will come back to you within the next half hour08:03
mzanettiwell... maybe not directly the solution. but can definitely help you08:03
dandradermzanetti, awesome! thanks!08:04
Saviqdandrader, DUDE! don't you sleep!?08:07
Saviq5am!? really?08:07
dandraderSaviq, yeah. woke up early so that I can go climbing in the afternoon :)08:10
Saviqdandrader, nice08:11
Saviqdandrader, but then...08:11
Saviq5am!? really?08:11
Saviq;)08:11
mzanettiSaviq: I keep on wondering too08:11
mzanetti:)08:11
dandraderactually, I started at 04:30 :)08:11
mzanettidandrader: I'd have time for you now08:11
mzanettilets not flood the channel...08:12
tsdgeosSaviq: http://bazaar.launchpad.net/~aacid/unity/sortfilterproxymodelqml_split/revision/611 and http://bazaar.launchpad.net/~aacid/unity/sortfilterproxymodelqml_split/revision/61208:16
Saviqtsdgeos, yup, looks good08:17
tsdgeoshe08:21
tsdgeosFound 2 license prolems:08:21
tsdgeostests/unittests/plugins/Utils/modeltest.hLGPL (v2.1)2013 Digia Plc and/or its subsidiary(-ies)08:21
tsdgeostests/unittests/plugins/Utils/modeltest.cppLGPL (v2.1)2013 Digia Plc and/or its subsidiary(-ies)08:21
tsdgeosHouston, we have a prolem08:22
tsdgeosSaviq: shall i simply remove the code and be happy with worse testing?08:23
Saviqtsdgeos, so it seems licensecheck checks for Canonical copyright explicitly?08:23
tsdgeosnot on my machine08:23
tsdgeosbut maybe on whatever that is runninng08:24
Saviqtsdgeos, yeah, it's set up so on jenkins, probably08:24
Saviqmzanetti, can you shed some light ^?08:24
* mzanetti reads backlog08:24
mzanettihehe08:24
* mzanetti adds Digia to the list of allowed licenses08:24
Saviqtsdgeos, see :)08:25
mzanettiSaviq: tsdgeos: https://code.launchpad.net/~mzanetti/pbuilderjenkins/add-digia-license/+merge/15931908:28
Saviqreboot08:41
tsdgeosuhoh08:42
tsdgeosmy kernel is oopsing08:42
tsdgeosin audio related stuff08:42
tsdgeosdid my HW just break?¿08:42
didrockssil2100: hey, what's the news on the street? on what are you working on? :)08:45
didrockssil2100: I'm afraid we won't be able to test your branch though, the power up/down machine is broken in the data center08:45
didrockssil2100: so we can't run autopilot…08:45
sil2100didrocks: hi! I have been continuuing some testing regarding HUD unit tests reliability and checking on merges and their failures08:49
sil2100didrocks: which branch do you have in mind?08:49
sil2100The one with AP fixes for raring?08:49
sil2100(armhf seems failing there for unknown reasons for me...)08:49
didrockssil2100: this one, but in paticular all the "add all the packages for autopilot to run"08:49
didrockssil2100: great! Keep hacking on the HUD, would be great to have something passing today ;)08:50
sil2100I also hope https://code.launchpad.net/~robru/camera-app/packaging/+merge/153651 will get in pretty soon!08:51
didrockssil2100: yeah, did you see the questions? if you have more QML knowledge than I…08:53
sil2100didrocks: btw. do you know the status of https://code.launchpad.net/~mterry/qtvideo-node/arches/+merge/158405 ? Since I remember it was blocked because of some issues with quantal...08:53
sil2100On which fginther was working I think?08:53
didrockssil2100: right, I don't know where fginther is with this08:53
didrockssil2100: if you don't mind tracking it :)08:53
sil2100Ok, will do that then - since I saw a discussion yesterday as well about it but didn't see if it got resolved ;)08:54
didrockssil2100: excellent, thanks!08:55
smspillazdidrocks: just got some foundations laid so that we can do acceptance tests on compiz plugins08:58
smspillazwhich is awesome08:58
didrockssmspillaz: oh! excellent news :) acceptance tests needs a UI though?08:59
smspillazdidrocks: xorg-gtest08:59
smspillazhttps://code.launchpad.net/~compiz-team/compiz/compiz.fix_1101026/+merge/159307 that should allow us to run move, resize, grid, decor without opengl09:00
smspillazso we can run them as a CI job09:00
didrockssmspillaz: ah, even better :)09:00
smspillazyep, been hoping to have that for a while09:00
didrockssmspillaz: this should be even some autopkgtests :)09:01
didrocksfor debian and ubuntu09:01
smspillazI just need to see how broken things are when running those plugins without opengl, as it hasn't worked in over a year09:01
didrockssmspillaz: yeah ;)09:01
tsdgeosand the oops went away09:01
tsdgeosnow i don't know if the HW is half dead or the SW is half broken :-/09:01
smspillazdidrocks: well, it should work as an autopkgtest as long as it works in CI. iirc xorg-gtest was in main now09:01
didrockssmspillaz: autopkgtests are running against the installed version of your software09:02
didrockssmspillaz: so it needs tweaking to support that :)09:02
didrockssmspillaz: those are triggered when a reverse dependency is uploaded for instance09:02
smspillazdidrocks: ah I see. I think that probably makes more sense for autopilot then09:02
smspillazhmm09:03
didrockssmspillaz: it's different from autopilot, but I agree the lines are not quite clear09:03
smspillazyeah I can see the difference, I'm just wondering what the implementation would look like09:03
didrocksyeah, quite hard to decipher some guidances09:04
smspillazyould probably do it, you'd just need to "install" the test binaries or something09:04
smspillaz*you could probably09:04
MacSlowsmspillaz, hey Sam!09:05
smspillazhowdy09:05
smspillazMacSlow: how goes things?09:06
MacSlowsmspillaz, very busy :)09:06
smspillazI can imagine ;-)09:06
tsdgeosmzanetti: i guess it'll take a while until that Digia fix gets to the jenkins slaves?09:16
mzanettitsdgeos: I have to apt-update and install it on all nodes manually :/09:16
tsdgeosreally?¿09:16
tsdgeosouch09:16
mzanettiyeah...09:17
mzanettiI think there was an approach to use landscape for the jenkins nodes. not sure why thats not the case any more09:17
mzanettitsdgeos: well... there might be a cron job that updates the pbuilderjenkins package nightly... but that won't happen before tomorrow09:18
tsdgeoshe he09:18
mzanettitsdgeos: is your MP prone to conflicts and should be merged asap or can it wait until it gets deployed somehow (i.e. someone else has a high prio fix and needs to update stuff)09:19
tsdgeosmzanetti: i guess it can wait09:19
sil2100didrocks: anything else I can help with in-between test runs?09:20
mzanettitsdgeos: ok... in case it kepps on failing after a reasonable amount of time, ping me and I'll update09:20
didrockssil2100: plenty of things! :) one sec09:20
sil2100:)09:21
didrockssil2100: do you want to do a hangout?09:29
didrocks(a quick one)09:29
sil2100Oh boy... didn't migrate to chromium yet! Is screensharing needed ;)? If it's not crucial, I could09:32
didrockssil2100: no screensharing I guess09:32
sil2100didrocks: ok, ready when you are then09:32
didrockssil2100: https://plus.google.com/hangouts/_/3a7c0c16c2cce6a379d806887326c30f5e5388ac?authuser=0&hl=it09:33
mzanettitsdgeos: could you just build and run this one please and let me know if its able to load the dashes? https://code.launchpad.net/~mterry/unity/phablet-greeter-lightdm/+merge/15824509:36
mzanettitsdgeos: it does not here... but I can't spot the difference to trunk - which works here09:36
tsdgeosmzanetti: device or pc?09:38
mzanettipc09:38
tsdgeosdoing09:39
tsdgeosmzanetti: seems to work here09:43
tsdgeoswhat problem are you having exacly?09:43
mzanettitsdgeos: any hints what could be wrong here?09:44
mzanettitsdgeos: ./run'ing trunk works fine here, but on this one it just doesn't load any dash09:44
mzanettitsdgeos: I already deleted builddir and built again09:44
tsdgeosmzanetti: so you get a totally empty thing after opening the greeter?09:45
mzanettitsdgeos: yes09:46
tsdgeosmzanetti: well, to be honest i'm not really running that branch alone, i got trunk and merged that branch in, maybe that makes a difference?09:46
* tsdgeos tries the pristine branch09:46
mzanettitsdgeos: I tried that too... says nothing to merge09:46
mzanettiI tried the other way round tho... cloned that branch and merged trunk into it09:47
Saviqmzanetti, I tried the set_target_properties()09:49
Saviqmzanetti, that doesn't get populated to the dependant targets09:49
Saviqmzanetti, and I agree on the cmake module, thought of that myself, and will do09:50
mzanettiSaviq: oh... that sucks... but afaics we would only need it on the real ctest ones...09:50
mzanettiSaviq: because the others are verbose themselves... while only make check uses ctest and hides output09:50
Saviqmzanetti, we would need to set it up on each add_test() manually :/09:50
mzanettiok... skip it then...09:50
Saviqmzanetti, I have creating a dummy Makefile in our root on my TODO09:51
Saviqmzanetti, that will forward whatever to builddir09:51
mzanettiuuhhh! nice one!09:51
Saviqmzanetti, and we can then put the env there09:51
mzanetti+109:52
Saviqok, so .cmake is remaining09:52
tsdgeosmzanetti: ran it on the device, works fine too09:54
tsdgeosmzanetti: honestly no idea how to debug it :-/09:54
mzanettitsdgeos: ok... thanks...09:56
Saviqcan I help?09:56
tsdgeosmzanetti: other than adding a million printfs of course :D09:56
mzanettiSaviq: https://code.launchpad.net/~mterry/unity/phablet-greeter-lightdm/+merge/15824509:56
mzanettiSaviq: running this on my PC does not load any lenses09:57
mzanettiSaviq: running trunk on my PC works fine09:57
mzanettiSaviq: says nothing to be merged09:57
Saviqmzanetti, checking09:58
mzanettiSaviq: well, it works on tsdgeos's machine... so probably something locally here... I would still like to understand whats happening09:58
mzanettibut not really sure how to approach09:58
tsdgeosmzanetti: which sdk do you have?09:59
mzanettitsdgeos: sdk?09:59
Saviqmzanetti, tried ./build_unity -c? I wonder if it's related to your having libunity installed09:59
tsdgeos qtdeclarative5-ubuntu-ui-toolkit-plugin09:59
tsdgeos0.1.42~raring1 ?09:59
tsdgeosmzanetti: ↑↑09:59
mzanettitsdgeos: same here10:00
mzanettiSaviq: no, didn't try10:00
mzanettiSaviq: still wouldn't explain trunk working, would it? (as the MP doesn't seem to change that part)10:00
tsdgeosmzanetti: ok10:00
Saviqmzanetti, true10:00
Saviqmzanetti, works here10:01
Saviqmzanetti, any interesting output on the console?10:01
mzanettiSaviq: nope. exactly the same except warnings caused by lenses are missing10:02
Saviqmzanetti, no idea here10:05
Saviqmzanetti, I'd go for a rm -R ../unity_build10:05
Saviqand start over10:05
tsdgeosmay it be related to you not having "some other lightdm" thing installed?10:05
* Saviq doesn't like the .h links, though10:05
tsdgeosmzanetti: just in case this is what i have http://paste.kde.org/~tsdgeos/726320/10:06
mzanettitsdgeos: the lightdm part is the only thing working :D10:06
mzanettianyways, thanks10:06
tsdgeosi know, but won't hurt to install a few packages :D10:07
CimiSaviq, saw your ping this morning but I'm with some headache cause I had insomnia last night and slept only a few, was for your cmake refactor? looks good to me10:19
Cimi(I took an aspirin btw)10:19
SaviqCimi, no, it was about yesterday's talk about dee in the mock plugin10:19
CimiSaviq, ok tell me10:20
SaviqCimi, there can be no dee in there, you just need a ListMOdel10:20
SaviqCimi, so you need to create/use something that's based on QAbstractListModel10:20
CimiSaviq, but it's categories from the c++ plugin?10:20
Cimiah ok10:20
SaviqCimi, it doesn't care about dee10:20
Cimiok10:20
SaviqCimi, it's an implementation detail the tests can't know about10:20
mzanettiCimi: you need DeeVariantText in a test?10:25
Cimimzanetti, I need a list modelk10:25
Cimimzanetti, lens.categories10:25
mzanettiCimi: ah... so why not use ListModel (I didn't get the full conversation so I might tell you outdated stuff now - just tell me to shut up in that case)10:26
Cimimzanetti, for me everything is ok, I have almost 0 experience in c++/qt, I have seen the real plugin was returning a dee model10:27
Cimimzanetti, and thought it was required the same10:27
Cimialso I need to see how to write a model in c++10:27
mzanettiCimi: not sure if it helps you... but here I mocked some parts of Dee in the most simplistic way there is: https://code.launchpad.net/~mzanetti/unity/phablet-test-filtergrids/+merge/15894110:28
Saviqmzanetti, uh, add_custom_target(check DEPENDS test) doesn't work...10:30
Saviqfreakin' test isn't a normal target10:30
Saviqat least not under make (it worked under ninja...10:30
mzanettihmpf...10:31
Saviqbut I did try to debuild it...10:33
* Saviq tries again10:33
Saviqhmm it works10:35
* Saviq doesn't get it10:36
Saviqmzanetti, why the --force-yes?10:38
mzanettiSaviq: already delete the branch again10:44
mzanettiSaviq: it kept on failing here... but while editing the comment in the MP I flashed todays image and it worked again10:44
Saviqmzanetti, k10:45
Saviqmzanetti, seem dh_auto_test actually uses the test target (at least for CMake projects)10:45
Saviqmzanetti, so the custom target wasn't helping anyway10:46
Saviqand the failures were silent anyway10:46
Saviqand!10:47
Saviqit sets CTEST_OUTPUT_ON_FAILURE by itself10:47
Saviqso we can get rid of the check target altogether10:48
Saviqyup10:49
mzanettiSaviq: yeah... the check target was just to keep syntax compatible to googletest, right?10:50
mzanettiI haven't seen that before either....10:50
Saviqmzanetti, I thought it was about debhelper...10:50
Saviqdidrocks, question - why the "check" target?10:50
mzanettibut jenkins executes "make check || make test" anyways... so fine with me10:50
didrocksSaviq: sorry, out of context, what check target? :)10:51
didrocksSaviq: jenkins merger?10:51
Saviqdidrocks, no, in general10:51
Saviqdidrocks, we had a custom "check" target in CMake10:51
Saviqfor compatibility with...10:51
Saviqsomething10:51
SaviqI thought it was about dh10:51
tsdgeosSaviq: so in the dash search we still have that occasional crash on search10:52
didrocksSaviq: dh runs make check, it's a general target we have in projects to run unit tests and tests that can be run with isolation10:52
Saviqdidrocks, actually dh runs make test10:52
tsdgeosi've checked and going to carousel+listview+patches still crashes10:52
Saviqdidrocks, at least under CMake10:52
tsdgeosso it's unrelated10:52
tsdgeosnot sure if carousel at all or not10:52
didrocksSaviq: right, it depends on the build tool10:52
Saviqtsdgeos, uhm, was hoping it is10:52
tsdgeosnah10:52
tsdgeosthe bt is weeeeird10:52
didrocksSaviq: autotools are more "make check": http://www.sourceware.org/autobook/autobook/autobook_98.html10:53
Saviqdidrocks, yeah exactly, so I was wondering if there's any reason to keep the check target if we have issues with keeping it :)10:53
tsdgeoshttp://paste.kde.org/~tsdgeos/726350/10:53
tsdgeosQQuickWindowIncubationController::incubate10:53
Saviqtsdgeos, that I never saw...10:53
tsdgeoswhat's a incubation controller? :D10:53
didrocksSaviq: if you have a make test, that's fine, use that one :)10:53
Saviqdidrocks, \o/10:53
tsdgeosSaviq: ah because you where not running with your own Qt so you didn't see the assert but the crash probably10:53
tsdgeoswhich may have been a bit different10:53
didrocksSaviq: let me just recheck that dh_tests is indeed running that one as well for cmake10:54
Saviqdidrocks, sbuild didn't complain10:54
Saviqtsdgeos, yeah, probably10:54
tsdgeoswhen running with the system qt i get http://paste.kde.org/~tsdgeos/726356/10:54
tsdgeosSaviq: ↑10:54
didrocksSaviq: didn't complain != run :p10:54
Saviqdidrocks, it did run the tests, yes10:54
didrocksSaviq: but yeah, dh_auto_test is running make test and make check, depending on what it found in the makefile10:55
Saviqdidrocks, ah, smarts!10:55
Saviqdidrocks, awesome, thanks10:55
didrocksSaviq: yw ;)10:55
Saviqtsdgeos, not even, I think10:55
tsdgeosok10:55
Saviqtsdgeos, so it might actually be related (i.e. you fixed one thing, but there's another happening anyway)10:55
tsdgeosyeah10:56
tsdgeosthis one gonna be harder i think10:56
Saviqmzanetti, will you tell me I should split out the cmake stuff out of that MP? (/me cries away for git)10:56
mzanettiSaviq: huh?10:57
mzanettiSaviq: from the flatten-qmltests one?10:57
Saviqmzanetti, yes10:57
mzanettiwhy that?10:57
Saviqmzanetti, 'cause it's not really related ;)10:57
mzanettino... just merge it as is... history isn't worth a thing with bzr anyways10:57
Saviqindeed10:58
Saviqmzanetti, what targets to we want "forwarded" to builddir?10:58
mzanettialltests10:58
mzanettiautopilot10:58
mzanettihmm.... thats the ones I use the most10:58
mzanettiSaviq: can you please check lines 1244 and 1257 in here https://code.launchpad.net/~mterry/unity/phablet-greeter-lightdm/+merge/15824511:00
mzanettiSaviq: I think there is something fishy - we should not need this11:00
Saviqmzanetti, indeed11:00
mzanettibesides the mock should probably live somewhere in tests/ not plugins/11:01
Saviqmzanetti, I'm thinking the plugin links to the mocks11:01
Saviqmzanetti, which are separate .so11:01
Saviqwhich would, of course, be bad11:01
mzanettiyeah... I guess so...11:01
Saviqmzanetti, is there a MP from you fixing the path/LD in autopilot?11:03
mzanettiSaviq: no...... only the builddir thingie...11:03
mzanettiSaviq: but I can do11:04
Saviqmzanetti, could there be?11:04
mzanettiyea11:04
Saviqmzanetti, yes please11:04
Saviqmzanetti, not sure about fixing it _in_ the .py, too11:04
mzanettiSaviq: where else?11:04
Saviqmzanetti, it's only relevant for local builds11:04
mzanettiyeah... the py distinguishes between local and installed builds11:05
Saviqmzanetti, ok, more local11:05
Saviqmzanetti, as in: if you have the deps installed11:05
Saviqmzanetti, the LD isn't required11:05
Saviqeven though you build locally11:05
mzanettimy case, that is11:05
Saviqyes ;)11:05
Saviqso I wonder if the Makefile I'm writing now could have bearing here11:06
Saviqmzanetti, with http://pastebin.ubuntu.com/5715632/11:09
Saviqmzanetti, make autopilot works here11:09
* Saviq needs to learn Ninja to do the same for it11:10
mzanettiSaviq: what if you execute "make autopilot" within the build dir?11:11
Saviqmzanetti, it won't11:11
Saviqof course11:11
Saviqmzanetti, but if you go ./qml-phone-shell in the build dir11:11
Saviqmzanetti, it will fail just as well11:11
dandradermzanetti, is there any way to known if ci is building (or is scheduled to build) my merge proposal?11:12
Saviqmzanetti, but it's temporary11:12
* dandrader is anxious11:12
Saviqdandrader, http://s-jenkins:8080/job/unity-phablet-ci/11:12
Saviqdandrader, you need to look through the running jobs11:13
Saviqdandrader, and look at their parameters11:13
Saviqthere's a link on the left11:13
Saviqdandrader, http://s-jenkins:8080/job/unity-phablet-ci/583/parameters/?11:13
mzanettidandrader: yeah... open parameters and then just use the "Previous build"/"Next build" links11:13
dandradermzanetti, Saviq  excellent! thanks!11:14
dandraderjesus.. it takes time "Started 42 min ago"11:15
mzanettiSaviq: I'm fine with it... if you're fine with it we're good I'd say11:15
Saviqdandrader, it's most probably queued11:15
=== MacSlow is now known as MacSlow|lunch
mzanettidandrader: hmm... let me check what's blocking the queue11:16
Saviqdandrader, http://10.97.2.10:8080/job/unity-phablet-qmluitests/11:16
Saviqmzanetti, http://10.97.2.10:8080/job/unity-phablet-qmluitests/376/console hangs11:16
Saviqmzanetti, we shouldn't need to do anything in run_on_device - build-dep are handling it (we need a release for that, though)11:19
Saviqmzanetti, beat me to it11:20
dandraderdamn whitespace...11:20
Saviqdandrader, http://vim.wikia.com/wiki/Highlight_unwanted_spaces ;)11:21
Saviqdandrader, also, MAKE TEST! ;P11:21
dandraderI have a "set listchars" for that but it doesn't seem to be working so well11:23
Saviqmzanetti, pushed the Makefile11:30
=== dandrader is now known as dandrader|lunch
mzanettiSaviq: pushed to where?11:30
Saviqmzanetti, same branch11:30
mzanettiSaviq: you sure?11:31
Saviqno11:31
Saviqmzanetti, now11:31
Saviqmzanetti, it's quite stupid, but saves the "-C builddir"11:34
Saviqand remembering it, too11:35
mzanettiI like it a lot11:35
mzanettijust for doing quick reviews the builddir thingie was quite annoying me11:35
mzanettiSaviq: approved11:36
Saviqmzanetti, cheers11:41
Saviqmzanetti, tsdgeos, what did we agree upon yesterday about the utils vs. mocks in /tests/ (and the tests for them)11:44
Saviq/tests/{utils,mocks} for the actual plugins11:44
Saviqand the tests for them? ;)11:44
mzanettiSaviq: yeah... regarding tests for utils we didn't come to a clear conclusion11:44
mzanettiSaviq: if you promise to keep them at one level you can put them to /tests/utils/tests :D11:45
Saviqlol11:45
mzanettianyone? https://code.launchpad.net/~mzanetti/unity/phablet-test-filtergrids/+merge/15894111:48
mzanettiCimi would like to get a second opinion before top-approving11:48
tsdgeos:D11:52
tsdgeoswhat mzanetti said11:52
tsdgeoslunch!11:52
mzanettiSaviq: do'h!12:01
mzanettiSaviq: Jenkins searches for a Makefile before it searches for a CMakeLists.txt12:01
Saviqmzanetti, jenkins or dh?12:01
mzanettiSaviq: probably both12:02
Saviqyeah, dh12:02
Saviqdamn12:02
Saviqlet's drop it for now12:02
SaviqI have an idea for later12:02
Saviqwill probably go for a Makefile.in in ${CMAKE_SOURCE_DIR}12:03
Saviqmzanetti, pushed12:03
=== MacSlow|lunch is now known as MacSlow
smspillazTrevinho: andyrock: got a question about the switcher if either of you are around12:10
=== dandrader|lunch is now known as dandrader
smspillazTrevinho: andyrock: do either of you know if the intended design for the switcher was that alt-tab (quickly) was supposed to take you to the next *application* or the next *window* (of the same application)12:11
Saviqmzanetti, what do you make of these failures https://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/585/console ?12:11
smspillazthere's an AP test that suggests its the former, although looking at SwitcherModel it doesn't even try to do that12:11
Saviqsmspillaz, AFAIK, alt+tab -> applications, alt+${key_over_tab} -> windows of the same application12:12
mzanettiSaviq: missing merge with trunk: line 1875 of https://code.launchpad.net/~dandrader/unity/phablet_remove_fakes_from_qml/+merge/15837012:12
smspillazthat's what I thought too12:12
mzanettiSaviq: not really a merge, but that file has been moved to Components12:13
Saviqmzanetti, k12:13
Saviqdandrader, you need to merge trunk again in your fakes branch12:15
dandraderSaviq, fixing it now12:15
dandraderdone. let's see what happens next12:16
Saviqlol12:16
mzanettidandrader: just merging is not enough... the file has been moved to Components. you need to adjust your .install file too12:19
mzanettidandrader: just remove line 1875 from your diff12:21
dandradermzanetti, I did that12:21
mzanettiok12:21
=== alan_g is now known as alan_g|lunch
sil2100fginther: ping! Give me a sign once you're online :)12:25
sil2100didrocks: can you access the Touch Apps docs?12:31
sil2100Since I get weird errors...12:31
didrockssil2100: do you have a link handy?12:32
didrocksas we are having a lot of docs :)12:32
mterrydidrocks, did you deploy the manual upload or shall I?12:35
mterrythe manual mode for Raring I meant12:35
didrocksmterry: no, I can do it, it would mean that for today and tomorrow, we need to manually publish though12:35
sil2100https://code.launchpad.net/~sil2100/qtubuntu-camera/add_bootstrapping/+merge/159372 <- if anything12:39
sil2100(wrong channel)12:40
mterrydidrocks, fair, if we don't like that, it can wait easy enough12:40
didrocksmterry: yeah, let's pull on Thursday?12:41
didrocksmterry: but at least, everything is prepared :)12:41
didrocksmterry: thanks btw!12:41
smspillazdidrocks: so the good news is that none of the WM related tests appear to be failing using lp:compiz12:52
mzanettiI guess its not possible to run old Unity windowed, is it?12:52
smspillazmzanetti: it is, dunno how tested it is12:52
didrockssmspillaz: \o/ excellent work, that's a first good step to trust the WM behavior :)12:52
smspillazmzanetti: unity-standalone12:53
mzanetticool, thanks12:53
mzanettiunity : Depends: libnux-abiversion-20121121.0112:59
mzanettiwhich is a virtual package provided by libnux-4.0-0 which is already installed12:59
mzanettistill it doesn't let me to install unity13:00
paulliuHi, I'm writing tests for DashPeople.qml. But still don't get how the data is passed from lens daemon to qml?13:00
mzanettipaulliu: let me check13:00
mzanettipaulliu: do you have a specific line of code you don't understand?13:01
paulliuFor example, Binding { target: lensView.lens... ,  who assigned the lensView.lens?13:01
tsdgeoswhat do you mean assigned?13:02
tsdgeoslensView is an object and lens is a property of the object, no?13:02
paulliulensView is the root element of DashPeople.qml13:03
paulliuBut it seems to me that it is inherit by LensView Object.13:03
paulliuinherit from.13:03
paulliuI'm thinking provide some mock-up data in tests and do some UI tests there? So I'd like to fake the data from lens?13:04
=== _salem is now known as salem_
tsdgeoswell DashPeople is a LensView13:05
mterrydidrocks, who was I supposed to ping with UTAH errors?13:06
mzanettipaulliu: yeah... DashPeople inherits from LensView, which means it inherits the property "lens"13:07
paulliumzanetti: ok, but that property is undefined when I create a instance of DashPeople.13:08
paulliumzanetti: Am I doing something wrong?13:08
fginthersil2100, pong chirp pong13:08
tsdgeospaulliu: of course it's undefined13:09
tsdgeos    property Lens lens13:09
tsdgeosnothing is assigned13:09
tsdgeosso it's undefined13:09
tsdgeosit does exist with no value13:09
mzanettipaulliu: in Dash/Dash.qml13:09
mzanettipaulliu: there is a Model holding all Lenses and assigning the appropriate LensViews as delegates13:10
paulliumzanetti: ah..ok. Let me check.13:10
mzanettipaulliu: sorry... DashContent.qml13:10
mzanettipaulliu: line 12413:11
paulliumzanetti: yeah.13:11
paulliumzanetti: ok. thanks.13:11
mzanettipaulliu: so you can just create your own "Lens" and assign it to DashPeople in your test13:12
tsdgeosdednick: the list deletes the delegates as its own discretion13:12
paulliumzanetti: Yeah, I'll do it.13:13
tsdgeosdednick: are we having problems when that happens?13:13
didrocksmterry: the CDU is dead13:14
mterrydidrocks, CDU?13:14
didrocksmterry: the hw rebooting the machines13:14
didrocksthat's why everything is broken13:14
mterrydidrocks, boo13:14
didrocksI just pinged you on the qa channel :)13:14
kgunnmzanetti: mterry oops....moving here13:19
mzanettiso... anyone has a hint why I can't install unity? http://paste.ubuntu.com/5715900/13:20
mzanettikgunn: yeah... same here... wanted to read through the launcher docs13:20
mzanettikgunn: anyways, mterry asks who will implement the Pinentry thingies13:21
tsdgeoslibnux-4.0-0 4.0.0phablet14bzr763raring013:21
tsdgeosbrrrr13:21
tsdgeosmzanetti: phablet-age stuff?13:21
tsdgeosdo you have the phone-ppa (the unsafe one) enabled?13:21
mzanettitsdgeos: yeah... but all the unity packages are not from there... and I would understand if it crashes because of the ABI breakage...13:22
kgunnmzanetti: mterry do either of you have a sense for size? e.g. days vs weeks vs month?.....what little i've witnessed so far would seem like ~1week13:23
kgunnmzanetti: are you wanting to do it? :)13:24
kgunnmterry: to answer your original question...not assigned at the moment....and we need to capture13:25
kgunnmterry: in a bp13:25
mzanettikgunn: yeah, sure... I'm just not exactly sure on the rest of the schedule and what exactly it includes13:25
tsdgeostsdgeos: disable that ppa, otherwise the versions conflict between eachothers and everyone is unhappy13:25
tsdgeoserr13:26
kgunnmzanetti: right...when gdocs are back, take a look at monthly first, see the launcher load...then let's chat13:26
tsdgeosmzanetti: ↑↑↑13:26
tsdgeosthat ppa is not for desktop usage13:26
mterrykgunn, mzanetti, yeah, I'm in the middle of trying to figure out bp and stuff13:26
kgunnmterry: should we just glom on to this one ? https://blueprints.launchpad.net/ubuntu/+spec/client-1303-unity-greeter13:27
kgunnmterry: or create a new one just for pin-entry13:27
mzanettikgunn: mterry: So in general yes, I'd be happy to help out with this if the schedule permits it - which we will find out once the requirements are captured13:28
kgunnmzanetti: mterry surely for 13.10 we can limit it to pin....leaving osk/passphrase for after13:29
mzanettiwouldn't entering a pin also require an OSK?13:30
mzanetti(depends on the design I guess)13:30
kgunnmzanetti: true...10 key most likely13:30
kgunnmzanetti: i was just trying to limit the ui effort to one layout vs doing 2 before 13.1013:31
mzanettikgunn: yeah... a matter of the layout, but still, the OSK needs to be there (unless they want us to implement something more fancy for just numbers ourselves)13:31
kgunnmzanetti: its basically the phone dial pad widget13:32
kgunn10 key that is13:32
mzanettiI see13:32
dednicktsdgeos: yes.13:32
tsdgeosdednick: ok, that's bd, let's do the standup and talk about it later again13:33
sil2100fginther: chirp ;)13:33
sil2100fginther: could you tell me what's the status with https://code.launchpad.net/~mterry/qtvideo-node/arches/+merge/158405 ? Since I remember the issue was the lack of stack for quantal or something?13:33
=== alan_g|lunch is now known as alan_g
tsdgeosdandrader: try if this helps https://code.launchpad.net/~nick-dedekind/unity/phablet-tests-menucontent/+merge/158562 (i think it's the "this" part for the Lens constructor)13:37
fginthersil2100, I'm not sure how we solve the quantal dependency problem at the moment. We almost have changes in place to use the daily build ppa, but that won't help here13:39
mterryfginther, sil2100: it will help; we just need to have the PPA used by CI, rebuild the PPA, then retrigger that merge job's CI13:43
Saviqhrm13:43
fginthermterry, sil2100 I'll get it setup13:44
mterrydidrocks, we will have the daily-release jobs building for quantal too, right?13:44
didrocksmterry: no, there is no gain of daily releasing touch for quantal13:46
didrocksknowing it will be dropped soon13:46
sil2100fginther: thanks!13:46
mterrydidrocks, :(  well, then we won't be able to CI a lot of these things.  Like, packages depend on platform-api for example.  If we don't daily-release platform-api...13:47
sil2100Well, we need the quantal packages fetched in CI at least as a 'dummy' workaround13:47
sil2100For quantal13:47
didrocksmterry: they still have a dput job in quantal, right?13:47
didrocksmterry: in a ppa13:47
didrockswithout daily release13:47
sil2100We can push manually to the daily-next PPA when there's need, right?13:48
didrocksmterry: then, I guess sergiusens will remove quantal support in the follow days when, he's happy with his raring image?13:48
sil2100As long as CI would use the daily-next PPA in CI13:48
sil2100Damn13:48
sil2100-CI13:48
didrockssil2100: ah, yeah, we can have a manual push, but TBH, I think it's not a priority and CI should continue commit putting to a ppa for quantal?13:48
fgintherdidrocks, that's what I thought we would be doing13:49
kgunnnic-doffay: http://qt-project.org/doc/qt-5.0/qtquick/scenegraph-openglunderqml.html13:49
mterrydidrocks, fginther: so CI will push to a PPA now?13:49
mterryI didn't know it did that13:49
fginthermterry, the upstream autolanding builds get dput into a ppa13:49
mterryfginther, fascinating.  I didn't realize we had two things pushing to PPAs (daily-release and CI)13:50
didrocksmterry: we are going to kill CI pushing to it once everything will be on the same page :)13:50
mterryfginther, that's why I was pushing for having the daily-release PPA included in CI jobs, because I thought it had to be to pick up previous builds13:50
mterryfginther, where will the CI quantal phablet builds go now?13:51
fginthermterry, so the ci jobs dput to ppa:phablet-team/ppa13:51
didrocksfginther: why you don't have a local repo?13:51
didrockslike for the rest13:51
didrocksmakes more sense as you build debs13:51
fgintherdidrocks, I think the ppa was setup first and there was too much inertia to change13:53
didrocksfginther: weird, the local repo was something we supported even before you redo the jenkins stuff :)13:54
sergiusensdidrocks: yup, lots of moving peices still, but its the plan13:55
sil2100Ok ok, so hm, what needs to be done still for the qtvideo-node merge to get in?13:56
sil2100;)13:57
fginthersil2100, I'll build ci again when I get the job updated. It's possible the broken dependency has been updated13:58
sergiusensdidrocks: local repo where? The PPA is later used to build from offspring13:58
sergiusensdidrocks: which has intense firewall rules13:58
sil2100fginther: \o/ excellent!13:59
sil2100Thanks13:59
didrockssergiusens: local repo so that CI always builds against latest13:59
didrockssergiusens: separate upstream merger discussion :)13:59
fginthersergiusens, we use a local repo for building unity.  We essentially save the debs the jenkins builds14:00
sergiusensfginther: didrocks that's fine, but we need the debs somewhere, to build the actual image14:00
sergiusensso local repo doesn't cut it here14:00
sergiusensif it is just for a jenkins build to speed things up, no problem with that, but the packages would still need to be accessible outside14:01
didrockssergiusens: right, forgot about that, but that won't be needed anymore once it daily releases14:01
Saviqnic-doffay, so, either you use a QML animation http://qt-project.org/doc/qt-5.0/qtquick/qtquick-qmltypereference.html#states-transitions-and-animations and use that in C++14:01
sergiusensdidrocks: yup, exactly.. and once we build in cdimage, everything will need to be in the archive anyways....14:01
Saviqnic-doffay, or you could probably use QVariantAnimation http://qt-project.org/doc/qt-5.0/qtcore/qvariantanimation.html14:02
didrockssergiusens: yep14:02
Saviqnic-doffay, but it's better to keep it in QML IMO14:02
nic-doffayAgreed Saviq it would simplify matters.14:02
Saviqnic-doffay, as it's not going to save us much to do it in C++, and will be less coupled to the actual behavior14:02
Saviqnic-doffay, so you will have a "mouseCoords" and "repelValue" from 0.0 to 1.014:03
Saviqnic-doffay, bound to the MouseArea and Animation14:03
Saviqrespectively14:03
Saviqnic-doffay, and those you will read in C++ and react accordingly14:03
=== hggdh_ is now known as hggdh
=== dandrader is now known as dandrader|afk
nic-doffaySaviq, will get back to you now, just busy sorting something out...14:15
Saviqnic-doffay, k14:15
nic-doffaySaviq, I just ran qmlscene on the phone, it seems to have executed correctly with no issues.14:16
nic-doffayHow can I check this on the actual phone? :P14:16
=== alan_g is now known as alan_g|tea
nic-doffayThere's a gesture, I just can't seem to pick up which one it is.14:16
nic-doffayIt's a bit erratic.14:17
Saviqnic-doffay, not sure what you mean :)14:24
nic-doffayThere's a certain swipe gesture which brings up the qmlscene14:24
nic-doffaywhich is being run14:24
nic-doffayI just want to clarify what it is haha14:25
nic-doffayBecause it's a bit erratic.14:25
Saviqnic-doffay, you just have to unlock the phone14:25
Saviqnic-doffay, by edge-swiping from the right14:25
nic-doffayOk brilliant.14:25
Saviqnic-doffay, and then switch to the apps lens14:25
Saviqnic-doffay, to see the qmlscene entry in "Running apps"14:25
Saviqnic-doffay, and just tap on it to bring it to the front14:25
Saviqnic-doffay, also, if you just keep the phone unlocked, the app you run should come to the front by itself14:26
kenvandinesmspillaz, when you have a moment, can you take a look at bug 116534314:26
ubot5bug 1165343 in Ubuntu UI Toolkit "Windows are placed with titlebar obscured by Unity menubar." [Critical,Triaged] https://launchpad.net/bugs/116534314:26
kenvandinesmspillaz, it was introduced by rev 3636 of lp:compiz/0.9.914:26
=== dandrader|afk is now known as dandrader
kenvandinesmspillaz, it breaks all Qt apps that don't set position to other than 0,014:27
CimiSaviq, Categories is a list of CategoryFilters?14:28
SaviqCimi, yes14:28
Cimiok14:29
SaviqCimi, actually14:29
SaviqCimi, it's a ListModel14:29
SaviqCimi, that exposes CategoryFilters as one of the roles14:29
CimiSaviq, mmm14:29
SaviqCimi, look in categories.{h,cpp}14:29
Cimione thing at a time14:29
CimiI am subclassinc qabstractlist14:30
Cimione of the private variables is a qlist14:30
Cimi    QList<Type*> m_categories;14:30
Cimiso Type might be CategoryFilter here14:30
Cimithen I'll have to copy categoryfilter to the fake plugin too I think14:31
SaviqCimi, but not the _actual_ CategoryFilter14:31
SaviqCimi, just something that has the same API14:31
Cimiok14:32
nic-doffayQuestion about the units quick.14:40
nic-doffayMy qmlscene shows up pretty small on the desktop at 25 unit width and height, however it seems pretty large on the phone.14:41
nic-doffaythis doesn't change no matter what unit amount I put.14:41
nic-doffayAny ideas?14:41
tsdgeosis it showing up maximized in the phone?14:43
tsdgeoswhat is big, the window or the things you draw?14:43
mterryfginther, so platform-api is raring-only in phablet-team/ppa, probably because it built when the configs were raring only.  How can we manually kick CI to rebuild packages for quantal?14:43
fginthersergiusens, ^^ We would need to create a release through a changelog update, correct?14:45
smspillazkenvandine: lovely, I'll have a look14:45
smspillazkenvandine: grrr, I hate it when this happens. Fix one app break another14:45
kenvandinehehe14:45
kenvandinesmspillaz, thanks!14:45
smspillazkenvandine: though tbh, if those Qt apps are setting StaticGravity then IMO they are broken14:46
kenvandinesmspillaz, hopefully bisecting it for you will make it easy to fix :)14:46
smspillazkenvandine: yeah, I know what the problem is, but I feel like its an app problem and not a window manager problem14:46
kenvandineperhaps qt should do that...14:46
nic-doffaytsdgeos, yeah maximum width14:46
smspillazkenvandine: is this every Qt window ?14:46
kenvandineyes14:46
tsdgeosnic-doffay: probably becuse the phone runs it maximixed14:46
kenvandinethat doesn't set the position14:46
kenvandinemany of the qt examples set position to other than 0,014:47
kenvandinethose are fine14:47
seb128kenvandine, is that breakage in raring?14:47
kenvandinebut if you drop that and try again14:47
kenvandineit breaks14:47
smspillazkenvandine: seb128 just back it out14:47
kenvandineseb128, yes14:47
nic-doffaytsdgeos, can I disable this?14:47
seb128kenvandine, shrug :/14:47
smspillazguake is not all that important14:47
seb128smspillaz, that will re-introduce the other bug though?14:47
smspillazseb128: yep14:47
smspillazsucks14:47
kenvandinethe fix was for guake14:47
smspillazyeah I know, guake is less important than this14:48
tsdgeosnic-doffay: no clue, to be honest, but just put another item on top of yours and then handle your item size on your own14:48
seb128kenvandine, why do you hate guake? :p14:48
smspillazand I feel like this is going to be nontrivial14:48
kenvandinehehe14:48
smspillazthough maybe it won't be14:48
tsdgeosnic-doffay: did i made any sense there?14:48
seb128kenvandine, can you propose the revert? or talk to sil2100 about it?14:48
kenvandinesure14:48
seb128smspillaz, well, hard freeze is tomorrow, we better revert and try to fix it properly in a SRU?14:48
smspillazmakes sense14:48
kenvandinesmspillaz, should i do that or maybe give you a few minutes to think about it?14:49
smspillazseb128: I have an idea. If you'd prefer a proper fix I can quickly check if one will work14:49
sergiusensfginther: changelog needs to say quantal14:49
nic-doffaytsdgeos, I think so :P14:49
Saviqmzanetti, tsdgeos, hmm, qmlRegisterSingleton<T>(...) → it seems you can't pass that back to C++ :/14:49
nic-doffaygiving it a go now.14:49
kenvandinesmspillaz, ping me and i can confirm the fix14:50
tsdgeosSaviq: ¿?14:50
sil2100kenvandine, seb128: what seems to be the problem?14:50
kenvandineand if it isn't a quick fix14:50
sergiusensfginther: but give me a sec, this might get me the leverage to move to raring quicker14:50
kenvandinei'll revert14:50
smspillazokay hang on14:50
Saviqtsdgeos, it comes back as QObject(0x0) :/14:50
tsdgeosSaviq: really?14:50
kenvandinesil2100, rev 3636 of lp:compiz/0.9.9 breaks qt apps14:50
fginthersergiusens, thanks14:50
Saviqtsdgeos, if I create the instance in QML14:50
Saviqtsdgeos, it comes back fine14:50
tsdgeosthat's weird14:50
kenvandinewindow decorations behind the panel14:50
* smspillaz hates working around all these broken applications14:51
kenvandinebug 116534314:51
ubot5bug 1165343 in compiz (Ubuntu) "Windows are placed with titlebar obscured by Unity menubar." [High,Confirmed] https://launchpad.net/bugs/116534314:51
smspillazkenvandine: actually, go ahead and revert it. I just realized that the only way we can fix it is to regress guake14:51
nic-doffaytsdgeos, as an example I tried moving the width and height to the child instead of the parent: https://pastebin.canonical.com/89362/14:51
nic-doffayIs this what you meant?14:51
nic-doffayBecause I'm not seeing a change.14:51
kenvandinesmspillaz, will do14:51
sergiusensfginther: well, it doesn't really matter as long as the devs do the changelog dance as they've been doing all along14:52
tsdgeosnic-doffay: that's what i meant14:52
tsdgeosnic-doffay: your code is "wrong" though14:52
kenvandinesmspillaz, thanks!14:52
* smspillaz represses some rage about broken applications that set _NET_FRAME_EXTENTS with StaticGravity14:52
tsdgeosnic-doffay: can't specify both width and height and say it has to fill the parent14:52
smspillaz(well _NET_REQUEST_FRAME_EXTENTS at least)14:54
smspillaz(that behaviour is just totally broken)14:54
tsdgeosSaviq: with qmlRegisterSingleton you mean qmlRegisterSingletonType ?14:54
nic-doffaySorted tsdgeos14:54
smspillaz(why would you ask to have window decorations pre-allocated and then ask to be placed as though you didn't have them ... and then get rid of the window decorations)14:54
tsdgeosnic-doffay: cool14:54
Saviqtsdgeos, yeah14:54
tsdgeosSaviq: can i see the code?14:55
Saviqtsdgeos, sure, incoming14:55
=== alan_g|tea is now known as alan_g
Saviqtsdgeos, https://code.launchpad.net/~saviq/unity/phablet.unity-test-module/+merge/15941015:01
tsdgeosSaviq: your callback function is wrong?15:05
tsdgeoshas to be15:05
tsdgeosQJSValue (*callback)(QQmlEngine *, QJSEngine *)15:05
Saviqtsdgeos, it converts automagically15:05
Saviqtsdgeos, actually15:06
Saviqtsdgeos, there's two15:06
tsdgeosah right15:06
kenvandinesmspillaz, sil2100: https://code.launchpad.net/~ken-vandine/compiz/0.9.9.fix_1165343/+merge/15941215:06
Saviqtsdgeos, there's another one for QObject *(*callback)(QQmlEngine *, QJSEngine *)15:06
kenvandinei am going to reopen the bug it reverts the fix for15:06
smspillazkenvandine: cool actually I just realized something15:07
Saviqtsdgeos, smells like a Qt bug, no?15:07
smspillazkenvandine: along with the revert, decor.cpp:157115:07
* kenvandine listens15:07
smspillazadd another check w->isViewable ()15:07
smspillazso w->frame () || w->isViewable () || w->hasUnmapReference ()15:08
smspillazum15:08
smspillaz(w->frame () && w->isViewable ()) || w->hasUnmapReference ()15:08
dandraderSaviq, omg, it passed! https://code.launchpad.net/~dandrader/unity/phablet_remove_fakes_from_qml/+merge/15837015:08
Saviqdandrader, omg! ;)15:08
sil2100smspillaz, kenvandine: the revert itself looks fine to me, so I'll approve it in its current state - feel free to modify it later on15:09
sil2100Since I have to jump out for practice now15:09
tsdgeosSaviq: so the problem is that if you put TestUtil somewhere that is a property of a C++ object that pointer is 0?15:09
Saviqtsdgeos, if I pass the object to C++ it's 015:10
Saviqtsdgeos, even if on QML side it's not15:10
Saviqtsdgeos, and even if an object created on the QML side is not, either15:10
tsdgeosSaviq: tried killing the "qmlRegisterType<TestUtil>(uri, 0, 1, "TestUtilObject"); // for debugging only" just in case?15:10
kenvandine-    const bool visible = (w->frame () ||15:11
kenvandine+    const bool visible = ((w->frame () && w->isViewable ()) ||15:11
kenvandine           w->hasUnmapReference ());15:11
kenvandinesmspillaz, ^^15:11
tsdgeosshouldn't matter but15:11
Saviqtsdgeos, it wasn't there before15:11
tsdgeosok15:11
Saviqtsdgeos, I only added it after I've discovered the issue15:11
tsdgeosyeah seems like you hit another bug :/15:11
smspillazkenvandine: yep15:11
tsdgeoswe shall create a testcase for it15:11
Saviqsimple to reproduce at least15:11
Saviqtsdgeos, unless15:11
Saviqtsdgeos, it might be that I'm passing the object to itself15:12
Saviqbut should work anyway15:12
smspillazkenvandine: seems to work here with guake15:12
tsdgeosit should not matter yeah15:12
smspillazkenvandine: can you check if that works with both guake and qt ?15:12
kenvandinesure15:12
smspillazkenvandine: (do that along with the revert)15:12
kenvandinei can update the MP after i check it15:12
kenvandineyeah15:12
smspillazyep15:12
Saviqtsdgeos, yeah, no difference15:14
tsdgeosSaviq: if you give me a testcase i can try it against 5.115:14
Saviqtsdgeos, testcase? you not want too much? :P15:15
Saviqthe Qt people...15:15
tsdgeosSaviq: or push something to that branch that uses it15:15
tsdgeosand tell me what to do15:15
Saviqtsdgeos, make testUnityTest15:16
tsdgeosin that branch?15:16
Saviqtsdgeos, yes15:16
Saviqtsdgeos, it is a testcase (not a Qt one, though)15:16
tsdgeosSaviq: i can't find where you're using TestUtil in QML in that branch?15:17
Saviqtsdgeos, did I not add the test file?15:17
tsdgeoshttp://bazaar.launchpad.net/~saviq/unity/phablet.unity-test-module/revision/61615:17
Saviqtsdgeos, pushed15:17
Saviqtsdgeos, sorry15:18
tsdgeosno worries15:19
Saviqtsdgeos, pushed some more debug15:19
smspillazkenvandine: any results ?15:20
kenvandinesmspillaz, still building :)15:20
kenvandinesmspillaz, i always build packages and test15:20
kenvandineso no quick rebuilds15:20
smspillazthat is the one thing I dislike about debian packages15:21
Saviqtsdgeos, sounds related https://bugreports.qt-project.org/browse/QTBUG-3009015:21
smspillazI would use a stronger word than "dislike" but I suspect that would start a fire15:21
* smspillaz waves goodbye to like 50 tests15:21
Saviqtsdgeos, the only bug about qmlRegisterSingletonType15:23
kenvandinehehe15:23
tsdgeosSaviq: nah, still fails with 5.115:24
Saviqtsdgeos, k, will build a simple test case tomorrow15:24
smspillazkenvandine: btw, if you have a bug in compiz make sure to mark it as affecting the upstream "compiz" project15:26
smspillazthat way I'll see it15:26
smspillazand squash it15:26
kenvandinesmspillaz, will do!15:26
mzanettiSaviq: haven't ever used registerSingleton15:27
mzanettire, btw15:27
smspillazkenvandine: I'm going to propose a similar merge to lp:compiz which will effectively revert the relevant bits, keep the extra tests and add that isViewable check15:27
Saviqmzanetti, yeah, me neither, really just asking if I'm doing something stupid15:27
mzanettiSaviq: well, you're using a singleton, so probably you are...15:27
kenvandinesmspillaz, cool, i noticed it wasn't trivial to revert that in trunk15:28
Saviqmzanetti, FU ;P15:28
Saviqmzanetti, but other than that, trying to save resources, ya know :P15:28
mzanettiSaviq: but no... can't give any useful info about singleton + qmlRegisterType15:28
mzanettiSaviq: that was trolling... I do well know where it makes sense15:28
Saviq;)15:28
Saviqgood15:28
smspillazkenvandine: yeah there's another similar fix stacked on top15:32
kenvandinesmspillaz, testing now15:32
* smspillaz wishes the NBN would hurry up and arrive in perth, it takes too long to branch lp:compiz15:33
smspillaz#firstworldproblem15:33
kenvandinesmspillaz, the additional check seems to have broken the qt apps again15:36
kenvandinebut it did fix guake :)15:36
smspillazkenvandine: ah really? hmm15:37
smspillazkenvandine: what's an example qt app I can try quickly ?15:37
kenvandinefriends-app15:37
smspillazkenvandine: can I just apt-get install it ?15:37
kenvandineyeah15:37
smspillazcool, lets see15:38
* kenvandine waits15:38
kenvandinesmspillaz, an even simpler test is http://paste.ubuntu.com/5716276/15:40
kenvandinesave that to test.qml15:40
kenvandineand run it with qmlscene15:40
smspillazcheers15:41
Saviqtsdgeos, actually got one now http://ubuntuone.com/6bdHFDTDRTktymrZsaO56w15:44
tsdgeosyep15:46
tsdgeosfails15:46
smspillazkenvandine: hmm, well this is particularly lame15:49
tsdgeosQIntrusiveList15:51
tsdgeosnice name :D15:51
smspillazkenvandine: I feel like this is tricky15:52
smspillazkenvandine: setting the StaticGravity hint means "don't move this window around to accomadate its decorations"15:52
smspillazand Qt is doing that15:52
kenvandine:/15:53
smspillazand we have to run the placement rules before we set the decorations on the window15:53
smspillazkenvandine: I mean, I could make an exception to the StaticGravity rule for windows that would effectively have their decorations be offscreen as a result of being decorated15:54
kenvandinesmspillaz, from the qtbase5 source:15:55
kenvandine    // Determine gravity from initial position. Do not change15:55
kenvandine    // later as it will cause the window to move uncontrollably.15:55
kenvandinethen it uses XCB_GRAVITY_STATIC15:55
smspillazkenvandine: another option is to not revert, then give windows any relevant decorations before we run the placement rules15:56
smspillazwhich kinda makes sense. When you're running the placement algorithm, having more information would work in your favor15:57
Saviqafk, more tomorrow15:57
Saviqcheers all!15:57
smspillazkenvandine: I wouldn't suggest doing it that way for raring though, what I'm suggesting is a fairly big change15:58
kenvandineok, so the revert for raring then?15:59
kenvandinewithout the decor.cpp change15:59
smspillazkenvandine: yeah do that for now16:01
kenvandineok16:02
smspillazkenvandine: I just noticed that what I'm suggesting is actually relatively simple, but I need to check if it works and give it some time to check for other potential regressions too16:02
kenvandinesmspillaz, ok, i just pushed my branch again16:02
=== alan_g is now known as alan_g|afk
smspillazkenvandine: actually, I'm starting to think more and more that Qt setting XCB_GRAVITY_STATIC is what's really broken here16:08
smspillazthe problem with guake was that its a normal window, hasn't yet undecorated itself, yet explicitly asked us not to consider its decorations when placing it16:08
smspillazbut Qt is basically doing the exact same thing, except that it intends to retain its decoratiosn16:08
kenvandinesmspillaz, perhaps, but based on that comment it looks like they did that for a reason16:09
smspillazkenvandine: well, the comment just said "we are basing the geometry of the window based on its gravity ... don't change this later"16:09
kenvandinewe could also patch qtbase516:09
smspillazkenvandine: would you happen to know the person to talk to in qt ?16:09
kenvandineMirv, ^^16:10
kenvandineMirv  maintains our packages16:10
smspillazthe other thing is that metacity and friends must implement some kind of workaround for this16:10
kenvandinesmspillaz, beyond that, i have no idea :)16:10
didrocksfginther: not sure what happened, but unity next was uploaded to the daily-build-next ppa :/16:10
didrocksfginther: https://launchpad.net/~ubuntu-unity/+archive/daily-build-next16:10
didrocksfginther: we are goint to push the unity compiz on it, I'm removing it16:10
kenvandinesmspillaz, what would you suggest besides XCB_GRAVITY_STATIC?16:11
didrocksfginther: same from unity-lens-mock16:11
smspillazkenvandine: XCB_GRAVITY_NORTHWEST16:11
didrocksand libunity16:11
didrocksfginther: /!\ urgent, remove that16:11
didrocksmterry: do you mind following that up with fginther once he's back?16:12
kenvandinesmspillaz, http://paste.ubuntu.com/5716339/16:12
didrocksmterry: and eventually, remove what we should meanwhile16:12
didrocks(I removed the 3 I mentionned)16:12
fgintherdidrocks, I'm a little confused, do you need me to remove them?16:12
kenvandinesays adapt geometry to match the WM expectations :)16:12
didrocksfginther: you shouldn't dput them in raring16:13
didrocksfginther: the daily release is doing it16:13
=== alan_g|afk is now known as alan_g
fgintherdidrocks, it came from CI? that's news to me16:13
didrocksfginther: seems so16:13
fgintherdidrocks, I'll dig into it16:14
didrocks unity - 7.81~phablet1bzr3258raring016:14
didrocksfginther: the version seems ci-ish? ^ :)16:14
fgintherdidrocks, true16:14
kenvandinesmspillaz, although it does look like it might do the right thing if i default it to northwest16:16
didrockskenvandine: for lp:compiz, do you know what to do?16:26
didrockskenvandine: with your branch16:26
kenvandinedidrocks, no, the revert isn't as trivial there16:27
kenvandinethere are more stacked changes on top of the same code16:27
kenvandinewhile waiting for CI to run again, i am doing a build of a patched qtbase5 too16:28
smspillazdidrocks: leave lp:compiz alone for now, I need more time to think about this16:29
smspillazand I don't want users of other apps (namely guake) complaining16:29
didrockssmspillaz: ok, can you get it written somewhere so that we don't forget about it?16:31
didrockssmspillaz: well, I think ken will notice the regression :p16:31
kenvandine:)16:31
didrocksso maybe that's enough to rely on kenvandine seeing it ;)16:31
* smspillaz throws a knife at the ICCCM16:31
kenvandinethe sdk guys will notice quickly too... it just took us a long time before blaming compiz :)16:32
smspillazI still think the sdk behaviour is questionable at best16:32
smspillazI'm just trying to figure out why it works with other window managers16:32
mterryfginther, I manually copied those packages over, my fault16:35
mterryfginther, I should have checked with didrocks16:35
fginthermterry, ahhhh. no worries, I was having some serious problems tracking it down ;-)16:36
mterryfginther, sorry :-/16:37
kgunnmzanetti: ping16:37
mzanettikgunn: pong16:38
=== salem_ is now known as _salem
=== _salem is now known as salem_
kgunndednick: ping16:43
smspillazah interesting, the other window managers don't work the way I think they do16:46
kgunnnic-doffay: ping16:46
mterryfginther, I did see you retrying the arches branch a few times16:49
mterryfginther, any luck there?16:49
didrockstedg: see, changing the name of the maintainer for yours is failing :) jenkins is rejecting you! :)17:01
didrocksmterry: phew, you made me afraid :)17:01
didrocksmterry: we are not going to land unity next in it at first17:01
dednickkgunn: pong17:07
smspillazkenvandine: ok, so I have something that fixes qt, guake and others17:14
smspillazthough it feels weird17:14
smspillazalthough, based on what I've read (and observed) with other window managers, the behaviour I'm doing seeks to be OK17:14
didrocksmterry: oh, as well, do you mind closely look at https://code.launchpad.net/~mandel/unity/plan-b/+merge/158383, it needs to get merged for tomorrow daily release17:15
didrocksfginther: as well ^17:15
didrocks(thanks!)17:15
smspillazkenvandine: basically, I've made it so that if you get undecorated (and aren't maximized) you get repositioned as though you were the same size as if you had decorations17:15
smspillazkenvandine: does that seem sane to you ?17:15
smspillaz(this is basically what kwin does)17:15
smspillazmetacity does something else, but that feels really broken, you basically just lose the decorations and end up with blank space above you17:16
=== olli_ is now known as olli
fginthermterry, is it safe to split a project branch into a raring branch at any time? Even if  there are recent changes in trunk that have not yet been released?18:24
mterryfginther, depends?  Do you want those changes in raring or not?18:25
mterryfginther, just depends on where in trunk you split I guess18:25
fginthermterry, yes they need to be in raring. The changes have been merged, but daily release has not run on them yet18:26
mterryfginther, so sure, split off raring and both branches will have the changes18:26
fginthermterry, excellent, that was my assumption.18:26
fginthermterry, cyphermox, one of you will need to deploy this when ready, correct? https://code.launchpad.net/~bregma/cupstream2distro-config/oif-raring-split/+merge/15946118:29
mterryfginther, yeah, but didrocks still needs to do final deployment (we need an archive-admin to do that)18:31
mterrybregma, why are there target_branch entries in raring in that branch?  Wouldn't everything move to head?18:31
fginthermterry, the raring stacks get the maintenance branches and head gets the trunk branches, correct?18:39
mterryfginther, yeah18:40
cyphermoxfginther: commented19:15
alecufginther: hi, thanks for the help yesterday fixing the jenkins issue. One more question: in lp:unity is it tarmac that takes care of landing branches with "Status: Approved"? I'm guessing that it might take some time for them to be landed, but I just wanted to make sure.19:26
alecu(I'm asking about the same branch as I asked yesterday: https://code.launchpad.net/~mandel/unity/plan-b/+merge/158383 )19:27
fgintheralecu, we use a jenkins process similar to the -ci jobs to do the merge19:27
fgintheralecu, that branch is building right now19:28
alecuexcellent!19:28
cyphermoxfginther: are you saying there are some changes in the oif stack that need to land ?19:32
fginthercyphermox, they have been merged into trunk, but daily release has not run on them yet19:33
cyphermoxfginther: mterry: I'll deploy the stuff in cu2d once it's mrged.19:33
cyphermoxok19:33
cyphermoxand it's stuff that needs to land in raring?19:33
cyphermoxfixing release critical bugs?19:33
fginthercyphermox, that's a question for bregma19:33
fgintherbregma, ping19:43
bregmawell, there's a major fix to libgrip that should land, the symptom being "it doesn't work"20:16
bregmasame for the geisview tool in geis, but that's a lower priority since it's really only for diagnostics20:17
fgintheralecu, plan-b has merged20:19
alecufginther: great!20:19
alecufginther: this means that it will be released in tomorrow's unity package, right?20:20
fgintheralecu, yes (as long as all the pieces work)20:21
fgintherbregma, I approved the oif-raring-split mp and will deploy the upstream jobs once it merges20:24
fgintherbregma, didrocks will have to do the final step to get this update into the daily release process20:26
bregmathanks20:57
kgunnmterry: ping21:09
mterrykgunn, hi21:10
=== salem_ is now known as _salem
SveinTanyone else here having problems with blur enabled on the Dash?21:55
SveinTperformance got horrible from 12.10 to 13.0421:55
=== hggdh_ is now known as hggdh

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