Skip to content

Sync run.sh and run.cmd options with run.py#125511

Draft
Copilot wants to merge 4 commits intomainfrom
copilot/sync-run-options
Draft

Sync run.sh and run.cmd options with run.py#125511
Copilot wants to merge 4 commits intomainfrom
copilot/sync-run-options

Conversation

Copy link
Contributor

Copilot AI commented Mar 12, 2026

Description

run.sh and run.cmd were missing several options accepted by run.py, and run.sh had a bug where it passed --tieringtest to run.py which only recognizes --tiering_test (causing the argument to be silently ignored or fail). Additionally, several flags accepted by both scripts were missing from their help menus, and one option was intentionally hidden.

run.sh

  • Bug fix: --tieringtest--tiering_test when forwarding to run.py
  • Added: --parallel=<type> option (none, collections, assemblies, all) — already present in run.cmd, missing from run.sh
  • Help fix: Added description for --limitedDumpGeneration (previously had an empty description)

run.cmd

  • Added: verbose — passes --verbose to run.py; already in run.sh, missing from run.cmd
  • Added: limitedDumpGeneration — passes --limited_core_dumps to run.py; already in run.sh, missing from run.cmd
  • Help fix: Added tieringtest, runnativeaottests, and runlargeversionbubblecrossgen2tests to the help menu — they were accepted by the argument parser but not documented
  • Help fix: Fixed alignment of verbose, limitedDumpGeneration, interpreter, and node entries in the help section
  • Removed: "intentionally undocumented" comment for runlargeversionbubblecrossgen2tests; option is now documented as experimental

Changes

  • src/tests/run.sh
  • src/tests/run.cmd

Testing

Shell script syntax verified (bash -n). No build artifacts affected.

Original prompt

This section details on the original issue you should resolve

<issue_title>src/tests/run.[sh|cmd] doesn't show all options present in run.py</issue_title>
<issue_description>There are a number of flags that run.py accepts that are not in src/tests/run.sh or src/tests/run.cmd. We should make sure they are in sync.</issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

- Fix bug in run.sh: passes '--tieringtest' to run.py but run.py expects '--tiering_test'
- Add '--parallel=<type>' option to run.sh (was in run.cmd but missing from run.sh)
- Add 'verbose' option to run.cmd (was in run.sh but missing from run.cmd)
- Add 'limitedDumpGeneration' option to run.cmd (was in run.sh but missing from run.cmd)

Co-authored-by: jtschuster <36744439+jtschuster@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix run.sh and run.cmd to include all options from run.py Sync run.sh and run.cmd options with run.py Mar 12, 2026
Copilot AI requested a review from jtschuster March 12, 2026 22:08
@jtschuster
Copy link
Member

@copilot make sure all the flags that are accepted are in the help menus too.

…; fix empty description in run.sh

Co-authored-by: jtschuster <36744439+jtschuster@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 12, 2026

@copilot make sure all the flags that are accepted are in the help menus too.

Done in b79eb42:

  • Added tieringtest and runnativeaottests to the run.cmd help menu (they were accepted but undocumented)
  • Fixed the empty description for --limitedDumpGeneration in run.sh
  • Fixed alignment of the newly added entries in run.cmd's help section

@jtschuster
Copy link
Member

@copilot Add the runlargeversionbubblecrossgen2tests option and remove the "intentionally hidden" comment. Just leave a note that it's experimental. I don't see why we should have any undocumented options.

… comment

Co-authored-by: jtschuster <36744439+jtschuster@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 12, 2026

@copilot Add the runlargeversionbubblecrossgen2tests option and remove the "intentionally hidden" comment. Just leave a note that it's experimental. I don't see why we should have an...

Done in 2c43f5e — removed the "intentionally undocumented" comment and added runlargeversionbubblecrossgen2tests to the help menu marked as (Experimental).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status
Status: No status

Development

Successfully merging this pull request may close these issues.

src/tests/run.[sh|cmd] doesn't show all options present in run.py

2 participants