Skip to content

test1477: verify that libcurl-errors.3 and public headers are synced#12424

Closed
bagder wants to merge 2 commits into
masterfrom
bagder/errors
Closed

test1477: verify that libcurl-errors.3 and public headers are synced#12424
bagder wants to merge 2 commits into
masterfrom
bagder/errors

Conversation

@bagder

@bagder bagder commented Nov 28, 2023

Copy link
Copy Markdown
Member

No description provided.

The script errorcodes.pl extracts all error codes from all headers and
checks that they are all documented, then checks that all documented
error codes are also specified in a header file.
@bagder bagder closed this in 66ec950 Nov 28, 2023
bagder added a commit that referenced this pull request Nov 28, 2023
The script errorcodes.pl extracts all error codes from all headers and
checks that they are all documented, then checks that all documented
error codes are also specified in a header file.

Closes #12424
@bagder bagder deleted the bagder/errors branch November 28, 2023 21:56
vszakats added a commit that referenced this pull request Jul 11, 2025
Fix test 1175 by passing the source root directory (was: tests).

Before this patch this caused silent Perl warnings and returning success
without executing the tests, due to:
```
readline() on closed filehandle $f at ../../tests/test1175.pl line 55.
readline() on closed filehandle $f at ../../tests/test1175.pl line 39.
```

Running the test revealed these issues:
```
CURLE_FUNCTION_NOT_FOUND is not in libcurl-errors.md
CURLE_HTTP_POST_ERROR is not in libcurl-errors.md
CURLE_TELNET_OPTION_SYNTAX is not in libcurl-errors.md
CURLM_CALL_MULTI_SOCKET is not in libcurl-errors.md
```

Apply fixes:
- mark `CURLE_FUNCTION_NOT_FOUND` deprecated by 7.53.0
- mark `CURLE_HTTP_POST_ERROR` deprecated by 7.56.0
- mark `CURLE_TELNET_OPTION_SYNTAX` deprecated by 7.78.0
- document `CURLM_CALL_MULTI_SOCKET` as a synonym for
  `CURLM_CALL_MULTI_PERFORM`
- test1477: exclude `CURLM_CALL_MULTI_SOCKET`.

But, these weren't officially deprecated. It may need more updates
to reflect that in other places, or fix the issues differently.

Follow-up to 66ec950 #12424
Follow-up to 74f441c #4628

Cherry-picked from #17877
Closes #17880
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant