From: Stefano Lattarini <stefano.lattarini@gmail.com>
To: Elia Pinto <gitter.spiros@gmail.com>
Cc: "Jonathan Nieder" <jrnieder@gmail.com>,
"Felipe Contreras" <felipe.contreras@gmail.com>,
git@vger.kernel.org, "Junio C Hamano" <gitster@pobox.com>,
"Jeff King" <peff@peff.net>, "Ævar Arnfjörð" <avarab@gmail.com>,
"Johannes Sixt" <j6t@kdbg.org>
Subject: Re: [PATCH] test-lib: avoid full path to store test results
Date: Wed, 31 Oct 2012 10:05:27 +0100 [thread overview]
Message-ID: <5090E9D7.1010706@gmail.com> (raw)
In-Reply-To: <CA+EOSBmTjwmf+dO-dgU+rGQaVEKDZw7u9ujrh5jYZkPM2zisOA@mail.gmail.com>
On 10/30/2012 11:17 PM, Elia Pinto wrote:
> Thanks. I know that posix support these usages, but exists some
> traditional shell that not support it.
>
True, but those shells are not POSIX shells -- the major example that
comes to mind is the accursed Solaris /bin/sh.
Since Git assumes a POSIX shell in its scripts and testsuite, use of
any POSIX feature should be fine -- until someone can show a real-world
POSIX shell that (likely due to a bug) fails to grasp such feature, in
which case a "pragmatic" workaround is needed.
Oh, and BTW, there are talks (and mostly consensus) among the Autotools
developers to start requiring a POSIX shell in the configure scripts
and Makefile recipes in the near future:
<http://lists.gnu.org/archive/html/bug-autoconf/2012-06/msg00009.html>
And also, related:
<http://lists.gnu.org/archive/html/automake/2012-08/msg00046.html>
<http://lists.gnu.org/archive/html/coreutils/2012-10/msg00127.html>
>These are described in the
> autoconf manual, last time i have checked. As the construct ; export
> var = x should be portable, but it is not.
>
I don't think POSIX requires that to be portable.
> If this is important these days i don't know.
>
I hope the above helps to clarify the matter a little.
Regards,
Stefano
prev parent reply other threads:[~2012-10-31 9:06 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-30 4:12 [PATCH] test-lib: avoid full path to store test results Felipe Contreras
2012-10-30 4:28 ` Jeff King
2012-10-30 4:39 ` Felipe Contreras
2012-10-30 4:46 ` Jonathan Nieder
2012-10-30 16:02 ` Felipe Contreras
2012-10-31 1:27 ` Jonathan Nieder
2012-10-31 1:59 ` Felipe Contreras
2012-10-31 2:13 ` Jonathan Nieder
2012-10-31 2:28 ` Felipe Contreras
2012-10-31 18:02 ` Johannes Sixt
2012-10-31 18:28 ` Felipe Contreras
2012-11-02 13:17 ` Jeff King
2012-11-02 15:17 ` Felipe Contreras
2012-11-02 15:20 ` Jeff King
2012-10-30 6:58 ` Elia Pinto
2012-10-30 7:01 ` Jonathan Nieder
2012-10-30 22:17 ` Elia Pinto
2012-10-31 9:05 ` Stefano Lattarini [this message]
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=5090E9D7.1010706@gmail.com \
--to=stefano.lattarini@gmail.com \
--cc=avarab@gmail.com \
--cc=felipe.contreras@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=gitter.spiros@gmail.com \
--cc=j6t@kdbg.org \
--cc=jrnieder@gmail.com \
--cc=peff@peff.net \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.