GeezerVison is showing github.com — Remove Circle CI and dashboard branch by o01eg · Pull Request #447 · python-cmake-buildsystem/python-cmake-buildsystem
Conversation
o01eg
commented
May 24, 2026
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Tweak "python_common.cmake" script to fix the error reported
below. Since "ctest_empty_binary_directory" returns:
(1) true if the directory does not exist
(3) false if the directory exists and does NOT contain a CMakeCache.txt
this commit moves the directory creation after the "ctest_empty_binary_directory"
call.
//------------
Clearing build tree...
CMake Error at scratch/python_common.cmake:306 (ctest_empty_binary_directory):
ctest_empty_binary_directory problem removing the binary directory:
C:\projects\python-cmake-buildsystem/../python-cmake-buildsystem/build
Call Stack (most recent call first):
scratch/appveyor_dashboard.cmake:31 (include)
//------------
…riable
If not set, it default to 8.
In both case build name will be prepended with "2.7.<PY_VERSION_PATCH>-".
…est-in-build-name
Indicate pull request source in CDash Build Name.
If the last commit topic includes the special comments of the form
"[cmake <ARGUMENT>]" without trailing spaces, the project will be
configured using all extracted arguments.
For example, using these comments:
[cmake -DFOO:STRING=Hello]
[cmake -DBAR:STRING=World]
[cmake -DANSWER=42 -DGOOD:STRING=Day]
The project will be built with -DFOO:STRING=Hello -DBAR=World -DANSWER=42 -DGOOD:STRING=Day
This commit will ensure CI builds (which are of type Experimental) do not
reset the source checkout to origin/master
…e < 3.1.0
This commit improve the fix introduced in 77f3479 to also work for
older version of CMake.
As explained in [1], for legacy reason setting variable like
LD_LIBRARY_PATH with paths containing spaces will not work even if
the spaces are escaped. For that reason, prefer build path without
spaces.
[1] http://ubuntuforums.org/showthread.php?t=1142062
This commit fixes the following error:
//------------
ctest -V -C %configuration% -S "%APPVEYOR_BUILD_FOLDER%\\scripts\\appveyor_dashboard.cmake"
CMake Error at scripts/appveyor_dashboard.cmake:10 (if):
if given arguments:
"STREQUAL" ""
Unknown arguments specified
Error in read script: C:/projects/python-cmake-buildsystem/scripts/appveyor_dashboard.cmake
Command exited with code -1
//------------
Default remains "Visual Studio 9 2008"
This is not required since the dashboard branch is checked out
within circle.yml
Follow-up of 6391889 ("cmake: Remove support for building CPython 2.7", 2025-04-28)
in which `appveyor.yml` was removed.
Since minimum required CMake version was updated in fbc3a81 ("Update minimum required CMake version from 2.8.6 to 3.13.5", 2022-01-03),
this commit partially reverts 7e11eb9 ("Ensure Experimental build do not reset the local repo when using CMake < 3.1.0", 2015-09-03)
CDash misinterprets colons (:) in the `CTEST_SITE` string, causing the
"Update" and "Configure/Build/Test" steps to appear as separate rows
in the dashboard.
This patch replaces colons with underscores as a workaround to ensure
all submission parts are grouped correctly.
Related issue: Kitware/CDash#2905
Make the build directory after using ctest_empty_binary_directory
because the command reports an error if asked to delete a directory
that exists but does not contain `CMakeCache.txt`.
Adapted from cmake/dashboard-scripts@cb56d71 ("cmake_common: Fix use of ctest_empty_binary_directory", 2016-01-21)
See https://gitlab.kitware.com/cmake/dashboard-scripts/-/commit/cb56d71
Since the submission is done by part, this hook is irrelevant.
Uh oh!
There was an error while loading. Please reload this page.
Assignees
No one assigned
Milestone
No milestone
Development
Successfully merging this pull request may close these issues.
Uh oh!
There was an error while loading. Please reload this page.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.