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

cmake: generate misc manpages and install mk-ca-bundle.pl #13197

Closed
wants to merge 3 commits into from

Conversation

vszakats
Copy link
Member

@vszakats vszakats commented Mar 27, 2024

  • install mk-ca-bundle.pl like autotools does.

  • generate and install mk-ca-bundle.1 and curl-config.1 like autotools. This fixes tests 1140 and 1173.

    Fixes Man page tests 1140 and 1173 fail on cmake #13194

  • add option BUILD_MISC_DOCS to control building the above two manpages. Enabled by default.

  • appveyor: stop disabling tests 1140 and 1173.

Closes #13197

- install `mk-ca-bundle.pl` like autotools does.

- generate and install `mk-ca-bundle.1` and `curl-config.1` like
  autotools. This fixes tests 1140 and 1173.

- add option `BUILD_MISC_DOCS` to control building the above two
  manpages. Enabled by default.

Fixes curl#13194
Closes #xxxxx
@github-actions github-actions bot added the CI Continuous Integration label Mar 27, 2024
@dfandrich
Copy link
Contributor

The tests pass fine for me with this!

@vszakats vszakats closed this in 5023ffa Apr 4, 2024
@vszakats vszakats deleted the cmake-misc-docs branch April 4, 2024 09:22
vszakats added a commit to vszakats/curl that referenced this pull request Apr 4, 2024
vszakats added a commit that referenced this pull request Apr 4, 2024
Fixing:
```
make[2]: Circular docs/curl-config.1 <- docs/curl-config.1 dependency dropped.
make[2]: Circular docs/mk-ca-bundle.1 <- docs/mk-ca-bundle.1 dependency dropped.
```
Ref: https://github.com/curl/curl/actions/runs/8559617487/job/23456740844?pr=13282#step:6:18

Follow-up to 5023ffa #13197
Closes #13283
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.

Man page tests 1140 and 1173 fail on cmake
3 participants