BUG: Remove accidentally committed CTest artifacts - #2021
Open
hjmjohnson wants to merge 1 commit into
Open
Conversation
Examples/Testing/Temporary/CTestCostData.txt and LastTest.log are transient CTest outputs, swept into d9b0d68 ("ENH: Quiet option.") alongside ten unrelated source changes. CTestCostData.txt holds only "---" and LastTest.log records an empty Nov 2015 run. Neither is referenced by the build system, and both are rewritten in the build tree on every ctest invocation. Ignore **/Testing/Temporary/ so the pattern cannot recur. The leading **/ is required: a pattern with an interior slash is anchored to the repository root and would miss Examples/Testing/Temporary/.
hjmjohnson
force-pushed
the
bug-remove-committed-ctest-artifacts
branch
from
August 22, 2026 16:12
7c3dc4c to
2bfc660
Compare
hjmjohnson
marked this pull request as ready for review
August 22, 2026 16:12
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Removes two transient CTest outputs that were accidentally committed to the source tree —
Examples/Testing/Temporary/CTestCostData.txtandLastTest.log— and adds a.gitignorerule so they cannot come back.Why these are accidental, not intentional fixtures
Both files entered the tree in d9b0d68 ("ENH: Quiet option."), a commit whose subject and remaining ten files are unrelated source changes —
antsRegistration.cxx,antsApplyTransforms.cxx,antsMotionCorr.cxx,itkantsRegistrationHelper.h/.hxx, and others. The signature of agit add -Asweep.Supporting evidence:
CTestCostData.txtcontains only---.LastTest.logrecords a test run with no tests:Start testing: Nov 12 21:41 PST/End testing: Nov 12 21:41 PST.*.cmake,CMakeLists.txt,*.sh, or*.pyin the repo mentions either path.Testing/Temporaryon every invocation, so the tracked copies are shadowed and never read.Why the ignore pattern needs the leading
**/The rule added is:
A gitignore pattern containing a slash anywhere other than at the end is anchored to the repository root. Writing
Testing/Temporary/would therefore match only<root>/Testing/Temporary/and would not matchExamples/Testing/Temporary/— the very path this PR removes. Verified withgit check-ignore -vagainst a probe file at that depth before and after the fix.Note also that
.gitignorehas no effect on files already tracked, which is why these two survived in the tree; untracking them is what makes the rule effective.