=== _salem is now known as salem_ === salem_ is now known as _salem === maclin_ is now known as maclin [05:07] Good morning [08:05] jibel: I'm also pondering to drop the "quote-shstring" testbed command again; that's not dependent on the testbed implementation, is it? that could just be a simple helper function in adt-run itself, no? === zequence_ is now known as zequence [08:30] pitti, with execute_raw instead of direct calls to subprocess in adt-run quote-shstring can go away [08:32] jibel: I'm currently working on cleaning up all this; this gets a whole lot simpler, but it's not just a 5 min small cleanup :) [08:33] pitti, after this clean up you can bump the version to 3.0 ;) [08:43] 35 files changed, 161 insertions(+), 2228 deletions(-) [08:43] jibel: ^ simplification since last Friday :) [09:38] jibel: I pushed the "remove shstring" bits [09:50] jibel: I pushed some more cleanups; I'm now happy enough with the "run command in testbed" parts [09:50] jibel: please let me know if anything is unclear now, or too complicated for the ssh runner === vrruiz_ is now known as rvr === chihchun_afk is now known as chihchun [10:12] pitti, thanks. I'll try it. === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === _salem is now known as salem_ [13:41] hi is anyone clued up about gnome-search-tool ? [13:41] I can install it with apt-get, but I can't find it in software centre === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk [15:04] brendand: I need a review here, please: [15:04] https://code.launchpad.net/~elopio/ubuntu-filemanager-app/clean_places_tests/+merge/222583 [15:07] elopio, looking at it === chihchun_afk is now known as chihchun [15:24] brendand: there are two daily landing meetings [15:24] one on US time, and one on EUR time. [15:24] so maybe you can attend one and I attend the other. [15:28] elopio, oh sure [15:29] brendand: it might be too much to attend the daily meetings. It might be better to spend that time making tests prettier, so lets see how it goes. === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun [15:56] elopio, brendand I've attended them on and off for a bit. Usually to give feedback. It's productive when you have things going on in the images, but if your tests are stable, there shouldn't be a reason for you to attend eh? :-) === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun === roadmr is now known as roadmr_afk [17:52] balloons: the needs fixing comment on the filemanager branch comes from running it on the phone? [17:52] elopio, I ran it on the latest image on flo.. I'm giving some time to debugging the whitescreen issue a bit more today.. going to fix all the null errors, etc [17:54] elopio, I ran it a couple times.. the second time I got the whitescreen issue again :-) [17:54] time to run the debug branch [17:58] elopio, I hope I ran everything correctly.. it is odd ; I would expect to see that type of error on the desktop too [18:00] balloons: are you running them with phablet-test-run? [18:01] elopio, yes [18:05] balloons: there was one bug, that I fixed. But it's not the one you are hitting. I think your problem is the same that we had with address book [18:05] phablet-test-run going crazy and selecting an old version. [18:06] elopio, yes, totally possible [18:08] elopio, hmm.. interesting I'm not see my new debug messages on this run either [18:09] however I do see the tests loaded the new qml, so that is consistent [18:09] same with your runs.. === chihchun is now known as chihchun_afk [18:24] elopio, quick retest ;-) [18:24] I think you should be good === chihchun_afk is now known as chihchun [18:37] balloons: sorry, you lost me. [18:37] retest of what? good with what? :) [18:37] elopio, lol.. sorry. I'm re-running your branch on my device and I think it should be fine now. I'll approve once it's finished [18:38] I mean, nothing new is being introduce :-) [18:38] balloons: oh, thanks. [18:39] just need to understand why qdebug isn't printing some lines [18:40] elopio, accepted [18:42] balloons: if I continue splitting the filemanager, will it cause conflicts with what you are doing? [18:43] elopio, I'm playing with the qml atm, so no, no worries [18:43] actually heh, the compiled modules but :-) === roadmr_afk is now known as roadmr [18:51] balloons: ok. On image #76 the ones that failed are related to context menu, so I'm going to split those out. [18:51] however, what seems to be happening is that it breaks randomly. [18:52] elopio, yes missing buttons, etc.. But be careful here. What I noticed playing with it originally (the missing popups) was that the app crashed. I suspect that may be happening with the context menu too.. but perhaps not [18:52] the weird thing is you can still introspect after it whitescreens.. you just don't get any values as the objects are gone [18:52] balloons: I don't see any crash files on the test artifacts. [18:52] elopio, right.. it doesn't generate any [18:53] the app just whites out, but appears to still be resident and running [18:53] as I said, you can still introspect it, but of course you won't get what you want [18:53] elopio, I've done the same just running the app manually.. the whitescreening is not a test issue [18:53] balloons: right. It seems a deeper issue. [18:53] I also see many of these: [18:53] void DirModel::setPath(const QString&) DirModel(0xca4fc8) path or url may not exist or cannot be read: "~" [18:54] do you know what that means? === roadmr is now known as roadmr_afk [18:54] elopio, yea, I'm playing with that atm.. Victor has a patch for the other null pathbar issue [18:54] https://code.launchpad.net/~vthompson/ubuntu-filemanager-app/fix-null-error-1316677/+merge/222698 [18:54] I pushed it since he never did [18:55] great. === chihchun is now known as chihchun_afk === chihchun_afk is now known as chihchun === roadmr_afk is now known as roadmr === DalekSec_ is now known as DalekSec === salem_ is now known as _salem