spotify / spotify/SPTPersistentCache
Change the default cache path
Nobody has claimed this yet.
- Dominant language
- Objective-C
- Stars
- 1.2k
- Forks
- 74
- PR merge metrics
- No merged PRs in 30d
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"];
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
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.
Assessment
- Tech stack
- objective-c
- Domain
- mobile
- Issue type
- Feature
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 35/100