* Re: mdocml: The sh(1) "test" builtin on Solaris 10 doesn't have -e, even
[not found] <1783826185453982422.enqueue@fantadrom.bsd.lv>
@ 2015-11-07 21:46 ` Joerg Sonnenberger
2015-11-07 22:07 ` Ingo Schwarze
0 siblings, 1 reply; 3+ messages in thread
From: Joerg Sonnenberger @ 2015-11-07 21:46 UTC (permalink / raw)
To: discuss
On Sat, Nov 07, 2015 at 04:31:52PM -0500, schwarze@mdocml.bsd.lv wrote:
> Log Message:
> -----------
> The sh(1) "test" builtin on Solaris 10 doesn't have -e,
> even though that's required by POSIX.
> Use -w and -r, that's just as good.
/bin/sh on Solaris 10 is simply broken, I don't think we should try to
workaround it...
Joerg
--
To unsubscribe send an email to discuss+unsubscribe@mdocml.bsd.lv
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: mdocml: The sh(1) "test" builtin on Solaris 10 doesn't have -e, even
2015-11-07 21:46 ` mdocml: The sh(1) "test" builtin on Solaris 10 doesn't have -e, even Joerg Sonnenberger
@ 2015-11-07 22:07 ` Ingo Schwarze
2015-11-07 22:15 ` Joerg Sonnenberger
0 siblings, 1 reply; 3+ messages in thread
From: Ingo Schwarze @ 2015-11-07 22:07 UTC (permalink / raw)
To: Joerg Sonnenberger; +Cc: discuss
Hi Joerg,
Joerg Sonnenberger wrote on Sat, Nov 07, 2015 at 10:46:56PM +0100:
> On Sat, Nov 07, 2015 at 04:31:52PM -0500, schwarze@mdocml.bsd.lv wrote:
>> Log Message:
>> -----------
>> The sh(1) "test" builtin on Solaris 10 doesn't have -e,
>> even though that's required by POSIX.
>> Use -w and -r, that's just as good.
> /bin/sh on Solaris 10 is simply broken, I don't think we should try to
> workaround it...
You mean, we should tell people to just run ./configure as
/usr/bin/ksh ./configure
on Solaris 10? Maybe...
You are certainly right that we shouldn't make things complicated
trying to work around broken shells.
This particular change doesn't do any harm elsewhere, though, and
it doesn't add a single byte to the code, so i don't think i need
to revert it.
Yours,
Ingo
--
To unsubscribe send an email to discuss+unsubscribe@mdocml.bsd.lv
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: mdocml: The sh(1) "test" builtin on Solaris 10 doesn't have -e, even
2015-11-07 22:07 ` Ingo Schwarze
@ 2015-11-07 22:15 ` Joerg Sonnenberger
0 siblings, 0 replies; 3+ messages in thread
From: Joerg Sonnenberger @ 2015-11-07 22:15 UTC (permalink / raw)
To: discuss
On Sat, Nov 07, 2015 at 11:07:26PM +0100, Ingo Schwarze wrote:
> Hi Joerg,
>
> Joerg Sonnenberger wrote on Sat, Nov 07, 2015 at 10:46:56PM +0100:
> > On Sat, Nov 07, 2015 at 04:31:52PM -0500, schwarze@mdocml.bsd.lv wrote:
>
> >> Log Message:
> >> -----------
> >> The sh(1) "test" builtin on Solaris 10 doesn't have -e,
> >> even though that's required by POSIX.
> >> Use -w and -r, that's just as good.
>
> > /bin/sh on Solaris 10 is simply broken, I don't think we should try to
> > workaround it...
>
> You mean, we should tell people to just run ./configure as
>
> /usr/bin/ksh ./configure
>
> on Solaris 10? Maybe...
>
> You are certainly right that we shouldn't make things complicated
> trying to work around broken shells.
>
> This particular change doesn't do any harm elsewhere, though, and
> it doesn't add a single byte to the code, so i don't think i need
> to revert it.
That or IIRC the xpg4 version they have around, which is nowhere near as
broken. The commit just seems to make things less readable.
Joerg
--
To unsubscribe send an email to discuss+unsubscribe@mdocml.bsd.lv
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-11-07 22:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1783826185453982422.enqueue@fantadrom.bsd.lv>
2015-11-07 21:46 ` mdocml: The sh(1) "test" builtin on Solaris 10 doesn't have -e, even Joerg Sonnenberger
2015-11-07 22:07 ` Ingo Schwarze
2015-11-07 22:15 ` Joerg Sonnenberger
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).