Skip to content

fix(docker): always provide apps-writable#1051

Merged
susnux merged 1 commit into
mainfrom
fix/apps
Jul 16, 2026
Merged

fix(docker): always provide apps-writable#1051
susnux merged 1 commit into
mainfrom
fix/apps

Conversation

@susnux

@susnux susnux commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Simplify the logic for custom apps folder:
Always use the apps folder just for the shipped apps, and use the custom apps_writable folder for custom mounted apps.

This should resolve issues in some cases like on server when mounting the whole apps folder but sometimes not cloning apps.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@susnux susnux requested a review from max-nextcloud July 16, 2026 19:22
@susnux susnux added 3. to review Waiting for reviews bug Something isn't working labels Jul 16, 2026
susnux added a commit to nextcloud/server that referenced this pull request Jul 16, 2026
ref: nextcloud-libraries/nextcloud-e2e-test-server#1051

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Simplify the logic for custom apps folder:
Always use the `apps` folder just for the shipped apps,
and use the custom `apps-writable` folder for custom
mounted apps.

This should resolve issues in some cases like on server when mounting
the whole `apps` folder but sometimes not cloning apps.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux changed the title fix(docker): always provide apps_writable fix(docker): always provide apps-writable Jul 16, 2026
@susnux

susnux commented Jul 16, 2026

Copy link
Copy Markdown
Contributor Author

(fixed tests and adjusted to suggestion of @kesselb to apps-writable (- vs _))

@susnux susnux merged commit 1efca0e into main Jul 16, 2026
10 checks passed
@susnux susnux deleted the fix/apps branch July 16, 2026 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants