rnpgp / rnp

RNP: high performance C++ OpenPGP library used by Mozilla Thunderbird
https://www.rnpgp.org
Other
192 stars 54 forks source link

Add CI tests to check whether man pages are built and installed correctly. #1437

Open ni4 opened 3 years ago

ni4 commented 3 years ago

Description

Once PR #1425 is merged we may add or update existing CI tests to make sure that man pages are build and installed if pandoc is present, and build doesn't error if there is no pandoc.

Some related discussion is here: https://github.com/rnpgp/rnp/issues/506#issuecomment-788530601

ronaldtse commented 3 years ago

Changed to use Asciidoctor instead of Pandoc. The macOS, ubuntu, FreeBSD environments should already work with #1425 via ci/install.sh.

These environments need installation of Asciidoctor to generate man pages:

maxirmx commented 1 year ago

CentOS/Fedora - CI tests for man pages are implemented in https://github.com/rnpgp/rnp/pull/2013 Ubuntu - CI tests for man pages are implemented in https://github.com/rnpgp/rnp/pull/2058

maxirmx commented 1 year ago

We have installation tests for man pages for CentOS/Fedora; Ubuntu; FreeBSD

CentOS/Fedora flows work as expected On Ubuntu rnp.1; rnpkeys.1; librnp.3 are packaged (and installed) instead of rnp.1.gz; rnpkeys.1.gz; librnp.3.gz On FreeBSD nothing is packaged (and installed) although man files are build correctly.

It looks like cmake/cpack issue that needs additional research. I was not able to find any relevant discussions. Not sure though if it is of significant importance. I belive package maintainers use some other scripts or tools, not cpack .