[PR #952] Skip writing client config if it is not writable #1101

New issue

Open

opened 2026-02-28 14:54:27 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 14:54:27 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Rigellute/spotify-tui/pull/952
Author: @amarshall
Created: 2/23/2022
Status: 🔄 Open

Base: master ← Head: handle-readonly-client-config

📝 Commits (1)

0bb19e4 Skip writing client config if it is not writable

📊 Changes

1 file changed (+3 additions, -2 deletions)

View changed files

📝 src/config.rs (+3 -2)

📄 Description

In some cases, the config file may be read-only. Since it is not
imperative that it be written (everything continues to work, it just
will not remember this device ID after a restart), just skip writing.

Ideally static and dynamic configuration should be totally separated,
however that is more complicated to implement and this is a quick fix.

Partial workaround for https://github.com/Rigellute/spotify-tui/issues/579.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Rigellute/spotify-tui/pull/952 **Author:** [@amarshall](https://github.com/amarshall) **Created:** 2/23/2022 **Status:** 🔄 Open **Base:** `master` ← **Head:** `handle-readonly-client-config` --- ### 📝 Commits (1) - [`0bb19e4`](https://github.com/Rigellute/spotify-tui/commit/0bb19e430a593fb838b47737a52058cbf528e5b1) Skip writing client config if it is not writable ### 📊 Changes **1 file changed** (+3 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/config.rs` (+3 -2) </details> ### 📄 Description In some cases, the config file may be read-only. Since it is not imperative that it be written (everything continues to work, it just will not remember this device ID after a restart), just skip writing. Ideally static and dynamic configuration should be totally separated, however that is more complicated to implement and this is a quick fix. Partial workaround for https://github.com/Rigellute/spotify-tui/issues/579. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>