Skip to content

Removes usage of Experimental (interactive) mode in ctest #892

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

Conversation

weslleyspereira
Copy link
Collaborator

The command ctest -D ExperimentalSubmit is not working properly in the Github Actions. See https://github.com/Reference-LAPACK/lapack/actions/runs/5588808844, and other recent runs failing at https://github.com/Reference-LAPACK/lapack/actions/workflows/cmake.yml. This PR removes the Experimental build and uses a regular CTest call for tests.

The Experimental mode is an interactive debug mode in CMake. "Prior to CMake 3.11, interactive mode on Windows allowed system debug popup windows to appear. Now, due to CTest's use of libuv to launch test processes, all system debug popup windows are always blocked." (https://cmake.org/cmake/help/latest/manual/ctest.1.html#cmdoption-ctest-interactive-debug-mode).

@weslleyspereira weslleyspereira force-pushed the try-disabling-experimental-ctest branch from e7d6aa4 to 7c7b380 Compare August 12, 2023 01:02
@codecov
Copy link

codecov bot commented Aug 12, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (47fd4fd) 0.00% compared to head (036ed5f) 0.00%.

❗ Current head 036ed5f differs from pull request most recent head 7c7b380. Consider uploading reports for the commit 7c7b380 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #892   +/-   ##
=======================================
  Coverage    0.00%    0.00%           
=======================================
  Files        1918     1918           
  Lines      188614   188614           
=======================================
  Misses     188614   188614           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@weslleyspereira weslleyspereira merged commit 9138a87 into Reference-LAPACK:master Aug 12, 2023
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