Skip to content

Conversation

@sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Nov 25, 2024

No description provided.

@sbc100 sbc100 force-pushed the also_with_wasmfs_and_rawfs branch 3 times, most recently from ce6fc7e to b5119cb Compare November 25, 2024 21:29
@sbc100 sbc100 force-pushed the also_with_wasmfs_and_rawfs branch 3 times, most recently from 124c977 to 8293f94 Compare November 25, 2024 22:04
Copy link
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd be ok with separately considering to remove FS.lchmod.

@sbc100 sbc100 force-pushed the also_with_wasmfs_and_rawfs branch from 8293f94 to 1b82cf3 Compare November 26, 2024 00:07
@sbc100 sbc100 force-pushed the also_with_wasmfs_and_rawfs branch from 1b82cf3 to fb81f62 Compare November 26, 2024 00:50
@sbc100 sbc100 merged commit 48b6742 into emscripten-core:main Nov 26, 2024
19 of 28 checks passed
@sbc100 sbc100 deleted the also_with_wasmfs_and_rawfs branch November 26, 2024 01:44
@brendandahl
Copy link
Collaborator

@sbc100 It looks like this is causing failures on windows https://chromium-review.googlesource.com/c/emscripten-releases/+/6051557

sbc100 added a commit to sbc100/emscripten that referenced this pull request Nov 26, 2024
This test depends on separate R/W/X permission bits that windows
lacks.

Followup to emscripten-core#23004
sbc100 added a commit that referenced this pull request Nov 26, 2024
This test depends on separate R/W/X permission bits that windows lacks.

Followup to #23004
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.

3 participants