EgonR|2Is there a roadmap or preliminary release date for the rewritten unity? (QT/QML)01:03
EgonR|2Also, will the QML version be free from all gtk* cruft?01:05
EgonR|2or will it still depend om gtk in any way?01:05
EgonR|2Not very lively today03:21
EgonR|2Ill wait a few more hours before i go to bed.03:21
dufluEgonR|2: Did you have a question that needs answering?03:24
EgonR|2I asked two and an half hour ago the following: Is there a roadmap publically available or preliminary release date for the rewritten unity? (QT/QML). Also, will the QML version be free from all gtk* cruft?03:25
EgonR|2perhaps you would know03:25
dufluEgonR|2: Yes *gtk* should be gone. And a basic roadmap is https://wiki.ubuntu.com/Mir/Spec#Roadmap03:27
EgonR|2Awesome! I am looking forward to is so much03:28
EgonR|2Will it be MIR exclusive? Or can you run the new unity on Xorg03:29
EgonR|2I mean, writing and entire displayserver takes time..03:30
dufluEgonR|2: Not totally sure. You should assume it's Mir-exclusive but we'll always aim to make things portable eventually. It's just not a priority right now03:31
EgonR|2Yeah, i understand that.03:31
dufluThis is where we need some Unity Next people online. I think they're mostly in Europe so won't wake up for a few hours yet03:32
EgonR|2Oh ok03:32
EgonR|2Looks like that is very much a pattern for most OSS projects on freenode.03:33
EgonR|2I think US programmers use EFNet a lot more than the europeans though.03:34
dufluMaybe... Though it's Canonical policy to have its public discussions on FreeNode03:36
dufluUbuntu policy even03:36
EgonR|2freenode seems like a better host for a software project. efnet is littered with strange channels as well..03:37
EgonR|2duflu: Thank you for your time, it was most helpful. I will go back to lurking.03:39
dufluEgonR|2: No problem. Sorry more people don't pay attention very often03:45
didrockshey Mirv, how are you?06:10
Mirvdidrocks: morning, fine, and you?06:16
didrocksMirv: I'm good, thanks! :)06:21
didrocksMirv: I know it's Qt4 and not Qt5, but do you mind having a look at bug #1155327 and tell us what's the best course of action for it?06:21
ubot5bug 1155327 in skype (Ubuntu Raring) "skype crashed with SIGSEGV in malloc@plt()" [High,In progress] https://launchpad.net/bugs/115532706:21
Mirvdidrocks: ok, looking..06:28
Mirvcommented, although can't reproduce myself as it needs nvidia hardware to happen (+ possibly prelink installed). I don't know what to do other than revert to qtwebkit-source 2.2.1 (or ask for rebuild/fix from Microsoft)06:56
Mirvor finding out skype is the only affected package and accepting the workaround for it06:57
didrocksMirv: do you mind discussing that with the kubuntu guys?06:57
didrocksMirv: I think we should, from your comment revert06:58
didrocksMirv: as you told, we are not sure if skype is the only affected package06:58
Mirvok, I'll ask around on kubuntu channel06:58
didrocksthanks :)06:58
Mirvriddell has looked into it a week ago, asked for an update07:01
didrocksMirv: excellent, do you mind keeping me in touch?07:01
Mirvdidrocks: ok07:05
didrockssil2100: good morning, how are you?07:20
sil2100didrocks: morning! Good, how about you?07:23
didrockssil2100: I'm good, thanks!07:24
didrockssil2100: do you have time for some touch apps boostrapping help for preparing that for S?07:24
sil2100didrocks: should be no problem, what can I do to help?07:25
didrockssil2100: should we have a quick hangout? It will maybe be easier07:25
sil2100didrocks: ok07:27
didrockssil2100: now is fine for you?07:27
sil2100didrocks: yep, ready when you are ;)07:29
Saviqmzanetti, whoa http://s-jenkins:8080/job/unity-phablet-ci/07:35
sil2100didrocks: just point me to the hangout when ready ;)07:35
mzanettiSaviq: yeah... I just cancelled them and am debugging right no07:35
didrockssil2100: yeah, finishing a discussion and will ping you soon :)07:36
Saviqmzanetti, thanks07:36
didrockssil2100: https://plus.google.com/hangouts/_/effa6372491bd8d9829359be5de48d6464cfbc83?authuser=0&hl=en07:40
dandraderSaviq, ping08:11
Saviqdandrader, pong08:14
dandraderSaviq, I've been working on removing those FooFake vs. FooWrapper switches and components throughout the qml code (e.g. get rid of ApplicationManagerFake). The plan is to have a fake implementation of Ubuntu.Application module. That way we can get the qml tree pretty much clean of those things. It started out as infrastructure to test RunningApplicationTile but then I though it would be better to just apply it on the whole code base08:18
dandraderSaviq, what do you think?08:18
dandraderThose FooFake vs. FooWrapper switches might be fine for running on the desktop but get in the way of testing08:20
Saviqdandrader, yeah, if we can make without those wrappers, I'm game08:24
Saviqdandrader, only so that the actual Ubuntu.Application is used elsewhere, make sure you put it in a separate module path08:25
Saviqand export QML2_IMPORT_PATH in the ./run script08:25
dandraderSaviq, right08:27
mzanettihey, anyone knows whats going wrong that after recent upgrades in raring the machine doesn't come up any more because of a read only rootfs?08:38
Saviqmzanetti, why would you shut down? it's so '90s ;)08:39
Saviqmzanetti, but seriously, nope, haven't had that08:40
mzanettiSaviq: let me rephrase: the machine doesn't come up after a quick reboot because of kernel upgrades08:40
mzanettiveebers has the same in his raring VM08:40
mzanettiveebers: yeah, if you could find out if its only with raring too or happens after inclusion of phablet ppa's that would be great08:42
Cimitsdgeos, hi :)08:42
veebersmzanetti: I'm just fire-ing up the VM now :-)08:42
tsdgeosCimi: hi there08:42
CimiI need a little help here if you have 5 mins08:42
CimiI was moving tests under subdirs08:43
Cimibut hud test complains of missing hudclient08:43
CimiI think is something to do with the makefiles08:43
Cimican you help me?08:43
tsdgeosgive me a sec08:45
Saviqmzanetti, we got a PASSED!08:48
mzanettiSaviq: huh?08:48
mzanettithe tests?08:48
Saviqmzanetti, http://s-jenkins:8080/job/unity-phablet-ci/349/08:48
mzanettiSaviq: yeah... seems one of the VM's is configured different than the other08:48
mzanettiSaviq: and makes the tests stall... I'm on it08:48
mzanettiSaviq: right now Mir is building on the "broken" one which takes 1.5 hours :/08:49
* mzanetti is happy that unity phablet ci runs in ~10 mins :D08:49
tsdgeosSaviq: so i got a patch by "the qml guy"08:52
Saviqtsdgeos, nice08:52
tsdgeosit does indeed fix the crash but i'm not sure his solution is great08:52
Saviqtsdgeos, does it help?08:52
Saviqtsdgeos, :)08:52
Saviqtsdgeos, is it in gerrit?08:52
tsdgeosproblem he is in US west coast, so it's going to take a while to ping-pong between us08:52
Saviqtsdgeos, can you think of a better solution?08:54
tsdgeosSaviq: it's not a better solution, it's that this one is only partial08:54
tsdgeosi.e. accounts for removals to the cache but not additions, not really sure that this is a problem, but it might08:55
tsdgeosand also the delayed stuff comes from somewhere that does "delete this" thus we effectively end up with a dangling pointer in the cache too08:55
tsdgeosit "might" not be a problem08:55
tsdgeosbut i'd want to make sure he has thought about these08:55
Saviqtsdgeos, that's what gerrit is for, isn't it :)08:56
tsdgeosit is08:56
tsdgeosi'm still perfecting my text :D08:56
Saviqtsdgeos, fortunately it's not awfully pressing08:56
Saviqtsdgeos, so we'll get there when we'll get there08:56
tsdgeosCimi: what happened is this "Panel/qml/HudClient/"08:57
Saviqtsdgeos, will you own getting this distro-patched?08:57
Cimitsdgeos, you have commit access if you can fix :)08:57
tsdgeosSaviq: once we agree on a solution upstream? sure, it's just messing with zoltan/mirv to get it into the packages, no?08:58
Saviqtsdgeos, yup08:58
tsdgeossure, no worries08:58
veebersmzanetti: after a dist-upgrade with no PPAs and a reboot, machine came up fine. I'll try with a single ppa now see if that breaks09:02
mzanettiveebers: cool, thanks09:02
Saviqnic-doffay, you joining the hangout?09:02
nic-doffayOne sec Saviq09:02
Saviqnic-doffay, cheers09:02
tsdgeosmzanetti: ping09:04
mzanettitsdgeos: yes sir09:04
tsdgeosmzanetti: you added/wanted to add a make -i/-k to qmluitests, right?09:05
mzanettitsdgeos: yes09:05
tsdgeosis that a good idea?09:05
mzanettiwhy not?09:05
tsdgeosi understand CI wise it makes sense09:05
tsdgeosbut if i'm sitting here09:05
MacSlowCimi, ping09:05
tsdgeosi run "make qmluitests"09:05
tsdgeosit writes lots of stuff in the shell09:05
CimiMacSlow, pong09:06
tsdgeoslike two screens09:06
tsdgeosand most probably i'll only look at the last test and see a "passed"09:06
mzanettitsdgeos: its only when you run "./runtests.sh"09:06
tsdgeosok :-)09:06
* tsdgeos hits head09:06
mzanettitsdgeos: if you run "make qmluitests" manually there is no -k :D09:06
tsdgeosyeah, sloooooow09:06
mzanettitsdgeos: hint: run "make alltests"09:06
tsdgeosCimi: pushed09:06
Cimitsdgeos, do panel tests work now?09:08
tsdgeosCimi: sure, why wouldn't they?09:08
Cimitsdgeos, thought thy needed qml subdir09:08
tsdgeosthey do09:08
tsdgeosit's still there09:08
tsdgeosthe part of it they need09:08
tsdgeosthis is good too09:09
tsdgeossince we're splitting the "qml" subdir that holded separate stuff for two separate tests09:09
Cimitsdgeos, because you modified cmakelist of panel09:09
Cimitsdgeos, the part you left doesn't need it?09:09
tsdgeosand putting it into the correct places09:09
tsdgeosCimi: no09:09
Cimitsdgeos, if everything works we're ready for MR09:09
luvmardy: ping09:10
Saviqnic-doffay, you with us? ;D09:10
CimiSaviq, mzanetti tsdgeos https://code.launchpad.net/~unity-team/unity/phablet.moving_tests/+merge/15782109:15
mzanettiCimi: ack09:15
Cimimzanetti, better sooner than peter so we have less tests to move afterwards :)09:16
veebersmzanetti: after the raring dist-upgrade, I added the phablet-team ppa and again did an update && dist-upgrade. I get this error: https://pastebin.canonical.com/88744/09:20
veebersmzanetti: running dist-upgrade again I get this error: http://pastebin.ubuntu.com/5691831/ (the apt-get -f install log: http://pastebin.ubuntu.com/5691833/)09:26
mzanettiveebers: hmm... somehow I can't figure how this relates to a read-only filesystem at boot09:27
veebersmzanetti: the dist-upgrade after that went fine, but after the reboot I get the read only filesystem09:27
mzanettiSaviq: any idea? ^09:28
veebersmzanetti, Saviq: this is what the phablet-team dist-upgrade wanted to install: http://pastebin.ubuntu.com/5691841/09:29
mzanettiveebers: mountall... that sounds like it could be involved09:29
mzanettiveebers: mind reverting your VM and just upgrade that package?09:29
tsdgeosCimi: can you merge master into https://code.launchpad.net/~unity-team/unity/phablet.test_responsiveflowview/+merge/157343 ? It's what making the qmluitests fail09:30
veebersmzanetti: yeah I can do that, I'll upgrade it using the phablet-team ppa09:31
mzanettiveebers: thanks09:31
Cimitsdgeos, ok09:37
Cimitsdgeos, done09:38
mardyluv: pong09:43
veebersmzanetti, Saviq: After adding phablet-team and installing mountall, rebooting gives me a readonly filesystem09:44
mzanettiveebers: \o/ thanks a lot!!!09:44
veebersmzanetti: no worries, good spotting with the mountall issue09:45
mzanettianyone knows what exactly this does, and expecially why we need a -phablet version for that?09:45
luvmardy: Hi! I've been messing around with accounts-sso and implementing the logout functionality to gnome-control-center-signon. I am trying to figure out an API call for libsignon to delete the stored password. What is also confusing me is that there is no signond running on my system.09:46
luvhttp://docs.accounts-sso.googlecode.com/git/libsignon-glib/html/SignonIdentity.html#signon-identity-remove just does not work for me09:46
veebersmzanetti: do you mind if I leave that with you? I need to finish for the night09:47
luvon the other hand, if I delete an account in gnome-control-center-signout the credentials are removed (I guess it communicates with signon via dbus - but no signond running?)09:47
mardyluv: it should work, because we are using that function call from inside libaccount-plugin to delete the identity when the account creation is aborted09:48
mardyluv: signond must be running, but it exits after a few seconds of inactivity09:48
mardyluv: set the debugging level to 2 in /etc/signond.conf, then keep an eye on the syslog09:49
luvyes, I was looking at libaccount-plugin to see how it deletes the account - but I didnt see a direct use of that function only a lot of dbus stuff09:49
mardyluv: you should be able to see if/why your call fails09:49
luvumm, I will try that09:49
mardyluv: http://bazaar.launchpad.net/~online-accounts/gnome-control-center-signon/trunk/view/head:/libaccount-plugin/oauth-plugin.c#L18709:50
=== dandrader is now known as dandrader|afk
luvthanks a lot, I will have another look tonight :-)09:50
mardyluv: but I'm a bit worried, why are you deleting identities? :-)09:51
luvalright - to implement "logout" functionality09:51
mardyluv: I don't think you should delete the identity in that case -- doing so will mess things up09:52
Cimitsdgeos, I've also rebased to the moving tests branch09:52
luvmardy: well sure, but you cant keep passwords and tokens around if you want to implement logout09:52
mardyluv: if you really want to, you could store an empty password on the identity, but I'm not 100% sure it's a good idea09:52
luvthat'd be useless09:52
mardyluv: why?09:53
luvkeeping passwords and tokens around would be useless09:53
mardyluv: ah, yes, agreed09:53
luvwell, I can modify gnome-control-center-signout to deal with a deleted identity properly09:54
luvand the apps themselves - they dont have access to identities, do they?09:55
mardyluv: you could retrieve the current IdentityInfo, clear the password (actually, I think you'll get it empty anyway), delete the record and then store the IdentityInfo as a new one09:55
mardyluv: they have access -- they use it when authenticating09:56
mardyluv: oh, a better solution, to make sure that there aren't race conditions, is to create the "cleared" identity record first, then store its ID into the account, and finally remove the previous ID09:57
mardyluv: maybe that's exactly what you wanted to do :-)09:58
=== dandrader|afk is now known as dandrader
luvmardy: sounds good!10:07
luvi will experiment with that for now ;-) and see where it goes10:07
mardyluv: but I think that it would be much better to implement a logout API directly in signond10:09
luvsure, but let me try this first10:12
mzanettiCimi: you know whats weird...10:13
Cimimzanetti, no :)10:13
mzanettiCimi: if I call "runtests.sh" on current trunk it says 35% coverage10:14
mzanettiCimi: if I run it on your branch its less10:14
Cimimzanetti, let's fixthe script then10:15
mzanettiCimi: ... (following code guidelines from mzanetti) ...10:15
mzanettiinteresting... which ones?10:15
Cimimzanetti, when I asked you yesterday10:16
mzanettiCimi: you mean the ordering of testcase and component10:16
mzanettiCimi: yeah... its fine... I was just wondering... my guidelines? :D10:16
Cimimzanetti, you now own a guideline, cheers :)10:16
mzanettiCimi: mystery solved... and approved. cheers10:23
Cimimzanetti, where was it? :)10:24
mzanettiuntested OnScreenKeyboard.qml was removed in trunk but still present in your branch10:24
Cimiah ok10:24
mzanettiwe just reached > 50% coverage!10:26
mzanettithanks everybody10:26
Saviqmzanetti, who told you to add phablet-team ppa? ;10:30
Saviqand to upgrade from it!10:30
Cimimzanetti, some tests are strikeout in the coverage document, means not needed?10:30
mzanettiSaviq: still, this is an issue10:30
mzanettiSaviq: I think our target should be to enable people to add the paa, no?10:31
Saviqmzanetti, not really, stuff should land in distro instead10:31
mzanettiSaviq: yeah... but I guess the packages in our ppa will be merged to the distro... as soon that happens it'll smash your system too10:32
mzanettiCimi: yep10:32
Saviqmzanetti, I don't know what's changed in mountall10:33
Saviqmzanetti, looking at https://code.launchpad.net/~phablet-team/phablet-extras/mountall-quantal I'd say this should not land in distro10:36
Saviqmzanetti, but ping rsalveti or awe about it10:37
mzanettiSaviq: yeah, I will! thanks for the research!10:37
nic-doffayGuys, how can I compile the Hud with qmlscene?10:48
nic-doffaytrying qmlscene Hud.qml10:48
Cimitsdgeos, ^10:55
dandradernic-doffay, you might need to pass the import path where the qml plugins it needs to run are located10:55
dandraderwith -I path10:55
Ciminic-doffay, there's a chance you nees -I and a path10:55
tsdgeosnic-doffay: the test?10:55
nic-doffaytsdgeos, not the test, the actual Hud.10:58
tsdgeosnic-doffay: why do you want to run the actual Hud with qmlscene?11:02
nic-doffaytsdgeos, just to take a look at it easily.11:02
tsdgeosi'm not sure it even works11:03
tsdgeosbut sure, you'll have to pass the correct path to the HudClient plugin11:03
=== MacSlow is now known as MacSlow|lunch
tsdgeosdednick: what creates unity.qmlproject.user ? QtCreator?11:13
dednicktsdgeos: yes11:13
dednicktsdgeos: yes11:14
dednickmy bad11:15
=== greyback is now known as greyback|lunch
Cimisomeone *did* test Panel/IndicatorItem.qml?11:26
Cimithere's a Y in the document, but it's not green nor link on MR11:27
tsdgeosi'd say it's "part" of the regulra Indicators test11:31
tsdgeosmaybe i'm mistaken11:31
dandraderCimi, if you don't see a test for it then it's not tested. I would guess that original list was generated by checking which files were read by the tests11:33
Cimidandrader, ok thx11:33
dednickanyone ever used the qml's item::mapFromItem ? it's driving me insane. trying to map a repeaters items to global position. i'm getting the correct location for the first item, but each subsiquent item is 2x the width of the previous item away...11:34
tsdgeosdednick: i used it once, worked without problems11:35
tsdgeosdednick: mapFromItem doesn't map width, you mean x?11:35
dednicktsdgeos: i mean the x of items after the first repeaters itms are 2xthe previous items width away.11:38
dednickso if the first items x's are 0, 28, 56, 84. when i do mapFromItem, i'm getting 0, 56, 112, 16811:38
tsdgeosright, you're probably maping it from the wrong item11:38
dednickbut it shouldnt change the x scale if the item is in the parent heirachy surely?11:39
tsdgeosnot sure11:39
tsdgeostry ignoring the repeater11:39
tsdgeosand map against the row11:39
tsdgeosor whatever is holding the repeaetr11:40
dednicki'm mapping to the indicator11:40
dednickindicator -> indicator row -> row -> row repeater -> item11:40
dednickbur i've tried doing it to a few of them with the same resules.11:41
nic-doffayWhat's the deal with the frantic key presses when running qmlscene?11:46
nic-doffayIt's really annoying.11:46
dednicktsdgeos: found the issue. me being a general idiot.11:46
dednicki was including the x,y of the item in the mapFromItem call, which was adding width...11:47
dednicktsdgeos: found the issue. I was including the x,y of the item in the mapFromItem call, which was adding width...11:49
dednickyeah. pretty stupid11:49
dednicki take solice in the fact that i'm not the first to do it. found answer on the web ;)11:50
Cimimzanetti, hey, I'm testing the Clock: I have two ways to get the text label: using objectName and grepping the label, or creating a readonly property alias of those labels11:50
Cimimzanetti, which is the preferred?11:50
mzanettiCimi: if possible, avoid API changes just for tests... prefer findChild()11:51
mzanettiCimi: if the API already offers that property for other reason, use that because its faster than findChild()11:51
dandraderSaviq, is it so that the ApplicationLauncher plugin is not used anymore?11:56
=== alan_g is now known as alan_g|ACCU
Saviqdandrader, yes11:56
dandraderSaviq, can we nuke it?11:56
Saviqdandrader, yup11:56
dandraderSaviq, will do11:56
Cimiwhere do I find qt. javascript functions?12:02
CimiI want to see if I can get a date from a string12:03
Cimi(testing Greeter's Clock)12:03
Cimithere's Qt.formatTime and Qt.formatDate, how about the opposite?12:03
mzanettiCimi: interesting... where?12:04
Cimibut it's not documented12:04
Cimiactually I'm not sure that works12:05
mzanettiCimi: Qt itself doesn't really ship any javascript functions... The javascript engine itself support the ECMA standard12:05
Cimiah ok12:05
mzanettiCimi: additionally Qt register a object (called "Qt") that offers a little functionality12:06
mzanettiCimi: its mostly a subset of the QDateTime C++ class12:06
CimiI was speaking of that12:06
Cimido you know how can I do the opposite?12:06
Cimicheck if a string is a regular date?12:06
=== MacSlow|lunch is now known as MacSlow
mzanettiCimi: not out of my head, no12:08
mzanettiCimi: but does that make sense? I mean... the string is the direct output of Qt.formatDate()... No need to test if formatDate() works. Qt folks should test that inside Qt12:09
Cimimzanetti, so I will check if those strings are not empty12:12
mzanettiCimi: yeah, thats good. you can also compare with a static string I guess12:13
Cimimzanetti, static string of that?12:13
mzanettisomething linke this in the data() function:12:14
mzanetti{ input: 123456; outputData: "Mon Feb. 12"; outputTime: "12:34" }12:14
mzanettiif you have a predefined input date (e.g. some unix timestamp) you can compare if the label holds that correct time12:15
mzanettijust an idea... not sure its really way to go...12:15
mzanettinot empty could in theory also be something like "Error converting date" which is not what we want12:16
mzanettiCimi: ^ (not sure if you still saw it)12:19
Cimimzanetti, do you think qt.formatTime returns that?12:20
* Cimi tests12:20
Cimimzanetti, just tested, qt returns empty string12:21
mzanettiCimi: for what=12:21
CimiI tested passing "__date" to Qt.formatTime and formatDate12:21
Cimiinstead the variable __date12:21
=== dednick is now known as dednick|lunch
=== dandrader is now known as dandrader|lunch
mzanettitsdgeos: https://code.launchpad.net/~mzanetti/unity/phablet-copyright-headers/+merge/15785812:34
Cimimzanetti, https://code.launchpad.net/~unity-team/unity/phablet.test_greeter-clock/+merge/15785912:35
Cimimzanetti, my only question here is if I should change width and height to something else, these are half width and height of greeter/dash tests12:36
=== greyback|lunch is now known as greyback
mzanettiCimi: I'd say its fine12:36
mzanettiCimi: if you change it, then I guess childrenRect.height/width would be ok12:37
CimiI should change to that?12:38
Cimiit does binding loop12:38
=== _salem is now known as salem_
mzanettiCimi: reviewed12:47
davmor2hey guys is there an actual package for autopilot if so what is it called or is it installed by default?12:47
mzanettidavmor2: autopilot-phablet12:48
mzanettidavmor2: if you're working with "the old unity" you can use python-autopilot12:48
mzanettidavmor2: both are in ppa:autopilot/ppa12:48
davmor2mzanetti: thanks12:48
mzanettiCimi: could you do a quick check on this please: https://code.launchpad.net/~mzanetti/unity/phablet-copyright-headers/+merge/15785812:49
mzanettiCimi: we need to merge this soon because I've enabled the check Jenkins and that will put Needs Fixing on all the other MP's until this is merged12:50
=== dednick|lunch is now known as dednick
tsdgeosmzanetti: done12:58
tsdgeoswe have 33 branches waiting for merge13:04
tsdgeosinstead of down we're going up :D13:04
tsdgeosmzanetti: you getting https://code.launchpad.net/~paulliu/unity/phablet-add_unit_test/+merge/156859 ?13:18
tsdgeosmzanetti: want me to do https://code.launchpad.net/~mzanetti/unity/phablet-pageheader-tests/+merge/157711 ?13:19
mzanettitsdgeos: sure13:19
mzanettitsdgeos: hmm... the openeffect thing is indeed a bit strange13:20
tsdgeosyeah, may an artifact of that the class hasn't really much to test13:20
mzanettitsdgeos: but it would have potential for the coolest looking test :D13:23
mzanettitsdgeos: well, dunno... I would have tested it quite differently... but not sure if its worth the effort to force paul to invest more time in it13:24
mzanettithere isn't much to test indeed. I guess he covered that bits. its not to too obvious13:24
Cimimzanetti, was sleeping like a baby during lunch break :)13:31
mzanettiCimi: :D13:32
Saviqguys, we'll be late13:32
mzanettiSaviq: should we just start or wait?13:32
Saviqmzanetti, go13:32
Saviqgreyback, your turn for notes13:33
greybackSaviq: ok13:33
Cimidednick, nic-doffay standup13:34
nic-doffayDon't have a mic here Cimi .13:35
CimiSaviq, are you going to merge the two dirs or shall I?13:50
SaviqCimi, you13:51
CimiSaviq, so to recap: qmluitests -> shell, unittests -> shell13:51
paulliuIs there copyright problem FTBFS?13:51
CimiSaviq, and two appropriare macro13:51
SaviqCimi, yeah, the macro and the correct calls are already in place13:52
SaviqCimi, you just need to movie it in a single CMakeLists.txt file13:52
tsdgeospaulliu: yes, merge with trunk13:55
tsdgeosCimi: can you merge trunk again in here? https://code.launchpad.net/~unity-team/unity/phablet.test_responsiveflowview/+merge/15783013:56
Cimitsdgeos, I will13:57
tsdgeosCimi: ah wait, sorry no need13:58
tsdgeoswe just need the copyright thing to work13:58
tsdgeosand i'll approve again13:58
mzanettipaulliu: what time is it at yours btw?13:58
mzanettitsdgeos: thanks13:58
CimiSaviq, in reality is slightly different http://paste.ubuntu.com/5692396/13:59
SaviqCimi, right, we don't support the path, just the component name14:00
SaviqCimi, but yeah14:00
SaviqCimi, that looks fine14:00
CimiSaviq, maybe I am not that into cmake14:00
SaviqCimi, that paste looks fine14:01
CimiSaviq, but is not correct14:01
* Saviq => meeting14:01
Cimiwho helps me here?14:01
mzanettiCimi: Saviq: that will break the default things14:01
mzanettiCimi: I can14:01
paulliumzanetti: 22:0114:01
Cimiok will push in a branch mzanetti14:01
paulliumzanetti: it's ok for me.14:01
mzanettipaulliu: poor guy daily standup at 9pm...14:01
Cimimzanetti, lp:~unity-team/unity/phablet.merge_qmluitests-unittests14:02
mzanettiCimi: all you need to do is to remove line 15 and 16 from the paste14:03
Cimimzanetti, but all files into this dir14:03
Cimimzanetti, will be moved under shell/Components14:03
Cimimzanetti, with the same CMakeFile used for qmluitests14:04
Cimihow can that work?14:04
=== dandrader|lunch is now known as dandrader
mzanettiCimi: you need to merge unittests/Components/CMakeLists.txt and qmluitests/Components/CMakeLists.txt too14:05
mzanettiCimi: "shell/Components" ?14:05
paulliutsdgeos: trunk is lp:unity/phablet, right?14:05
mzanettipaulliu: yes14:05
Cimimzanetti, tests/shell/Components14:05
Cimiqmluitests => shell14:05
paulliumzanetti: hmm, I merged. But CI build failed.14:06
mzanettiCimi: I don't know why you want to rename that too... just leave it qmluitests14:06
Cimimzanetti, following michal mail14:06
mzanettiCimi: ok then... well, just rename it then14:06
mzanettiwhats the problem?14:06
Cimimzanetti, the problem is that to me they will be treated like qmluitests14:07
Cimimzanetti, because the cmakelist is the same14:07
Cimimzanetti, so they have one target?14:07
Cimimzanetti, otherwise for each subdirectory I need to add a special target for unittests14:07
mzanettiCimi: yeah14:07
mzanettiisn't that what you want?14:07
CimiI'll try to do it and ask you for a review14:08
CimiI don't know how this cmake works14:08
mzanettiCimi: I don't see the point in moving the files together but keeping separate make targets14:08
mzanettiactually I don't see the point in moving them together at all...14:09
Cimimzanetti, so why we agreed on that on the standup?14:09
mzanettiI'm not against it... If you all want it... just said I don't see the point14:09
nic-doffaySome QML file structure queries if anyone has a second. I have infographics currently with those dots representing days in the Infographics.qml, however I'd like to make the "day dot" a separate component. Firstly is this overkill?14:12
mzanettinic-doffay: we can't know unless we see how much code it actually is14:13
nic-doffayIt will probably be a fair amount mzanetti14:13
mzanettinic-doffay: sounds ok to me...14:13
nic-doffayOk, in that case how should the folder hierarchy be structured mzanetti ? Shall I put everything in Infographics/ (I imagine there will be more than one component) or just keep all the separate components in /Greeter?14:15
nic-doffayIf other components are created for the Greeter this could create clutter.14:15
nic-doffayAnd confusion.14:15
mzanettinic-doffay: dunno... can't tell without seeing the code. I wouldn't worry too much as long as you have only 3 files. its easy to move them around afterwards if it turns out to get cluttered14:17
nic-doffayThanks for the comments mzanetti I'll keep everything in one file until it grows a sizeable amount.14:19
Cimimzanetti, my new cmake doesn't work as the previous... so I'll gently skip that :)14:19
Cimi(make test doesn't run unittests anymore)14:20
mzanettiCimi: ?14:20
mzanettiCimi: yeah... that was expected14:20
mzanettiCimi: well... It still should run qsortfiltermodeltest14:20
Cimiit does14:20
Cimimzanetti, anyway, let's keep separate for now14:20
mzanettiCimi: then you're fine I'd say14:20
CimiI'll adapt tests with the discussed guidelines14:21
Cimiand update readme14:21
mzanettiCimi: thats was the reason for the separation... to have tests that can run as unittests in that target14:21
mzanettiCimi: but in jenkins we execute all tests anyways, that's why I don't have a stron opinion on it14:21
mzanettidednick: FYI: lp:~nick-dedekind/unity/phablet-test-indicators requires a merge of trunk to pass CI again14:37
dednickmzanetti: hm. i just did it about 30 min ago...14:38
dednicklet me check again14:39
mzanettidednick: ok... sorry... I was just browsing through the failed jenkins jobs14:39
mzanettithe job in question was started 37 mins ago...  :)14:39
dednickmzanetti: i just triggered a rebuild. i pushed about 40 minutes ago14:40
dednickmazhm. i need to run a test suite once for fullscreen, and once for not.14:46
dednickmzanetti: i need to run a test suite once for fullscreen, and once for not. what would be the best way to do that?14:46
nic-doffayWhat's the gu to pixel for the phone?14:47
mzanettidednick: Father ted?14:47
mzanettidednick: what kind or tests?14:47
dednickmzanetti: fullscreen just being a flag for the indicators14:47
dednickmzanetti: panel tests. it's just a flag that needs to be set, but the behaviour should be the same. so dont really want to replicate tests.14:48
mzanettidednick: a _data() function?14:48
mzanetti{ tag: "windowed"; fullscreenFlag: false }14:49
mzanetti{ tag: "fullscreen"; fullscreenFlag: true }14:49
mzanettidednick: you understand what I mean?14:49
mzanettinic-doffay: depends on the screen14:49
dednickmzanetti: nope :)14:49
mzanettidednick: let me write a pastebin snippet... one sec14:50
nic-doffayFor the Nexus 7 lets say mzanetti14:50
mzanettiI think that was 1614:50
nic-doffayCool thanks, just wanted to get some point of reference.14:50
dednicki remember seeing a list somewhere.14:50
mzanettinic-doffay: the Galaxy Nexus has 18 iirc14:50
mzanettinic-doffay: I use 20 on my Laptop with Retina screen14:51
mzanettinic-doffay: bottomline: your code should not care about it14:51
nic-doffayIt doesn't.14:51
mterrydidrocks, cu2d-config has been updated, needs your archive-admin powers to make it final.  Also, maybe you want to review https://code.launchpad.net/~mterry/cupstream2distro-config/media-signals/+merge/157726 while you're thinking on cu2d-config stuff14:51
Cimimzanetti, you can kill me https://code.launchpad.net/~unity-team/unity/phablet.tests_coding-style/+merge/15788814:52
* Cimi runs14:52
Cimiwell the refactoring branch is there ^14:52
mzanettidednick: https://pastebin.canonical.com/88768/14:53
mzanettidednick: the test_seomething() function will now be called for every row in the _data() function14:53
mzanettidednick: so you run the same test twice with different parameters14:54
dednickmzanetti: ah. awesome14:55
didrocksmterry: two questions, is that what's need qt5?14:56
mterrydidrocks, can you rephrase that?14:56
didrocksmterry: you uploaded qt5 to the ppa14:57
didrocksis it needed for anything?14:57
didrocksthose patches can't be into raring?14:57
mterrydidrocks, the patch was for qtubuntu.  I didn't put it in raring because it changes which symbols are exposed14:57
didrocksmterry: ok, and now looking at qtubuntu-media-signals14:58
mterrydidrocks, it just adds symbols (doesn't take them away), but it still felt like a feature break14:58
didrocksmterry: this is qtubuntu-media?14:58
Cimidandrader, https://code.launchpad.net/~unity-team/unity/phablet.tests_coding-style/+merge/15788814:58
mterrydidrocks, this is qtubuntu-media-signals14:58
mterrydidrocks, I'm still draining the swamp to get to my own package14:59
mterrydidrocks, once this is in the PPA I can propose a patch for qtubuntu-media itself14:59
didrocksmterry: ok, I don't see it on the spreadsheet though14:59
dandraderCimi, checking15:00
didrocksmterry: like, in term of tests, integration tests and so on15:00
mterrydidrocks, ah yeah.  This was a late addition (it wasn't on our roadmap until I realized it was needed).  It's a super tiny package, like 20 lines of code.  So it doesn't have any tests  :-/  But I can add to spreadsheet15:00
didrocksmterry: ok, just add it please and add that you have the feeling tests are not needed :)15:01
didrocksmterry: then, you ping me so that I pull for that stack or something else?15:01
didrocksmterry: I saw that qtubuntu-sensors has been added when we shouldn't as per the spreadsheet15:02
mterrydidrocks, added to spreadsheet.  What's that about pulling?15:02
mterrydidrocks, qtubuntu-sensors is really xnox's package.  I added it to the stack though in order to get to my own packages15:03
mzanettitsdgeos: autolanding the header fixes failed :/ I fear we were too early with thelling people to merge trunk15:03
didrocksmterry: the tests are not read apparently :/15:03
didrocksmterry: so I'm not confortable adding it15:03
tsdgeosmzanetti: :-/15:03
mterryxnox, ^15:03
tsdgeosmzanetti: what went wrong?15:03
* xnox is getting a raring chroot up on nexus7 to test out sensors.15:04
tsdgeosmzanetti: can we kill all the autoloanding jobs until that one is the first again?15:04
mterrydidrocks, it would be really nice to decouple a "tests are acceptable for distro" check-off and the work to get these packages into the PPA in the first place15:04
mzanettitsdgeos: ci failed because of the hanging qmluitests and posted a needs fixing just before autolanding would have merged15:04
didrocksmterry: you mean pushing the stuff in the ppa without having tests?15:04
mzanettitsdgeos: yeah... I'll kill the queue15:04
mterrydidrocks, tests are something distro can't really write for PS, so it's blocking our work to daily-release these15:04
mterrydidrocks, they're in a PPA now without tests.  We're just changing the PPA15:04
didrocks1mterry: sorry, got disconnected15:07
mterrydidrocks, I said...15:07
mterry<mterry> didrocks, tests are something distro can't really write for PS, so it's blocking our work to daily-release these15:07
mterry<mterry> didrocks, they're in a PPA now without tests.  We're just changing the PPA15:07
didrocks1mterry: yeah, one time push is fine, just not daily release ;)15:08
didrocks1mterry: the spreadsheet is there to make the separation, right?15:08
didrocks1mterry: do you mind putting qtubuntu-sensors if not good to to_transition:15:09
mterrydidrocks, I just think we're making things harder on ourselves.  Distro can make the packaging better and get everything under distro release ourselves.  But tests should be driven by PS.  We can tell PS that we won't copy the PPA into the archive until tests are finished, but I don't see why we block our *own* work on PS15:09
didrocks1mterry: and feel free to upload it once :)15:09
didrocks1mterry: if you stay and everyone stays on that line, that's fine by me15:10
mterrydidrocks1, or at least, if we *are* going to block our own work, can we light a fire under xnox?  :)15:10
didrocks1mterry: but I know what will happen…15:10
didrocks1"put that in distro now"15:10
=== didrocks1 is now known as didrocks
didrocksmterry: that's why "one manual upload" is unblocking you, isn't it?15:10
mterrydidrocks, yeah it would.  But I'm leery of future changes making more manual uploads necessary15:11
mterrybut OK15:11
didrocksmterry: let's do manual and ensure we are pushing upstream with deadlines15:11
didrocksas I had to do with unity upstream15:12
didrocksand we can see the result now :)15:12
didrocksotherwise, we'll slip over and nothing good will be out of this15:12
didrocks(like for indicators)15:12
xnoxdidrocks: mterry: i think with sensors & platform-api it's not that tests are not there, it's non-trivial to have tests as they are tied in against android hw layer.15:12
mterrydidrocks, alright, I pushed to config trunk a change to make qtubuntu-sensors in transition15:12
xnoxand solving how those will be run on daily basis is the key point.15:12
didrocksxnox: if we are telling "we can't run them by now", I'm fine with it as long as it's logged and known15:12
didrocksmterry: ok, let me pull that, thanks! you are deploying, isn't it?15:13
xnoxcause there is no way to run them at package build-time nor on distro-builders.15:13
mterrydidrocks, guh!  no, let me deploy.  I forgot that was a step15:13
mterrydidrocks, now deployed15:14
mterrydidrocks, but there's nothing to do anymore, since that was my one change15:14
mterrydidrocks, (enabling it then disabling it)15:14
didrocksmterry: you didn't deploy the media stuff?15:14
mterrydidrocks, it's not landed in trunk yet15:14
mterrydidrocks, just MP15:14
didrocksmterry: approved15:15
didrocksmterry: so you can deploy :p15:15
mterrydidrocks, I've been trying to do things in serial, but I guess I should just do a lot of one-off uploads and then do a mass migration in the config15:15
didrocksmterry: that's fine as well15:15
mterryheh, whoops15:17
mterrydidrocks, OK.  Media stack added and deployed15:25
didrocksmterry: I will pull the branch ASAP :)15:26
nic-doffayAre there any generally accepted trends when commenting QML files?15:26
mterrynic-doffay, you mean like gtk-doc syntax or not or just whether to use // or /*?15:27
didrocksmterry: thanks!15:27
nic-doffaygtk-doc syntax mainly mterry15:27
mterrynic-doffay, I've mostly seen // but I haven't seen anything commented as rigorously as gtk-doc style15:27
nic-doffaySo simple // wherever is generally fine?15:28
mterrynic-doffay, but I've just poked around unity-next, not an expert15:28
mterrynic-doffay, that's what I've seen15:28
=== dandrader is now known as dandrader|afk
mzanettinic-doffay: yeah... explaining internal stuff is fine with // or /* */15:29
mzanettinic-doffay: I don't think you need an API doc for what you do right now15:30
nic-doffayCool thanks mzanetti15:32
nic-doffayAnother quick question15:32
nic-doffayWhy is an assignment made in this: https://pastebin.canonical.com/88773/15:32
nic-doffay(It's off the QML tuts)15:33
nic-doffayI read elsewhere yesterday that assignments weren't done in QML.15:33
nic-doffayOn the same docs.15:33
mzanettinic-doffay: because onClicked: is executed as in imperative code15:33
mzanettinic-doffay: the cellColor for example is read at startup and cellColor: "red" will remain throughout the whole lifetime15:34
mzanettinic-doffay: the helloText.color = cellColor is only exected once when you click on it15:34
nic-doffayThanks mzanetti I found a page detailing it.15:34
nic-doffayIf anyone else is interested: http://qt-project.org/doc/qt-5.0/qtqml/qtqml-syntax-objectattributes.html#imperative-value-assignment15:35
didrocksmterry: if you see more components that you need, do not hesitate to mark them as WI (you did the work after all!) and to request help from sil2100 if needed15:43
=== dandrader|afk is now known as dandrader
Cimidandrader, https://code.launchpad.net/~unity-team/unity/phablet.tests_coding-style/+merge/15788815:52
Cimimzanetti, so, what to test of /Panel/IndicatorItem.qml ?15:59
Cimitsdgeos, can you reapprove this? https://code.launchpad.net/~unity-team/unity/phablet.test_responsiveflowview/+merge/15790116:01
sil2100mterry: indeed - push anything that you I could work on to me ;)16:02
mzanettiCimi: set an icon/text and check if the width adjusts accordingly16:03
tsdgeosCimi: sure, i dreamt i did it already16:07
tsdgeoswhat did i approve if not that?16:07
* tsdgeos checks16:07
Cimitsdgeos, you did, but I merged trunk16:07
Cimitsdgeos, and superseeded16:07
tsdgeosi mean 5 minutes ago16:07
Cimiah ok16:07
tsdgeosyou did this 15 min ago16:08
tsdgeosmaybe my 5 minutes was 16 minutes ago :D16:08
Cimitsdgeos, I'm glad I'm not a girl and I'm not having a date with you :)16:08
tsdgeosCimi: i'd be more careful :-P16:09
tedgpete-woods, Thinking, we'll need to choose what data set to show by default in the greeter....16:15
tedgpete-woods, Thinking the default setting should be the data set with the highest standard deviation.16:15
tedgpete-woods, That one is likely to have the most interesting graphic.16:16
tedgAnyway, random thought.16:16
pete-woodstedg: it'll have to be something statistical, yes!16:21
pete-woodstedg: I believe there is the idea that some sets are more important, like I don't know 3G data usage, or high battery usage?16:21
mterrydidrocks, if I'm uploading to PPA manually, and the UNRELEASED version in trunk is 0.3, will I screw daily-release up if I upload with 0.3~manual1?16:21
tedgpete-woods, Yeah, but I imagine you'd want to choose the one that's the most interesting for that user.  It's about personalization.16:22
tedgpete-woods, They user would of course be able to explicitly set.  But, by default we're going to have to choose one.16:22
didrocksmterry: no, it will try to upload the first time 0.3daily<…>16:23
didrocksmterry: so good :)16:23
pete-woodstedg: yep, I don't know for sure if SD is necessarily the right choice, but there does need to be some indicator of activity16:24
tedgHmm, got an e-mail from LinkedIn that pete-woods visited my profile.  LinkedIn is creepy.16:24
mterrydidrocks, thanks16:24
tedgpete-woods, Sure, I was thinking interesting graphic there.16:24
pete-woodstedg: yes, I've started getting connection requests from Canonical folks now16:24
pete-woodsa whole new cluster of linked in folks16:25
didrocksmterry: yw! it's only if things hit the "dest" ppa (like if it hitted "distro") that he wants it16:25
didrocks(on the changelog)16:25
tedgpete-woods, Ha!  That is a cool visualization.16:26
Cimimzanetti, mmm16:27
Cimimzanetti, IndicatorItem16:27
Cimimzanetti, has label and iconSource16:28
Cimibut where are thos properties defined?16:28
mzanettiCimi: thats the API to be tested16:28
Cimimzanetti, there are not defined16:28
mzanettiCimi: ?16:28
Cimimzanetti, iconSource is what?16:28
mzanettiCimi: you set that from the outside16:29
Cimimzanetti, but item doesn't have this property16:29
dandraderCimi, they probaly come from the context, injected from the model, if IndicatorItem is being used as a delegate16:29
Cimiaaaahn ok16:29
dandraderyes, it's cryptic16:30
Cimidandrader, so how do I use them in tests?16:31
Cimidandrader, I simply add IndicatorItem { iconSource = "...." }16:31
Cimiassuming is used?16:31
dandraderCimi, I don't know, I never faced this situation myself16:32
Cimiif someone knows, here we are :)16:32
dandraderI think dednick had this problem before16:32
dednickCimi: they are defined in the PluginModel. Items of a Repeater can access the model roles in the same way as properties16:33
dednickItems of a model i should say.16:34
Cimimmm shall I use a model then?16:34
dednickCimi: there is a fake one defined in Panel/qml/Ubuntu/ChewieUI/PluginModel.qml. It's used to mimic behaviour of the indicator model in chewieui.16:35
dednickwhich is what you want?16:35
mzanettiCimi: hmmm... now I understand... sorry16:36
dednickwhat are you testing? IndicatorItem?16:36
Cimidednick, yes16:36
dednickthen yes, you can use that16:36
mzanettiCimi: dednick: shouldnt the item be fixed to offer those two properties as an API?16:37
dednickmzanetti: yeah. i think i was told not to when i wrote the tests ;). although i didnt really do it in a nice way.16:37
mzanettiyeah... its a thin path16:38
mzanettiimho this one would justify a fix16:38
dednickalthough i'm not really sure if you can just alias the model roles as a property. something you can try Cimi.16:38
mzanettiyeah, alias would work16:38
mzanettijust add property alias iconSource: image.source16:39
mzanettiand then, where the IndicatorItem is used, set those properties from the outside16:39
mzanettias that one is the file where the model actually lives (hopefully :D)16:39
mzanettiCimi: is that ok for you?16:39
dednickmzanetti: not sure if that will work without an attached model? the icon.source is bound to the model's iconSource role. wont that just either cause a failure, or some crazy recursion?16:41
dednicksource: iconSource16:41
mzanettidednick: no... local context has priority16:41
mzanettidednick: if there are conflicts, you can access model stuff with model.iconSource16:42
Cimiother question, I never have to add the @18.png just .png is fine right?16:42
mzanettidednick: but if you just call iconSource and there is a local property with that name, the local one is used16:42
dednickbut if the local one is an alias?16:43
mzanettiCimi: yep. never add that. its the job if the SDK internals to do that16:43
mzanettidednick: doesn't matter... its a property. no matter of which type16:43
Cimimzanetti, but I don't have Ubuntu components imported16:43
Cimimzanetti, so first I have to import them I guess, right?16:43
mzanettiCimi: yeah. import them16:43
mzanettidednick: Cimi: https://pastebin.canonical.com/88782/16:45
mzanettiand where you use it, just set "text: label" and "iconSource: model.iconSource"16:46
Cimimzanetti, public pastebin pls16:47
Cimi(don't have my phone here :-)))16:47
mzanettiCimi: oh... ok16:47
Cimimzanetti, anyway if it's for the alias, I've already used it16:47
Cimimzanetti, or you can do bzr diff Panel/IndicatorItem.qml | pastebinit16:47
dednickCimi: http://pastebin.ubuntu.com/5692847/16:48
Cimiyep was exactly what I did16:48
mzanettiCimi: http://paste.ubuntu.com/5692848/16:48
Cimithe trick above is pro though :)16:48
Cimibzr diff filename | pastebinit16:48
Cimibe sure to apt-get install pastebinit16:49
mzanettiCimi: I have an icon in my panel ans when I drag&drop stuff on it it pastes it16:49
Cimithat's even more pro16:49
* bschaefer installs pastebinit16:50
mterrycyphermox, you around today?16:54
didrocksfginther: do you mind having a look at https://code.launchpad.net/~robru/cupstream2distro-config/webbrowser-app/+merge/157917 for the CI part?16:55
didrocksrobru: ^16:55
cyphermoxmterry: yeah16:56
cyphermoxI'm always around16:56
mterrycyphermox, :)  I see that hud failed to build in the PPA16:56
cyphermoxwhether i'm temporarily not answering due to debugging is another matter though16:56
cyphermoxdon't worry about it, I'm looking at this with sil right now16:56
mterrycyphermox, cool!16:56
cyphermoxmterry: I'll be doing another upload and hopefully that one will be good, then we'll know it builds fine on all arches and we can start the daily release16:57
mterrycyphermox, awesome, thanks man!16:57
cyphermoxmterry: hud is waiting on a ci job right now, which may or may not be failing due to tests16:59
* cyphermox runs off to grab food16:59
mzanettiCimi: I'd have a test now that does not require any window... should I put it to unittests or qmluitests then?17:09
Cimimzanetti, I believe we won't move to a new structure until we all agree and we don't have functionality regressions17:09
mzanettiCimi: ok... I wrote my opinion on it to the ML. I'm fine with either way.17:10
Cimimzanetti, https://code.launchpad.net/~unity-team/unity/phablet.test_IndicatorItem/+merge/15791917:17
CimiI'm done for today ;)17:17
Cimilike I don't think I can start a new test and finish in good time, better go to the gym :D17:18
mzanettiCimi: https://code.launchpad.net/~mzanetti/unity/phablet-test-sidestage/+merge/15792117:19
mzanettiI'm done too17:19
Cimimzanetti, I can review that tomo morning17:19
mzanettiCimi: sure. no problem17:19
mzanettiCimi: have fun at the gym17:19
Cimimzanetti, I bet no fun and lots of sweat17:20
* Cimi is doing heavy HIIT17:20
* mzanetti tries to have fun when doing sports17:20
mzanettibut thats why I don't go to the gym but rather running or biking17:20
Cimimzanetti, running is fun, running with HIIT is not fun17:20
Cimiunless you're masochist :)17:21
Cimihigh intensity interval training17:21
mzanettiCimi: FYI, your test doesn't need to be a qmluitest17:21
mzanettiCimi: it would make a nice unittest :D17:21
CimiI basically run 1 min at 15-16Km/h, 1 min 11km/h... that for 40 min17:21
cyphermoxNot that beep test thing?17:21
Cimiit's deadly hard17:22
Cimimzanetti, why not? it required a window no?17:22
cyphermoxThought it would be something like running/hiking with weights17:22
mzanettiCimi: no not really17:22
mzanettiCimi: you don't do any mouse or keyboard interaction17:22
mzanettiCimi: just calculating some integers can be done in memory only17:23
Cimimzanetti, I thought it was depending on showing things on screen17:23
mzanettiCimi: yeah... you would need to remove the "when: windowShown" then it would execute the same test without a window17:23
Cimimzanetti, so if I move the file to unittests and move TestCase to parent, will work?17:23
mzanettiCimi: no need to move the TestCase to parent either17:23
mzanettiCimi: just remove "when: windowSowhn"17:24
Cimimzanetti, ok I'll move it under unittest17:24
Cimiso it's ready for your review tomo17:24
mzanettiCimi: ok then. I'm off too. see you all tomorrow!17:24
Cimi(I'm finding excuses to postpone the HIIT)17:25
Cimi(ok that was quick, pushed)17:27
Cimimzanetti, in reality I need the findChild17:28
CimiI'll add the proper import17:28
mzanettiCimi: wait17:28
mzanettiCimi: I have the same problem too17:28
mzanettiCimi: I just did the quick&dirty way of importing ../../qmluitests17:29
Ciminot nice?17:29
mzanettiCimi: but if this is a common thing we should probably move the UnityTestCase away to tests/imports/testhelpers.js or something like this17:29
mzanettiCimi: and then import testhelpers.js wherever we need findChild17:30
Cimibut let's do this in a separate branch17:30
mzanettiack. then lets merge our two first with the quick&dirty one17:30
mzanettiCimi: if you want you can take care of that tomorrow... otherwise I'll do it at some point tomorrow17:31
mterrydidrocks, are we not upstream for hybris?17:58
didrocksmterry: I don't think so from what I understood from ogra17:59
mterrydidrocks, oh, hm.  We have some code in there anyway17:59
didrocksmterry: right, it's part of the "patched" things17:59
mterrydidrocks, yar18:00
didrocksmterry: one sec, sergio is assigned to push it to distro18:00
mterrydidrocks, it's homepage is apparently not up now18:01
didrocks[rsalveti] Update the libhybris codebase to be upstream compatible (we want the upstream based version at raring): INPROGRESS18:01
didrocksmterry: ^18:01
=== jhodapp is now known as jhodapp|lunch
=== jhodapp|lunch is now known as jhodapp
=== racarr is now known as racarr|curry
slangasekmterry: hey, so didrocks intimated that you might be the one to look at whatever unity integration test failures are blocking my super-important compiz fix from reaching raring20:07
slangasekmterry: do you have a handle on it?  are the failures my fault?  anything I can do to help?20:08
mterryslangasek, hrm, maybe.  Or at least point you at someone else  :)20:08
mterryslangasek, can you point me at a log?20:08
slangasekmterry: only an IRC log ;P20:08
mterryslangasek, oh I see what you mean, sure.  give me a sec20:08
mterryfginther, I'm having trouble connecting to the jenkins VPN20:10
mterryslangasek, sorry, technical difficulties20:12
fginthermterry, sorry, I can't get to it either20:15
mterryslangasek, there is problem with the QA labs network right now, I can't check for you20:17
slangasekmterry: oh bother20:17
mterryslangasek, if it's super urgent, you can always just distro-patch it like old times20:18
=== racarr|curry is now known as racarr
mterryslangasek, it won't screw up daily-releasing, we'll just have a bit of paperwork to do to sync back up20:18
slangasekmterry: nah, it's not super-urgent, it's just that I spent a lot of time tracing that bug down and I want instant gratification ;)20:18
mterryslangasek, :)20:18
slangasekmterry: for my edification, if the QA lab weren't having trouble, where would I look to see the test failures?20:18
mterryslangasek, there's an internal-VPN that has a jenkins web UI to look at20:19
mterryslangasek, if you expect you have access already, I can give you a URL20:19
slangasekmterry: I do have access, but don't have the url20:20
=== salem_ is now known as _salem
=== luv_ is now known as luv
fginthercyphermox_, ping21:45
fgintherbregma, compiz/0.9.9 ?21:54
cyphermox_fginther: pong23:35

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