From: Felipe Contreras <firstname.lastname@example.org> To: "Jeff King" <email@example.com>, "Martin Ågren" <firstname.lastname@example.org> Cc: "brian m. carlson" <email@example.com>, Bagas Sanjaya <firstname.lastname@example.org>, Git Users <email@example.com> Subject: Re: [RFC suggestion] Generate manpage directly with Asciidoctor Date: Mon, 10 May 2021 23:27:54 -0500 [thread overview] Message-ID: <firstname.lastname@example.org> (raw) In-Reply-To: <YJmykGWaWi03+WoW@coredump.intra.peff.net> Jeff King wrote: > On Sun, May 09, 2021 at 10:20:37AM +0200, Martin Ågren wrote: > > In general, I agree. I do think it's important that "most people > > contributing to Git", whatever that means, can build the documentation > > to check the part they're adding/modifying and not find it broken left > > and right. They would then (quite rightly) not even bother building it. > > Agreed. But I think that is mostly the case (asciidoctor seems no harder > to acquire on most modern systems than asciidoc; there are system > packages in most cases, and decent binary-package systems for both ruby > and python if you really need it). > > It does create a situation where people like Randall on NonStop might > need to do part of their dev work on another, more mainstream platform. > But I suspect that is already the case. Or use distributed tarballs with already built documentation. I don't see any big issue with cross-compilation though. > > When we looked at xmlto-less rendering around two years ago , we > > found various asciidoctor bugs up to and around version 2.0. We would > > likely need to require some >=2.0.x. The exact requirements will > > probably only become clear when someone really does the work. > > That does make things a little less convenient; Debian stable, for > instance, still has 1.5.8. And it has git 2.20.1, released at the end of 2018. I've never understood developers worried about how the bleeding edge would build in ancient platforms, when ancient platforms don't care about the bleeding edge. > It's not too hard to install an updated gem, but not quite as nice as > using the system package (it also makes things weird for building the > stable Debian package itself, which would want to rely only on other > packages; but of course any proposed change to the doc toolchain would > be for new versions, and would not get backported there anyway). Anyone trying to build git master on top of Debian stable 1. probably can live with the output of the current doc toolchain, and 2. probably doesn't exist. I'm not sure how much the git project gains by worrying about such a hypothetical person. > > I think what I'm arguing for is > > > > 1) switch the default to asciidoctor, > > 2) enable optionally using it without xmlto, > > 3) figure out what broke and fix it, and document which is the minimum > > asciidoctor version we're going to bother with for (2), > > 4) lather, rinse, repeat (3), > > 5) switch the default to not using xmlto, > > 6) drop the xmlto way of generating the manpages(?). > > I'm unclear when support for python asciidoc goes away here. Is it part > of step 6 (because it does not have another way of generating them)? Or > does it live on forever as a non-default legacy system? I'd prefer not, > but as long as we are clear about the primary target and leave it up to > people interested in the legacy to do the compat fixes, that might be > OK. How about we leave the legacy system in place as an alternative, and decide later what to do with it? -- Felipe Contreras
next prev parent reply other threads:[~2021-05-11 4:28 UTC|newest] Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-05-07 6:06 Bagas Sanjaya 2021-05-07 12:02 ` Randall S. Becker 2021-05-07 22:55 ` Felipe Contreras 2021-05-07 22:57 ` brian m. carlson 2021-05-08 1:42 ` Randall S. Becker 2021-05-07 12:27 ` Đoàn Trần Công Danh 2021-05-07 12:47 ` Bagas Sanjaya 2021-05-07 23:03 ` Felipe Contreras 2021-05-08 4:27 ` Bagas Sanjaya 2021-05-07 20:25 ` brian m. carlson 2021-05-07 22:19 ` Jeff King 2021-05-08 4:22 ` Bagas Sanjaya 2021-05-09 8:20 ` Martin Ågren 2021-05-09 18:46 ` Felipe Contreras 2021-05-10 18:43 ` Martin Ågren 2021-05-10 22:24 ` Jeff King 2021-05-11 4:27 ` Felipe Contreras [this message] 2021-05-11 6:13 ` Jeff King 2021-05-11 8:03 ` Felipe Contreras 2021-05-11 12:44 ` Ævar Arnfjörð Bjarmason 2021-05-11 19:00 ` Felipe Contreras 2021-05-11 19:09 ` Jeff King 2021-05-11 20:22 ` Felipe Contreras 2021-05-11 23:14 ` brian m. carlson 2021-05-12 1:44 ` Felipe Contreras 2021-05-11 18:45 ` Martin Ågren 2021-05-11 19:07 ` Jeff King 2021-05-11 19:11 ` Martin Ågren 2021-05-11 20:14 ` Felipe Contreras 2021-05-11 9:04 ` Jean-Noël Avila 2021-05-11 18:54 ` Martin Ågren 2021-05-07 23:35 ` Felipe Contreras 2021-05-07 23:57 ` brian m. carlson 2021-05-08 3:10 ` Jeff King 2021-05-08 3:23 ` Jeff King 2021-05-09 0:22 ` brian m. carlson 2021-05-09 8:29 ` Martin Ågren 2021-05-07 22:48 ` Felipe Contreras
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 List information: http://vger.kernel.org/majordomo-info.html * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [RFC suggestion] Generate manpage directly with Asciidoctor' \ /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
Code repositories for project(s) associated with this inbox: https://80x24.org/mirrors/git.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).