From 92a1b44d7859e641380171be6b0faf24838fa4b0 Mon Sep 17 00:00:00 2001 From: Joshua Hughes Date: Mon, 16 Jul 2012 13:27:50 -0400 Subject: Fix issue 2856. This change fixes issue 2856 by making scons consider the .def file only if it's explicitly given or insertion is explicitly requested. --- src/engine/SCons/Tool/mingw.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/engine/SCons/Tool/mingw.py b/src/engine/SCons/Tool/mingw.py index 5827aae..83f8c93 100644 --- a/src/engine/SCons/Tool/mingw.py +++ b/src/engine/SCons/Tool/mingw.py @@ -100,12 +100,12 @@ def shlib_emitter(target, source, env): target.append(env.fs.File(targetStrings)) # Append a def file target if there isn't already a def file target - # or a def file source. There is no option to disable def file - # target emitting, because I can't figure out why someone would ever - # want to turn it off. + # or a def file source or the user has explicitly asked for the target + # to be emitted. def_source = env.FindIxes(source, 'WINDOWSDEFPREFIX', 'WINDOWSDEFSUFFIX') def_target = env.FindIxes(target, 'WINDOWSDEFPREFIX', 'WINDOWSDEFSUFFIX') - if not def_source and not def_target: + skip_def_insert = env.subst("$WINDOWS_INSERT_DEF") in ['', '0', 0] + if not def_source and not def_target and not skip_def_insert: # Create list of target libraries and def files as strings targetStrings=env.ReplaceIxes(dll, 'SHLIBPREFIX', 'SHLIBSUFFIX', -- cgit v0.12 From 593b92958c6438d2b8a07d39efae97add8d90209 Mon Sep 17 00:00:00 2001 From: Joshua Hughes Date: Mon, 23 Jul 2012 15:32:36 -0400 Subject: Add test for issue 2856 fix. This change also corrects a typo found in the WINDOWS_INSERT_DEF description/help entry. --- src/engine/SCons/Tool/mslink.xml | 2 +- test/MinGW/WINDOWS_INSERT_DEF.py | 61 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 62 insertions(+), 1 deletion(-) create mode 100644 test/MinGW/WINDOWS_INSERT_DEF.py diff --git a/src/engine/SCons/Tool/mslink.xml b/src/engine/SCons/Tool/mslink.xml index 35e8b64..80c5530 100644 --- a/src/engine/SCons/Tool/mslink.xml +++ b/src/engine/SCons/Tool/mslink.xml @@ -201,7 +201,7 @@ A deprecated synonym for &cv-link-WINDOWSEXPSUFFIX;. When this is set to true, a library build of a Windows shared library -(.dllfile) +(.dll file) will also build a corresponding .def file at the same time, if a .def file diff --git a/test/MinGW/WINDOWS_INSERT_DEF.py b/test/MinGW/WINDOWS_INSERT_DEF.py new file mode 100644 index 0000000..31c0707 --- /dev/null +++ b/test/MinGW/WINDOWS_INSERT_DEF.py @@ -0,0 +1,61 @@ +#!/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__" + +""" +Make sure that WINDOWS_INSERT_DEF isn't ignored when using MinGW. +""" + +import sys +import TestSCons + +test = TestSCons.TestSCons() + +if sys.platform not in ('cygwin', 'win32'): + test.skip_test( + "Skipping mingw test on non-Windows platform: %s" % sys.platform) + +test.write('hello.c', r""" +#include +int hello_world(void) { return printf("Orbis, te saluto!\n"); } +""") + +test.write('SConstruct', """ +env = Environment(TOOLS = ['mingw']) +hello_dll = env.SharedLibrary(WINDOWS_INSERT_DEF = 0, + target = 'hello', source = 'hello.c') +""" % locals()) + +test.run(arguments = ".") + +test.must_not_exist('hello.def') + +test.pass_test() + +# Local Variables: +# tab-width:4 +# indent-tabs-mode:nil +# End: +# vim: set expandtab tabstop=4 shiftwidth=4: -- cgit v0.12 From 1a8576e9eaf4329247ff522e2e687fd7011185a7 Mon Sep 17 00:00:00 2001 From: Joshua Hughes Date: Tue, 24 Jul 2012 10:17:56 -0400 Subject: Make sure things aren't rebuilt. SCons should not rebuild things after the first successful build when using WINDOWS_INSERT_DEF = 0. --- test/MinGW/WINDOWS_INSERT_DEF.py | 1 + 1 file changed, 1 insertion(+) diff --git a/test/MinGW/WINDOWS_INSERT_DEF.py b/test/MinGW/WINDOWS_INSERT_DEF.py index 31c0707..78aa26e 100644 --- a/test/MinGW/WINDOWS_INSERT_DEF.py +++ b/test/MinGW/WINDOWS_INSERT_DEF.py @@ -51,6 +51,7 @@ hello_dll = env.SharedLibrary(WINDOWS_INSERT_DEF = 0, test.run(arguments = ".") test.must_not_exist('hello.def') +test.up_to_date(arguments = ".") test.pass_test() -- cgit v0.12