Skip to content

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

Merged
nilsding merged 1 commit intostable-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 intostable-4.0from
backport/9634/stable-4.0

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Mar 17, 2026

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

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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant