=== maclin1 is now known as maclin
=== maclin1 is now known as maclin
[09:57] <sil2100> Saviq: ping
[09:58] <Saviq> sil2100, hey
[09:58] <davmor2> Saviq: I'll tell you it's all your fault ;)
=== dandrader is now known as dandrader|afk
[15:11] <pstolowski> mterry, hey, can you take a look at https://code.launchpad.net/~stolowski/unity8-session-snap/thumbnailer-fixes/+merge/310756 ?
[15:12] <mterry> pstolowski: looking, thanks
[15:30] <pstolowski> mterry, thanks!
[15:45] <pete-woods> Saviq: hey. I'm planning to add a new indicator widget for etheret links, I just wanted to check if there were any big MRs is progress for that stuff?
[15:45] <pete-woods> (doing the touch mode / desktop mode change)
[15:45] <pete-woods> (or otherwise)
[15:46] <mterry> pstolowski: so to see the bug before I test your MP, I need a scope that uses thumbnails.  Does the normal Music scope do that?  I'm seeing a Music scope with album art from 7digital
[15:46] <mterry> pstolowski: or wikipedia?
[15:47] <mterry> scope
[15:47] <Saviq> pete-woods, yeah, please coord with Trevinho
[15:47] <pete-woods> Saviq: right, I'm assuming that's a "yes" then :)
[15:48] <Saviq> yes
[15:51] <pete-woods> Saviq: in lieu of a reply yet, does this look like the right MR? https://code.launchpad.net/~3v1n0/ubuntu-settings-components/touch+pointer-styles/+merge/307447
[15:52] <Saviq> pete-woods, yes, you can base on that
[15:52] <pete-woods> Saviq: cool, that's really all need (I think)
[15:52] <pete-woods> the change looks intelligable enough
[15:53] <Saviq> pete-woods, can use https://bileto.ubuntu.com/#/ticket/2195 for reference, too
[15:53] <pete-woods> Saviq: this is planned to land as soon as it's ready, right?
[15:56] <pstolowski> mterry, the only scopes that use image://thumbnailer/ uris for art are music and videos and the problem would only be visible if you had local media files, which I'm not sure is possible to have atm with unity8 snap (due to mediascanner). I noticed the problem with the unity-scope-fake which uses a bunch of uris like this. so for testing i installed it in the unity8 snap
[15:57] <mterry> pstolowski: hmm, I think we have a fixed mediascanner in silo 2129, though I haven't tested local media.  Let me try that...
[15:58] <pstolowski> mterry, ah, cool, i didn't know that
[15:58] <mterry> pstolowski: could use a review: https://code.launchpad.net/~mterry/mediascanner2/snap-root/+merge/310602
[15:58] <mterry> I don't know if that fixes everything...  but it lets it run anyway
[16:00] <Saviq> pete-woods, yeah, it's already under britney
[16:00] <pstolowski> mterry, cool. i'm not familiar with mediascanner code, but that looks simple enough
[16:00] <pete-woods> Saviq: awesome. will stop bugging you now :)
[16:00] <mterry> pstolowski: heh, you mentioned it by name, so I assumed you were an expert  :P
[16:00] <pstolowski> lol
[16:03] <pstolowski> mterry, i'm pretty sure the problems I found with mime & gdk loaders are genuine. I added some debug to thumbnailer code and found out it considered PNGs to be "text/plain" (and ignored them); after fixing mime problem it failed when processing them clearly pointing at gdk pixbuf modules missing
[16:04] <pstolowski> mterry, only thing I'm wondering about is what it didn't break any other stuff (especially the lack of mime data)
[16:04] <mterry> pstolowski: yeah doing some digging, I *think* query-loader is run by dpkg hooks.  And I suspect the mime stuff is too.  So it does seem like something we should do.  But couldn't we also do query-loader like we do the mime stuff?  (i.e. as a static cache file in the snap, instead of HOME?)
[16:07] <pstolowski> mterry, the generated loaders.cache file has absolute target paths in it (e.g. "/usr/lib/x86_64-linux-gnu/gdk-pixbuf-2.0/2.10.0/loaders/libpixbufloader-pnm.so")
[16:09] <pstolowski> which makes me wonder if it's not going to break when snap is upgraded and the version in home is only generated once
[16:15] <mterry> pstolowski: ugh yeah
[16:30] <mterry> pstolowski: I'm testing my suggested changes to the mime updating code...  I can hit that bug by just noticing that mediascanner ignores my local music (wrong mime type)
[16:40] <pstolowski> mterry, ah, good
[16:41] <pstolowski> mterry, we could generate loaders.cache during the build, then (cough) postprocess it with sed..
[16:41] <mterry> pstolowski: during run time?
[16:42] <pstolowski> mterry, ah, gosh, you're right, we only know the final path at runtime. so no
[16:42] <mterry> I mean... we do do that with the fonts file...
[16:43] <mterry> It's not pretty
[16:43] <pstolowski> eek
[16:44] <pstolowski> mterry, i can't top-approve https://code.launchpad.net/~mterry/mediascanner2/snap-root/+merge/310602; apparently i'm community ;) . will need to ask jamesh
[16:47] <pstolowski> mterry, i'm about eod. i'll continue working on this tomorrow morning to get rid of custom plugin and use postprocess.mk instead; not sure about gdk loaders yet, need to think about it
=== dandrader|afk is now known as dandrader
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== jjb is now known as oao