Re: [PATCH 1/2] MyFirstContribution: clarify asciidoc dependency

From: Taylor Blau <me@ttaylorr.com>
Date: Oct 16, 2020, 21:52 UTC
Message-ID: <20201016215235.GA2208862@nand.local>
In-Reply-To: <xmqqo8l1amtq.fsf@gitster.c.googlers.com>

On Fri, Oct 16, 2020 at 02:21:53PM -0700, Junio C Hamano wrote:

Show 8 quoted lines

> As this is only moving the existing note around in the
> documentation, it is not making things any worse, so I am OK to take
> the patch as-is, but if somebody (it is fine if it were done by you,
> Emily) can double check "apt-get install asciidoc" on a vanilla box
> does bring in what we need, that would be quite good.  FWIW, we
> write in our top-level INSTALL file that we require asciidox/xmlto
> toolchain (the latter is needed if you format for manpage, i.e. if
> you do "git subcmd --help").

I was curious myself, and surprised to learn that 'apt-get install asciidoc' installs more than 2GB of dependencies. Yikes. Unsurprisingly, somewhere in those 2GB we manage to fit in everything that seems to matter, since:

  $ cat /etc/os-release | grep PRETTY
  PRETTY_NAME="Debian GNU/Linux 9 (stretch)"
  $ apt-get update && apt-get install asciidoc
  $ cd copy-of-git
  $ make -C doc

...works just fine.

Thanks, Taylor