Add separate $RANLIBCOM and $RANLIBCOMSTR variables. (Christoph Schulz)
authorstevenknight <stevenknight@fdb21ef1-2011-0410-befe-b5e4ea1792b1>
Tue, 29 Nov 2005 06:52:44 +0000 (06:52 +0000)
committerstevenknight <stevenknight@fdb21ef1-2011-0410-befe-b5e4ea1792b1>
Tue, 29 Nov 2005 06:52:44 +0000 (06:52 +0000)
git-svn-id: http://scons.tigris.org/svn/scons/trunk@1399 fdb21ef1-2011-0410-befe-b5e4ea1792b1

src/CHANGES.txt
src/engine/SCons/Tool/__init__.py
src/engine/SCons/Tool/ar.py
src/engine/SCons/Tool/ar.xml
test/AR/ARCOM.py
test/AR/ARCOMSTR.py
test/RANLIB/RANLIB.py [moved from test/RANLIB.py with 100% similarity]
test/RANLIB/RANLIBCOM.py [new file with mode: 0644]
test/RANLIB/RANLIBCOMSTR.py [new file with mode: 0644]
test/RANLIB/RANLIBFLAGS.py [moved from test/RANLIBFLAGS.py with 100% similarity]

index 9b0154ca5828a8981f0d39829254830d46c8a77d..3c3ab20fe24972e6bfc5688df2d1780799121151 100644 (file)
@@ -722,6 +722,9 @@ RELEASE 0.97 - XXX
 
   - Add support for a $INSTALLSTR variable.
 
+  - Add support for $RANLIBCOM and $RANLIBCOMSTR variables (which fixes
+    a bug when setting $ARCOMSTR).
+
   From Craig Scott:
 
   - Have the Fortran module emitter look for Fortan modules to be created
index 528ee4d16cf726306d1f957275d35bd114c8b94d..77da0c71daf2214492707311c0a5b6c85ab35f9b 100644 (file)
@@ -169,8 +169,12 @@ def createStaticLibBuilder(env):
     try:
         static_lib = env['BUILDERS']['StaticLibrary']
     except KeyError:
