Bug 235701 - security/py-certbot: Add man page
Summary: security/py-certbot: Add man page
Status: Open
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-python mailing list
URL:
Keywords: needs-patch
Depends on:
Blocks:
 
Reported: 2019-02-12 16:19 UTC by freebsd
Modified: 2019-02-13 22:22 UTC (History)
1 user (show)

See Also:
koobs: maintainer-feedback+


Attachments
svn-diff-manpage (1.93 KB, patch)
2019-02-13 06:44 UTC, Walter Schwarzenfeld
no flags Details | Diff
svn-diff-manpage_v2 (1.98 KB, patch)
2019-02-13 07:17 UTC, Walter Schwarzenfeld
no flags Details | Diff
svn-diff-manpage_v3 (2.00 KB, patch)
2019-02-13 08:29 UTC, Walter Schwarzenfeld
no flags Details | Diff
svn-diff-alternate (2.24 KB, patch)
2019-02-13 14:36 UTC, Walter Schwarzenfeld
no flags Details | Diff
svn-diff-manpage_v4 (1.46 KB, patch)
2019-02-13 15:51 UTC, Walter Schwarzenfeld
no flags Details | Diff
svn-diff-manpage_v5 (1.49 KB, patch)
2019-02-13 16:00 UTC, Walter Schwarzenfeld
no flags Details | Diff
svn-diff-manpage_v6 (1.49 KB, patch)
2019-02-13 17:03 UTC, Walter Schwarzenfeld
no flags Details | Diff
svn-diff-manpage_v7 (1.34 KB, patch)
2019-02-13 17:27 UTC, Walter Schwarzenfeld
no flags Details | Diff
svn-diff-manpage_v8 (1.27 KB, patch)
2019-02-13 18:51 UTC, Walter Schwarzenfeld
no flags Details | Diff
svn-dff-manpage_v9 (1.34 KB, patch)
2019-02-13 22:22 UTC, Walter Schwarzenfeld
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description freebsd 2019-02-12 16:19:47 UTC
py27-certbot has no man page

11.2-RELEASE-p9 FreeBSD 11.2-RELEASE-p9 #0: Tue Feb  5 15:30:36 UTC 2019     root@amd64-builder.daemonology.net:/usr/obj/usr/src/sys/GENERIC  amd64
Comment 1 Walter Schwarzenfeld freebsd_triage 2019-02-12 17:17:02 UTC
Try certbot-2.7 --help.
Comment 2 Walter Schwarzenfeld freebsd_triage 2019-02-12 17:35:00 UTC
...and the manpage is the same as the helptext.
Comment 3 Kubilay Kocak freebsd_committer freebsd_triage 2019-02-13 01:47:31 UTC
docs/Makefile contains a man: target that uses sphinx to convert docs/man* into man pages. While the contents is the same as the command line help, its desirable and preferable that ports install man pages if they are (or can be) provided.
Comment 4 Walter Schwarzenfeld freebsd_triage 2019-02-13 06:44:51 UTC
Created attachment 201975 [details]
svn-diff-manpage

sphinx has warnings.

man 7 certbot has a syntax error:

      free, so thereas no need to arrange payment.

