Comment 5 for bug 1276658

For the record:

2014-08-15 18:17:31 ybon jdstrand: QtLocation seems to store cache in QStandardPaths::GenericCacheLocation is that a constant familiar for you?
2014-08-15 18:17:45 ybon cf http://qt-project.org/doc/qt-5/qstandardpaths.html
2014-08-15 18:18:46 jdstrand ybon: I am going to point you at kalikiana. he made sure that all this worked right in other parts of the toolkit
2014-08-15 18:19:20 ybon jdstrand: thanks :)
2014-08-15 18:19:26 ybon kalikiana: around by chance? :)
2014-08-15 18:19:45 jdstrand ybon: can you paste the path that QtLocation is using which is getting denied by apparmor for kalikiana?
2014-08-15 18:19:54 ybon sure
2014-08-15 18:20:09 ybon for reference, the line in the QtLocation I'm refering to http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/utopic/qtlocation-opensource-src/utopic-proposed/view/head:/src/location/maps/qgeotilecache.cpp#L119
2014-08-15 18:21:12 ybon kalikiana, jdstrand: here is the path /home/phablet/.cache/QtLocation/
2014-08-15 18:22:45 jdstrand kalikiana: I've not looked personally, but it appears QtLocation may not be honoring applicationName or similar
2014-08-15 18:22:53 jdstrand ybon: you are using location-service?
2014-08-15 18:23:11 ybon jdstrand: I'm not sure what you call location-service
2014-08-15 18:23:13 jdstrand well, I think QtLocation is supposed to use location-service
2014-08-15 18:23:16 ybon I'm using a QtMap object
2014-08-15 18:23:18 ybon with a plugin
2014-08-15 18:23:49 jdstrand kalikiana: might need to ask tvoss if you are curious about location-service
2014-08-15 18:23:50 ybon here is the code https://github.com/yohanboniface/OSMTouch/blob/master/OSMTouch.qml#L80
2014-08-15 18:28:07 kalikiana ybon: that code is technically correct, xdg_cache_home is shared on the desktop - jdstrand I can't check right now, is it set on the phone? It's expected that no app name is passed but it must be set by our launcher I think
2014-08-15 18:29:33 jdstrand kalikiana: I'm not sure. ybon asked me about the denial and I said that the denial should not be fixed and that applicationName was probably not correctly set.
2014-08-15 18:29:49 ybon I sware it's set :)
2014-08-15 18:29:55 jdstrand right
2014-08-15 18:29:57 ybon https://github.com/yohanboniface/OSMTouch/blob/master/OSMTouch.qml#L24
2014-08-15 18:30:06 jdstrand which is why I pointed you at kalikiana
2014-08-15 18:30:09 ybon maybe it's not done the proper way, though
2014-08-15 18:30:33 jdstrand that looks ok to me
2014-08-15 18:30:51 jdstrand ybon: does it use an C++ or is it all qml?
2014-08-15 18:30:58 ybon all qml
2014-08-15 18:31:08 jdstrand yeah, that is all you should have to do
2014-08-15 18:31:11 jdstrand kalikiana: ^
2014-08-15 18:31:15 ybon I've no C++ licence yet
2014-08-15 18:31:43 jdstrand ybon: maybe file a bug and ping kalikiana so he can look at it when he has time?
2014-08-15 18:32:18 ybon kalikiana: I'm not sure to get you well, are you saying that ~/.cache is the expected final path? or that the QtLocation code is correct, but the QStandardPaths::GenericCacheLocation value is missing or something?
2014-08-15 18:32:41 ybon this is the related issue on OSMTouch https://bugs.launchpad.net/osmtouch/+bug/1276658
2014-08-15 18:32:43 ubot5 Ubuntu bug 1276658 in OSMTouch "Add offline cache" [Undecided,Confirmed]
2014-08-15 18:35:13 kalikiana ybon: jdstrand what I mean is that the code in qtlocation is correct and that xdg_cache_home should contain the app specific path already - otherwise all existing code using QStandardPaths::GenericCacheLocation would have to be ifdeffed
2014-08-15 18:35:42 kalikiana Because the semantics on the desktop are different
2014-08-15 18:35:51 ybon kalikiana: should I do something on my app to get xdg_cache_home set?
2014-08-15 18:36:39 kalikiana Ybon no, definitely not (unless you want a work-around for now)
2014-08-15 18:36:57 jdstrand this seems to be working ok for non-QtLocation though, otherwise we'd have massive breakage on touch
2014-08-15 18:37:33 ybon depending on how this can be fixed, I'd be happy to have a workaround, if you know some, kalikiana, thanks :)
2014-08-15 18:37:50 ybon but if there is an easy proper fix, no need of course
2014-08-15 18:39:36 kalikiana ybon is your app available somewhere? I can take a closer look on Monday
2014-08-15 18:39:58 kalikiana Meanwhile please file a bug and assign to me
2014-08-15 18:40:06 ybon kalikiana: sure, it's on the store, name OSMTouch, and the code is here https://github.com/yohanboniface/OSMTouch/blob/master/OSMTouch.qml#L24
2014-08-15 18:40:17 ybon ok, thanks!
2014-08-15 18:41:02 ybon kalikiana: you should be assigned on https://bugs.launchpad.net/osmtouch/+bug/1276658 thanks a lot :)
2014-08-15 18:41:04 ubot5 Ubuntu bug 1276658 in OSMTouch "Add offline cache" [Undecided,Confirmed]