D4549: localrepo: move check for existing repo into createrepository()

2018-09-12 Thread martinvonz (Martin von Zweigbergk)
This revision was automatically updated to reflect the committed changes. Closed by commit rHGe471cb2852ea: localrepo: move check for existing repo into createrepository() (authored by martinvonz, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D4549: localrepo: move check for existing repo into createrepository()

2018-09-12 Thread martinvonz (Martin von Zweigbergk)
martinvonz updated this revision to Diff 10977. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D4549?vs=10973=10977 REVISION DETAIL https://phab.mercurial-scm.org/D4549 AFFECTED FILES mercurial/localrepo.py CHANGE DETAILS diff --git

D4549: localrepo: move check for existing repo into createrepository()

2018-09-12 Thread martinvonz (Martin von Zweigbergk)
martinvonz added a comment. In https://phab.mercurial-scm.org/D4549#69559, @pulkit wrote: > test-init.t and test-mq.t says hi! Sorry! :( Looks like I forgot to drop the `.hg` argument when I switched from wdirvfs to hgvfs. I'll fix right away. REPOSITORY rHG Mercurial

D4549: localrepo: move check for existing repo into createrepository()

2018-09-12 Thread pulkit (Pulkit Goyal)
pulkit added a comment. test-init.t and test-mq.t says hi! --- /storage/pulkit/repo/hgpush/tests/test-init.t +++ /storage/pulkit/repo/hgpush/tests/test-init.t.err @@ -99,7 +99,7 @@ test failure $ hg init local - abort: repository local already exists!

D4549: localrepo: move check for existing repo into createrepository()

2018-09-12 Thread martinvonz (Martin von Zweigbergk)
martinvonz created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY For symmetry with the check for existence of a repo in localrepository.__init__, we should check for the non-existence in createrepository(). We could