Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Follow the Freedesktop spec for thumbnails location #30

Merged
merged 1 commit into from
Jun 17, 2013
Merged

Follow the Freedesktop spec for thumbnails location #30

merged 1 commit into from
Jun 17, 2013

Conversation

evanpurkhiser
Copy link
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
Copy link
Member

daf commented Jun 17, 2013

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
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
Copy link
Member

daf commented Jun 17, 2013

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