https://github.com/dyung created https://github.com/llvm/llvm-project/pull/207476
PR #205586 refactored existing tests and created a new test `clang-tools-extra/test/clang-doc/html/long-name.cpp` which when run on Windows generates a file that exceeds the maximum path length, leading to the directory not being able to be easily deleted. From my bot: ``` Z:\b\llvm-clang-x86_64-sie-win>rmdir /S /Q build build\tools\clang\tools\extra\test\clang-doc\html\Output\long-name.cpp.tmp\html\GlobalNamespace\_ZTV243ThisStructHasANameThatResultsInAMangledNameThatIsExactly250CharactersLongThatIsSupposedToTestTheFilenameLengthLimitsWithinClangDocInOrdertoSeeifclangdocwillcrashornotdependingonthelengthofthestructIfTheLengthIsTooLongThenClangDocWillCrashAnd12.html - The system cannot find the path specified. build\tools\clang\tools\extra\test\clang-doc\html\Output\long-name.cpp.tmp\html\GlobalNamespace\_ZTV244ThisStructHasANameThatResultsInAMangledNameThatIsExactly251CharactersLongThatIsSupposedToTestTheFilenameLengthLimitsWithinClangDocInOrdertoSeeifclangdocwillcrashornotdependingonthelengthofthestructIfTheL29DE8558215A13A506661C0E01E50AA3E5C9C7FA.html - The system cannot find the path specified. build\tools\clang\tools\extra\test\clang-doc\html\Output\long-name.cpp.tmp\json\GlobalNamespace\_ZTV243ThisStructHasANameThatResultsInAMangledNameThatIsExactly250CharactersLongThatIsSupposedToTestTheFilenameLengthLimitsWithinClangDocInOrdertoSeeifclangdocwillcrashornotdependingonthelengthofthestructIfTheLengthIsTooLongThenClangDocWillCrashAnd12.json - The system cannot find the path specified. build\tools\clang\tools\extra\test\clang-doc\html\Output\long-name.cpp.tmp\json\GlobalNamespace\_ZTV244ThisStructHasANameThatResultsInAMangledNameThatIsExactly251CharactersLongThatIsSupposedToTestTheFilenameLengthLimitsWithinClangDocInOrdertoSeeifclangdocwillcrashornotdependingonthelengthofthestructIfTheL29DE8558215A13A506661C0E01E50AA3E5C9C7FA.json - The system cannot find the path specified. ``` When looking into this, I noticed that there was a practically identical test checked in as `clang-tools-extra/test/clang-doc/json/long-name.cpp` that was marked as `UNSUPPORTED: system-windows`, so this change extends that to this similar test as well. This should fix build bots being unable to successfully clean the build directory between runs. See https://lab.llvm.org/buildbot/#/builders/46/builds/37142 for an example of a build where the deletion of the build directory failed. The previous run was where the change was first run on the bot and would have introduced the problematic files. >From 027d94fb31f6178be0eed8bb8b87b9b84c5b4438 Mon Sep 17 00:00:00 2001 From: Douglas Yung <[email protected]> Date: Fri, 3 Jul 2026 22:20:24 -0400 Subject: [PATCH] Mark test introduced in #205586 as unsupported on Windows because it creates paths that are too long. --- clang-tools-extra/test/clang-doc/html/long-name.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/clang-tools-extra/test/clang-doc/html/long-name.cpp b/clang-tools-extra/test/clang-doc/html/long-name.cpp index 3ba73a13c9d34..556a66842e422 100644 --- a/clang-tools-extra/test/clang-doc/html/long-name.cpp +++ b/clang-tools-extra/test/clang-doc/html/long-name.cpp @@ -1,3 +1,4 @@ +// UNSUPPORTED: system-windows // RUN: rm -rf %t && mkdir -p %t // RUN: clang-doc --output=%t --format=html --executor=standalone %S/../Inputs/long-name.cpp // RUN: ls %t/html/GlobalNamespace | FileCheck %s -check-prefix=CHECK-HTML _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