:<standard input>:562: `\&' is not allowed in a name
grotty:<standard input> (<standard input>):11450: character above first line discarded
grotty:<standard input> (<standard input>):11454: character above first line discarded
grotty:<standard input> (<standard input>):11458: character above first line discarded
grotty:<standard input> (<standard input>):11462: character above first line discarded
grotty:<standard input> (<standard input>):11466: character above first line discarded
       How you use Certbot depends on the configuration of  your  web  server.
       The  best way to get started is to use our interactive guide. It gener-
Comment 5 Walter Schwarzenfeld freebsd_triage 2019-02-13 07:17:40 UTC
Created attachment 201976 [details]
svn-diff-manpage_v2

PORT_OPTIONS:MMAN was not right.
Comment 6 Walter Schwarzenfeld freebsd_triage 2019-02-13 08:29:45 UTC
Created attachment 201978 [details]
svn-diff-manpage_v3

Forgot FLAVOR for sphinx.
Comment 7 Walter Schwarzenfeld freebsd_triage 2019-02-13 14:36:19 UTC
Created attachment 201984 [details]
svn-diff-alternate

Alternate patch fixes output cetbot.7.
Comment 8 Tobias Kortkamp freebsd_committer 2019-02-13 15:11:19 UTC
Comment on attachment 201984 [details]
svn-diff-alternate

1. MAN should be named MANPAGES as it usually is.

2. The new options and helpers are badly ordered [1].

3. Remove .include <bsd.port.options.mk> again and use options helpers
   targets.

4. This should probably be post-build-MAN-on (or post-build-MANPAGES-on after fixing the option name):
+pre-install:

5. gmake is not declared as a build dependency, so this will probably fail:
+	${LOCALBASE}/bin/gmake -C ${WRKSRC}/docs man

6. Use the INSTALL_MAN macro to install files into STAGEDIR, not CP. [2]

7. What does this do and why does it require gsed?
+	${SED_CMD} -i '1i\.if n .pl 2000v' ${STAGEDIR}${PREFIX}/man/man7/certbot.7

   If the input for creating certbot.7 is already broken can it not
   be fixed instead?

8. Why is patch-docs_conf.py needed?

[1] https://www.freebsd.org/doc/en/books/porters-handbook/porting-order.html
[2] https://www.freebsd.org/doc/en/books/porters-handbook/install.html
Comment 9 Tobias Kortkamp freebsd_committer 2019-02-13 15:20:45 UTC
(In reply to Tobias Kortkamp from comment #8)
> 5. gmake is not declared as a build dependency, so this will probably fail:

Ah, nevermind. I see now that there is MAN_USES=gmake. In which case

+	${LOCALBASE}/bin/gmake -C ${WRKSRC}/docs man

can be

+	${DO_MAKE_BUILD} -C ${WRKSRC}/docs man

to not ignore MAKE_{ARGS,ENV,FLAGS}, etc.
Comment 10 Walter Schwarzenfeld freebsd_triage 2019-02-13 15:38:31 UTC
(In reply to Tobias Kortkamp from comment #8)


>7. What does this do and why does it require gsed?
It does not work with "standard" sed.
Comment 11 Walter Schwarzenfeld freebsd_triage 2019-02-13 15:50:18 UTC
(In reply to Tobias Kortkamp from comment #8)

Why is patch-docs_conf.py needed?
Was a wrong idea, sphinx does not work with extension repoze.sphinx.autointerface.
I obolete this patch, and use BUILD_DEPENDS= ....devel/py-repoze.sphinx.autointerface
Comment 12 Walter Schwarzenfeld freebsd_triage 2019-02-13 15:51:51 UTC
Created attachment 201985 [details]
svn-diff-manpage_v4
Comment 13 Walter Schwarzenfeld freebsd_triage 2019-02-13 16:00:24 UTC
Created attachment 201986 [details]
svn-diff-manpage_v5

Forgot to change MAN to MANPAGES.
Comment 14 Walter Schwarzenfeld freebsd_triage 2019-02-13 17:03:36 UTC
Created attachment 201989 [details]
svn-diff-manpage_v6

Change targets.
Comment 15 Walter Schwarzenfeld freebsd_triage 2019-02-13 17:27:15 UTC
Created attachment 201990 [details]
svn-diff-manpage_v7

Sent the wrong svn diff.
Comment 16 Walter Schwarzenfeld freebsd_triage 2019-02-13 18:51:21 UTC
Created attachment 201992 [details]
svn-diff-manpage_v8

Remove PLIST_FILES cause of duplicate file listing.
Comment 17 Walter Schwarzenfeld freebsd_triage 2019-02-13 22:22:08 UTC
Created attachment 201997 [details]
svn-dff-manpage_v9

Revert last change and accept duplicate listing (seems autoplist makes nonsense).