Properly define (or not) LSB_FIRST and FRAMESKIP instead of setting them to

1 or 0. Also, if the LSB_FIRST option is unspecified, check the host system's
byte order instead of assuming little-endian.

SConstruct

@@ -1,18 +1,17 @@
 import os
+import sys
 
 # XXX path separator fixed right now
 opts = Options()
 opts.AddOptions(
   BoolOption('PSS_STYLE', 'Path separator style', 1),
-  BoolOption('LSB_FIRST', 'Least significant byte first?', 1),
+  BoolOption('LSB_FIRST', 'Least significant byte first?', None),
   BoolOption('FRAMESKIP', 'Enable frameskipping', 1),
   BoolOption('OPENGL',    'Enable OpenGL support', 1)
 )
 
 env = Environment(options = opts,
-                  CPPDEFINES={'PSS_STYLE' : '${PSS_STYLE}',
+                  CPPDEFINES={'PSS_STYLE' : '${PSS_STYLE}'})
-                              'LSB_FIRST' : '${LSB_FIRST}',
-                              'FRAMESKIP' : '${FRAMESKIP}'})
 
 if os.environ.has_key('CC'):
   env.Replace(CC = os.environ['CC'])
@@ -47,6 +46,17 @@ if env['OPENGL'] and conf.CheckLibWithHeader('GL', 'GL/gl.h', 'c++', autoadd=1):
 
 env = conf.Finish()
 
+# option specified
+if env.has_key('LSB_FIRST'):
+  if env['LSB_FIRST']:
+    env.Append(CCFLAGS = ' -DLSB_FIRST')
+# option not specified, check host system
+elif sys.byteorder == 'little':
+  env.Append(CCFLAGS = ' -DLSB_FIRST')
+
+if env['FRAMESKIP']:
+  env.Append(CCFLAGS = ' -DFRAMESKIP')
+
 # parse SDL cflags/libs
 env.ParseConfig('sdl-config --cflags --libs')