Pepper_15 examples don't build under Windows

[GoogleCodeExporter]

(Reported by jonovos:)

Problem with pepper_15  in Windows: (SCons)
1.: My environment directory tree is C:\NaCl_sdk    ==> Open a dos
box.
2.: ==> cd  \NaCl_sdk\pepper_15\examples
3.: ==> C:\NaCl_sdk\pepper_15\examples> scons
4.: ==>  RESULT == "Error: NACL_SDK_ROOT is not set.",        HOWEVER,
immediately following this error echo text,        the dos-box EXITS
with an "exit 1" at the end of the .BAT
5.: ==> So, add this line to windows environment var's:       
NACL_SDK_ROOT=C:\NaCl_sdk\        * Warning! this MUST END with the
back-slash
6.: ==> REPEAT: scons
7.: ==> ImportError:
C:\NaCl_sdk\pepper_15\examples>scons
Traceback (most recent call last):  File "C:\NaCl_sdk
\pepper_15\third_party\scons-2.0.1\script\scons",  line 198, in
<module> import SCons.ScriptImportError: No module named SCons.Script
C:\NaCl_sdk\pepper_15\examples>
8.: ==> Possible solution ...        - ISSUE  PRAYERS TO CAT GOD-DESS 
      - BURN 12-GRAMS OF TUNA FLAVORED INCENSE        - BEWARE: This
solution might not work.

Original issue reported on code.google.com by mball@google.com on 25 Sep 2011 at 4:22

[GoogleCodeExporter]

This affects pepper_15, revision 1199 with sdk_tools 1.4

Original comment by mball@google.com on 25 Sep 2011 at 4:27

[GoogleCodeExporter]

Fixed suggested by jonovos:

Under Windows, to compile the examples in  pepper_15,  the following
MUST be done:

1.: Set environment variable in dos, WITH TRAILING Back-Slash(\):
    ===> set NACL_SDK_ROOT=C:\NaCl\sdk\

2.: EDIT the SCons batch file:  ......\pepper_15\examples\scons.bat

    At approx. Line#26: DELETE the QUOTE-MARKS:

   ===> INCORRECT: set NACL_TARGET_PLATFORM="pepper_15"
   ===> FIXED: set NACL_TARGET_PLATFORM=pepper_15

Original comment by mball@google.com on 25 Sep 2011 at 4:28

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

Original comment by dsprin...@chromium.org on 26 Sep 2011 at 6:01

• Changed state: Started

[GoogleCodeExporter]

Fixed with CL: http://codereview.chromium.org/8044023/
Pushed as r1208.

Original comment by dsprin...@chromium.org on 26 Sep 2011 at 10:39

• Changed state: Fixed