-        ar_action = SCons.Action.Action("$ARCOM", "$ARCOMSTR")
-        static_lib = SCons.Builder.Builder(action = ar_action,
+        action_list = [ SCons.Action.Action("$ARCOM", "$ARCOMSTR") ]
+        if env.Detect('ranlib'):
+            ranlib_action = SCons.Action.Action("$RANLIBCOM", "$RANLIBCOMSTR")
+            action_list.append(ranlib_action)
+
+        static_lib = SCons.Builder.Builder(action = action_list,
                                            emitter = '$LIBEMITTER',
                                            prefix = '$LIBPREFIX',
                                            suffix = '$LIBSUFFIX',
index 92cab59ed8f9ec047c38edc3e3833fd01026576b..913da5fdd0db159102a36b2313ccdc2da77f99b9 100644 (file)
@@ -42,18 +42,16 @@ def generate(env):
     """Add Builders and construction variables for ar to an Environment."""
     SCons.Tool.createStaticLibBuilder(env)
 
-    arcom = '$AR $ARFLAGS $TARGET $SOURCES'
-    ranlib = 'ranlib'
-    if env.Detect(ranlib):
-        arcom = arcom + '\n$RANLIB $RANLIBFLAGS $TARGET'
-
     env['AR']          = 'ar'
-    env['ARFLAGS']     = SCons.Util.CLVar('r')
-    env['RANLIB']      = ranlib
-    env['RANLIBFLAGS'] = SCons.Util.CLVar('')
-    env['ARCOM']       = arcom
+    env['ARFLAGS']     = SCons.Util.CLVar('rc')
+    env['ARCOM']       = '$AR $ARFLAGS $TARGET $SOURCES'
     env['LIBPREFIX']   = 'lib'
     env['LIBSUFFIX']   = '.a'
 
+    if env.Detect('ranlib'):
+        env['RANLIB']      = 'ranlib'
+        env['RANLIBFLAGS'] = SCons.Util.CLVar('')
+        env['RANLIBCOM']   = '$RANLIB $RANLIBFLAGS $TARGET'
+
 def exists(env):
     return env.Detect('ar')
index baebd272afb6a8bab04f3bffe01ffd0b0dfa709d..24f886be8df70b5d2ed07c0820c57128cdd290d3 100644 (file)
@@ -46,6 +46,23 @@ The archive indexer.
 </summary>
 </cvar>
 
+<cvar name="RANLIBCOM">
+<summary>
+The command line used to index a static library archive.
+</summary>
+</cvar>
+
+<cvar name="RANLIBCOMSTR">
+<summary>
+The string displayed when a static library archive is indexed.
+If this is not set, then &cv-RANLIBCOM; (the command line) is displayed.
+
+<example>
+env = Environment(RANLIBCOMSTR = "Indexing $TARGET")
+</example>
+</summary>
+</cvar>
+
 <cvar name="RANLIBFLAGS">
 <summary>
 General options passed to the archive indexer.
index cf46559b48c0aa09fb814d9c74d383612a5df6be..c8ac1d87026662cc23f21eaa0510dfc6211204cd 100644 (file)
@@ -46,13 +46,17 @@ for f in sys.argv[2:]:
 sys.exit(0)
 """)
 
+test.write('myranlib.py', """
+""")
+
 test.write('SConstruct', """
 env = Environment(tools=['default', 'ar'],
                   ARCOM = r'%s myar.py $TARGET $SOURCES',
+                  RANLIBCOM = r'%s myranlib.py $TARGET',
                   LIBPREFIX = '',
                   LIBSUFFIX = '.lib')
 env.Library(target = 'output', source = ['file.1', 'file.2'])
-""" % python)
+""" % (python, python))
 
 test.write('file.1', "file.1\n/*ar*/\n")
 test.write('file.2', "file.2\n/*ar*/\n")
index 933a4e18ec292335ad206901765c834db2515a14..f20adb52fb368435bb7788ff6115be76e3bb114e 100644 (file)
@@ -47,21 +47,26 @@ for f in sys.argv[2:]:
 sys.exit(0)
 """)
 
+test.write('myranlib.py', """
+""")
+
 test.write('SConstruct', """
 env = Environment(tools=['default', 'ar'],
                   ARCOM = r'%s myar.py $TARGET $SOURCES',
                   ARCOMSTR = 'Archiving $TARGET from $SOURCES',
+                  RANLIBCOM = r'%s myranlib.py $TARGET',
                   LIBPREFIX = '',
                   LIBSUFFIX = '.lib')
 env.Library(target = 'output', source = ['file.1', 'file.2'])
-""" % python)
+""" % (python, python))
 
 test.write('file.1', "file.1\n/*ar*/\n")
 test.write('file.2', "file.2\n/*ar*/\n")
 
 test.run(stdout = test.wrap_stdout("""\
 Archiving output.lib from file.1 file.2
-"""))
+%s myranlib.py output.lib
+""" % python))
 
 test.must_match('output.lib', "file.1\nfile.2\n")
 
similarity index 100%
rename from test/RANLIB.py
rename to test/RANLIB/RANLIB.py
diff --git a/test/RANLIB/RANLIBCOM.py b/test/RANLIB/RANLIBCOM.py
new file mode 100644 (file)
index 0000000..25f2f61
--- /dev/null
@@ -0,0 +1,76 @@
+#!/usr/bin/env python
+#
+# __COPYRIGHT__
+#
+# Permission is hereby granted, free of charge, to any person obtaining
+# a copy of this software and associated documentation files (the
+# "Software"), to deal in the Software without restriction, including
+# without limitation the rights to use, copy, modify, merge, publish,
+# distribute, sublicense, and/or sell copies of the Software, and to
+# permit persons to whom the Software is furnished to do so, subject to
+# the following conditions:
+#
+# The above copyright notice and this permission notice shall be included
+# in all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY
+# KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE
+# WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+# NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+# LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+# OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+# WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+#
+
+__revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
+
+"""
+Test the ability to configure the $RANLIBCOM construction variable.
+"""
+
+import TestSCons
+
+python = TestSCons.python
+
+test = TestSCons.TestSCons()
+
+
+
+test.write('myar.py', """
+import sys
+outfile = open(sys.argv[1], 'wb')
+for f in sys.argv[2:]:
+    infile = open(f, 'rb')
+    for l in filter(lambda l: l != '/*ar*/\\n', infile.readlines()):
+        outfile.write(l)
+sys.exit(0)
+""")
+
+test.write('myranlib.py', """
+import sys
+lines = open(sys.argv[1], 'rb').readlines()
+outfile = open(sys.argv[1], 'wb')
+for l in filter(lambda l: l != '/*ranlib*/\\n', lines):
+    outfile.write(l)
+sys.exit(0)
+""")
+
+test.write('SConstruct', """
+env = Environment(tools=['default', 'ar'],
+                  ARCOM = r'%s myar.py $TARGET $SOURCES',
+                  RANLIBCOM = r'%s myranlib.py $TARGET',
+                  LIBPREFIX = '',
+                  LIBSUFFIX = '.lib')
+env.Library(target = 'output', source = ['file.1', 'file.2'])
+""" % (python, python))
+
+test.write('file.1', "file.1\n/*ar*/\n/*ranlib*/\n")
+test.write('file.2', "file.2\n/*ar*/\n/*ranlib*/\n")
+
+test.run(arguments = '.')
+
+test.must_match('output.lib', "file.1\nfile.2\n")
+
+
+
+test.pass_test()
diff --git a/test/RANLIB/RANLIBCOMSTR.py b/test/RANLIB/RANLIBCOMSTR.py
new file mode 100644 (file)
index 0000000..4e348c0
--- /dev/null
@@ -0,0 +1,81 @@
+#!/usr/bin/env python
+#
+# __COPYRIGHT__
+#
+# Permission is hereby granted, free of charge, to any person obtaining
+# a copy of this software and associated documentation files (the
+# "Software"), to deal in the Software without restriction, including
+# without limitation the rights to use, copy, modify, merge, publish,
+# distribute, sublicense, and/or sell copies of the Software, and to
+# permit persons to whom the Software is furnished to do so, subject to
+# the following conditions:
+#
+# The above copyright notice and this permission notice shall be included
+# in all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY
+# KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE
+# WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+# NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+# LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+# OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+# WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+#
+
+__revision__ = "__FILE__ __REVISION__ __DATE__ __DEVELOPER__"
+
+"""
+Test that the $RANLIBCOMSTR construction variable allows you to
+customize the displayed archive indexer string.
+"""
+
+import TestSCons
+
+python = TestSCons.python
+
+test = TestSCons.TestSCons()
+
+
+
+test.write('myar.py', """
+import sys
+outfile = open(sys.argv[1], 'wb')
+for f in sys.argv[2:]:
+    infile = open(f, 'rb')
+    for l in filter(lambda l: l != '/*ar*/\\n', infile.readlines()):
+        outfile.write(l)
+sys.exit(0)
+""")
+
+test.write('myranlib.py', """
+import sys
+lines = open(sys.argv[1], 'rb').readlines()
+outfile = open(sys.argv[1], 'wb')
+for l in filter(lambda l: l != '/*ranlib*/\\n', lines):
+    outfile.write(l)
+sys.exit(0)
+""")
+
+test.write('SConstruct', """
+env = Environment(tools=['default', 'ar'],
+                  ARCOM = r'%s myar.py $TARGET $SOURCES',
+                  RANLIBCOM = r'%s myranlib.py $TARGET',
+                  RANLIBCOMSTR = 'Indexing $TARGET',
+                  LIBPREFIX = '',
+                  LIBSUFFIX = '.lib')
+env.Library(target = 'output', source = ['file.1', 'file.2'])
+""" % (python, python))
+
+test.write('file.1', "file.1\n/*ar*/\n/*ranlib*/\n")
+test.write('file.2', "file.2\n/*ar*/\n/*ranlib*/\n")
+
+test.run(stdout = test.wrap_stdout("""\
+%s myar.py output.lib file.1 file.2
+Indexing output.lib
+""" % python))
+
+test.must_match('output.lib', "file.1\nfile.2\n")
+
+
+
+test.pass_test()