Skip to content

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

Merged
nilsding merged 1 commit intostable-33.0from
backport/9634/stable-33.0
Mar 17, 2026
Merged

[stable-33.0] fix(filesystem): deal with win32 long paths when recursively removing folders#9640
nilsding merged 1 commit intostable-33.0from
backport/9634/stable-33.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-9640.zip

Digest: sha256:fdcb8cfe859b184a7e128b02e6b49109662766c230c018ab68fed410b974143a

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.

@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

@nilsding nilsding merged commit 14876c3 into stable-33.0 Mar 17, 2026
20 of 22 checks passed
@nilsding nilsding deleted the backport/9634/stable-33.0 branch March 17, 2026 11:59
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