KEMBAR78
Fix GH_001923_filesystem_long_path_support/custombuild.pl for the MSVC-internal test harness by StephanTLavavej · Pull Request #5799 · microsoft/STL · GitHub
Skip to content

Conversation

@StephanTLavavej
Copy link
Member

In toolset update #5783 I added test coverage for <filesystem> long path support. This requires a manifest to be embedded into the executable with a special linker option. The MSVC-internal test harness uses Perl for this, and I used a single cl command. It turns out that in certain configurations (of course only run in CI checks, not in PR checks where I would have discovered this earlier), the MSVC-internal test harness adds /FastFailDump. When compiling with Clang, that gets sent to lld-link, which emits

lld-link: error: could not open '/FastFailDump': no such file or directory

I should have followed precedent and used separate compile and link commands, which avoids confusing lld-link (as in this case I believe we invoke MSVC link.exe):

Run::ExecuteCL("Test.cpp /c");
Run::ExecuteLink("/machine:$machine /out:$cwd.exe Test.obj");

@StephanTLavavej StephanTLavavej requested a review from a team as a code owner October 23, 2025 07:00
@StephanTLavavej StephanTLavavej added test Related to test code filesystem C++17 filesystem labels Oct 23, 2025
@github-project-automation github-project-automation bot moved this to Initial Review in STL Code Reviews Oct 23, 2025
@StephanTLavavej StephanTLavavej moved this from Initial Review to Final Review in STL Code Reviews Oct 23, 2025
@StephanTLavavej StephanTLavavej merged commit eb024e0 into microsoft:main Oct 23, 2025
39 checks passed
@github-project-automation github-project-automation bot moved this from Final Review to Done in STL Code Reviews Oct 23, 2025
@StephanTLavavej StephanTLavavej deleted the fix-fs-perl branch October 23, 2025 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filesystem C++17 filesystem test Related to test code

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants