comparison mercurial/statichttprepo.py @ 39695:cb2dcfa5cade

localrepo: move requirements reasonability testing to own function

Just because we know how to handle each listed requirement doesn't
mean that set of requirements is reasonable.

This commit introduces an extension-wrappable function to validate
that a set of requirements makes sense.

We could combine this with ensurerequirementsrecognized(). But I think
having a line between basic membership testing and compatibility
checking is more powerful as it will help differentiate between
missing support and buggy behavior.

Differential Revision: https://phab.mercurial-scm.org/D4571

39694:6192980553b4

                raise error.RepoError(msg)

        supportedrequirements = localrepo.gathersupportedrequirements(ui)
        localrepo.ensurerequirementsrecognized(requirements,
                                               supportedrequirements)

        # setup store
        self.store = store.store(requirements, self.path, vfsclass)
        self.spath = self.store.path
        self.svfs = self.store.opener

39695:cb2dcfa5cade

                raise error.RepoError(msg)

        supportedrequirements = localrepo.gathersupportedrequirements(ui)
        localrepo.ensurerequirementsrecognized(requirements,
                                               supportedrequirements)
        localrepo.ensurerequirementscompatible(ui, requirements)

        # setup store
        self.store = store.store(requirements, self.path, vfsclass)
        self.spath = self.store.path
        self.svfs = self.store.opener