Skip to content

Remove DETERMINISTIC setting#26648

Open
sbc100 wants to merge 1 commit intoemscripten-core:mainfrom
sbc100:remove_DETERMINISTIC
Open

Remove DETERMINISTIC setting#26648
sbc100 wants to merge 1 commit intoemscripten-core:mainfrom
sbc100:remove_DETERMINISTIC

Conversation

@sbc100
Copy link
Copy Markdown
Collaborator

@sbc100 sbc100 commented Apr 7, 2026

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: #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
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 87a404d to 8deca85 Compare April 12, 2026 06:31
@sbc100 sbc100 marked this pull request as ready for review April 12, 2026 06:31
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

1 participant