From: Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: "Weiny, Ira" <ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: Jason Gunthorpe
<jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>,
"Hefty,
Sean" <sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
"linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH 05/13] Have cmake run man pages through text substitution
Date: Mon, 26 Sep 2016 10:01:53 +0300 [thread overview]
Message-ID: <20160926070153.GI4088@leon.nu> (raw)
In-Reply-To: <2807E5FD2F6FDA4886F6618EAC48510E24EEBF1A-8k97q/ur5Z2krb+BlOpmy7fspsVTdybXVpNB7YpNyf8@public.gmane.org>
[-- Attachment #1: Type: text/plain, Size: 1862 bytes --]
On Mon, Sep 26, 2016 at 06:30:21AM +0000, Weiny, Ira wrote:
> >
> > On Fri, Sep 23, 2016 at 09:23:42PM +0000, Hefty, Sean wrote:
> > > > rsocket.7 had an errant text substitution that never worked, it is a
> > > > good idea to have the man pages use the correct paths, so let us
> > > > have cmake run them through.
> > > >
> > > > Any man page ending in '.in' will be substituted automatically.
> > > >
> > > > Signed-off-by: Jason Gunthorpe <jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
> > >
> > > Acked-by: Sean Hefty <sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> > >
> > > FWIW, I'm in favor of replacing the man pages with markdown and then
> > > generating the man pages from that. I volunteer someone else to do
> > > that work.
> >
> > I concur, and someone else should do the work ..
> >
> > We could begin by requiring all new man pages to be in markdown format.
> > Do you do this in libfabric? What conversion tool do you like?
> >
> > I've used rst for this in the past with reasonable results..
>
> Infiniband-diags uses rst through rst2man. But if I had to do it over I would probably do it different. rst2man is not always installed and has caused build issues for some people. Whatever tool we use I would recommend it be something common. I'm not too familiar with markdown. Is this really common now?
Among hipsters - yes :)
>
> Ira
>
> >
> > Jason
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the
> > body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at
> > http://vger.kernel.org/majordomo-info.html
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2016-09-26 7:01 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-23 19:16 [PATCH 00/13] Policy changes and packaging support Jason Gunthorpe
[not found] ` <1474658228-5390-1-git-send-email-jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-23 19:16 ` [PATCH 01/13] Be consistent about defining NDEBUG Jason Gunthorpe
2016-09-23 19:16 ` [PATCH 02/13] Switch valgrind memcheck.h to on by default Jason Gunthorpe
2016-09-23 19:16 ` [PATCH 03/13] Do not produce libtool .la files Jason Gunthorpe
2016-09-23 19:16 ` [PATCH 04/13] Do not produce static libraries by default Jason Gunthorpe
2016-09-23 19:17 ` [PATCH 05/13] Have cmake run man pages through text substitution Jason Gunthorpe
[not found] ` <1474658228-5390-6-git-send-email-jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-23 21:23 ` Hefty, Sean
[not found] ` <1828884A29C6694DAF28B7E6B8A82373AB08E9D8-P5GAC/sN6hkd3b2yrw5b5LfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2016-09-23 21:38 ` Jason Gunthorpe
[not found] ` <20160923213819.GC8490-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-23 21:48 ` Hefty, Sean
2016-09-26 6:30 ` Weiny, Ira
[not found] ` <2807E5FD2F6FDA4886F6618EAC48510E24EEBF1A-8k97q/ur5Z2krb+BlOpmy7fspsVTdybXVpNB7YpNyf8@public.gmane.org>
2016-09-26 7:01 ` Leon Romanovsky [this message]
2016-09-26 15:19 ` Hefty, Sean
[not found] ` <1828884A29C6694DAF28B7E6B8A82373AB08EFE3-P5GAC/sN6hkd3b2yrw5b5LfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2016-09-26 17:47 ` Jason Gunthorpe
[not found] ` <20160926174713.GF22965-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-26 18:17 ` Weiny, Ira
[not found] ` <2807E5FD2F6FDA4886F6618EAC48510E24EEC6E9-8k97q/ur5Z2krb+BlOpmy7fspsVTdybXVpNB7YpNyf8@public.gmane.org>
2016-09-26 18:21 ` Jason Gunthorpe
[not found] ` <20160926182154.GA27094-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-29 18:37 ` ira.weiny
2016-09-23 21:49 ` Hefty, Sean
2016-09-23 19:17 ` [PATCH 06/13] Consolidate definitions for paths into config.h Jason Gunthorpe
[not found] ` <1474658228-5390-7-git-send-email-jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-23 21:22 ` Hefty, Sean
2016-09-23 19:17 ` [PATCH 07/13] Pass ancillary files that use paths through cmake substitution Jason Gunthorpe
2016-09-23 19:17 ` [PATCH 08/13] srp_daemon: Move lock file into /var/run/ Jason Gunthorpe
2016-09-23 19:17 ` [PATCH 09/13] verbs: Move the providers into /usr/lib.../libibverbs by default Jason Gunthorpe
2016-09-23 19:17 ` [PATCH 10/13] ipathverbs: Move truescale-serdes.cmds to /usr/libexec/ Jason Gunthorpe
2016-09-23 19:17 ` [PATCH 11/13] srp_daemon: Add the debian initscripts as an option Jason Gunthorpe
[not found] ` <1474658228-5390-12-git-send-email-jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-28 18:27 ` Doug Ledford
[not found] ` <57EC0B75.3000101-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-09-28 18:47 ` Jason Gunthorpe
[not found] ` <20160928184708.GA31472-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-28 18:55 ` Doug Ledford
[not found] ` <57EC120F.7070506-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-09-28 21:13 ` Jason Gunthorpe
2016-09-28 20:18 ` Bart Van Assche
[not found] ` <07810a28-9db1-6dca-0666-d811fe246b7e-XdAiOPVOjttBDgjK7y7TUQ@public.gmane.org>
2016-09-28 21:01 ` Jason Gunthorpe
[not found] ` <20160928210137.GB2533-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-28 21:11 ` Doug Ledford
2016-09-23 19:17 ` [PATCH 12/13] Document a versioning strategy and use it Jason Gunthorpe
2016-09-23 19:17 ` [PATCH 13/13] Remove ibsupport-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org from MAINTAINERS Jason Gunthorpe
[not found] ` <1474658228-5390-14-git-send-email-jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-24 13:43 ` Dalessandro, Dennis
[not found] ` <1474724600.6625.1.camel-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2016-09-24 16:56 ` Jason Gunthorpe
[not found] ` <20160924165645.GA16206-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-24 19:18 ` Dalessandro, Dennis
[not found] ` <1474744719.6625.3.camel-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2016-09-26 18:12 ` Jason Gunthorpe
[not found] ` <20160926181206.GA28992-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-09-26 19:12 ` Dalessandro, Dennis
2016-09-28 18:27 ` [PATCH 00/13] Policy changes and packaging support Doug Ledford
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160926070153.GI4088@leon.nu \
--to=leon-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
--cc=ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
--cc=jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org \
--cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox