[Boost-commit] svn:boost r48364 - trunk/tools/build/v2/tools

From: jurko.gospodnetic_at_[hidden]
Date: 2008-08-24 22:25:47

Author: jurko
Date: 2008-08-24 22:25:47 EDT (Sun, 24 Aug 2008)
New Revision: 48364
URL: http://svn.boost.org/trac/boost/changeset/48364

Log:
Cleaned up several error and configuration debugging messages in Boost Build's msvc.jam toolset. Minor comment changes.
Text files modified: 
   trunk/tools/build/v2/tools/msvc.jam |    45 +++++++++++++++++++++++---------------- 
   1 files changed, 26 insertions(+), 19 deletions(-)

Modified: trunk/tools/build/v2/tools/msvc.jam
==============================================================================
--- trunk/tools/build/v2/tools/msvc.jam	(original)
+++ trunk/tools/build/v2/tools/msvc.jam	2008-08-24 22:25:47 EDT (Sun, 24 Aug 2008)
@@ -146,20 +146,32 @@
         case "all" :
             if $(options)
             {
-                errors.error "msvc: Options should be empty when '$(version)'"
-                    "is specified" ;
+                errors.error "MSVC toolset configuration: options should be"
+                    "empty when '$(version)' is specified." ;
             }
 
             # Configure (i.e. mark as used) all registered versions.
             local all-versions = [ $(.versions).all ] ;
-            for local v in $(all-versions)
+            if ! $(all-versions)
             {
-                # Note that there is no need to skip already configured
-                # versions here as this will request configure-really rule to
-                # configure the version using default options which will in turn
-                # cause it to simply do nothing in case the version has already
-                # been configured.
-                configure-really $(v) ;
+                if $(.debug-configuration)
+                {
+                    ECHO "notice: [msvc-cfg] Asked to configure all registered"
+                        "msvc toolset versions when there are none currently"
+                        "registered." ;
+                }
+            }
+            else
+            {
+                for local v in $(all-versions)
+                {
+                    # Note that there is no need to skip already configured
+                    # versions here as this will request configure-really rule
+                    # to configure the version using default options which will
+                    # in turn cause it to simply do nothing in case the version
+                    # has already been configured.
+                    configure-really $(v) ;
+                }
             }
 
         case "default" :
@@ -185,7 +197,6 @@
     # options are off by default. If we are sure that the msvc version is at
     # 7.*, add those options explicitly. We can be sure either if user specified
     # version 7.* explicitly or if we auto-detected the version ourselves.
-    # above).
     if ! [ MATCH ^(6\\.) : $(version) ]
     {
         toolset.flags $(toolset).compile CFLAGS $(conditions) : /Zc:forScope /Zc:wchar_t ;
@@ -603,8 +614,8 @@
         # identical sets of options are used.
         if $(options) && ( $(options) != [ $(.versions).get $(version) : options ] )
         {
-            errors.error "msvc: Toolset version '$(version)' already"
-                "configured." ;
+            errors.error "MSVC toolset configuration: Toolset version"
+                "'$(version)' already configured." ;
         }
     }
     else
@@ -837,7 +848,7 @@
             {
                 for local cpu-condition in $(cpu-conditions)
                 {
-                    ECHO "msvc: condition: '$(cpu-condition)', setup: '$(setup-$(c))'" ;
+                    ECHO "notice: [msvc-cfg] condition: '$(cpu-condition)', setup: '$(setup-$(c))'" ;
                 }
             }
 
@@ -1113,7 +1124,7 @@
         {
             if $(.debug-configuration)
             {
-                ECHO "notice: msvc-$(version) detected, command: '$(command)'" ;
+                ECHO "notice: [msvc-cfg] msvc-$(version) detected, command: '$(command)'" ;
             }
 
             $(.versions).register $(version) ;
@@ -1209,13 +1220,9 @@
 .version-7.1toolkit-envpath = "bin" ;
 
 
-# We try to auto-detect all the available msvc installations on the system
-# directly on module import.
+# Auto-detect all the available msvc installations on the system.
 auto-detect-toolset-versions ;
 
 
 # And finally trigger the actual Boost Build toolset registration.
-#
-# TODO: This should later be triggered lazily to avoid registering the toolset
-# in case none of its versions have been configured successfully.
 register-toolset ;