[tahoe-lafs-trac-stream] [tahoe-lafs] #730: unify Windows and non-Windows "make quicktest"

tahoe-lafs trac at tahoe-lafs.org
Mon Jan 17 03:49:54 UTC 2011


#730: unify Windows and non-Windows "make quicktest"
------------------------------------+---------------------------------------
     Reporter:  zooko               |       Owner:  zooko               
         Type:  defect              |      Status:  closed              
     Priority:  minor               |   Milestone:  1.7.0               
    Component:  dev-infrastructure  |     Version:  1.4.1               
   Resolution:  fixed               |    Keywords:  windows test cleanup
Launchpad Bug:                      |  
------------------------------------+---------------------------------------

Comment (by davidsarah):

 Actually for the use in {{{make quicktest}}} and {{{make quicktest-
 coverage}}}, it was incorrect to make the change in [3910] (I was wrong
 about that in comment:4), because it would have caused the make command to
 exit leaving the test running. Probably no-one noticed because they don't
 use the Makefile on Windows, since it requires gmake.

 Anyway, [4927/ticket1306] fixes that, because {{{bin\tahoe.pyscript}}}
 doesn't use {{{os.execve}}}. The optimization that was made in [3891] is
 applicable to {{{bin/tahoe}}}, but it would have to fall back to
 {{{subprocess.call}}} on Windows. (This optimization should not be made in
 1.8.2 since it is a potentially destabilizing change. I'd rather get rid
 of the extra subprocess completely on all platforms by munging
 {{{sys.path}}} instead of setting PYTHONPATH.)

-- 
Ticket URL: <http://tahoe-lafs.org/trac/tahoe-lafs/ticket/730#comment:7>
tahoe-lafs <http://tahoe-lafs.org>
secure decentralized storage


More information about the tahoe-lafs-trac-stream mailing list