Project

General

Profile

Actions

Bug #2612

closed

Use POSIX shell in Makefile

Bug #2612: Use POSIX shell in Makefile

Added by brianegge (Brian Egge) almost 16 years ago. Updated over 14 years ago.

Status:
Rejected
Assignee:
-
Target version:
-
ruby -v:
ruby 1.9.2dev (2010-01-19 trunk 26352) [i386-solaris2.10]
Backport:
[ruby-core:27616]

Description

=begin
For some systems, including Solaris 10, /bin/sh is not a POSIX shell. Certain features like the continue statement are missing from the /bin/sh shell on Solaris. Autoconf contains a macro which will set the appropriate POSIX shell for a system.

http://www.gnu.org/software/hello/manual/autoconf/The-Make-Macro-SHELL.html

The following allows ruby to be built, and more specifically, cleaned without modification on Solaris 10.

Index: Makefile.in

--- Makefile.in (revision 26352)
+++ Makefile.in (working copy)
@@ -1,4 +1,4 @@
-SHELL = /bin/sh
+SHELL = @SHELL@
NULLCMD = :
RUNCMD = $(SHELL)
CHDIR = @CHDIR@
=end

Updated by nobu (Nobuyoshi Nakada) almost 16 years ago Actions #1

=begin
Hi,

At Tue, 19 Jan 2010 11:49:56 +0900,
Brian Egge wrote in [ruby-core:27616]:

For some systems, including Solaris 10, /bin/sh is not a
POSIX shell. Certain features like the continue statement
are missing from the /bin/sh shell on Solaris. Autoconf
contains a macro which will set the appropriate POSIX shell
for a system.

I've heard that /bin/sh in Solaris 10 has continue statement.

What error occurs actually?

--
Nobu Nakada

=end

Updated by nobu (Nobuyoshi Nakada) almost 16 years ago Actions #2

  • Status changed from Open to Feedback

=begin

=end

Updated by mame (Yusuke Endoh) over 15 years ago Actions #3

  • Status changed from Feedback to Rejected

=begin
Hi, Brian Egge

I close this ticket because of absence of your response.

If you have any actual problem, please reopen this ticket with
answer to nobu.
Unless there is actual problem, we are not keen to change such
a build-related part because it may affect another platform.

--
Yusuke Endoh
=end

Actions

Also available in: PDF Atom