/srv/irclogs.ubuntu.com/2014/07/25/#ubuntu-app-devel.txt

=== shuduo_afk is now known as shuduo
=== _salem is now known as salem_
=== salem_ is now known as _salem
bzoltannik90: hello, do you want to test drive the latest QtC plugin from the https://launchpad.net/~ubuntu-sdk-team/+archive/ubuntu/tools-development?05:02
=== chihchun_afk is now known as chihchun
dholbachgood morning07:02
justCarakasmorning all07:14
JamesTaitGood morning all; happy Friday, and happy SysAdmin Appreciation Day! :-D08:18
=== renato is now known as Guest59000
nik90bzoltan: hey, good morning09:09
nik90bzoltan: I forgot to remove the tools-development ppa after the previous testing09:09
nik90bzoltan: so I must be running the latest QtC plugin already?09:09
nik90bzoltan: what would you like me  to test specifically?09:09
bzoltannik90:  if you have run dist-upgrade in the last ~15 hours then you have it09:11
bzoltannik90: mostly the  the emulator startup, html5/webapp packaging and deployment, correct fw list on the Publish page. Generic stuff.09:12
bzoltannik90: thanks for your help09:12
nik90bzoltan: will check it out09:12
nik90np09:12
nik90t1mp: hey, in your https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/rtm-fit-finish/+merge/227696 branch are you also planning to change the color of the listitem text to midAubergine like in the design docs of apps?09:53
nik90t1mp: or would you provide a property to change the text color?09:53
dpmdholbach, I've been pinged on #ubuntu-app-devel to get gallery-app approved in the store, as popey is not there. Is this something you could do? I could have a look myself (I've got the permissions, I think), but you've done more reviews/approvals and know better what you're doing.09:54
dpmLet me know if it works for you09:54
dpmsorry, I meant I've been pinged on #ubuntu-ci-eng09:54
dholbachdpm, I don't see it in the review queue09:55
dholbachdpm, right now we basically just run the review tools on the packages09:56
dholbachdpm, I just checked - I replied on it "2014-07-23 11:06 - 1 day, 22 hours ago"09:56
dpmdholbach, ah, so it's indeed in the queue?09:57
dholbachno09:57
dholbachit was09:57
dholbachI sent it back because it used an old framework09:57
dpmdholbach, ah, thanks09:58
t1mpnik90: good question. I have a review with design in a few minutes, I'll ask what should be the default. I'll get back to you10:05
nik90t1mp: thnx10:10
nik90mzanetti: I got an answer to your question you had yesterday about changing the time via the clock app10:12
nik90mzanetti: https://docs.google.com/presentation/d/1JvDyhsW17d1-Mz8OY1YMBKwfRI2z9qgyRjbujEsxEMk/edit#slide=id.g18895458d_02410:12
nik90mzanetti: we will provide a settings option in the clock app for that. On clicking it, the user will be taken to the system settings date&time page where they can change it10:13
mzanettinik90: nice!10:14
nik90:010:15
WangWeiWhen I make a scope app, I receive error" package 'libunity-scopes>=0.5.0' not found10:16
WangWei"10:16
WangWeihelp10:19
t1mpnik90: text in list items is dark grey by default, so apps with purple need to override it11:41
t1mpnik90: I guess that's not possible now, did you report a bug for it?11:42
nik90t1mp: no I haven't reported a bug about it. I wonder if exposing would be better or if using the style API is better11:42
nik90exposing via property or via the style api*11:42
* nik90 is super excited about https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/125477511:45
ubot5Launchpad bug 1254775 in Unity 8 "Desktop file can specify a splash image and background color that shell will show until app is ready" [Medium,Triaged]11:45
nik90t1mp: reported bug at https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/134860411:50
ubot5Launchpad bug 1348604 in Ubuntu UI Toolkit "Provide a property to change the font color of listitems" [Undecided,New]11:50
nik90karni: oh btw, I cannot fix the docs with no images since those point to the original qt docs. Only mhall119 can fix that :)11:54
karniright, thanks nik90 :)11:54
nik90karni: I am more into fixing our own SDK documentation11:55
karniack11:55
nik90rpadovani: hey, got a minute?12:20
rpadovaninik90, for you always ;-)12:20
nik90rpadovani: :) can you review https://code.launchpad.net/~nik90/ubuntu-clock-app/improve-hour-hand-accuracy/+merge/22788512:20
nik90rpadovani: 1 liner12:20
rpadovaninik90, looks good, 3 miuntes that I think to corner cases...12:24
rpadovani*minutes12:24
nik90ok12:24
rpadovaninik90, just to be sure, analogTime.getMinutes() / 2 becomes a float right? I mean, 00:00 has angle of 0 and 00:01 has angle of 0.5?12:28
nik90rpadovani: I would assume yes12:29
nik90rpadovani: since rotation is a float, it should convert it automatically12:29
rpadovaninik90, approved :-)12:29
nik90rpadovani: thnx12:30
nik90mzanetti: ping12:35
mzanettinik90: pong12:43
=== _salem is now known as salem_
nik90mzanetti: hey I did a small test where I changed the timezone in the system settings app and returned to the clock app. I noticed that the clock app time was incorrect. However on restarting the clock app, it then showed the correct time.12:43
nik90mzanetti: could this be the same bug you reported where timezone changes are not transfered properly to the javascript date object?12:44
mzanettinik90: I don't think so...12:44
mzanettinik90: but no idea what it is12:44
nik90mzanetti: hmm ok. I am going to make the plugin output the local time and see if that shows correctly after the timezone change.12:45
mzanettiyeah, try that..12:45
t1mpnik90: yeah, maybe using the style api is better12:53
nik90t1mp: is there a documentation for that or any examples that I can take a look at?12:54
t1mpnik90: no, we didn't write new docs yet on the new (since one year?) theming engine12:57
t1mpnik90: it is a bit confusing for me too, zsombor knows it all but he is away now12:58
t1mpnik90: perhaps there are other apps that use a custom theme12:58
t1mpnik90: oh wait I am talking nonsense12:58
t1mpnik90: the list items are not themable12:58
t1mpnik90: so we'll have to fix your bug12:58
nik90t1mp: :)12:58
nik90t1mp: I will see what the address-book is doing for instance12:59
nik90t1mp: address-book defined their own custom listitem since they needed one with the new delete swipe actions13:00
t1mpnik90: yes, it is completely custom13:21
t1mpnik90: zsombor is working on completely new list items for after RTM13:21
nik90t1mp: Yeah he told me...but its too late for me. I am thinking about temporary solutions for now.13:22
jdstranddholbach: hi!13:38
dholbachhey jdstrand13:39
jdstranddholbach: ok, I think we are ready to cut click-reviewers-tools 0.813:39
dholbachyeeehaw!13:39
dholbachjdstrand, great work13:39
dholbachjdstrand, shall I take care of it?13:39
jdstranddholbach: snag r214 and do whatever testing you normally do. please ping me if I need to fix something13:40
dholbachawesome13:40
jdstranddholbach: oh, heh, yes please :)13:40
dholbachwill do13:40
dholbachthanks!13:40
jdstrandthank you :)13:40
jdstranddholbach: this should be the one that works without needing any apparmor deps13:40
dholbach*\o/*13:41
jdstrandso that will be nice13:41
jdstrandtrusty users, unite!13:41
jdstranddholbach: I'm happy about that, but I'm most happy that now if an unrecognized hook is found, it will WARN13:43
jdstrandwe ended up getting so far behind with all the new hooks13:43
dholbachthat's great! good thinking!13:43
jdstrandsome had security impact too, so I'm glad we are caught up13:44
om26ert1mp, Hi! Is the current height of the header as designed ? With new fonts changes header just looks bigger13:57
dholbachjdstrand, can you have a quick look at https://code.launchpad.net/~dholbach/click-reviewers-tools/pep8-fixes/+merge/228315?14:03
t1mpom26er: the header icon sizes became smaller also, but they will become bigger again14:03
t1mpom26er: the header is the same as before, but it will be 0.5 grid units smaller soon14:03
om26ert1mp, alright, great. I see the header size in dash is smaller than what we have in apps :)14:04
t1mpom26er: the height and auto-hiding of the header in dash is something custom (only what's inside the header they took from sdk)14:08
t1mpom26er: but we'll make it a little bit less high in the uitk14:08
jdstranddholbach: huh-- are you running utopic?14:09
jdstranddholbach: trusty's pep8 was clean14:10
dholbachjdstrand, yes - let me give you the output14:10
jdstranddholbach: that's ok, I have a schroot14:10
jdstrandoh, I fixed some of those in the past14:11
jdstrand(different files)14:11
balloonsgerlowskija, get your adt-run question answered?14:11
nik90mzanetti: bug confirmed :/ .. the c++ plugin returns the correct time while the javascript date object that I use to display the time in the main clock fails miserable14:15
nik90mzanetti: atleast I know now how to fix it. I might create a new plugin whose only job is to return the local time14:15
mzanettinik90: meh... if its different than my example, feel free to add more info to the bug report14:16
nik90mzanetti: ok14:16
nik90mzanetti: I have one other performance issue with our timezone model14:16
mzanettinik90: performance issue?14:17
nik90mzanetti: yeah. The xmllistmodel that we feed into the c++ plugin now has about 150 cities. So when I load it, it takes about 5 seconds to display the page14:17
balloonsmardy, any luck having a look at https://bugs.launchpad.net/reminders-app/+bug/1347905?14:17
ubot5Launchpad bug 1347905 in Ubuntu Reminders app "Autopilot tests fail to create fake account on device" [Critical,Triaged]14:17
nik90mzanetti: I will still need to do some debugging to see if it might be a bottleneck elsewhere14:18
jdstranddholbach: approved. thanks!14:18
mzanettinik90: parsing that xml model takes 5 secs?14:18
mzanettiwow... seems I messed up badly then14:18
mzanettinik90: can you please add this at selected places of the code:14:19
nik90mzanetti: it could be my fault as well since what I do now is send the results of our timezonemodel into a sorted model provided by the sdk. The sorted model sorts it alphabetically.14:19
mzanettiQDateTime start = QDateTime::currentDateTime();14:19
mzanettiand later14:19
mzanettiqDebug() << "operation took" << start.msecsTo(QDateTime::currentDateTime());14:20
mzanettithen you can find out where it takes long14:20
nik90mzanetti: do you want that to be added inside the data() function?14:20
mzanettinik90: I'd suspect the xml parsing code14:20
mzanettinik90: but yeah... beginning and end of each function that could potentially be the bottleneck14:21
nik90mzanetti: ok will do and let you know the results14:21
mzanettidata() might be a bit odd for that, as its called rowCount * rolesCount times14:21
mzanettiso will spam quite a bit14:21
nik90mzanetti: yeah I think the loadTimeZonesFromXml() might be a good starting point14:22
mardyballoons: nope, sorry, was busiy with something else. Monday morning, I promise :-)14:24
dholbachjdstrand, all done14:26
dholbachpopey, click-reviewers-tools should be happy on trusty again14:26
jdstrand\o/14:27
balloonsmardy, :-)14:28
=== daker_ is now known as daker
nik90mzanetti: confirmed. the bottleneck is loadFromXmlModel() which takes about 4.772 secs on the phone.14:35
nik90mzanetti: on the emulator it takes about 0.7 secs14:35
mzanettinik90: really weird...14:36
mzanettinik90: ok. need to debug that... don't have an idea atm14:37
nik90mzanetti: ok, I will also do some reading about this14:37
=== boiko_ is now known as boiko
gerlowskijaballoons: Nope, not yet.  Still around? (and if so, up for an adt-run question? ;-p)14:58
balloonsgerlowskija, better to ask the question, than ask to ask it ;)15:00
gerlowskijajust checking you were still around.  adt-run has a bunch of options for specifying the source of the tests to run, and the click/deb package to run them against.  But I was having trouble putting together a command to run the use a local checkout both for the source of the tests, and of the app-code.15:10
gerlowskijai.e. What I want to do is bzr pull lp:~gerlowskija/ubuntu-calendar-app/my-branch-blah-blah15:11
gerlowskijaand then use the version of both the tests, and the app that are in that checkout15:11
gerlowskijaI had a couple examples of things that I'd tried and their results, but those are on my home machine.15:11
dpmjdstrand, I re-read the bug about terminal/file manager security, as there was something I didn't quite understand. Does this diagram reflect the workflow in the requirements? https://ubuntu.mybalsamiq.com/projects/ubuntuphonecoreapps/Terminal%20security15:13
ogra_dpm, just imagine it runs under gksudo now ;)15:14
ogra_(even thought it doesnt, since it doesnt elevate privileges, just allows access at all)15:14
dpmogra_, that's what I thought, but I'm trying to figure out how to do authentication in the Qt world. We can check if there is a password via dbus, but I don't know how to prompt and check for the password from a Qt app15:15
ogra_dpm, there is some code you can use as example in system-settings ... also mterry should be able to help you15:16
gerlowskijaballoons: oops, meant to tag you in one of those posts ^^15:27
=== chihchun is now known as chihchun_afk
balloonsgerlowskija, you would build the click file and pass that15:30
balloonsso bzr branch lp:ubuntu-calendar-app15:30
balloonsclick-buddy --dir ubuntu-calendar-app15:30
balloonsadt-run ubuntu-calendar-app com.ubuntu.calendar*.click . . .15:31
dpmballoons, thanks for the workaround for bug 1347905 - have you had to talk to mardy for the proper fix?15:32
ubot5bug 1347905 in Ubuntu Reminders app "Autopilot tests fail to create fake account on device" [Critical,Triaged] https://launchpad.net/bugs/134790515:32
dpmgerlowskija, hey, thanks a lot for your awesome work and your patience with our tools! When your calendar branch lands we need to celebrate :)15:33
balloonsdpm, I spent some time looking at it and prodding mardy about it. He's promised to have a look monday15:36
brendandballoons, i commented :)15:36
brendandballoons, you don't actually have to have the trailing ) i just found out15:37
brendandballoons, it can go at the end of the second line too15:37
balloonsbrendand, :-) The reason I wasn't concerned about the line is it shouldn't be there. It should be a method call to the class15:37
dpmballoons, do you think we can get reminders on the dashboard already, with your workaround?15:38
balloonssee the commented line above, it gives an error and I wasn't sure why lasy night15:38
balloonsdpm, we should be able to move it over15:38
dpmexcellent15:38
dpmballoons, independently of the dashboard, could you organize a reminders upload? A critical bug fix (camera not saving pictures) landed, and it'd be good to get it in the store15:39
balloonsdpm, sure thing15:42
balloonsbrendand, if you look at the MP can you understand why the self.app.main_view.no_account_dialog.open_account_settings() doesn't work?15:43
balloonswe can land it as-is, but I'd like to understand that. Also a review from you so it can land would be helpful.. poor Leo has enough to look at15:43
dpmawesome, thanks balloons!15:44
=== gatox is now known as gatox_lunch
gerlowskijaballoons: I think I tried something similar yesterday evening when I was playing with adt-run.  It saw the click, but reported that there were no tests in the manifest for the click, and exited15:47
gerlowskijaballoons: errr...actually forget I said that.  I might've made some other mistake in my command...i'm useless w/o my bash history around ;-p15:47
balloonsgerlowskija, well :-)15:53
brendandballoons, i'll have a link. elopio isn't working today anyway15:53
brendands/link/look/15:53
balloonsbrendand, excellent thanks. In the interim in case we land it as-is, I'll fix the flake8 issue15:54
jdstranddpm: "Does the device have a password set? No" -> terminal *starts*16:08
dpmjdstrand, do I get this right? So if my phone doesn't have a password set and my little cousin grabs it he can just do whatever he wants with the terminal?16:10
jdstranddpm: I think you want to use 'unix_chkpwd' if you have to write your own, but might check with mdeslaur16:10
gerlowskijaballoons: belated thanks balloons; I'll try your steps as soon as I get home.16:10
jdstranddpm: that is correct16:10
jdstrandif you don't want your little cousin to have access, set a pin/password or keep it out of his hands :)16:11
jdstrandhttps://wiki.ubuntu.com/SecurityAndPrivacySettings/ProtectingUserData16:11
dpmbetter said than done ;)16:11
dpmok, let me update the diagram16:11
jdstranddpm: I don't understand the note "Or it could remain in the password prompt to allow retries"16:12
jdstranddpm: we definitely want to allow retries16:12
dpmjdstrand, ah, so in that case the app should not quit, but it should remain open with the password prompt dialog16:13
jdstranddpm: it should probably follow what we end up using in https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/134825116:13
ubot5Launchpad bug 1348251 in lightdm (Ubuntu) "please make use of pam_tally2 for Touch login and screenunlock" [High,Incomplete]16:13
balloonsdpm, the new reminders is in the store. We'll try and push yet another one today with the workaround16:13
jdstranddpm: well, it could quit. that meets the requirement16:13
jdstranddpm: it seems not as nice though16:13
jdstranddpm: we should not allow arbitrary retries without the application quitting16:14
jdstrandie, if you are going to allow retries, it should quite after a while16:14
jdstrandquit*16:14
dpmjdstrand, something like that? https://ubuntu.mybalsamiq.com/projects/ubuntuphonecoreapps/Terminal%20security16:15
dpmawesome, thanks balloons16:15
jdstranddpm: looks good (did you see my comment on unix_chkpwd?)16:16
dpmjdstrand, I saw it, I was going to ask you, I'd rather use code than calling binaries in the app. What step were you suggesting it for? For "password is set" or "password prompt"?16:17
mdeslaurdpm, jdstrand: someone needs to write a library to check the password using pam that can be used from qml16:17
jdstranddpm: password is set is in the bug. password prompt is what I was talking about16:18
jdstrandmdeslaur: unix_chkpwd should be using PAM, but I agree, a library is the way to go16:19
dpmmdeslaur, jdstrand, if that is the case, that's a big job, not sure we should be able to write a QML authentication library from the community core apps side of things16:19
dpmInitially I saw we could do the password check via DBUS16:19
mdeslaurjdstrand: no, unix_chkpwd is what pam_unix uses, you can't use that directly16:19
dpmbut password prompt and check is an entirely different beast16:19
jdstrandoh I see that now in the man page16:19
mhall119jdstrand: is there any reason why this check and prompt can't be done via unity-app-launch of apparmor itself for unconfined apps?16:27
jdstrandapparmor can't prompt16:28
mhall119why not?16:28
jdstrandit is all kernel16:28
jdstrandual could, I guess16:29
mhall119jdstrand: don't we prefix an app's Exec= with aa-exec or something like that?16:29
jdstrandmhall119: that would only work on non-unity816:29
mhall119non?16:30
ogra_jus write a QTsudo and hook that into the .desktop Exec line ?16:30
mhall119ogra_: we don't want to run them as root16:30
jdstrandunity8 doesn't use those desktop files16:30
mhall119jdstrand: it doesn't use ~/.local/share/applications?16:31
mhall119Exec=aa-exec-click -p com.ubuntu.developer.mhall119.ureadit_ureadit_0.9.8~1 -- qmlscene @$ uReadIt.qml16:31
jdstrandmhall119: no, it doesn't16:32
jdstrandthat is for non-unity816:32
mhall119so..what does Unity 8 use? Saviq ?16:32
jdstrandif the application itself isn't doing it, then you need trust session support16:32
brendandballoons, reminders app seems to be broken more fundamentally16:35
brendandballoons, it's using the wrong desktop file to launch16:35
brendandballoons, ap tries to use /usr/share/application/reminders.desktop16:35
brendandballoons, which doesn't exist16:35
balloonsbrendand, well on the device / dash we'll be using launch_test_click16:36
dpmmhall119, perhaps there is something we can use from Qt or directly from C++ - ogra_ was pointing to the ubuntu-system-settings code for an example, but I've not yet had a look and I'm not too familiar with the code16:37
jdstrandmdeslaur: so you are saying that rather than using unix_chkpwd directly, we should use the PAM library (which will ultimately use unix_chkpwd itself), and all that should be in a library so it can be abstracted from the terminal and filemanager16:37
balloonsbrendand, that said, it seems the deb build the .desktop file as /usr/share/applications/com.ubuntu.reminders.desktop16:38
mdeslaurjdstrand: yes16:41
balloonsbrendand, I can fix that with a simple cmake change16:42
mdeslaurjdstrand: perhaps reusing this: http://bazaar.launchpad.net/~unity-team/unity/trunk/view/head:/lockscreen/UserAuthenticatorPam.cpp16:42
balloonsor we just update the test16:42
jdstranddpm: ^16:43
jdstrandmdeslaur: that is the bit that sarnold reviewed?16:43
mdeslaurjdstrand: I have no idea if sarnold reviewed that or not16:43
jdstrandno, that is unity16:43
mdeslauryeah, that's what the unity lock screen does16:43
mdeslaurit can be used as an example16:44
jdstrandcool16:44
jdstrandthanks16:44
dpmmzanetti, quick question as a Qt expert: do you think this http://bazaar.launchpad.net/~unity-team/unity/trunk/view/head:/lockscreen/UserAuthenticatorPam.cpp could be made into a QML authentication plugin that the Terminal and File Manager app could use? The context is that we need to add a password prompt/check for those 2 apps16:45
jdstranddpm: I think you would want something based on that broken out into a library/plugin, not that specifically16:46
jdstrandbut yeah-- that is definitely the approach to take (I added a comment to the bug)16:47
dpmjdstrand, yeah, that's my question. Something based on that made into a QML plugin or a C++ wrapper that the apps can use16:47
mzanettidpm: yes, but you'd need to get away with things like "const pam_message** msg" in the api and use Qt data types instead16:48
mzanettibut otherwise sure. I'm not entirely sure what it does though16:48
=== gatox_lunch is now known as gatox
mzanettidpm: yeah, seems rather simple16:49
dpmmzanetti, ok thanks. As per what it does, I assume it does password authentication using PAM, I don't understand the code myself16:50
mzanettidpm: also not using callback like this does, but switching it over to signals/slots16:50
mzanettiafter reading a bit it just seems some glue code to talk to pam and ask a gui for input and display16:51
mzanettiyeah. shouldn't be too hard I suppose16:51
dpmmzanetti, do you think it would make more sense to make it into a QML plugin, or to keep it only accessible from C++? Both apps have some C++ code already, but I guess it'd be nicer to use it from QML16:51
mzanettidpm: I'd definitely vote for both... but if you're doing it a bit clever I guess you can get a nice class that can be used in qml and c++ the same way16:52
mzanettiso you only write one class that c++ can include and is also qmlRegisterType'd16:53
dpmok, thanks16:53
mhall119dpm: is the intention that we would make this available to other apps?16:56
dpmmhall119, I think so, otherwise each app should ship a copy of this plugin. We should probably make it a QML plugin installed by default, that those apps requiring authentication can use16:57
mhall119jdstrand: would we want to allow apps to call PAM? are there any potential security concerns to allowing that?16:58
jdstrandmhall119: it is fine cause they can't authenticate anyone else but the user17:00
jdstrandmhall119: and confined apps won't be able to do it17:01
jdstrandso a confined app wouldn't be able to brute force the password, for example17:01
jdstranddpm: do keep that in mind ^ (only unconfined apps could leverage the plugin)17:02
dpmoh, good point about confinement, so perhaps we should not make it a system-wide plugin17:02
jdstrandbut having terminal and filemanager need it is justification alone17:02
jdstrandwe could, apparmor would just block it17:03
dpmok17:03
jdstrandjust like if a C++ app tried to access the PAM library itself17:03
jdstrandthat didn't come out perfectly. if a C++ app tried to use the PAM library, there would be denials at some point that would prevent it from succeeding17:05
mhall119there's no point in making it available when most apps won't be able to use it17:06
mhall119especially when it's easy enough to just include it with the terminal and filemanager packages17:06
* jdstrand doesn't have an opinion17:08
jdstrandwhatever you decide-- it really shouldn't be hardcoded in both though. ie, embed the library if you must, but use a library :)17:08
dpmmhall119, yeah, you've got a point. I think I'd favour that too, although we'll have to think how to deal with code duplication. If we create a separate project for the pam plugin, I'm not sure how Jenkins will deal with fetching the code from there to put it in each app's click package17:15
balloonsbrendand, this should be ready for a final review: https://code.launchpad.net/~nskaggs/reminders-app/workaround-1347905/+merge/228213. I ended up removing the code that caused the flake8 discussion17:28
brendandballoons, i need to look at it more closely and unfortunately don't have the time now. i'll try and look at it later, but you might have to get elopio to look at it on monday (i'm off till wednesday)17:31
balloonsbrendand, ahh, ty17:31
balloonsmzanetti, are you still about?17:33
mzanettiballoons: yeah17:33
balloonsmzanetti, want to give https://code.launchpad.net/~nskaggs/reminders-app/workaround-1347905/+merge/228213 a quick peek? It's basically skip the tests that involve faking the account and services in an effort to get reminders onto the dashboard17:34
balloonsI'd like to land this today so we can get it on the dash and monitored17:34
balloonsthings keep breaking out from under us17:34
mzanettiballoons: interesting... for me it fails exactly the other way round17:36
mzanettiballoons: not that code snippet, but creating accounts etc17:36
mzanettiballoons: do you want to disable test_go_to_account_settings everywhere?17:37
balloonsmzanetti, yea.. I'm giving up on getting it to work on the device as well17:37
balloonssadly had these been running this month, we would have been informed when they broke17:38
mzanettiballoons: added a small comment17:38
mzanettiballoons: not saying you need to fix it. If you're still sure this is how you want it, I'm ok with it17:39
balloonsideally I want everything to work, but yes, I'd like it to get in today, so having some of the tests running is better than nothing :-)17:39
balloonsmzanetti, replied. thanks17:41
Diogohi guys21:09
=== salem_ is now known as _salem

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