Change the default cache path

Open
#67 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
35/100
Issue type
Feature
Clarity
Clearly specified
Activity status
Stale
Tech stack
objective-c
Domain
mobile

Research direction

Open Sources/SPTPersistentCacheOptions.m at line 51 and inspect how the default cache path is assembled. Update the default location to the proposed generic path when the project is ready for the version 2.0 breaking release; done means the default path no longer uses the old cache-specific component.

Written by the indexing model from the issue text.

Description

We should change the default cache location the next time we release a breaking update. I think the earliest we can do that is with version 2.0 (which is a far ways off, not even planned).

Currently the path used is the temporary directory appended with a path component that is very specific.

[NSTemporaryDirectory() stringByAppendingPathComponent:@"/com.spotify.temppersistent.image.cache"];

(SPTPersistentCacheOptions.m#L51)

It would be better to set it to a more generically named path, such as:

[NSTemporaryDirectory() stringByAppendingPathComponent:@"/com.spotify.persistentcache.default"];
Dominant language
Objective-C
Stars
1.2k
Forks
74
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from spotify/SPTPersistentCache

All issues in spotify/SPTPersistentCache

Similar issues

More Objective-C issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.