On Mon, Oct 20, 2008 at 07:19:20PM -0500, Shawn Walker wrote:

>> Why not make the directory check take --refresh-index into account?
>
> Because refresh-index, etc. are command-line options to depot.py and do not 
> control or alter the actual behaviour of server configuration or the depot 
> functionality itself.  In other words, the fact that depot.py doesn't care 
> about content_root if --refresh-index is passed does not translate in a 
> generically applicable fashion to the server/catalog forking off a depot 
> process.
>
>> Perhaps content_root passed as None into the SvrConfig constructor could
>> indicate it could be ignored, and set it to None if reindex is True?
>
> That seems like a viable solution for the moment.

... which seems to me like it's taking --refresh-index into account.

server/config.py:

  - line 111: Why not just "if root:"?

Danek
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to