cmake: defer building unitprotos.h till a test target needs it#18086
Closed
vszakats wants to merge 1 commit intocurl:masterfrom
Closed
cmake: defer building unitprotos.h till a test target needs it#18086vszakats wants to merge 1 commit intocurl:masterfrom
unitprotos.h till a test target needs it#18086vszakats wants to merge 1 commit intocurl:masterfrom
Conversation
3 tasks
vszakats
changed the title
unitprotos.h till a target needs itunitprotos.h till a test target needs it
bagder
reviewed
Jul 29, 2025
jhauga
added a commit
to jhauga/curl
that referenced
this pull request
Jul 30, 2025
* tests: merge clients into libtests, drop duplicate code libtests and clients were built the same way after recent overhauls. libtests are used by runtests, clients by pytests. Merge clients into libtests, aligning their entry function signature, dropping common utility functions, and simplifying the build. Note: After this patch `CURLDEBUG` applies to cli tests, when enabled. Also: - lib552: drop local copy-paste debug callback in favor of testtrace. - lib552: drop local copy-paste dump function in favor of testtrace. - clients: use `long` for HTTP version, drop casts. - clients: replace local dump function in favor of testrace clone. - sync cli test entry function prototype with libtests'. - h2_serverpush: replace local trace callback with testtrace. - de-duplicate 3 websocket close, ping, ping, functions. Kept the pong iteration from `ws_pingpong`. Note: the pong clone in `lib2304` was returning an error when `curl_ws_recv()` returned non-zero and the payload matched the expected one anyway. After this patch, this case returns success, as it does in `ws_pingpong`. `lib2304` keeps passing, but I'm not sure if the previous behavior was intentional. - display full value in websocket close, ping, pong, drop casts. Closes curl#18079 * cmake: defer building `unitprotos.h` till a test target needs it Follow-up to c9bb9cd curl#17750 Ref: curl#17750 (comment) Closes curl#18086 * curl: add -h cheat-sheet using curl-cheat-sheet as text Resolve failing checks. --------- Co-authored-by: Viktor Szakats <commit@vsz.me>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to c9bb9cd #17750
Ref: #17750 (comment)