Skip to content

Run test_unistd_isatty under deno. NFC#26601

Merged
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:deno_fix
Apr 15, 2026
Merged

Run test_unistd_isatty under deno. NFC#26601
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:deno_fix

Conversation

@sbc100
Copy link
Copy Markdown
Collaborator

@sbc100 sbc100 commented Apr 1, 2026

The upstream issue has now been fixed: denoland/deno#32995

The upstream issue has now been fixed: denoland/deno#32995
@sbc100 sbc100 requested a review from brendandahl April 1, 2026 20:55
@brendandahl
Copy link
Copy Markdown
Collaborator

I guess this hasn't made it into the deno release yet?

@sbc100 sbc100 merged commit b9e9469 into emscripten-core:main Apr 15, 2026
28 of 38 checks passed
@sbc100 sbc100 deleted the deno_fix branch April 15, 2026 23:11
@brendandahl
Copy link
Copy Markdown
Collaborator

Haven't we disabled deno testing? Do we know if this fix made it into deno yet?

@brendandahl
Copy link
Copy Markdown
Collaborator

Oh I see we ran test-deno above.

@sbc100
Copy link
Copy Markdown
Collaborator Author

sbc100 commented Apr 15, 2026

Oh I see we ran test-deno above.

Ah.. yes, that as not deliberate.. But it worked because I did not rebase this branch so when i did "re-run all tests" it ran all the tests that were enabled when I created this branch. Lucky.

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.

tty.isatty() reports wrong results for opened files

2 participants