* 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).