[Boost-commit] svn:boost r40163 - in branches/build/python_port/python: . boost/build

From: ghost_at_[hidden]
Date: 2007-10-18 15:58:47

Author: vladimir_prus
Date: 2007-10-18 15:58:47 EDT (Thu, 18 Oct 2007)
New Revision: 40163
URL: http://svn.boost.org/trac/boost/changeset/40163

Log:
Revive 'command-line-free-features'.

Text files modified: 
   branches/build/python_port/python/TODO.txt                    |     7 +++++++                                 
   branches/build/python_port/python/boost/build/build_system.py |    11 +----------                             
   branches/build/python_port/python/boost/build/manager.py      |     7 +++++++                                 
   3 files changed, 15 insertions(+), 10 deletions(-)

Modified: branches/build/python_port/python/TODO.txt
==============================================================================
--- branches/build/python_port/python/TODO.txt	(original)
+++ branches/build/python_port/python/TODO.txt	2007-10-18 15:58:47 EDT (Thu, 18 Oct 2007)
@@ -1,6 +1,13 @@
 
 - Document that using strings for classes will not be carried over.
 
+- Review absolutely all 'raise' statements and decide what do do
+about them.
+
+- Test dependency properties. Probably need to fix them!
+
+- Review logging in target.py
+
 
 2. The current way to make rules available to Jamfiles is nasty --
 only functions defined in ProjectContext class are exported.

Modified: branches/build/python_port/python/boost/build/build_system.py
==============================================================================
--- branches/build/python_port/python/boost/build/build_system.py	(original)
+++ branches/build/python_port/python/boost/build/build_system.py	2007-10-18 15:58:47 EDT (Thu, 18 Oct 2007)
@@ -34,14 +34,6 @@
 #}
 
 # FIXME:
-command_line_free_features = []
-# Returns the property set with the
-# free features from the currently processed
-# build request.
-#rule command-line-free-features ( )
-#{
-#    return $(.command-line-free-features) ;
-#}
 
 def get_boolean_option(name):
     match = "--" + name
@@ -330,9 +322,8 @@
     # so we need to record which targets are produced.
     results_of_main_targets = []
 
-    global command_line_free_features
     for p in expanded:
-        command_line_free_features = property_set.create(p.free())
+        manager.set_command_line_free_features(property_set.create(p.free()))
         
         for t in targets:
             g = t.generate(p)

Modified: branches/build/python_port/python/boost/build/manager.py
==============================================================================
--- branches/build/python_port/python/boost/build/manager.py	(original)
+++ branches/build/python_port/python/boost/build/manager.py	2007-10-18 15:58:47 EDT (Thu, 18 Oct 2007)
@@ -35,6 +35,7 @@
         self.argv_ = bjam.variable("ARGV")
         self.boost_build_path_ = bjam.variable("BOOST_BUILD_PATH")
         self.errors_ = Errors()
+        self.command_line_free_features_ = property_set.empty()
         
         # Object Map.
         # TODO: This is a kludge: maps object names to the actual instances.
@@ -78,6 +79,12 @@
     def boost_build_path(self):
         return self.boost_build_path_
 
+    def command_line_free_features(self):
+        return self.command_line_free_features_
+
+    def set_command_line_free_features(self, v):
+        self.command_line_free_features_ = v
+
     def register_object (self, value):
         """ Stores an object in a map and returns a key that can be used to retrieve it.
         """