[PR #472] [MERGED] fix: do not override XDG_DATA_HOME/XDG_CONFIG_HOME if they are already set #785

Open
opened 2025-10-09 16:57:12 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pocket-id/pocket-id/pull/472
Author: @ItalyPaleAle
Created: 4/22/2025
Status: Merged
Merged: 4/22/2025
Merged by: @kmendell

Base: mainHead: fix-env-vars-override


📝 Commits (1)

  • 925b1df fix: do not override XDG_DATA_HOME/XDG_CONFIG_HOME if they are already set

📊 Changes

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

View changed files

📝 scripts/docker/entrypoint.sh (+2 -2)

📄 Description

Patch would break setups like mine in case the env vars are defined already.


🔄 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/pocket-id/pocket-id/pull/472 **Author:** [@ItalyPaleAle](https://github.com/ItalyPaleAle) **Created:** 4/22/2025 **Status:** ✅ Merged **Merged:** 4/22/2025 **Merged by:** [@kmendell](https://github.com/kmendell) **Base:** `main` ← **Head:** `fix-env-vars-override` --- ### 📝 Commits (1) - [`925b1df`](https://github.com/pocket-id/pocket-id/commit/925b1df3f9d97b3058b0c2601710bf24c9546571) fix: do not override XDG_DATA_HOME/XDG_CONFIG_HOME if they are already set ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `scripts/docker/entrypoint.sh` (+2 -2) </details> ### 📄 Description Patch would break setups like mine in case the env vars are defined already. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2025-10-09 16:57:12 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/pocket-id-pocket-id-2#785