git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
To: Thomas Rast <trast@inf.ethz.ch>
Cc: "Jeff King" <peff@peff.net>, "Junio C Hamano" <gitster@pobox.com>,
	"Thomas Rast" <trast@student.ethz.ch>,
	"Carlos Martín Nieto" <cmn@elego.de>,
	"Matthieu Moy" <Matthieu.Moy@imag.fr>,
	git@vger.kernel.org
Subject: Re: "docs: stop using asciidoc no-inline-literal" breaks asciidoc 8.2.5
Date: Tue, 29 May 2012 14:31:38 +0200	[thread overview]
Message-ID: <CACBZZX5iOB5PCZUCX5ksR4hpw+DTLTaP4jOnyHAJ8hZVe9xo5w@mail.gmail.com> (raw)
In-Reply-To: <87ehq3mbxg.fsf@thomas.inf.ethz.ch>

On Tue, May 29, 2012 at 2:03 PM, Thomas Rast <trast@inf.ethz.ch> wrote:
> Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
>
>> Jeff King's 6cf378f0cbe7c7f944637892caeb9058c90a185a broke my Git
>> build on CentOS 5.5. The patch suggests that it only breaks
>> compatibility with asciidoc 7 but that isn't actually the case.
>
> The commit message of 6cf378f0 is not correct in that respect.  My
> ancient 71c020c5 has the right numbers: asciidoc 8.4.1 is the one that
> introduced the new inline-literal behavior.  Based on my little survey
> in
>
>  http://thread.gmane.org/gmane.comp.version-control.git/191738/focus=191790
>
> we decided that "nobody" really uses such an old asciidoc any more.
> Evidently you are a counterexample.

Well, me and and others using current production RHEL releases:

    $ cat /etc/redhat-release && yum info asciidoc.noarch|grep Version
    CentOS release 5.5 (Final)
    Version    : 8.1.0
    $ cat /etc/redhat-release && yum info asciidoc.noarch|grep Version
    CentOS release 5.5 (Final)
    Version    : 8.2.5

I can't quite recall how I ended up with 8.2.5 on one of the boxes,
but it seems 8.1.0 may actually be the stock version.

>> I suggest just ejecting this patch and trying again, these RedHat
>> systems are still used in a lot of environments, especially by various
>> companies.
>
> Do you/they have to *build* the docs, as opposed to using the prebuilt
> ones coming from Junio?  Perhaps we can make it so 'make man' refuses to
> run if asciidoc is too old, and give a message to the effect that you
> should 'make quick-install-man' instead.
>
> Otherwise we'll be working around the f{asterisk}{asterisk}{asterisk}ing
> quoting rules for years to come.

I could personally change my build process to do something else, but
the reason I'm running "all doc" occasionally on this older toolchain
is to smoke out issues like these for other users on slightly older
toolchains as well.

I think it's too soon to break the stock toolchain on systems as
recent as RHEL 5.5 in the name of slightly nicer asciidoc syntax.

We could also keep the nice syntax and have some simple sed-based
pre-processor that converts the syntax to the older and more widely
supported version.

Or we could just decide to break RHEL 5 and systems released at a
similar time, but that isn't what the patch suggested it was doing, so
we should probably step back and ponder whether that's something we
want to do.

Personally I think having a hard dependency on a version of a software
package released more recently than 2007 (8.2.5 came out on
2007-11-18) is way too soon, it would be similar to breaking on a GCC
older than 4.2.0 which came out around the same time.

  reply	other threads:[~2012-05-29 12:32 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-29 10:19 "docs: stop using asciidoc no-inline-literal" breaks asciidoc 8.2.5 Ævar Arnfjörð Bjarmason
2012-05-29 12:03 ` Thomas Rast
2012-05-29 12:31   ` Ævar Arnfjörð Bjarmason [this message]
2012-05-29 19:45     ` Junio C Hamano
2012-05-29 22:01       ` Jeff King
2012-05-29 21:56     ` Jeff King
2012-05-29 22:05       ` Dennis Kaarsemaker
2012-05-29 22:42       ` Ævar Arnfjörð Bjarmason
2012-05-29 22:51         ` Ævar Arnfjörð Bjarmason
2012-05-30  8:51         ` Jeff King
2012-05-30  9:02           ` Matthieu Moy
2012-05-30 10:17       ` [PATCH 0/3] docs: asciidoc version tweaks Jeff King
2012-05-30 10:18         ` [PATCH 1/3] INSTALL: update asciidoc recommendation Jeff King
2012-05-30 10:22         ` [PATCH 2/3] docs: drop asciidoc7compatible flag Jeff King
2012-05-30 10:22         ` [PATCH 3/3] docs: drop antique comment from Makefile Jeff King

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=CACBZZX5iOB5PCZUCX5ksR4hpw+DTLTaP4jOnyHAJ8hZVe9xo5w@mail.gmail.com \
    --to=avarab@gmail.com \
    --cc=Matthieu.Moy@imag.fr \
    --cc=cmn@elego.de \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=peff@peff.net \
    --cc=trast@inf.ethz.ch \
    --cc=trast@student.ethz.ch \
    /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;
as well as URLs for NNTP newsgroup(s).