Skip to content

Follow the Freedesktop spec for thumbnails location#30

Merged
daf merged 1 commit into
l3ib:masterfrom
evanpurkhiser:fixup/xdg-thumbnails-dir
Jun 17, 2013
Merged

Follow the Freedesktop spec for thumbnails location#30
daf merged 1 commit into
l3ib:masterfrom
evanpurkhiser:fixup/xdg-thumbnails-dir

Conversation

@evanpurkhiser

Copy link
Copy Markdown
Contributor

According to the Freedesktop specifications for thumbnails, the thumbnails directory should be stored in $XDG_CACHE_HOME/thumbnails, not in $HOME/.thumbnails. This fixes that.

According to the Freedesktop specifications for thumbnails [1], the
thumbnails directory should be stored in $XDG_CACHE_HOME/thumbnails, not
in $HOME/.thumbnails. This fixes that

[1]: http://specifications.freedesktop.org/thumbnail-spec/thumbnail-spec-latest.html#DIRECTORY
@daf

daf commented Jun 17, 2013

Copy link
Copy Markdown
Member

Wow. I SWEAR we used to have this at some point, but I can't find it in history anywhere (look at my comments in #5). The weird part is, on my system, it IS ~/.thumbnails/normal that has my cache from Nautilus, with pretty recent timestamps (from this year at least). Do you think it's distro dependent? (ubuntu here)

@evanpurkhiser

Copy link
Copy Markdown
Contributor Author

I believe the thumbnail specification was updated in May 2012 which is actually after your comment in #5.

I'm on Arch, but I'm pretty sure the Freedesktop spec should be distribution independent. Perhaps Nautilus doesn't yet follow the spec for thumbnail cache yet?

@daf

daf commented Jun 17, 2013

Copy link
Copy Markdown
Member

Looks like you're right, and my memory is pretty fuzzy. Thanks for the patch.

daf added a commit that referenced this pull request Jun 17, 2013
Follow the Freedesktop spec for thumbnails location
@daf daf merged commit 3c3b032 into l3ib:master Jun 17, 2013
@Stebalien Stebalien mentioned this pull request Jul 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants