Skip to content

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Dec 21, 2025

In unity builds the source filename (via __FILE__) has no path (or
uses slashes?), while in non-unity ones it does contain backslashes
on Windows, with MSVC. Fix the test to recognize backslashes in the
stripfile regexp.

Seen in MSVC jobs in CI:

-MEM tool_cfgable.c[LF]
-MEM tool_paramhlp.c[LF]
-MEM tool_cfgable.c[LF]
-MEM tool_cfgable.c[LF]
-MEM tool_cfgable.c[LF]
-MEM tool_cfgable.c[LF]
+MEM D:\a\curl\curl\src\tool_cfgable.c[LF]
+MEM D:\a\curl\curl\src\tool_paramhlp.c[LF]
+MEM D:\a\curl\curl\src\tool_cfgable.c[LF]
+MEM D:\a\curl\curl\src\tool_cfgable.c[LF]
+MEM D:\a\curl\curl\src\tool_cfgable.c[LF]
+MEM D:\a\curl\curl\src\tool_cfgable.c[LF]

Ref: https://github.com/curl/curl/actions/runs/20408366058/job/58641468316?pr=20061#step:13:303
Ref: https://github.com/curl/curl/actions/runs/20408522070/job/58641826216?pr=20064#step:13:298


Seen in msvc, CM x64-windows openssl +examples job:
```
--- log/8/check-expected	2025-12-21 10:20:56.062527200 +0000
+++ log/8/check-generated	2025-12-21 10:20:56.062527200 +0000
@@ -1,6 +1,6 @@
-MEM tool_cfgable.c[LF]
-MEM tool_paramhlp.c[LF]
-MEM tool_cfgable.c[LF]
-MEM tool_cfgable.c[LF]
-MEM tool_cfgable.c[LF]
-MEM tool_cfgable.c[LF]
+MEM D:\a\curl\curl\src\tool_cfgable.c[LF]
+MEM D:\a\curl\curl\src\tool_paramhlp.c[LF]
+MEM D:\a\curl\curl\src\tool_cfgable.c[LF]
+MEM D:\a\curl\curl\src\tool_cfgable.c[LF]
+MEM D:\a\curl\curl\src\tool_cfgable.c[LF]
+MEM D:\a\curl\curl\src\tool_cfgable.c[LF]
```
Ref: https://github.com/curl/curl/actions/runs/20408366058/job/58641468316?pr=20061#step:13:303
@vszakats vszakats added tests Windows Windows-specific labels Dec 21, 2025
@github-actions github-actions bot added the CI Continuous Integration label Dec 21, 2025
@vszakats vszakats changed the title test96: fix to accept non-unity memdump content on Windows test96: fix to accept non-unity memdump content with MSVC Dec 21, 2025
@vszakats vszakats closed this in 4df7269 Dec 21, 2025
@vszakats vszakats deleted the t96 branch December 21, 2025 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration tests Windows Windows-specific

Development

Successfully merging this pull request may close these issues.

1 participant