Skip to content

Fix symlink support in IDBFS#26408

Merged
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:test_idbfs_autopersist
Mar 7, 2026
Merged

Fix symlink support in IDBFS#26408
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:test_idbfs_autopersist

Conversation

@sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Mar 6, 2026

Previously symlinks would be followed when saving to IBDFS and then
restored as essentially duplicates of the files they pointed to, which is
not correct.

Fixes: #26407

@sbc100 sbc100 requested review from brendandahl and kripken March 6, 2026 23:10
@sbc100 sbc100 force-pushed the test_idbfs_autopersist branch from 8ec9f49 to b9f1a1d Compare March 6, 2026 23:33
@sbc100
Copy link
Collaborator Author

sbc100 commented Mar 6, 2026

Actually it turned out that the fix wasn't too hard.. so this change now just fixes the issue!

@sbc100 sbc100 changed the title Update test_idbfs_autopersist regarding symlinks. NFC Fix symlink support in IDBFS Mar 6, 2026
@sbc100 sbc100 force-pushed the test_idbfs_autopersist branch 2 times, most recently from 0952577 to a78f496 Compare March 6, 2026 23:35
Previously symlinks would be followed when saving to IBDFS and then
restored as essentially duplicates of the files they pointed to.

Fixes: emscripten-core#26407
@sbc100 sbc100 force-pushed the test_idbfs_autopersist branch from a78f496 to ed47309 Compare March 6, 2026 23:41
@sbc100 sbc100 requested review from brendandahl and juj March 6, 2026 23:41
@sbc100
Copy link
Collaborator Author

sbc100 commented Mar 7, 2026

Landing with actual fix TBR since this is currently blocking #26398

@sbc100 sbc100 merged commit 81b238d into emscripten-core:main Mar 7, 2026
37 checks passed
@sbc100 sbc100 deleted the test_idbfs_autopersist branch March 7, 2026 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

IDBFS does not correctly store symlinks

2 participants