Skip to content

Remove DETERMINISTIC setting#26648

Merged
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:remove_DETERMINISTIC
Apr 21, 2026
Merged

Remove DETERMINISTIC setting#26648
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:remove_DETERMINISTIC

Conversation

@sbc100
Copy link
Copy Markdown
Collaborator

@sbc100 sbc100 commented Apr 7, 2026

This setting was marked as deprecated in #26653 and nobody on the mailing list showed interested in keeping it around.

Fixes: #26647

@sbc100 sbc100 marked this pull request as draft April 7, 2026 22:31
sbc100 added a commit to sbc100/emscripten that referenced this pull request Apr 8, 2026
sbc100 added a commit that referenced this pull request Apr 8, 2026
This does not remove anything, just generates a warning if the setting
is used.

See #26647 and #26648
@sbc100 sbc100 force-pushed the remove_DETERMINISTIC branch from 87a404d to 8deca85 Compare April 12, 2026 06:31
@sbc100 sbc100 marked this pull request as ready for review April 12, 2026 06:31
I'm not sure there are any users of this settings, but in case there
are I have left the `src/deterministic.js` file in place which can be
used via `--pre-js` instead.

Fixes: emscripten-core#26647
@sbc100 sbc100 force-pushed the remove_DETERMINISTIC branch from 8deca85 to 2ffe78e Compare April 21, 2026 20:11
@sbc100 sbc100 requested a review from kripken April 21, 2026 20:47
@sbc100 sbc100 merged commit 8351b0a into emscripten-core:main Apr 21, 2026
29 checks passed
@sbc100 sbc100 deleted the remove_DETERMINISTIC branch April 21, 2026 22:10
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.

Tracking issue for the deprecation and removal of -sDETERMINISTIC

2 participants