Skip to content

Conversation

@hjmjohnson
Copy link
Contributor

@hjmjohnson hjmjohnson commented Dec 18, 2025

@hjmjohnson
Copy link
Contributor Author

@bradking I was unable to replicate this error on my local Windows VS 2022 build of this same codebase.

I am not familiar with how this system works, but it seems like the CI is giving a warning ab the use of "-c" flag that is not used in windows builds that is corrupting the expected output.

This is a failing test, even though I think that the code would working.

I'd love some advice in how to get this failing test to pass on the github CI.

Thanks,
Hans

  51/5242 Test   #51: cmd.fbracket-depth ...........................................***Failed    0.26 sec
CMake Error at D:/a/CastXMLSuperbuild/CastXMLSuperbuild/build/castxml-prefix/src/castxml/test/run.cmake:117 (message):
  result does not match that expected.

  stderr does not match that expected.

  Command was:

   command> "D:/a/CastXMLSuperbuild/CastXMLSuperbuild/build/castxml-prefix/src/castxml-build/bin/castxml.exe" "-fbracket-depth=300" "-std=c11" "-c" "D:/a/CastXMLSuperbuild/CastXMLSuperbuild/build/castxml-prefix/src/castxml/test/input/fbracket-depth.c"

  Expected result to match:

   expect-result> 1

  Expected stderr to match:

   expect-stderr> [^
   expect-stderr> ]*/test/input/fbracket-depth.c:[0-9:]+: fatal error: bracket nesting level exceeded maximum of 300

  Actual result:

   actual-result> Access violation

  Actual stdout:

   actual-stdout> 

  Actual stderr:

   actual-stderr> warning: argument unused during compilation: '-c' [-Wunused-command-line-argument]

@bradking
Copy link
Member

That test was added by CastXML/CastXML#219, and the failure looks much like the problem that change was meant to fix. Please check the -stack:... argument with which the superbuild is linking castxml.exe.

@bradking
Copy link
Member

Or, I wonder if GitHub Actions is somehow limiting the stack size of the process.

@hjmjohnson hjmjohnson marked this pull request as draft December 19, 2025 23:48
@hjmjohnson hjmjohnson changed the title ENH: Explicitly specify to run 1 test at a time WIP DO NOT MERGE ENH: Explicitly specify to run 1 test at a time Dec 19, 2025
@hjmjohnson hjmjohnson changed the title WIP DO NOT MERGE ENH: Explicitly specify to run 1 test at a time WIP DO NOT MERGE ENH: Update CastXML to fix MSVC Stack size issue Dec 19, 2025
@hjmjohnson
Copy link
Contributor Author

@thewtex @bradking This PR will need to be modified after the CastXML PR

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