Skip to content

[stable-4.0] fix(filesystem): deal with win32 long paths when recursively removing folders - #9639

Merged
nilsding merged 1 commit into
stable-4.0from
backport/9634/stable-4.0
Mar 17, 2026
Merged

[stable-4.0] fix(filesystem): deal with win32 long paths when recursively removing folders#9639
nilsding merged 1 commit into
stable-4.0from
backport/9634/stable-4.0

Conversation

@backportbot

@backportbot backportbot Bot commented Mar 17, 2026

Copy link
Copy Markdown

Backport of PR #9634

… folders

`std::filesystem::remove` etc. seem to behave the same way as the pure
win32-APIs: sometimes long paths work, sometimes they won't.  In this
case it seems to be able to deal with long UNC-style paths just fine ...

Resolves #9293

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
@nilsding
nilsding enabled auto-merge March 17, 2026 10:25
@github-actions

Copy link
Copy Markdown
Contributor

Artifact containing the AppImage: nextcloud-appimage-pr-9639.zip

Digest: sha256:bcc514153d62b0d662dd04b42d268e5181c242bbd13bea8a63d0b9c2ba3c432d

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@nilsding
nilsding merged commit 899f395 into stable-4.0 Mar 17, 2026
19 checks passed
@nilsding
nilsding deleted the backport/9634/stable-4.0 branch March 17, 2026 11:04
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Maintainability Rating on New Code (required ≥ A)
1 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment