/srv/irclogs.ubuntu.com/2014/06/18/#ubuntu-app-devel.txt

=== chriadam|away is now known as chriadam
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== chihchun_afk is now known as chihchun
mihirGood Morning05:28
=== chriadam is now known as chriadam|away
dholbachgood morning07:14
justCarakasgood mornign08:01
justCarakasmaybe someone can use this for an World cup app08:19
justCarakashttp://worldcup.sfg.io/08:19
JamesTaitGood morning all; happy International Panic Day! :-D08:26
justCarakasAAAAAAAAAAAAGGGGHHHHHHHHH RRRUUUUUUUUUUNNNNN08:29
dpmkalikiana, does U1Db has a public C++ API?08:38
* DanChapman would like to know that ^^ one aswell :-D 08:40
DanChapmanGood Morning all08:42
justCarakasis U1Db accessable for HTML5 apps ?08:51
brendandpopey, https://code.launchpad.net/~brendan-donegan/ubuntu-calendar-app/remove_osk_hide/+merge/22352309:06
popeybrendand: will take a look now.09:08
brendandpopey, it's not guaranteed to fix the failures, because i could never reproduce them locally anyway09:11
popeyI'm running Ap on a device with that branch now09:12
popeybrendand: all passed09:23
brendandpopey, good09:29
popeybrendand: does the music app do the same?09:31
popeyoh, no09:31
brendandpopey, clock does09:32
brendandpopey, clock doesn't seem to have many issues though. except for todays hang09:32
t1mpoSoMoN: hello09:59
t1mpoSoMoN: do you do reviews for notes-app? I have one https://code.launchpad.net/~tpeeters/notes-app/remove_get_header/+merge/22321810:00
t1mpoSoMoN: it is needed for a change I will do in UITK10:00
oSoMoNt1mp, I can review it10:00
oSoMoNI’ll take a look in a minute10:01
t1mpoSoMoN: thanks10:01
mihirpopey: ping10:17
mihirpopey: it fails on desktop10:17
popeymihir: the one I just commented on?10:29
popeytrusty desktop failure?10:29
popey(sorry, commented on a bunch of merges, gimmie a link for context) :D10:29
mihirpopey: on AP fails10:33
mihirhttps://code.launchpad.net/~gang65/ubuntu-calendar-app/ubuntu-calendar-autopilot-edit-event/+merge/22112710:33
popeyyeah, is it missing something that's only in the sdk on uptopic we wondered10:33
mihirpopey: on repetition option , does it function well on device ?10:33
popeylet me test here.. gimmie 5 mins10:33
mihirpopey: sure :)10:34
popeythanks mihir10:34
popeymihir: with that branch I'm getting the odd thing where it doesn't fit on screen - http://popey.mooo.com/screenshots/device-2014-06-18-113549.png10:36
mihirpopey: sorry , i lost which branch give you this output ?10:37
popey11:33:09 < mihir> https://code.launchpad.net/~gang65/ubuntu-calendar-app/ubuntu-calendar-autopilot-edit-event/+merge/22112710:37
popeythat one10:37
mihirpopey: hmm that's weird10:37
popeymihir: left a comment on the merge10:38
mihirpopey: the thing is , it is just updating AP no UI changes i can see in change log10:38
popeyhm.10:44
popeylemme remove the app and re-install, i have all kinds of rubbish on this device10:45
popeysame10:47
popeymihir: does it need remergeing from trunk?10:48
popeydidnt we fix this before.10:48
mihirpopey: may be10:50
mihirit required latest merge10:50
=== MacSlow is now known as MacSlow|lunch
=== MacSlow|lunch is now known as MacSlow
=== _salem is now known as salem_
brendandpopey, are you inclined to land the calendar-app change?12:37
=== chihchun is now known as chihchun_afk
karniI'm having problem click packaging a scope. Can anyone advise? http://paste.ubuntu.com/7663564/ scope name has been replaced with "Foo"12:52
dpmkarni, I've not yet packaged any scope with click, but perhaps mhr3 on #ubuntu-unity can help? Or perhaps some of the security or click guys (jdstrand or cjwatson)12:54
karnidpm: yes, I pinged both, will also poke Michal, thanks12:55
dpmok12:55
dpmrpadovani, mzanetti, sorry, I'll have to skip the Reminders meeting today, I'll reschedule for Friday13:01
mzanettiack13:01
dholbachbzoltan, ever seen this before? http://paste.ubuntu.com/7663604/13:03
dholbachzbenjamin, ^ too13:03
rpadovanidpm, k13:03
dholbachthat's a crash on startup on current utopic13:03
zbenjamindholbach: no, but looks like the sensors plugin does something bad13:05
zbenjamindholbach: is this when starting QTCreator?13:07
dholbachzbenjamin, yes13:07
zbenjamindholbach: what qtcreator-plugin-ubuntu package version do you have installed?13:08
dholbachzbenjamin, http://paste.ubuntu.com/7663636/13:09
* zbenjamin wonders why QtC would initialize a accelerometer13:09
zsombi_nik90: have you had time to test the pullToRefresh stuff?13:10
zbenjamindholbach: looks like you have the most recent version installed13:10
t1mpnik90: can you check that this branch has everything you need for the pull-to-add-alarm in clock-app? https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/pullToRefresh/+merge/21909013:11
zbenjamindholbach: and it just starts and immediately dies?13:11
nik90zsombi_: sry, no13:11
nik90zsombi_: Can I test it this evening?13:11
dholbachzbenjamin, http://paste.ubuntu.com/7663650/13:12
t1mpnik90: I'm doing a final review now. If you tell me this evening that it is good for you I can happrove it13:13
zbenjamindholbach: that looks suspicious "Ubuntu Platform API: Unable to find module configuration file -- Aborting"13:13
zbenjaminzsombi_: ^^13:13
zsombi_nik90: t1mp is reviewing it again, so if you have some issues, make a note in the MR13:13
zbenjaminzsombi_: can you make any sense from that error? Ubuntu Platform API: Unable to find module configuration file -- Aborting13:14
zsombi_zbenjamin: no idea...13:14
zsombi_zbenjamin: or at least I don't remember what is teh issue there...13:14
dholbachzbenjamin, the warning is from ./src/ubuntu/application/base_module.h in platform-api13:15
dholbachlooking for /etc/ubuntu-platform-api/application.conf13:15
dholbachwhich is not shipped in any package13:15
dholbachwhy would we do that?13:15
zbenjaminhmm yeah i remember i saw that before but i could not reproduce it. Maybe its some special plugin that only loads when its installed?13:16
dholbachzbenjamin, looks like a question for ricmm13:16
dholbachI'm going to go over to the office now - should be back in 10m13:17
zbenjamindholbach: ok13:17
=== dholbach_ is now known as dholbach
balloonsvthompson, ahh you are in here now.. I looked earlier15:01
=== chihchun_afk is now known as chihchun
=== wendar_ is now known as wendar
=== gatox is now known as gatox_lunch
ahayzenvthompson, o/17:01
vthompson'ello!17:01
ahayzenvthompson, hows things going?17:02
vthompsonvthompson, crummy!17:02
vthompsonha, man--I also need sleep17:02
ahayzenvthompson, has balloons got any hair left? or does he need some of mine17:02
vthompsonahayzen, still trying to battle the tests17:02
ahayzenoh god :(17:02
balloonsahayzen, howdy17:03
ahayzenballoons, o/17:03
ahayzenballoons, vthompson, so what is the situation now? (i've been at work all day)17:03
balloonsI think as vthompsons has discovered, nothing is wrong with the test setup really.. instead, we don't know how to set it up to run17:03
balloonsmeaning, the implementation is fine, it's just wrong17:03
vthompsonahayzen, we have the backing up of the dirs working now. Even adding a start and stop of ms2 and a sleep doesn't cause the app to see the new db17:04
ahayzenvthompson, yeah i tried start/stopping msw17:04
ahayzen*ms2 before17:04
vthompsonahayzen, if you look in the ms2 logs it's picking up the files, it's just that the plugin isn't reporting the new db17:05
ahayzeni started looking into it and it uses some Qt thing for $HOME? i couldn't really figure it out17:05
* ahayzen can't remember if the dbus part of the service was a separate process that didn't get stopped or something?17:06
ahayzeni'll fire up my VM so i can battle along with u guys :)17:06
vthompsonwhat I was suggesting to balloons over in #ubuntu-touch was that we perhaps ship the image with the test songs in ~/Music and then also create our own script that we run before and after we execute the tests manually. We'd also need to fix the desktop tests--which I haven't done yet17:06
ahayzenvthompson, so why doesn't moving ~/Music to ~/Music.bak and then filling ~/Music with the test data work?17:07
ahayzenvthompson, or does it bit thats the 2nd option in Alan's mail?17:07
vthompsonahayzen, we're trying to do #2 and it still isn't reflecting what's in the new db17:08
ahayzenvthompson, ok17:08
vthompsonahayzen, balloons, I'll be back in a bit--getting some lunch17:10
ahayzenvthompson, cool :)17:10
balloonspopey, so you are in the loop, see above. The backup and restore still doesn't work :-)17:11
ahayzenballoons, popey was 'on holiday' for this afternoon no?17:12
balloonsahayzen, yes.. he'll see the log later17:12
ahayzenballoons, ah right :)17:12
ahayzenballoons, so we are trying to do option 2 right?17:12
nik90t1mp: ping17:13
balloonsahayzen, yes17:13
ahayzenballoons, ok i'm just updating my VM then i'm ready to rock until dinner :)17:13
=== gatox_lunch is now known as gatox
ahayzenballoons, that was odd it actually nearly worked the first time on utopic desktop17:21
ahayzenballoons, but is was weird because mediascanner2 crashed...apport came up... then music-app with the correct test library!... then the test failed because of the apport dialog... but when i run again it displays no music so i wonder what was different/caused mediascanner2 to crash?17:22
ahayzenballoons, it was as if the delay that apport created in starting the music-app was long enough for ms2 to rescan or something?!17:23
balloonsahayzen, so if mediascanner2 isn't running it worked eh?17:29
balloonsvictor reportedly tried stopping it before the test17:29
ahayzenballoons, well i haven't been able todo it again but it basically crashed while ap was starting17:30
ahayzenballoons, and then i guess as it was restarting it saw the test music files17:30
ahayzenhmmm17:30
ahayzenballoons, you have tried restart ms2 service after copying the files across right?17:33
vthompsonahayzen, that's what I'm doing when testing on the device, also waiting 10 seconds afterward17:34
ahayzenvthompson, does it work?17:34
vthompsonahayzen, nope17:34
ahayzenhmmm17:34
ahayzentht is where i got up to last time IIRC17:35
vthompsonahayzen, I take it you are just testing in your VM now17:35
ahayzenyep17:35
ahayzenvthompson, but it worked the first run! then won't do it again17:35
nik90ahayzen: just to confirm one last time, you wouldn't be able to make it to the meeting tomorrow afternoon?17:35
ahayzenvthompson, like ms2 crashed while ap was starting up and then it showed the correct library17:35
vthompsonahayzen, odd17:36
ahayzennik90, yeah sorry as we have music/coreapps back2back and i only get 1hr for lunch17:36
nik90ahayzen: no worries. I am going to cancel this week's meeting then17:36
nik90ahayzen: I will send a mail out later tonight17:36
ahayzennik90, unless vthompson can make it :)17:36
nik90ahayzen: well even then we will only have 3 people in it which is less to start of with.17:37
vthompsonnik90, ahayzen, I *could* try to make it, but I should be out of the house by then17:37
ahayzennik90, vthompson maybe we should try a different day/time that works a bit better?17:37
ahayzendamn work getting in the way of fun :P17:37
nik90ahayzen: tbh I have an exam next week tuesday, so I am going to need time to prepare for that. This is why I don't want to take work items this week anyway17:38
ahayzennik90, ah good luck :)17:38
nik90ahayzen: also I really want to get bzoltan again in the meeting17:38
nik90ahayzen: thnx17:38
ahayzennik90, yeah it was really useful having someone from SDK in the meeting17:41
ahayzenvthompson, ok so why does stopping and starting ms2 and waiting  for a long period of time  after doing create_music_library() not work?17:43
vthompsonahayzen, if I knew I'd have fixed it by now17:44
vthompson:)17:44
ahayzenvthompson, hah17:44
vthompsonahayzen, I was suggesting earlier that it must be some sort of dbus/contention thing17:44
ahayzenvthompson, ok so wht about doing the process manually does that work?17:45
ahayzenvthompson, as in manually go and move your music folder, patch and restart ms2?17:45
* ahayzen tries17:45
vthompsonahayzen, like how I was suggesting? copying the files first then doing a restart of ms2? Yes that works--I haven't ran the whole test suite though17:45
ahayzenvthompson, yeah basically hmmm ok so what is different then :P17:46
* balloons is following along17:46
ahayzenvthompson, how are you restarting ms2 just doing os.system("stop mediascanner-2.0") right?17:46
vthompsonahayzen, some sort of issue with the plugin not seeing the updated db when the tests are running... I blame dbus17:46
vthompsonahayzen, yep17:46
ahayzenvthompson, but wht is different in/out side of autopilot...17:47
vthompsonahayzen, both are using dbus and, again, I blame dbus17:48
ahayzenhah17:48
vthompsonahayzen, we probably just need someone who knows about this sort of stuff to help us out. I have no idea, thus I blame dbus17:48
ahayzenvthompson, any ideas on the 'someone who knows' :)17:49
ahayzenvthompson, why do we need to run patch_mediascanner_home() if we are just copying directly into ~/Music ?17:50
vthompsonmaybe Thomas Voss?17:50
balloonsvthompson, could you push to the branch?17:50
vthompsonahayzen, we don't my local isn't doing it anymore17:50
vthompsonballoons, will do17:50
* ahayzen is just looking at whtever was pushed17:51
vthompsonballoons, ahayzen, pushed.17:52
ahayzenvthompson, thanks17:52
ahayzenvthompson, yeah tht is basically wht i'm doing to restart it17:53
vthompsonahayzen, you can also "tail -f /home/phablet/.cache/upstart/mediascanner-2.0.log" to see that the files are being added17:53
vthompsonso it looks like all should be right in the world17:53
t1mpnik90: hello17:54
nik90t1mp: hey, I was testing zsombi's branch but got stuck at a few things17:54
t1mpnik90: zsombi has many branches ;)17:55
nik90t1mp: sry, I was referring to the pulltorefresh branch that you wanted to review this evening17:55
nik90t1mp: I had trouble executing an example he listed in the MP. I will need to talk to him tomorrow morning17:55
ahayzenvthompson, nothing appears in the log when i run ap17:56
nik90t1mp: would you still be doing the reviewing today?17:56
t1mpnik90: now that you remind me, yes :)17:56
vthompsonahayzen, running from your VM? maybe the added stop/start is for click only17:56
vthompsonmaybe17:56
nik90t1mp: lol17:56
t1mpnik90: if something is wrong or unclear, please leave a comment on the mr17:57
nik90t1mp: ok. Will do in another 15 mins..having dinner atm17:57
t1mpnik90: sure17:57
t1mpthanks17:57
ahayzenvthompson, so i got this pastebin.ubuntu.com/766487917:58
ahayzenvthompson, so it looks like the files are appearing...but they don't appear in music-app ?!17:58
vthompsonahayzen, yep.17:59
ahayzenvthompson, do u get the same for u on device?17:59
vthompsonahayzen, you have now joined us in our confusion17:59
vthompsonyes17:59
vthompsonthat's why I think the plugin just doesn't know that the db is updated until AP is done18:00
ahayzenvthompson, at least it is consistently weird rather than randomly weird like before :P18:00
vthompsonbrb, I have to dial into another mtg18:00
ahayzenvthompson, cool i'll probably have dinner in a bit18:00
ahayzenvthompson, balloons, there is a process on my machine called mediascanner-dbus-2.0 that doesn't stop when i call 'stop mediascanner-2.0' ... could this be the issue?18:03
balloonsok, I have time now to dive deeper in this again.. heh, here goes18:04
ahayzenwhere did that process even come from i can't get it back now :P grrr18:05
ahayzenballoons, do u have a mediascanner-dbus-2.0 on your machine?18:05
balloonsahayzen, I have mediascanner-service-2.018:06
ahayzenballoons, i had another one it was weird...18:07
ahayzenballoons, yeah if u start the music-app notice that a dbus process starts up18:07
balloonsahayzen, yep /usr/lib/x86_64-linux-gnu/mediascanner-2.0/mediascanner-dbus-2.018:08
ahayzenballoons, could that be anything todo with it18:08
ahayzenballoons, also when the music-app closes dbus crashes at the moment.... and tht process doesn't disappear?18:09
balloonsahayzen, I just realized ms2 still thinks I have all the music that got wiped18:09
balloonshaha18:09
ahayzenhah18:09
balloonsand it's playing?18:09
balloonswhat on earth...18:10
ahayzenwtf as in playing or attempting to play?18:10
balloonsno it's playing18:10
ahayzenmaybe it has stored all ur music in a magical place18:10
ahayzenballoons, look at the music-app log it'll tell you the URL of the file it thinks it is playing18:10
balloonsahayzen, ohh interesting18:11
balloonsit cataloged stuff from a phone backup18:12
ahayzenah18:12
balloonsit found music I didn't realize I had, so :-)18:13
ahayzenballoons, so running the mediascanner-dbus-2.0 manually works fine.... but if i run inside autopiliot it claims the the schemas of the db is an 'unsupported version'18:15
ahayzenballoons, ^^ maybe causing the 'no music found' issue18:15
ahayzenballoons, anyway i've go dinner now i'll be back in ~1hr18:16
balloonsahayzen, I'm gonna play with it manually18:16
=== chihchun is now known as chihchun_afk
balloonsenjoy18:16
ahayzenballoons, i think it *may* have something todo with the dbus process not starting correctly18:16
balloonsahayzen, I can follow that line of thought18:17
balloonssomehow it's not using the new db.. old process, whatever18:17
* popey waves18:27
popeyhows it going chaps18:27
popeysummarise it for me in one word18:27
vthompsonF18:27
popeyhttps://www.youtube.com/watch?v=CH1Q738UEsQ18:27
vthompsonsorry, that's a letter18:27
vthompsonpopey, I'm reading the latest between balloons and ahayzen, but really it seems that the plugin isn't getting updated when we make these changes while running the tests18:28
bluezonethis packaging process is ROUGH18:29
bluezoneHere for the distribution: http://bpaste.net/show/VwD6SRP5j9D3HvlxYq1b/ Do i put the lowest possible that i can support?18:31
popeybluezone: debian packaging?18:34
bluezonepopey, well yeah i'm following this  http://developer.ubuntu.com/publish/apps/other-forms-of-submitting-apps/packaging-commercial-apps-part-1-get-set-up-overview-of-debian-packaging/18:34
popeyah18:35
bluezoneThat's the reviewer guy told me to double check because i seem to not be following it exactly18:35
bluezoneHe told me i'm only allowed to save to home and /opt/<appname> too?18:36
balloonsvthompson, I'm on to something in regards to this18:37
balloonsvthompson, it's important the mediascanner process also thinks HOME is where it should be18:37
popeybluezone: yes, for commercial apps thats right18:37
balloonsso I'm going to tweak things to start it with the proper env.. I notice it's still writing in home18:37
vthompsonballoons, but we aren't fiddling with the location of $HOME anymore18:37
bluezonepopey, so then how do i go about adding a launcher if i can't save in /usr/share/applications18:38
balloonsvthompson, yes I know.. still18:38
popeybluezone: not sure, I'm not a packaging expert, davmor2 may be able to help if he's around18:39
bluezoneIt's not necessarily commercial but i guess i selected proprietary during the process haha18:39
bluezoneOk thanks18:39
davmor2bluezone: so Your application needs to live in /opt/appname and then anything it saves can live in /home/$user/appname preferably18:40
davmor2bluezone: what did you need to know18:40
davmor2bluezone: launcher is /usr/share/applications is fine18:41
davmor2bluezone: it's the main app that needs to live in /opt/  see the dos and donts on this end of this link http://developer.ubuntu.com/publish/apps/other-forms-of-submitting-apps/packaging-commercial-apps-part-2-packaging-software-additional-notes/18:42
bluezoneok, i can also save to /usr/share/icons ? (Although for some reason my icon never appears as the launcher icon but that's a different problem)18:42
bluezoneok18:42
davmor2bluezone: again icons in /usr/share/icons/hi-color iirc is fine18:43
davmor2bluezone: in your .desktop file you only need to name the icon not the path to the icon that might be the issue18:43
bluezoneyeah haha i'm going to try that thanks so much :)18:44
balloonsvthompson, is this trunk for ms2? https://code.launchpad.net/~mediascanner-team/mediascanner/v218:47
balloonsno updates in awhile.. surely I must be looking in the wrong place18:47
vthompsonballoons, no18:47
vthompsonlet me get it18:47
balloonsty18:47
vthompsonhttps://code.launchpad.net/~mediascanner-team/mediascanner2/trunk18:48
balloonsso vthompson one of the issues with backup / restore is it doesn't just look inside the Music folder18:48
vthompsonballoons, what do you mean?18:49
balloonsvthompson, I'd really like to have better control over the ms service..18:50
balloonsI mean if you backup the Music folder, it's going to find and index that music anyway18:50
balloonsif you have any music on the device other than the provided songs it will find it and index it18:51
vthompsonbecause it's still under home?18:51
balloonsafaict18:51
balloonsyes18:51
vthompsonyea, can we move it to /tmp?18:51
balloonshehe vthompson .. move every bit of media? I'm still messing with the patching concept18:51
balloonsI believe if I can get the scanner to start with the proper context i should be good.. as it stands, it's ignoring me18:52
balloonsI have your backup code included.. I'm happy to go either way.. I just don't see it as working18:52
balloonssadly ;-(18:52
vthompsonballoons, doesn't it seem to be the same issue as why the backup/restore isn't taking hold as well18:52
vthompson?18:52
balloonsvthompson, yes.. I believe the issue is the same no matter how you do it. namely ms is ignoring us and indexing things and using it's db as it feels like18:53
vthompsonagreed, if we can get this issue resolved, going back to mocking seems possible and is clearly a better option18:53
balloonsso again, I'm for the simplest thing possible.. I just need to find more about forcing ms to scan certain ways18:54
vthompsonI mean we shouldn't neven need to scan. we are patching the mediascanner db18:54
vthompsonI think I'll look at the ms2 code to see what mediascanner-dbus-2.0 is doing18:54
balloonsvthompson, right.. most certainly we should be able to get ms to behave for us18:55
balloonsI'm looking at the code now for insights ;-)18:55
vthompsonballoons, I almost think it's autopilot preventing mediascanner2 from being able to get things proper18:55
balloonsvthompson, well actually since ap isn't launching it, that seems a bit off..18:56
balloonsthat was my concern the other way.. that we were launching music inside AP with all our tweaks, but the service wasn't being done that way18:56
vthompsonballoons, I'm going to at least draft up an email to the ms2 guys and explain our testing issue and see if we can "command" ms2 to restart it's dbus connection, or otherwise pickup the latest db or a mocked db18:58
balloonsvthompson, james should be around in a few hours18:59
balloonsI'll pounce on him too18:59
vthompsonballoons, perfect. I'll add you to the email I send19:00
bluezoneIf my copyright file is empty, so i still have to place an empty file in /usr/share/doc/$(package_name)/copyright ?19:00
bluezoneso -> do*19:00
balloonsvthompson, mm.. tasty.. time to dive in and see if ms is taking our /home or not19:08
vthompsonballoons, good luck! Don't let it also take your hair!19:09
ahayzenOMGOMGOMGMOG19:09
ahayzeni can see my music!!!!!!!!!!!!119:09
vthompsonahayzen, you mean test music?19:09
ahayzenyeah!19:09
vthompsonhow, through what voodoo?19:09
balloonsahayzen is excitied :-)19:09
ahayzenby why is autopilot not clicking things...19:09
ahayzenbut the music !19:09
ahayzenwin19:09
vthompsonhow did you get it to pick it up?19:10
ahayzenah its clicking things now just a massive delay19:10
ahayzenvthompson, it'll cost u ;) ... no i'll push/post a diff in a bit19:10
ahayzen'Run 1 test in 92s Ok' :D19:10
ahayzennow to optimise19:10
vthompsonwhat's the tl;dr of the fix?19:10
ahayzenrun dbus manually after waiting for ms to write the db19:11
vthompsonmediascanner-dbus-2.0?19:11
ahayzenyah19:11
vthompsoncool, I had tried to kill it and restart it, but that wasn't doing anything for me19:11
vthompsonso, that means we might be able to go back to mocking19:12
ahayzenhmm maybe19:12
balloonsterminate called after throwing an instance of 'std::runtime_error'19:13
balloons  what():  Tried to open a db with an unsupported schema version.19:13
balloonsAborted (core dumped)19:13
vthompsonthis whole time I assumed the fix was 1. setup 2. shotgun to dbus 3. run test19:13
balloonsis our mocked db correct?19:13
ahayzenballoons, put a sleep between starting ms2 and running dbus19:13
vthompsonlooks like I was right ;)19:13
ahayzenballoons, i put a 10s sleep and it was fine19:13
ahayzenballoons, i assume it is because ms2 has the db open and is writing to it so if u wait for it to finish then it works19:14
* ahayzen would post a diff but xterm won't copy grrr19:14
balloonsahayzen, you aren't calling dbus first? If I call mediascanner first it seems to not work19:15
ahayzenno u start mediascanner2 first19:15
ahayzenit rescans19:15
ahayzenthen start dbus19:15
ahayzenballoons, pastebin.ubuntu.com/766519419:15
ahayzenvthompson, balloons ^^ is working for me it is a bit slow but gets there19:16
ahayzenballoons, i can commit'n'push if u think it looks good but it needs cleanups etc19:16
vthompsonahayzen, I'm going to try reverting the backup/restore and add the dbus call19:17
ahayzenvthompson, cool shall i make mine cleanup etc so it is ready as a fallback?19:17
balloonsvthompson, I confirmed ms sees the fakehome19:17
ahayzen\o/19:18
balloonscool.. ok, so it's not writing files in /home anymore19:21
ahayzenballoons, so is this it ? we're on the way to getting it working lol?19:21
balloonsok, so if you use subprocess and copy the env and launch with it, all is well with the mocking it seems..19:23
vthompsonballoons, awesome!19:23
balloonsahayzen, this line is going to be a problem.. it's arch specific19:23
balloonsos.system('/usr/lib/x86_64-linux-gnu/mediascanner-2.0/mediascanner-dbus-2.0 &')19:23
ahayzenballoons, can we not detect?19:24
balloonsahayzen, we'll do what we need to do :)19:24
vthompsonyea, also 'killall' isn't found on phone19:24
ahayzenvthompson, pkill ?19:24
vthompsonahayzen, yep, pkill is there19:24
ahayzen\o/19:24
vthompsonCalling this seems to work: /usr/lib/*/mediascanner-2.0/mediascanner-dbus-2.0 &19:26
ahayzenballoons, so we need it to do arm-linux-gnueabihf/x86_64-linux-gnu/i386-linux-gnu? is there no way of it just ... oh ^^19:26
balloonsahayzen, yes there's many ways.. vthompson's way my end up being fine19:26
balloonsit could call several dbus's :-)19:27
balloonsor blow up.. unsure which would happen19:27
balloonsperhaps blow up19:27
ahayzenvthompson, yeah tht works :) on my machines19:27
vthompsonin looking at what kill options were avail on the device I accidentally ran killall5--which was new to me19:27
vthompsonneedless to say it killsall in a very brutal fashion19:28
ahayzenvthompson, i think the only reason that process stays around is because our app crashes with dbus errors at the moment?19:28
ahayzenvthompson, because if u restart ur machine and start ms that dbus process doesn't exist?19:29
vthompsonahayzen, not sure19:29
ahayzenvthompson, but it need restarting anyway19:29
vthompsonahayzen, balloons, I can't seem to kill it on the device19:33
ahayzenvthompson, same19:33
ahayzenvthompson, hmmm how can we kill it on device then?19:36
vthompsonwell, you can kill -9 it19:36
ahayzenvthompson, does that work on the desktop as well?19:39
vthompsonahayzen, my bash-foo is not very good this afternoon. we can get the pid here: "ps -L u n | tr -s " " | grep mediascanner-dbus-2.0 | grep -v grep | cut -d " " -f3" and then do a kill -9 of the pid19:41
ahayzenvthompson, blimey....19:42
ahayzenvthompson, pidof /usr/lib/arm-linux-gnueabihf/mediascanner-2.0/mediascanner-dbus-2.0 ?19:44
vthompsonahayzen, yea, that works19:45
ahayzen:)19:45
ahayzenvthompson, so now to put the /*/ and add it to autopilot.... then win ?!19:45
ahayzenvthompson, are you adding it to the autopilot.... is balloons? or am i ...i'm lost who is trying/doing wht now lol19:47
vthompsonahayzen, I'm just seeing if I can get it to work on the device19:47
balloonsI'm still having fun chasing env -)19:48
ahayzenhah19:49
=== karni is now known as karni-holiday
balloonsok, I think last thing is the provided db isn't correct19:55
balloonsis backup / restore working on device too?19:55
ahayzenballoons, for the env u'll have to use Victor sql patching right?19:55
balloonsahayzen, for mocking yes.. dbus complains the db is invalid.. and I get no music. However, it writes the files properly, etc. So I think that's the only thing left to solve19:56
ahayzenhmmm19:56
balloonsmm.. well, whoops, one more thing to change19:56
vthompsonballoons, could you push that?19:56
balloonsvthompson, it is quite a mess, but yes I'll push it, heh19:56
ahayzenballoons, we can tidy it up :)19:57
vthompsonballoons, I don't know what I did, but suddenly my device can't find the tests module for music_app...19:57
vthompsonso I might revert to trying to test on my vm19:57
balloonsvthompson, probably should push the tests again19:58
balloonsadb push tests/autopilot /home/phablet/autopilot19:58
vthompsonballoons, doesn't click-buddy --dir . --provision do this?20:01
balloonsvthompson, yes it does20:02
vthompsonballoons, ahayzen, this is like pulling teath20:23
ahayzenvthompson, whats up?20:24
* ahayzen is patiently waiting for all the ap tests to run on his VM20:24
vthompsonahayzen, still can't get the tests to run on the device20:24
vthompsonahayzen, I'm trying to go the route of mocking still though20:25
ahayzenvthompson, what happens?20:25
balloonsvthompson, there's alot you need to mock20:25
balloonsvthompson, let me clean and push.. I'm fairly confident I've mocked properly now.. it took a bit20:25
ahayzenballoons, then we can all run on our devices/vms \o/20:26
vthompsonahayzen, same issue as before--not using the test music20:26
balloonshowever, it doesn't like the db we are supplying20:26
ahayzenvthompson, hmmm :/20:26
ahayzenah20:26
balloonsso the service blows up.. but that's progress :-)20:26
vthompsonahayzen, even better then that, then we can push to trunk, to the store and solve world hunger!20:26
ahayzenand i can propose my mp to remove grilo :)20:27
ahayzenyeah aha all our bugs will be fixed! ... not20:27
balloonsI guess I'll push straight to the branch? I don't want to overlay what you might find useful20:31
balloonsvthompson, ahayzen what do you think? ^^20:31
ahayzenballoons, yeah thts fine with me20:32
vthompsonballoons, fine w/ me. I've stashed away the changes I was making20:33
balloonsk, pushing20:33
balloonsrev 50520:33
ahayzen\o/20:33
ahayzen'slightly saner version ' :)20:34
balloonsso vthompson to further the mocking effort I use subprocess env=env and also set the upstart env HOME..it's crazy20:34
vthompsonwell, it is insane that the tests would actually run20:34
ahayzenballoons, shouldn't 'killall mediascanner-dbus-2.0' be ... 'kill -9 `pidof /usr/lib/*/mediascanner-2.0/mediascanner-dbus-2.0`' ?20:35
balloonsI didn't bother fixing the horrible os.system calls at the top20:35
ahayzenballoons, as the device doesn't have killall20:35
balloonsahayzen, yes.. it should be whatever you and vthompson determined to work20:36
ahayzenballoons, ah cool20:36
ahayzenballoons, vthompson, shall i mod those and add the cleanups?20:36
balloonsahayzen, feel free to clean at will. I'm going to look at the db now20:36
ahayzen'Ran 17 tests in 1277s ... 1 failed' .... so close20:36
vthompsonahayzen, sure go for it20:36
ahayzencool20:37
balloonsnotice the output from launching ms " terminate called after throwing an instance of 'std::runtime_error'\n  what():  Tried to open a db with an unsupported schema version"20:37
ahayzenballoons, are u waiting long enough?20:37
balloonsahayzen, I'm assuming there's a db problem.. maybe not though20:37
ahayzenballoons, where does urs start dbus?20:38
balloonsmm.. actually since dbus doesn't use usptart we could pull that bit out20:38
vthompsonthe starting of dbus needs to change so the armhf version will also work20:38
balloonshehe more tweaks20:39
balloonslet me split them again20:39
ahayzenballoons, shall i continue moding or wait lol20:39
balloonsno we won't overwrite each other20:39
balloonskeep going20:40
balloonsit'll merge fine20:40
ahayzenballoons, cool shall i resolve ur long lines....PEP8 is gonna catch us otherwise ;)20:40
balloonsI'm not worried till it's done20:40
balloonsi pushed20:41
ahayzencool i'll pull20:41
ahayzenyey text conflicts :)20:42
balloonsguess I lied20:42
ahayzenballoons, pushed my cleanups could u check over them20:46
ahayzenballoons, also don't think we needed killall mediascanner-service-2.0 as stop mediascanner-2.0 does tht20:46
=== salem_ is now known as _salem
ahayzenballoons, changes here http://bazaar.launchpad.net/~music-app-dev/music-app/use-mediascanner2.0/revision/50720:47
balloonscool20:47
vthompsonahayzen, balloons, agreed that we don't need killall ms220:47
ahayzen:)20:47
ahayzenballoons, right i'm just pulling down to my VM so ur issue is the db ?20:48
vthompsonsomething about starting it isn't working on my device though20:48
balloonsvthompson, I dumped the schema and it does seem different than what's in there now20:48
ahayzenballoons, vthompson, it is running on my VM and is showing the correct library...haven't seen the console output yet though20:49
ahayzenballoons, why do we need to '#wait a few seconds for the magic' ?20:49
ahayzenballoons, is that as the mocking stuff is running?20:50
balloonsahayzen, it probably isn't needed...20:50
balloonsit's working for you though?20:50
ahayzenballoons, at the moment it is just sitting with the app launched doing nothing...20:51
balloonsi'm working on changing the schema to match what I see now20:51
ahayzenballoons, but i can see that the app has the correct library20:51
ahayzenballoons, note this is on a utopic VM .. not device20:51
ahayzenwtf i move the window (so i can see the console) and then it says 'ran 1 tests in xxxx Ok'20:52
balloonsahayzen, yea, I'm running on utopic too20:54
ahayzenuhoh something has corrupted my ~/Music so it contains the test music :P FFS20:54
ahayzenfew at least the old patching is working i thought i may have mistaken it for working when it wasn't....20:56
ahayzen*phew20:57
ahayzenvthompson, what state is yours in?20:58
vthompsonahayzen, not working the device tests stop after starting ms221:00
ahayzenvthompson, ok have u got a VM to test in? ... i'm just about to try my device...21:00
vthompsonahayzen, I do. I'll run a test there as well21:02
vthompsonahayzen, on my vm it didn't like the kill command and my library is still shown21:03
ahayzenthe kill cmd will appear in the console if the dbus process wasn't running21:04
ahayzenvthompson, and yeah my library is broken in that as well :/21:04
ahayzenvthompson, i thought it was working for a second but a previous run had copied the test music to ~/Music lol21:04
vthompsonahayzen, I'll put my update to the kill in and push21:05
ahayzenvthompson, is there a way to get it to do nothing if the proc isn't found?21:05
balloonsk, time to try with the new db21:05
vthompsonahayzen, yea, my branch kinda does nothing in that case21:06
ahayzenvthompson, ok21:06
ahayzenvthompson, same21:06
vthompsonahayzen, pushed. the test still hangs for me on the device21:07
ahayzenvthompson, cool looks good21:08
vthompsonahayzen, oh I need to fix it a bit21:09
vthompsonahayzen, why are we doing a kill -9 on the mediascanner service?21:10
ahayzenvthompson, mediascanner-dbus-2.021:11
ahayzenvthompson, the dbus service21:11
ahayzenvthompson, if u run the music-app and then close it this process is still running... and it needs to be stopped or bad things happen21:11
ahayzenvthompson, i think it doesn't stop because of those dbus errors on exit of the app21:11
ahayzenvthompson, oh no u mean my cleanup21:12
ahayzenvthompson, yeah thts a typo that should be -dbus- not -service-21:12
* ahayzen fixes21:13
vthompsonahayzen, there are a few things wrong here. We also need to use balloons's env stuff in the subprocess calls21:14
* ahayzen pushes...21:14
ahayzenvthompson, which subprocess calls?21:14
vthompsonI'll try to fix em21:16
ahayzenvthompson, i just fixed the -service- which should have been -dbus-21:16
balloonshaving a little trouble making the db21:19
ahayzenballoons, whats happening?21:23
balloonsahayzen, vthompson sorry.. t-storm here lost power21:25
ahayzenuhoh21:25
vthompsonballoons, mother nature does not like the idea that we are getting closer21:27
ahayzenhah21:27
balloonslol21:28
vthompsonon my VM I get the wrong music, but the test I'm using says it passes after 74 seconds21:32
ahayzenvthompson, yeah same lol21:33
ahayzenvthompson, balloons, so what is going wrong is the patching of the DB?21:35
balloonsahayzen, well I'm not able to get it to build a db that passes through patching21:41
ahayzen:/21:41
balloonsvthompson, you don't build the whole db on the fly, but just massage a table or two21:42
vthompsonballoons, what you might need to do is redump the db to an sql file of commands and use the same table I have in the sql file21:42
balloonsthat's what I've tried21:43
vthompsonwhat's changed?21:43
balloonsI guess I should say.. dbus doesn't like it21:43
* ahayzen hides21:43
balloonsvthompson, turns out I think it was the same.. your media table tho is smaller. and the schema is 6 now not 521:43
balloonsbut your bigger dump seems to have all the columns missing from media21:43
balloonsit's weird why your insert is different21:44
vthompsoncan you post your dump to a paste?21:44
balloonsohh.. that's a problem21:46
balloonsstill has your old schemversion in it :-)21:46
balloonsso it has 2 rows, hehe21:46
balloonsthere's lots of extra junk in here actually21:46
vthompsonlooks like schema 6 added a width, height, latitude, and longitude field to the media table21:47
balloonsyep, that's pretty much it21:48
balloonsgonna remove the extra row and try21:48
balloonsok, trying again21:58
vthompsonso you're trying to 1) caputure the new db with the test songs, 2) dump the db to an sql file and 3) update that file to just have the media table?22:00
balloonsvthompson, so I updated your old db to have the new schema version, and I emptied all the other tables22:01
vthompsonhm22:01
vthompsonthat might work, it would have been easier to put the test songs on the device and caputure the resulting db file22:01
balloonslol22:01
balloonsperhaps I'll do that now22:01
balloonsnow mediascanner doesn't seem to finish starting up22:02
balloonserr wait no22:02
balloonsit started,,22:02
vthompsonthen you can dump it to sql and truncate the file to just drop and create the media table22:02
* balloons deletes pics and videos22:06
balloonsok, trying again with the new db22:08
balloonsvthompson, I pushed what I have22:10
balloonsbah, diverged branches?22:11
vthompsonyou prob need to pull22:11
balloonsyea, done.. rev 50922:11
balloonsso mediascanner-dbus doesn't crash on load anymore, but it's spinning22:12
ahayzenballoons, mine is just sitting there at the bit trying to kill mediascanner-dbus22:13
balloonsI'm going to try on the device now heh22:13
ahayzenballoons, oh no i can see it is stuck at 'launching mediascanner-dbus' when running with -v not killing22:15
balloonsbtw, if backup / restore is / was successful, let's have it ready to slide in :-)22:15
balloonsahayzen, yes it hangs there now22:15
ahayzenballoons, shall i add that as a fallback branch?22:15
balloonslooks like the same result on the device22:15
ahayzenballoons, mine was working other than 1 test IIRC22:15
balloonswell we have to call it at some point22:15
balloonsso, next I guess we should trace the run for dbus and see what'sup22:16
balloonsahayzen, yea go ahead and incoporate what's needed and what we're learned22:19
balloonsI think at this point we can push it on top.. unless vthompson has other ideas22:20
vthompsonso ahayzen what you had working on the vm was for backing up/restoring right?22:20
ahayzenvthompson, yeah22:21
ahayzenvthompson, hang on i'll figure out the best way of getting the code off the VM ....22:21
vthompsonlet's try to get that working on the device as well and we can call it good22:21
balloonsyep, I'm down with that22:21
ahayzenvthompson, gimme like 5-10 mins need to revert/copy code across push22:21
vthompsonahayzen, sounds good22:23
ahayzenslowest bzr push ever!22:27
balloonstracing the launch shows dbus launches and looks good.. weird22:27
ahayzenballoons, vthompson, does this work for u on ur VM? https://code.launchpad.net/~music-app-dev/music-app/use-mediascanner2.0-fallback-plan22:29
ahayzenit does take a while to click the first thing which is weird...22:29
balloonsahayzen, tyring on device22:30
ahayzenballoons, i've never tried tht one on device that could be interesting ;)22:31
ahayzenballoons, it runs on my VM :)22:31
DrewHi22:31
vthompsonahayzen, I'll try it on my VM22:32
ahayzenvthompson, cool :)22:32
balloonssh: 1: /usr/lib/*/mediasscanner-2.0/mediascanner-dbus-2.0: not found22:32
ahayzenvthompson, it doesn't have ur protection to kill mediascanner-dbus22:32
balloonssee the error.. that said, it works22:32
ahayzenballoons, yeah tht should be fine? thts because the process it not running when it tries to kill it22:32
ahayzenballoons, the branch just needs vthompson's protection he added to the other22:33
balloonsahayzen, ohh, heh.. right :-) I thought it was happening later lol22:33
balloonsanyways, looks good.. I say we ship it22:33
ahayzenballoons, but did it run correctly?22:33
ahayzenhah win \o/22:33
balloonsmy head hurts on this mocking stuff again.. but we are much closer22:33
ahayzenballoons, only problem is i think 1 test failed when i tried on my VM22:33
vthompsonahayzen, doesn't appear to show the correct music on my VM22:34
vthompsonand the calls to kill and start dbus don't seem to work22:34
ahayzenvthompson, wht? :(22:34
ahayzenvthompson, yeah the kill i would expect but it should show the right music22:34
vthompsoneh, pastebin is down22:35
balloonswell.. works on my device.. ship it!22:36
balloons:-)22:36
ahayzen'works for me' lol22:37
ahayzenah damn i just broke it missing import...22:37
balloonsi'm surprised it's not working for you22:38
balloonsok, I take that back.. that's not surprising I guess22:38
ahayzenvthompson, i tried added ur try except in and it still outputs the error in the console ... did that not work correctly or something22:39
ahayzenballoons, so it works for me and u...thts 2 vs 1...ship it!22:39
balloonswait.. I'm an idiot22:39
ahayzenballoons, you left the files in ~/Music didn't u?22:40
balloonsahayzen, I like your style22:40
balloonsno, nothing wrong with yours ahayzen .. I confirmed it's backing up properly tas well22:40
vthompsonahayzen, well if it isn't working on the device thats 2 vs 4, so I win22:40
balloonsmy branch has a wait on the dbus call22:40
balloonsit works too22:40
balloonslolololol22:41
ahayzenballoons, hang on wht the preferred branch is working?22:41
balloonsahayzen, vthompson the dbus call was check_output.. it was just waiting forever22:41
ahayzenballoons, omg explaining it just sitting there :)22:42
balloonsROFL22:42
ahayzenballoons, push it22:42
vthompsonballoons, ha22:42
vthompsonman22:42
ahayzen"given enough eyeballs, all bugs are shallow"22:43
ahayzen....as long as they aren't sleepy eyeballs22:43
vthompsonIf we're dealing with shallow bugs, I don't want to see the deep ones22:44
ahayzenhaha good point ;)22:44
balloons+122:44
ahayzenbrb22:45
balloonsi pushed rev 510. see what you think22:46
balloonswe're shipping something22:46
vthompsonooo, looking good on VM22:47
vthompson^ first time that's been true for me22:47
balloonsvthompson, heh. woot22:47
* ahayzen returns22:48
balloonsrunning on my device now22:49
balloonsandrew's finished.. one error on test_artists_tab_album22:49
balloonsperhaps something we need to fix22:49
vthompsonThis is what I get on the device: http://paste.ubuntu.com/7666084/22:49
ahayzenballoons, what was the error i'll take a look22:50
ahayzenballoons, ur branch is working on my VM aswell \o/22:50
ahayzen# stop mediascanner-2.022:50
ahayzenstop: Unknown job: mediascanner-2.022:50
ahayzenroot@ubuntu-phablet:~# start mediascanner-2.022:50
ahayzenstart: Unknown job: mediascanner-2.022:50
ahayzen^^ tht isn't right is it? ... on device?22:51
vthompsoncan't do it as root22:51
ahayzenoh22:51
vthompsonit's a user service22:51
vthompsonneed to be phablet22:51
ahayzenvthompson, did urs pass in the VM ... mine 'passed' but didn't actually click anything22:51
vthompsonahayzen,  same.22:52
vthompsonwhich would make me believe it did nothing22:52
ahayzenballoons, any idea wht is going on? the app starts... then it just sits there not clicking anything and after 75s outputs 'pass'22:52
vthompsonahayzen, was your branch doing the same22:53
balloonsok, so that sounds quite odd22:53
vthompson?22:53
ahayzenvthompson, mine was clicking things fine IIRC22:53
balloonson the device ahayzen's branch ran through things22:53
ahayzenballoons, its the main branch which has this oddity22:54
vthompsonhm22:54
balloonswell back to shipping ahayzen :-)22:54
* vthompson moves back over to testing ahayzen's branch22:54
ahayzenone does not simply 'ship ahayzen'22:54
balloonswe can pick up the mocking silliness later.. should look at the one failed test and fix it22:54
ahayzenok22:55
vthompsonahayzen, we might need to fix your kil/start of dbus22:55
ahayzenvthompson, i added ur bits to my branch if u pull22:55
vthompsonseems like it might have worked this time around, but it didn't start dbus and it also isn't clicking anything22:55
vthompsonahayzen, ah22:55
vthompsonstill couldn't start dbus... but that might be OK?22:56
vthompsonstill doesn't click anything22:56
ahayzenvthompson, yeah i think so...note it takes like 30s to click something22:56
ahayzenidk whts going on there22:57
balloonsahayzen, vthompson I'm an idiot again22:57
balloonsI have a sys.exit() at the top of test_music22:57
balloonslovely22:57
ahayzen\o/22:57
balloonsI must have pushed it also22:57
ahayzen3 != 2 ... ok ...22:57
ahayzenlooks like one of our tests is actually properly failing?22:58
ahayzenmaybe one of the filters with mediascanner2 is slightly differnt....22:58
* ahayzen investigates22:58
ahayzenthe test_artists_tab_album one22:58
vthompsonwell, I think I ctrl+c'd out of a test and things wheren't restored22:59
ahayzenah i think i did tht before i'm assuming thts how my ~/Music suddenly had the test files in it23:00
ahayzenballoons, does the main branch now work ? lol23:00
balloonsahayzen, stay focused :-)23:00
ahayzento many things!23:01
ahayzenballoons, ok so on the fallback branch why does autopilot take like 30s to click something?23:01
balloonsahayzen, no idea23:01
balloonsany messages?23:01
balloonsdo we have a sleep stuck somewherE?23:02
ahayzeni can't see the console its behind the app hang on23:02
balloonsahayzen, main branch does the same I see23:02
balloonsif it's the same, it's dbus timeouts23:02
vthompsonballoons, I'm running verbose and I don't see any console logs23:02
ahayzensame23:03
ahayzenoh no yeah dbus timeout23:03
ahayzenjust saw one23:03
balloonsok, well on the main branch it sits for a long time.. prints 19:01:43.443 ERROR proxies:410 - Introspect error on :1.304:/com/canonical/Autopilot/Introspection: dbus.exceptions.DBusException: org.freedesktop.DBus.Error.NoReply: Message did not receive a reply (timeout by message bus) and then it goes23:03
ahayzenballoons, yeah thts wht i get then after a few seconds it starts clicking things23:03
ahayzenvthompson, can u look at the code for test_artists_tab_album why does it think there should be 3 tracks in the queue when there are only 2 tracks for the artist it is playing?23:04
ahayzenvthompson, is that because our music has changed?23:04
vthompsonI'll take a look23:04
ahayzenballoons, is that anything todo with us or is it autopiliot itself?23:05
vthompsonI do see console errors now, but paste is doen23:05
vthompsondown23:05
ahayzenvthompson, about line 523 in test_music.py is where it fails23:05
vthompsonHere's the error I get before it clicks on stuff: http://pastebin.com/gepn8CWF23:06
ahayzenvthompson, yeah same23:06
vthompsonahayzen, yea, that should be 2. Really it should be 1, but one of the other dummy files is counted as being in the same album23:11
balloonsthat might be because of our messing with dbus.. which autopilot also uses23:11
balloonssince it eventually works, I don't see it as a blocker23:12
ahayzenagreed and agreed23:12
ahayzenvthompson, should i or u update it to 2?23:12
vthompsonI can do it, running that way now23:12
ahayzencool23:13
balloonsyou could add back in the sleep I had before laucnhing23:13
balloonsthat might help.. but likely you'll end up waiting either way23:14
ahayzenyeah i guess ... but it 'works' thts the main thing :)23:14
ahayzenballoons, vthompson, right i propose to propose the fallback for merging so jenkins can have a go at it?23:15
ahayzenballoons, vthompson, oh pep9 may spectacularly fail first23:16
* ahayzen starts patching23:16
vthompsonwould it be hard to just merge it into the currently proposed use-mediascanner2.0?23:16
balloonsahayzen, ohh yea23:16
ahayzenvthompson, could we not keep the use-mediascanner2.0 as a WIP solution to getting the isolation working?23:16
vthompsonwe could, but it also has all the activity up until this point23:17
vthompsonahayzen, either way is probably fine23:17
ahayzenhmm23:18
ahayzenand it has the bugs linked...23:18
ahayzenbut then it has the progress of the isolation which nearly works :/23:18
vthompsonyours only differs in that it doesn't have the isolation?23:18
vthompsonIt's just that we already have been doing a code review on this, to review another merge is kinda silly23:19
balloonswe could merge back ahayzen's branch and include both ways in __init__.py to be solved later23:19
vthompsonit should be the same merge23:19
balloonsthe main branch does work (since I took out the sys.exit, lol), but mocking doesn't seem to work on the device (story of our lives)23:20
ahayzenballoons, vthompson, pep8 fixes done :)23:20
ahayzenballoons, does the fallback work on device?23:20
vthompsonahayzen, your branch times out before launching on my device23:20
balloonsahayzen, yes we've established that23:20
vthompsonI'm running the test_mp3 test23:21
balloonswell it works for me23:21
ahayzenhmmm conflicting info there23:21
balloonsvthompson, feel free to try the main branch on your device if you wish23:21
balloonsI'm going to merge back ahayzen's branch and have backup/restore be the default23:21
balloonsand we can merge it23:21
balloonsthat way the isolation code stays in23:21
ahayzenballoons, yeah i don't wanna lose that23:21
ahayzenballoons, as it is soo close23:21
vthompsonhm, I get timeout on the device for the main branch as well23:22
vthompsontimeout: http://paste.ubuntu.com/7666207/23:23
vthompsonballoons, I pushed the test_artists_tab_album test fix to the main branch, r51423:26
balloonsvthompson, what image #?23:26
vthompsonballoons, I'm on 87, IIRC23:27
vthompsonsystem settings won't run right now23:27
ahayzenvthompson, yeah i usually have to restart the phone after running autopilot some crazy stuff happens23:27
ahayzenballoons, vthompson, ok just ran the fallback on my device and got 'no music found'23:28
vthompsonyea, rebooted. Could also have been why I was timing out during the test23:28
balloonsvthompson, same23:28
balloonsok, I think I've merged the branches23:28
ahayzenhere are my two logs...install http://pastebin.ubuntu.com/7666215/ and the running http://pastebin.ubuntu.com/7666213/23:29
vthompsonballoons, rebooting the phone also causes the app not to time out anymore...23:30
vthompsontrying andrew's branch23:30
ahayzenmagic23:30
vthompsonahayzen, balloons, fantastic, andrew's branch works on the device. trying the main branch now23:32
balloonsI'm inprocess merging andrews branch back.. just testing it now for sanity23:33
ahayzenballoons, vthompson, ok i can see my branch working now ... the restart fixed it :)23:33
vthompsonfunny that we'd see the same issue23:34
ahayzenvthompson, probably because we both tried a failed ap earlier?  i know i did23:34
ahayzen'Ran 1 test in 94.622s23:35
ahayzenOK' \o/23:35
vthompsonon your branch right?23:35
ahayzenyeah23:35
vthompsonthe main branch still sees the user's library23:35
vthompsonon my device23:35
ahayzennow wht was wrong with the main branch that means we need to merge mine...23:35
ahayzenah tht23:35
ahayzenvthompson, have u restarted ;) lol23:35
vthompsonballoons, ahayzen, something small is probably just missing from the merge23:35
balloonsmain branch merge with andrews needs more tweaks23:36
balloonsworking on it23:36
ahayzenvthompson, if it is something small tht is stopping the preferred branch can we not investigate tht .... or is it something more serious?23:36
balloonsgot it23:38
vthompsonahayzen, I was assuming it was small. balloons is looking into what it is I'm sure23:38
ahayzenvthompson, yeah it does the same on my VM just displays whtever is in ur real library23:38
ahayzenballoons, so the main branch is now actually the fallback branch with the isolation code still there but not being used?23:39
ahayzenor have i misunderstood whts been going on lol23:39
vthompsonballoons shoots and he scores!23:40
ahayzenyep!23:40
balloonsahayzen, yes.. rev 51723:40
ahayzen:D23:40
ahayzennow time to test on device...23:41
vthompsonso, one minor thing is that our call to start dbus with the * in it doesn't work on my vm... but maybe we don't need that call?23:41
* ahayzen wonders how long jenkins will take to catch up23:41
vthompsonahayzen, device actually worked for me23:41
balloonsflake8 that and I think we're good.. I blocked the two methods so it should be as simple as removing one block and uncommenting the other in a few places23:41
ahayzenwhich call in particular?23:41
ahayzenawesome23:41
vthompsonthe call to start mediascanner-dbus-2.023:42
vthompsonI think we just need to kill it23:42
vthompsonstarting ms2 probably starts it if it needs it23:42
ahayzenhmmm maybe i thought the starting was important...23:43
vthompsonalso, do stopping and starting the ms2 service need to be a cleanup task?23:43
ahayzenvthompson, yep tht restores it back to the users one23:44
ahayzenvthompson, you should end up with the same processes/state after running the tests then23:44
vthompsonahayzen, ah right23:44
ahayzenvthompson, definitely need the kill of mediascanner-dbus23:44
vthompsonbut the cleanup to kill dbus is probably not needed23:44
ahayzenhah23:44
vthompsonreally23:45
vthompsonwhy23:45
ahayzenvthompson, as tht is the one tht causes the issue in the first place isn;t it?23:45
vthompsonyea, but only because of our dbus issue while running23:45
ahayzenvthompson, yeah but i think it is cleaner to stop the processes we are manually starting within ap23:45
vthompsonwell, I also dont think we need to start dbus, just axe it23:46
vthompsonexecution style23:46
ahayzenvthompson, try taking it out... i think maybe just the first kill is important if it is already running then?23:46
vthompsonahayzen, that's my theory23:47
ahayzen:)23:47
* ahayzen tries on his VM23:47
balloonsok, flake8 is happy now23:48
* balloons runs on device again23:49
ahayzenvthompson, so remove the manual start of dbus and the cleanup... i've left the restart of mediascanner2... tht works for me23:49
vthompsonI pushed my cleanup23:49
ahayzenvthompson, check that if u start the app normally afterwards tht u can see ur real lib23:50
vthompsonI left the restart of ms2 as well23:50
ahayzenvthompson, yep agreed23:50
vthompsonahayzen, good point23:50
ahayzenvthompson, time to try on device23:50
vthompsonahayzen, ok, I did have to kill dbus again23:51
vthompsonI'll readd it23:51
ahayzeninteresting23:51
ahayzenthought there was a reason it was there...but i assume if our app closed correctly tht process would be closed if no other apps were using the QML plugin?23:52
ahayzenvthompson, yeah it does need killing my device just finished23:54
vthompsonahayzen, yea, I just repushed23:55
vthompsonreverted it23:55
ahayzenvthompson, cool retest time23:55
balloonsok jenkins should come back happen next run23:56
vthompsonballoons, so we also need to figure out what's up with jenkins23:56
ahayzenvthompson, pyflakes etc23:56
vthompsonhm23:56
ahayzenvthompson, it is a few rev's behind23:56
balloonsvthompson, yep and it's fixed23:56
balloonsI've kicked off 520 manually to catch it up :-)23:57
ahayzen:)23:57
ahayzenballoons, ok so if this is actually good and passes... will it then land in the store?23:57
balloonsahayzen, ohh yea.. fore sure23:58
ahayzenballoons, i'm asking as i need to know when to propose my branch to remove grilo from the image?23:58
balloonsI'll push it asap23:58
balloonsas soon as it's merged, it's store time23:58
ahayzenballoons, ok and then ^^ mp will unblock Qt5.3 :D23:58
balloonsahayzen, as soon as this is merged, propose it :-)23:58
ahayzenballoons, will do :D23:58
ahayzenballoons, then time to tick things off in trello23:58
* ahayzen hopes Jenkins doesn't cause us pain23:59
vthompson+123:59
ahayzen#520 = perfect on device for me23:59

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