mbox series

[00/23] doc: cleanups and asciidoctor direct man pages

Message ID 20210621163110.1074145-1-felipe.contreras@gmail.com (mailing list archive)
Headers show
Series doc: cleanups and asciidoctor direct man pages | expand

Message

Felipe Contreras June 21, 2021, 4:30 p.m. UTC
I already sent the first part of this series separately multiple times,
the last one is 4 simple obviously correct patches [1], but since the
maintainer has admitted he has no problem ignoring valid work
entirely due to personal animus [2], I'm sending the whole chain.

There's no point in carefully selecting multiple series of patches to be
merged one by one when all of them are going to be ignored. So I'm
sending all 3 series at once.

Hopefully by sending it all at once some people will be able to realize
that:

 1. They are valid
 2. They are helpful
 3. They make the code more maintainable
 4. They enable new features
 5. They enable the new features to be easily tested
 6. They reduce the doc-diff of the new feature, as well as others
 7. They are superior to the competing series currently in seen
 8. They include work of multiple contributors

Any fair and impartial maintainer would attempt to pick them up.

Cheers.

[1] https://lore.kernel.org/git/20210618215231.796592-1-felipe.contreras@gmail.com/
[2] https://lore.kernel.org/git/xmqqmtrmjpa8.fsf@gitster.g/

Felipe Contreras (21):
  doc: remove GNU troff workaround
  doc: use --stringparam in xmlto
  doc: simplify version passing
  doc: asciidoc: remove unnecessary attribute
  doc: asciidoctor: remove unnecessary require
  doc: asciidoctor: remove cruft
  doc: asciidoctor: reorganize extensions
  doc: asciidoctor: use html-prefix only for html
  doc: asciidoctor: refactor macro registration
  doc: asciidoctor: improve string handling
  doc: asciidoctor: split the format from the code
  doc: asciidoctor: specify name of our group
  doc: doc-diff: set docdate manually
  doc: use asciidoctor to build man pages directly
  doc: asciidoctor: add linkgit macros in man pages
  doc: add man pages workaround for asciidoctor
  doc: asciidoctor: add hack for xrefs
  doc: asciidoctor: add hack to improve links
  doc: asciidoctor: add support for baseurl
  doc: asciidoctor: cleanup man page hack
  doc: asciidoctor: add hack for old versions

Jeff King (1):
  doc-diff: support asciidoctor man pages

Martin Ă…gren (1):
  doc-diff: drop --cut-footer switch

 Documentation/.gitignore                |   1 -
 Documentation/Makefile                  |  32 ++++----
 Documentation/asciidoc.conf             |  20 -----
 Documentation/asciidoctor-extensions.rb | 103 +++++++++++++++---------
 Documentation/doc-diff                  |  38 ++++-----
 Documentation/manpage-base-url.xsl.in   |  10 ---
 Documentation/manpage-quote-apos.xsl    |  16 ----
 Makefile                                |   8 +-
 8 files changed, 104 insertions(+), 124 deletions(-)
 delete mode 100644 Documentation/manpage-base-url.xsl.in
 delete mode 100644 Documentation/manpage-quote-apos.xsl

Comments

Bagas Sanjaya June 22, 2021, 5:35 a.m. UTC | #1
On 21/06/21 23.30, Felipe Contreras wrote:
> I already sent the first part of this series separately multiple times,
> the last one is 4 simple obviously correct patches [1], but since the
> maintainer has admitted he has no problem ignoring valid work
> entirely due to personal animus [2], I'm sending the whole chain.
> 
> There's no point in carefully selecting multiple series of patches to be
> merged one by one when all of them are going to be ignored. So I'm
> sending all 3 series at once.
> 
> Hopefully by sending it all at once some people will be able to realize
> that:
> 
>   1. They are valid
>   2. They are helpful
>   3. They make the code more maintainable
>   4. They enable new features
>   5. They enable the new features to be easily tested
>   6. They reduce the doc-diff of the new feature, as well as others
>   7. They are superior to the competing series currently in seen
>   8. They include work of multiple contributors
> 
> Any fair and impartial maintainer would attempt to pick them up.
> 
> Cheers.
> 

I think the next step after this patch series is to add asciidoctor 
direct man pages generation option to ./configure script (maybe 
--enable-asciidoctor-manpage?). But before that, we need to add 
--enable-asciidoctor option, which sets USE_ASCIIDOCTOR=YesPlease.
Felipe Contreras June 23, 2021, 1:37 a.m. UTC | #2
Bagas Sanjaya wrote:
> On 21/06/21 23.30, Felipe Contreras wrote:
> > I already sent the first part of this series separately multiple times,
> > the last one is 4 simple obviously correct patches [1], but since the
> > maintainer has admitted he has no problem ignoring valid work
> > entirely due to personal animus [2], I'm sending the whole chain.
> > 
> > There's no point in carefully selecting multiple series of patches to be
> > merged one by one when all of them are going to be ignored. So I'm
> > sending all 3 series at once.
> > 
> > Hopefully by sending it all at once some people will be able to realize
> > that:
> > 
> >   1. They are valid
> >   2. They are helpful
> >   3. They make the code more maintainable
> >   4. They enable new features
> >   5. They enable the new features to be easily tested
> >   6. They reduce the doc-diff of the new feature, as well as others
> >   7. They are superior to the competing series currently in seen
> >   8. They include work of multiple contributors
> > 
> > Any fair and impartial maintainer would attempt to pick them up.
> > 
> > Cheers.
> > 
> 
> I think the next step after this patch series is to add asciidoctor 
> direct man pages generation option to ./configure script (maybe 
> --enable-asciidoctor-manpage?). But before that, we need to add 
> --enable-asciidoctor option, which sets USE_ASCIIDOCTOR=YesPlease.

Maybe. I don't use ./configure, so perhaps somebody that does should
take charge of this.