Re: [PATCH v2 0/7] Macros for Asciidoctor support
On Wed, Jan 25, 2017 at 06:30:00PM -0500, Jeff King wrote: > On Wed, Jan 25, 2017 at 11:19:26PM +, brian m. carlson wrote: > > > On Wed, Jan 25, 2017 at 04:35:44PM -0500, Jeff King wrote: > > > On Wed, Jan 25, 2017 at 02:28:55PM +0100, Johannes Schindelin wrote: > > > > > > > > The need for the extensions could be replaced with a small amount of > > > > > Ruby code, if that's considered desirable. Previous opinions on doing > > > > > so were negative, however. > > > > > > > > Quite frankly, it is annoying to be forced to install the extensions. I > > > > would much rather have the small amount of Ruby code in Git's > > > > repository. > > > > > > Me too. Dependencies can be a big annoyance. I'd reserve judgement until > > > I saw the actual Ruby code, though. :) > > > > I've sent the patch before, but I can send it again. It's relatively > > small and self-contained. I'm also happy to be responsible for > > maintaining it. > > Ah, it's: > > > http://public-inbox.org/git/1413070656-241955-5-git-send-email-sand...@crustytoothpaste.net/ > > (and note there is some surrounding discussion there). > > The code is not _too_ bad. The main thing is that it would have to be > kept up to date with changes to asciidoc.conf's version of the linkgit > macro. But it's not like that changes frequently. Yes. I think I can actually simplify it some more, since we always seem to use the argument to linkgit, so I'll send out a simplified patch in a few minutes. -- brian m. carlson / brian with sandals: Houston, Texas, US +1 832 623 2791 | https://www.crustytoothpaste.net/~bmc | My opinion only OpenPGP: https://keybase.io/bk2204 signature.asc Description: PGP signature
Re: [PATCH v2 0/7] Macros for Asciidoctor support
On Wed, Jan 25, 2017 at 11:19:26PM +, brian m. carlson wrote: > On Wed, Jan 25, 2017 at 04:35:44PM -0500, Jeff King wrote: > > On Wed, Jan 25, 2017 at 02:28:55PM +0100, Johannes Schindelin wrote: > > > > > > The need for the extensions could be replaced with a small amount of > > > > Ruby code, if that's considered desirable. Previous opinions on doing > > > > so were negative, however. > > > > > > Quite frankly, it is annoying to be forced to install the extensions. I > > > would much rather have the small amount of Ruby code in Git's repository. > > > > Me too. Dependencies can be a big annoyance. I'd reserve judgement until > > I saw the actual Ruby code, though. :) > > I've sent the patch before, but I can send it again. It's relatively > small and self-contained. I'm also happy to be responsible for > maintaining it. Ah, it's: http://public-inbox.org/git/1413070656-241955-5-git-send-email-sand...@crustytoothpaste.net/ (and note there is some surrounding discussion there). The code is not _too_ bad. The main thing is that it would have to be kept up to date with changes to asciidoc.conf's version of the linkgit macro. But it's not like that changes frequently. -Peff
Re: [PATCH v2 0/7] Macros for Asciidoctor support
On Wed, Jan 25, 2017 at 04:35:44PM -0500, Jeff King wrote: > On Wed, Jan 25, 2017 at 02:28:55PM +0100, Johannes Schindelin wrote: > > > > The need for the extensions could be replaced with a small amount of > > > Ruby code, if that's considered desirable. Previous opinions on doing > > > so were negative, however. > > > > Quite frankly, it is annoying to be forced to install the extensions. I > > would much rather have the small amount of Ruby code in Git's repository. > > Me too. Dependencies can be a big annoyance. I'd reserve judgement until > I saw the actual Ruby code, though. :) I've sent the patch before, but I can send it again. It's relatively small and self-contained. I'm also happy to be responsible for maintaining it. -- brian m. carlson / brian with sandals: Houston, Texas, US +1 832 623 2791 | https://www.crustytoothpaste.net/~bmc | My opinion only OpenPGP: https://keybase.io/bk2204 signature.asc Description: PGP signature
Re: [PATCH v2 0/7] Macros for Asciidoctor support
On Wed, Jan 25, 2017 at 02:28:55PM +0100, Johannes Schindelin wrote: > > The need for the extensions could be replaced with a small amount of > > Ruby code, if that's considered desirable. Previous opinions on doing > > so were negative, however. > > Quite frankly, it is annoying to be forced to install the extensions. I > would much rather have the small amount of Ruby code in Git's repository. Me too. Dependencies can be a big annoyance. I'd reserve judgement until I saw the actual Ruby code, though. :) -Peff
Re: [PATCH v2 0/7] Macros for Asciidoctor support
Hi Brian, On Sun, 22 Jan 2017, brian m. carlson wrote: > There are two major processors of AsciiDoc: AsciiDoc itself, and > Asciidoctor. Both have advantages and disadvantages, but traditionally > the documentation has been built with AsciiDoc, leading to some > surprising breakage when building with Asciidoctor. Partially, this is > due to the need to specify a significant number of macros on the command > line when building with Asciidoctor. > > This series cleans up some issues building the documentation with > Asciidoctor and provides two knobs, USE_ASCIIDOCTOR, which controls > building with Asciidoctor, and ASCIIDOCTOR_EXTENSIONS_LAB, which > controls the location of the Asciidoctor Extensions Lab, which is > necessary to expand the linkgit macro. I like it. I reviewed all the patches and think they are good (except the XSLT patch, which made me just feel incompetent because I do not know enough to have an opinion about it). > The need for the extensions could be replaced with a small amount of > Ruby code, if that's considered desirable. Previous opinions on doing > so were negative, however. Quite frankly, it is annoying to be forced to install the extensions. I would much rather have the small amount of Ruby code in Git's repository. Thanks, Johannes
Re: [PATCH v2 0/7] Macros for Asciidoctor support
"brian m. carlson" writes: > There are two major processors of AsciiDoc: AsciiDoc itself, and Asciidoctor. > Both have advantages and disadvantages, but traditionally the documentation > has > been built with AsciiDoc, leading to some surprising breakage when building > with > Asciidoctor. Partially, this is due to the need to specify a significant > number > of macros on the command line when building with Asciidoctor. > > This series cleans up some issues building the documentation with Asciidoctor > and provides two knobs, USE_ASCIIDOCTOR, which controls building with > Asciidoctor, and ASCIIDOCTOR_EXTENSIONS_LAB, which controls the location of > the > Asciidoctor Extensions Lab, which is necessary to expand the linkgit macro. > > The need for the extensions could be replaced with a small amount of Ruby > code, > if that's considered desirable. Previous opinions on doing so were negative, > however. > > In the process, I found several issues with cat-texi.perl, which have been > fixed. It has also been modernized to use strict, warnings, and lexical file > handles. I also made an attempt to produce more diffable texi files; I may > follow up with additional series along this line to make the documentation > build > reproducibly. Thanks. We'd probably want INSTALL to talk about Asciidoctor once this matures, as it is very simple requirement for the builder to have to just set USE_ASCIIDOCTOR, but the version requirement and stuff might be still confusing.
[PATCH v2 0/7] Macros for Asciidoctor support
There are two major processors of AsciiDoc: AsciiDoc itself, and Asciidoctor. Both have advantages and disadvantages, but traditionally the documentation has been built with AsciiDoc, leading to some surprising breakage when building with Asciidoctor. Partially, this is due to the need to specify a significant number of macros on the command line when building with Asciidoctor. This series cleans up some issues building the documentation with Asciidoctor and provides two knobs, USE_ASCIIDOCTOR, which controls building with Asciidoctor, and ASCIIDOCTOR_EXTENSIONS_LAB, which controls the location of the Asciidoctor Extensions Lab, which is necessary to expand the linkgit macro. The need for the extensions could be replaced with a small amount of Ruby code, if that's considered desirable. Previous opinions on doing so were negative, however. In the process, I found several issues with cat-texi.perl, which have been fixed. It has also been modernized to use strict, warnings, and lexical file handles. I also made an attempt to produce more diffable texi files; I may follow up with additional series along this line to make the documentation build reproducibly. Changes from v1: * Fix a brown-paper-bag bug. brian m. carlson (7): Documentation: fix warning in cat-texi.perl Documentation: modernize cat-texi.perl Documentation: remove unneeded argument in cat-texi.perl Documentation: sort sources for gitman.texi Documentation: add XSLT to fix DocBook for Texinfo Documentation: move dblatex arguments into variable Makefile: add a knob to enable the use of Asciidoctor Documentation/Makefile | 22 ++ Documentation/cat-texi.perl | 21 - Documentation/texi.xsl | 26 ++ Makefile| 6 ++ 4 files changed, 62 insertions(+), 13 deletions(-) create mode 100644 Documentation/texi.xsl