Skip to content
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

test 1307 failures due to fnmatch differences #2825

Closed
bagder opened this issue Aug 1, 2018 · 2 comments
Closed

test 1307 failures due to fnmatch differences #2825

bagder opened this issue Aug 1, 2018 · 2 comments
Labels

Comments

@bagder
Copy link
Member

bagder commented Aug 1, 2018

I did this

Check autobuilds, like this.

Test 1307 fails and shows fice test patterns that error.

 Tested with system fnmatch(), linux-style
Curl_fnmatch("[[FF]]", "[FF]") should return FAIL (returns MATCH) [42]
Curl_fnmatch("[![FF]]", "") should return FAIL (returns NOMATCH) [62]
Curl_fnmatch("[![FF]]", "[FF]") should return FAIL (returns NOMATCH) [63]
Curl_fnmatch("[![FF]]", "a") should return FAIL (returns MATCH) [64]
Curl_fnmatch("**]*[*[[13]]**[*[13])]*]*[**[*[13]~r-]*]**[.*]*[[E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3][E3]*[[13]]**[*[13])]*]*[*[[13]]*[~r]*]*[BA][13][A6]~b-]*", "a") should return FAIL (returns NOMATCH) [156]

Interestingly, all the actual returns seem to match what the internal fnmatch function would return.

Presumably, this happens because this is run on an older libc with a different fnmatch implementation that differs from what a modern Linux glibc fnmatch returns for these patterns.

Tests on travis and on my local dev hosts all run this test fine. (glibc 2.27)

This seems complicated to fix for real and I'm leaning towards simply removing those test patterns that differ like this from the test.

I expected the following

All tests to run OK.

curl/libcurl version

current git master but this was introduced in a115c6b when we switched to using the native version when available.

operating system

Linux, but possibly other systems could show similar issues.

@bagder bagder added the tests label Aug 1, 2018
@bagder
Copy link
Member Author

bagder commented Aug 1, 2018

Yeah, also Solaris failures here

@bagder
Copy link
Member Author

bagder commented Aug 1, 2018

Fixing this "properly" will be painful. I'm going to disable test 1307 for now.

@bagder bagder closed this as completed in b7bdf21 Aug 1, 2018
xquery pushed a commit to xquery/curl that referenced this issue Aug 9, 2018
Turns out that since we're using the native fnmatch function now when
available, and they simply disagree on a huge number of test patterns
that make it hard to test this function like this...

Fixes curl#2825
falconindy pushed a commit to falconindy/curl that referenced this issue Sep 10, 2018
Turns out that since we're using the native fnmatch function now when
available, and they simply disagree on a huge number of test patterns
that make it hard to test this function like this...

Fixes curl#2825
@lock lock bot locked as resolved and limited conversation to collaborators Oct 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

1 participant