git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Jeff King <peff@peff.net>
To: Armin Kunaschik <megabreit@googlemail.com>
Cc: git@vger.kernel.org
Subject: Re: Portability of git shell scripts?
Date: Wed, 4 May 2016 17:20:28 -0400	[thread overview]
Message-ID: <20160504212028.GG21259@sigill.intra.peff.net> (raw)
In-Reply-To: <CALR6jEh5dAcnqiyo4kXkj+8imfQQd0nT=baPOW_qbJpJwmFsyw@mail.gmail.com>

On Wed, May 04, 2016 at 08:17:38PM +0200, Armin Kunaschik wrote:

> I'm trying to compile/test/use git 2.8.2 on AIX 6.1 with no bash available.
> /bin/sh is a hard link to /bin/ksh which is a ksh88, a posix shell.
> Is this supposed to work?

We aim for a practical subset of Bourne shells, including bash, dash,
ash, ksh, etc. There's at least one Bourne-ish shell known not to work,
which is Solaris /bin/sh[1]. POSIX is usually a good guide, but we aim
for practical portability more than adhering strictly to the standards
document.

I've tested with mksh in the past (though it's possible that we've
introduced a regression since then). But I think we've run into problems
with ksh93[2]. I don't know about ksh88, or what construct it doesn't
like.  It may or may not be easy to work around.

> As an example: make test fails on nearly every t34* test and on tests
> which contain rebase.
> The installation of bash (and manually changing the shebang to
> /bin/bash) "fixes" all rebase test failures. So obviously git-rebase
> is not portable at some point.

Right. Any modern-ish Bourne shell will do, so moving to bash is one way
to fix it.

> Does it make any sense to put work into making these scripts portable,
> that is, work with posix shells?

Maybe. :) If you can find what it is that ksh88 is unhappy with, we can
see how painful it is to adapt to. But given my looking into ksh93 in
[2], I suspect it will be easier to just use a more modern shell.

> And, as last resort, is it possible to configure git use bash in some
> or all shell scripts?

You can set SHELL_PATH in your config.mak file.

-Peff

[1] Solaris /bin/sh doesn't even understand $(), so we declared it as
    hopeless long ago. I think most people just replace it with bash,
    but I suspect /usr/xpg6/bin/sh probably works, too.

[2] http://thread.gmane.org/gmane.comp.version-control.git/268657/focus=268666

  parent reply	other threads:[~2016-05-04 21:20 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-04 18:17 Portability of git shell scripts? Armin Kunaschik
2016-05-04 18:35 ` Torsten Bögershausen
2016-05-04 21:20 ` Jeff King [this message]
2016-05-06 16:57   ` Armin Kunaschik
2016-05-07  2:53     ` 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

  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 \
    --in-reply-to=20160504212028.GG21259@sigill.intra.peff.net \
    --to=peff@peff.net \
    --cc=git@vger.kernel.org \
    --cc=megabreit@googlemail.com \
    /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.
Code repositories for project(s) associated with this public 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).