Skip to content

Fix Windows expr syntax error in CORE/MEM comparison#6932

Merged
smlambert merged 1 commit intoadoptium:masterfrom
annaibm:addMissingWindo
Mar 11, 2026
Merged

Fix Windows expr syntax error in CORE/MEM comparison#6932
smlambert merged 1 commit intoadoptium:masterfrom
annaibm:addMissingWindo

Conversation

@annaibm
Copy link
Copy Markdown
Contributor

@annaibm annaibm commented Mar 10, 2026

  • Update PowerShell command to take int

closes: #6711

@annaibm
Fix Windows expr syntax error in CORE/MEM comparison
dd8999c 
- Update  PowerShell command to take int

closes: adoptium#6711
Signed-off-by: Anna Babu Palathingal <anna.bp@ibm.com>
@annaibm
Copy link
Copy Markdown
Contributor Author

annaibm commented Mar 10, 2026

Before fix (Grinder #58867):

 set TEST_FLAG to JFR
13:25:09  Running testList ...
13:25:09  There are 2 test targets in testList
13:25:09  "C:/Users/jenkins/workspace/Grinder_testList_1/jdkbinary/j2sdk-image\\bin\\java" -version
13:25:10  java version "17.0.19-beta" 2026-04-21
13:25:10  IBM Semeru Runtime Certified Edition 17.0.19+5-202603092245 (build 17.0.19-beta+5-202603092245)
13:25:10  Eclipse OpenJ9 VM 17.0.19+5-202603092245 (build 17.0.19-beta+5-202603092245-master-205f6016d0, JRE 17 Windows 11 amd64-64-Bit Compressed References 20260309_1189 (JIT enabled, AOT enabled)
13:25:10  OpenJ9   - 205f6016d0
13:25:10  OMR      - 189e2e54c
13:25:10  JCL      - 9518b8d950 based on jdk-17.0.19+5)
13:25:11  make[3]: Entering directory '/cygdrive/c/Users/jenkins/workspace/Grinder_testList_1/aqa-tests'
13:25:12  make[4]: Entering directory '/cygdrive/c/Users/jenkins/workspace/Grinder_testList_1/aqa-tests/openjdk'
13:25:16  expr: non-integer argument
13:25:16  expr: syntax error: missing argument after '>'
13:25:17

After fix:

  • Windows (Grinder #58880):✅ No expr errors, test execution proceeds normally
 Running testList ...
15:41:28  There are 2 test targets in testList
15:41:28  "C:/Users/jenkins/workspace/Grinder_testList_1/jdkbinary/j2sdk-image\\bin\\java" -version
15:41:30  java version "17.0.19-beta" 2026-04-21
15:41:30  IBM Semeru Runtime Certified Edition 17.0.19+5-202603092245 (build 17.0.19-beta+5-202603092245)
15:41:30  Eclipse OpenJ9 VM 17.0.19+5-202603092245 (build 17.0.19-beta+5-202603092245-master-205f6016d0, JRE 17 Windows Server 2022 amd64-64-Bit Compressed References 20260309_1189 (JIT enabled, AOT enabled)
15:41:30  OpenJ9   - 205f6016d0
15:41:30  OMR      - 189e2e54c
15:41:30  JCL      - 9518b8d950 based on jdk-17.0.19+5)
15:41:30  make[3]: Entering directory '/cygdrive/c/Users/jenkins/workspace/Grinder_testList_1/aqa-tests'
15:41:31  make[4]: Entering directory '/cygdrive/c/Users/jenkins/workspace/Grinder_testList_1/aqa-tests/openjdk'
15:41:35  
15:41:35  ===============================================
  • s390x, AIX, Mac (Grinder #58881): ✅ No regressions on non-Windows platforms

@annaibm annaibm marked this pull request as ready for review March 10, 2026 21:32
smlambert
smlambert approved these changes Mar 11, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @annaibm !

@smlambert smlambert merged commit 64ee832 into adoptium:master Mar 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@karianna karianna karianna approved these changes

@smlambert smlambert smlambert approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

syntax error: missing argument after '>' in parallel runs on windows

3 participants

@annaibm @karianna @smlambert