Skip to content
Snippets Groups Projects
user avatar
Georges Racinet authored
It was a mistake to depend on `load_repo`, as it will fail if,
e.g., the `.hg` sub directory does not exist (seen in Rails
RSpec tests) and various corrupted cases that a *removal* should
clean up.

The price to pay is the transactionality of the roster, but we
cannot lock it properly without a repo. In theory in should be
possible to instantiate the `Vfs` separately, but it does not seem
to matter so much.

With this change, if removing the working directories fails, it
will not prevent removing the trashed repository (but of course
we risk a later collision in workdirs).

Backs out in particular changeset 1c735d40cfe7
10d6717c
History
Name Last commit Last update