Skip to content

make all libcurl man page examples compile - and verify them in CI #12448

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

Closed
wants to merge 5 commits into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Dec 3, 2023

  • the gcc invocation uses the flags -Wunused -Werror -Wno-unused-but-set-variable
  • also run checksrc,pl on the code to make them follow our code style

@bagder bagder changed the title make all libcurl examples compile - and verify them in CI make all libcurl man page examples compile - and verify them in CI Dec 3, 2023

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg
makes it identify the correct source file and line
@bagder bagder force-pushed the bagder/verify-man-examples branch from 2462468 to 8ae0ea5 Compare December 4, 2023 07:55

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg
Closes #12448
@bagder bagder force-pushed the bagder/verify-man-examples branch from 8ae0ea5 to 97a7090 Compare December 4, 2023 07:57

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg
@bagder
Copy link
Member Author

bagder commented Dec 4, 2023

I think this is pretty much done for merge.

@bagder bagder closed this in cb521d1 Dec 4, 2023
@bagder bagder deleted the bagder/verify-man-examples branch December 4, 2023 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

1 participant