assert failure in <c:cppunit> if srcdir is defined

[anonymous]

Python gives this error if srcdir is specified in <c:cppunit> command:

[ERROR   ] global name 'posixpath' is not defined
Traceback (most recent call last):
  File "/usr/local/python2.4/lib/python2.4/asyncore.py", line 69, in read
    obj.handle_read_event()
  File "/usr/local/python2.4/lib/python2.4/asyncore.py", line 391, in handle_read_event
    self.handle_read()
  File "/usr/local/python2.4/lib/python2.4/asynchat.py", line 137, in handle_read
    self.found_terminator()
  File "build/bdist.linux-i686/egg/bitten/util/beep.py", line 278, in found_terminator
  File "build/bdist.linux-i686/egg/bitten/util/beep.py", line 311, in _handle_frame
  File "build/bdist.linux-i686/egg/bitten/util/beep.py", line 465, in handle_data_frame
  File "build/bdist.linux-i686/egg/bitten/slave.py", line 152, in handle_msg
  File "build/bdist.linux-i686/egg/bitten/slave.py", line 176, in execute_build
  File "build/bdist.linux-i686/egg/bitten/recipe.py", line 125, in execute
  File "build/bdist.linux-i686/egg/bitten/recipe.py", line 70, in run
  File "build/bdist.linux-i686/egg/bitten/build/ctools.py", line 84, in cppunit
NameError: global name 'posixpath' is not defined

The attached patch will fix it.

[joel@…]

fix for ctools.py

[osimons]

Latest trunk already imports posixpath.

However, we shouldn't really use that module directly - instead os.path should be used to get the correct path module for the platform Python runs on. I've changed this in [677].

[osimons]

Replying to osimons:

However, we shouldn't really use that module directly - instead os.path should be used to get the correct path module for the platform Python runs on. I've changed this in [677].

Just for the record: Not the best of ideas, and reverted again in [679]... :-)