Issue 2162: YACC fix for OS X
authorGregNoel <GregNoel@fdb21ef1-2011-0410-befe-b5e4ea1792b1>
Thu, 14 Aug 2008 23:33:59 +0000 (23:33 +0000)
committerGregNoel <GregNoel@fdb21ef1-2011-0410-befe-b5e4ea1792b1>
Thu, 14 Aug 2008 23:33:59 +0000 (23:33 +0000)
git-svn-id: http://scons.tigris.org/svn/scons/trunk@3275 fdb21ef1-2011-0410-befe-b5e4ea1792b1

src/engine/SCons/Tool/yacc.py
test/YACC/live.py

index 0b648e816633e73ae5257912bb5444e3eae5122c..3d13dd0e8892b596e87acd0649ac5d2fae7036f0 100644 (file)
@@ -109,12 +109,15 @@ def generate(env):
     env['YACCCOM']   = '$YACC $YACCFLAGS -o $TARGET $SOURCES'
     env['YACCHFILESUFFIX'] = '.h'
 
-    if env['PLATFORM'] == 'darwin':
-        # Bison on Mac OS X just appends ".h" to the generated target .cc
-        # or .cpp file name.  Hooray for delayed expansion of variables.
-        env['YACCHXXFILESUFFIX'] = '${TARGET.suffix}.h'
-    else:
-        env['YACCHXXFILESUFFIX'] = '.hpp'
+    # Apparently, OS X now creates file.hpp like everybody else
+    # I have no idea when it changed; it was fixed in 10.4
+    #if env['PLATFORM'] == 'darwin':
+    #    # Bison on Mac OS X just appends ".h" to the generated target .cc
+    #    # or .cpp file name.  Hooray for delayed expansion of variables.
+    #    env['YACCHXXFILESUFFIX'] = '${TARGET.suffix}.h'
+    #else:
+    #    env['YACCHXXFILESUFFIX'] = '.hpp'
+    env['YACCHXXFILESUFFIX'] = '.hpp'
 
     env['YACCVCGFILESUFFIX'] = '.vcg'
 
index 3c5050ec565dd35ae8e17ff313ebefa950784c96..88cbafbe5c60cbd16d359cb8865bb92cd9f0d303 100644 (file)
@@ -56,7 +56,7 @@ yacc = foo.Dictionary('YACC')
 bar = Environment(YACC = r'%(_python_)s wrapper.py ' + yacc)
 foo.Program(target = 'foo', source = 'foo.y')
 bar.Program(target = 'bar', source = 'bar.y')
-foo.Program(target = 'hello', source = ['hello.cpp']) 
+foo.Program(target = 'hello', source = ['hello.cpp'])
 foo.CXXFile(target = 'file.cpp', source = ['file.yy'], YACCFLAGS='-d')
 foo.CFile(target = 'not_foo', source = 'foo.y')
 """ % locals())
@@ -103,11 +103,14 @@ graph:        GRAPH_T
 %%
 """)
 
-import sys
-if sys.platform[:6] == 'darwin':
-   file_hpp = 'file.cpp.h'
-else:
-   file_hpp = 'file.hpp'
+# Apparently, OS X now creates file.hpp like everybody else
+# I have no idea when it changed; it was fixed in 10.4
+#import sys
+#if sys.platform[:6] == 'darwin':
+#   file_hpp = 'file.cpp.h'
+#else:
+#   file_hpp = 'file.hpp'
+file_hpp = 'file.hpp'
 
 test.write("hello.cpp", """\
 #include "%(file_hpp)s"