Skip to content

meson: fix tests under Windows #192

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 6, 2025
Merged

meson: fix tests under Windows #192

merged 3 commits into from
Apr 6, 2025

Conversation

neheb
Copy link
Contributor

@neheb neheb commented Apr 3, 2025

No description provided.

@neheb
Copy link
Contributor Author

neheb commented Apr 3, 2025

@bgilbert nope. Probably the solution is to replace runtest.sh with python.

neheb added 2 commits April 2, 2025 21:48
Need to use find_program to see if we can run this.

Signed-off-by: Rosen Penev <[email protected]>
Signed-off-by: Rosen Penev <[email protected]>
@neheb
Copy link
Contributor Author

neheb commented Apr 3, 2025

hrm sh works. a couple of seconds slower. not too bad though.

@neheb neheb marked this pull request as ready for review April 3, 2025 08:35
@bgilbert
Copy link
Contributor

bgilbert commented Apr 3, 2025

neheb#2 has one more fix.

diff fails when run from a tarball: the unittest programs produce CRLF
line endings and baseline_*.txt in the tarball uses LF.  Reproduce this
in CI by preventing Git from converting the text files to CRLF, then
ensure the unittest programs produce Unix line endings.
@neheb neheb marked this pull request as draft April 3, 2025 20:22
@neheb neheb marked this pull request as ready for review April 3, 2025 20:22
@neheb
Copy link
Contributor Author

neheb commented Apr 3, 2025

Interesting...

@benhoyt benhoyt merged commit dcb0446 into benhoyt:master Apr 6, 2025
3 checks passed
@neheb neheb deleted the msvc branch April 6, 2025 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants