[PATCH] Subrepository support

[timeless]

2009/2/10 Alexander Solovyov <piranha at piranha.org.ua>:
> +def sublist(ui, repo, **opts):
> +    '''List all subrepositories with their desired revisions'''

'with' is odd, do you mean 'and'?

> +    def subrepos(self, rev=None, usecache=True):

> +        Revisions of subrepos are stored inside a parent repository

a parent => the parent ?

> +        store as a text file containing revision in a hex format. This

revision => (the | a) revision ?

> +        file is placed exactly at subrepo path:

^ this is awkwardly phrased

> +    def subrepo(self, path, rev=None):
> +        if not path in self.subrepos(rev):
> +            raise error.RepoError('Subrepository %s unavailable' % path)

typically i'd want a verb ("is")

> +def subupdate(repo, path, rev=None):
> +    '''Update subrepository to necessary revision.

necessary => requested | required ?

> +def subremove(repo, path):
> +    '''Remove subrepository, saving its store in a cache.

"a cache"? which cache?