Boost logo

Boost Testing :

From: Markus Schöpflin (markus.schoepflin_at_[hidden])
Date: 2005-11-29 11:45:56

Rene Rivera wrote:
> Markus Schöpflin wrote:
>>Hello all,
>>the function update_source() in currently looks like this:
>>def update_source( user, tag, proxy, args, **unused ):
>> if user is not None or os.path.exists( os.path.join( boost_root, 'CVS'
>>) ):
>> open( timestamp_path, 'w' ).close()
>> log( 'Updating sources from CVS (%s)...' % timestamp() )
>> retry(
>> cvs_update
>> , ( user, tag, args )
>> )
>> else:
>> get_source( user, tag, proxy, args )
>>Obviously the first line is supposed to check whether a CVS update can be
>>done. For this to work correctly, I think the 'or' must be changed to an
>>'and', right?
> No. The check should be equivalent to: if there's a CVS user passed in
> *or* if there's a CVS user already encoded in boost-root/CVS/Root file;
> then one can do a CVS update. Perhaps it should be change to
> specifically check for the CVS/Root file instead of just the CVS subdir.

But you can't run CVS update when no CVS folder is present, can you?

Boost-testing list run by mbergal at