[MERGE] Enable setting and getting make_working_trees for all repositories
Aaron Bentley
aaron at aaronbentley.com
Thu Apr 10 03:41:12 BST 2008
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Andrew Bennetts wrote:
> Aaron Bentley wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> Hi all,
>>
>> This patch enables set_make_working_trees on all repository
>> implementations. For repository implementations where it cannot be
>> changed, it raise ValueError if called with "False".
>
> I don't like using ValueError for this.
I can do a custom error, then.
> For that matter, I don't really
> understand how this is a significant improvement over raising
> NotImplementedError.
NotImplementedError usually means someone forgot to implement something.
I think that kind of error should trigger a test suite failure.
ValueError means that the supplied value was unsuitable, and for
AllInOne repositories, False is not a usable value. (True is, because
that's the value they always use.)
Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFH/X5I0F+nu1YWqI0RArdiAJ9fy1CA2A1MiI1yTL0aBz6SxxCOHwCfffyW
lvP94Gtz6+UO4jcHsLQVwzo=
=6v35
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list