yugui	2012-08-27 10:22:53 +0900 (Mon, 27 Aug 2012)

  New Revision: 36828

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=36828

  Log:
    * nacl/GNUmakefile.in (CC, LD, NM, AR, AS, RANLIB, OBJDUMP, OBJCOPY)
      Rewrites these variables instead of PATH.
      NaCl port uses a toolchain which is specified by NACL_SDK_ROOT
      environment variable. Originally, NaCl build added the toolchain
      under the NACL_SDK_ROOT to the PATH. But updating PATH doesn't work
      on Mac.
      (RBCONFIG): Replaces configs with the variable updates above.
    
    * configure.in: Thus it is no longer necessary to check $PATH.
    
    Based on a patch by Takashi Toyoshima <toyoshim AT gmail.com>.

  Modified files:
    trunk/ChangeLog
    trunk/configure.in
    trunk/nacl/GNUmakefile.in

Index: configure.in
===================================================================
--- configure.in	(revision 36827)
+++ configure.in	(revision 36828)
@@ -111,11 +111,7 @@
   AC_MSG_RESULT(${NACL_TOOLCHAIN})
 
   AC_MSG_CHECKING([path to SDK])
-  if echo -- "${PATH}" | grep -F "${NACL_SDK_ROOT}/toolchain/${NACL_TOOLCHAIN}/bin" > /dev/null; then
-    AC_MSG_RESULT([set])
-  else
-    AC_MSG_RESULT([no])
-    AC_MSG_WARN([Add "${NACL_SDK_ROOT}/toolchain/${NACL_TOOLCHAIN}/bin" to \$PATH])
+  if ! echo -- "${PATH}" | grep -F "${NACL_SDK_ROOT}/toolchain/${NACL_TOOLCHAIN}/bin" > /dev/null; then
     PATH="${PATH}:${NACL_SDK_ROOT}/toolchain/${NACL_TOOLCHAIN}/bin"
   fi
 
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 36827)
+++ ChangeLog	(revision 36828)
@@ -1,3 +1,17 @@
+Sun Aug 26 09:22:33 2012  Yuki Sonoda (Yugui)  <yugui@y...>
+
+	* nacl/GNUmakefile.in (CC, LD, NM, AR, AS, RANLIB, OBJDUMP, OBJCOPY)
+	  Rewrites these variables instead of PATH.
+	  NaCl port uses a toolchain which is specified by NACL_SDK_ROOT
+	  environment variable. Originally, NaCl build added the toolchain
+	  under the NACL_SDK_ROOT to the PATH. But updating PATH doesn't work
+	  on Mac.
+	  (RBCONFIG): Replaces configs with the variable updates above.
+
+	* configure.in: Thus it is no longer necessary to check $PATH.
+
+	Based on a patch by Takashi Toyoshima <toyoshim AT gmail.com>.
+
 Sun Aug 26 16:53:00 2012  Nobuyoshi Nakada  <nobu@r...>
 
 	* insns.def (checkmatch): suppress warnings.  [ruby-core:47310]
Index: nacl/GNUmakefile.in
===================================================================
--- nacl/GNUmakefile.in	(revision 36827)
+++ nacl/GNUmakefile.in	(revision 36828)
@@ -7,7 +7,14 @@
 NACL_SDK_ROOT=@NACL_SDK_ROOT@
 NACL_TOOLCHAIN=@NACL_TOOLCHAIN@
 NACL_TOOLCHAIN_DIR=$(NACL_SDK_ROOT)/toolchain/$(NACL_TOOLCHAIN)
-PATH+=:$(NACL_TOOLCHAIN_DIR)/bin
+CC:=$(NACL_TOOLCHAIN_DIR)/bin/$(CC)
+LD:=$(NACL_TOOLCHAIN_DIR)/bin/$(LD)
+NM:=$(NACL_TOOLCHAIN_DIR)/bin/$(NM)
+AR:=$(NACL_TOOLCHAIN_DIR)/bin/$(AR)
+AS:=$(NACL_TOOLCHAIN_DIR)/bin/$(AS)
+RANLIB:=$(NACL_TOOLCHAIN_DIR)/bin/$(RANLIB)
+OBJDUMP:=$(NACL_TOOLCHAIN_DIR)/bin/$(OBJDUMP)
+OBJCOPY:=$(NACL_TOOLCHAIN_DIR)/bin/$(OBJCOPY)
 PYTHON=@PYTHON@
 
 PPROGRAM=pepper-$(PROGRAM)
@@ -40,6 +47,19 @@
 	$(Q) $(OBJCOPY) --weaken-symbol=rb_file_load_ok $@.tmp $@
 	@-$(RM) $@.tmp
 
+$(RBCONFIG):
+	@sed \
+	  -e 's!CONFIG\["CC"\] = .*!CONFIG\["CC"\] = "$(CC)"!' \
+	  -e 's!CONFIG\["LD"\] = .*!CONFIG\["LD"\] = "$(LD)"!' \
+	  -e 's!CONFIG\["NM"\] = .*!CONFIG\["NM"\] = "$(NM)"!' \
+	  -e 's!CONFIG\["AR"\] = .*!CONFIG\["AR"\] = "$(AR)"!' \
+	  -e 's!CONFIG\["AS"\] = .*!CONFIG\["AS"\] = "$(AS)"!' \
+	  -e 's!CONFIG\["RANLIB"\] = .*!CONFIG\["RANLIB"\] = "$(RANLIB)"!' \
+	  -e 's!CONFIG\["OBJDUMP"\] = .*!CONFIG\["OBJDUMP"\] = "$(OBJDUMP)"!' \
+	  -e 's!CONFIG\["OBJCOPY"\] = .*!CONFIG\["OBJCOPY"\] = "$(OBJCOPY)"!' \
+	-i.bak rbconfig.rb
+	@touch .rbconfig.time
+
 all: pprogram
 main: $(PROGRAM_NMF)
 pprogram: showflags $(PPROGRAM) $(PPROGRAM_NMF)
@@ -53,8 +73,13 @@
 showflags: show_naclflags
 
 show_naclflags:
-	@echo "      PATH = $(PATH)"
-	@echo "      NACL_SDK_ROOT = $(NACL_SDK_ROOT)"
+	@echo "        NACL_SDK_ROOT = $(NACL_SDK_ROOT)"
+	@echo "        NM = $(NM)"
+	@echo "        AR = $(AR)"
+	@echo "        AS = $(AS)"
+	@echo "        RANLIB = $(RANLIB)"
+	@echo "        OBJDUMP = $(OBJDUMP)"
+	@echo "        OBJCOPY = $(OBJCOPY)"
 
 clean-local::
 	-$(RM) $(PPROGRAM) pepper_main.$(OBJEXT) $(PROGRAM_NMF) $(PPRGORAM_NMF)

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/