Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Warn when setting used on the command line more than once (#21464)" #21535

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Mar 14, 2024

This reverts commit 696b4f2.

After some discussion we decided it was best not to issue a warning here, since many folks
expect later flag to silently override earlier ones.

@sbc100 sbc100 requested review from kripken and dschuff March 14, 2024 20:43
@sbc100 sbc100 enabled auto-merge (squash) March 19, 2024 00:09
@sbc100 sbc100 merged commit 8236e54 into emscripten-core:main Mar 19, 2024
29 checks passed
@sbc100 sbc100 deleted the revert branch March 19, 2024 01: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.

2 participants