Mercurial > hg
changeset 28867:ffcc649a4e8f
debugobsolete: style fixes to debugobsolete that slipped from original commit
| author | Kostia Balytskyi <ikostia@fb.com> |
|---|---|
| date | Tue, 12 Apr 2016 03:40:53 -0700 |
| parents | 02be5fc18c0c |
| children | 445a25bb70be |
| files | mercurial/commands.py mercurial/obsolete.py tests/test-obsolete.t |
| diffstat | 3 files changed, 23 insertions(+), 31 deletions(-) [+] |
line wrap: on
line diff
--- a/mercurial/commands.py Wed Mar 09 10:47:33 2016 -0500 +++ b/mercurial/commands.py Tue Apr 12 03:40:53 2016 -0700 @@ -3063,29 +3063,22 @@ 'node identifiers') if opts.get('delete'): - try: - indices = [int(v) for v in opts.get('delete')] - except ValueError: - raise error.Abort(_('invalid index value'), - hint=_('use integers fro indices')) + indices = [] + for v in opts.get('delete'): + try: + indices.append(int(v)) + except ValueError: + raise error.Abort(_('invalid index value: %r') % v, + hint=_('use integers for indices')) if repo.currenttransaction(): - raise error.Abort(_('Cannot delete obsmarkers in the middle ' + raise error.Abort(_('cannot delete obsmarkers in the middle ' 'of transaction.')) - w = repo.wlock() - l = repo.lock() - try: - tr = repo.transaction('debugobsolete') - try: - n = repo.obsstore.delete(indices) - ui.write(_('Deleted %i obsolescense markers\n') % n) - tr.close() - finally: - tr.release() - finally: - l.release() - w.release() + with repo.lock(): + n = repo.obsstore.delete(indices) + ui.write(_('deleted %i obsolescense markers\n') % n) + return if precursor is not None:
--- a/mercurial/obsolete.py Wed Mar 09 10:47:33 2016 -0500 +++ b/mercurial/obsolete.py Tue Apr 12 03:40:53 2016 -0700 @@ -629,7 +629,7 @@ return len(new) def delete(self, indices): - """Delete some obsmarkers from store and return the number of them + """Delete some obsmarkers from store and return how many were deleted Indices is a list of ints which are the indices of the markers to be deleted."""
--- a/tests/test-obsolete.t Wed Mar 09 10:47:33 2016 -0500 +++ b/tests/test-obsolete.t Tue Apr 12 03:40:53 2016 -0700 @@ -1106,18 +1106,17 @@ $ hg debugobsolete --index --rev "3+7" 1 6fdef60fcbabbd3d50e9b9cbc2a240724b91a5e1 d27fb9b066076fd921277a4b9e8b9cb48c95bc6a 0 \(.*\) {'user': 'test'} (re) 3 4715cf767440ed891755448016c2b8cf70760c30 7ae79c5d60f049c7b0dd02f5f25b9d60aaf7b36d 0 \(.*\) {'user': 'test'} (re) - $ cd .. - $ cd .. Test the --delete option of debugobsolete command - $ hg init dorepo - $ cd dorepo - $ echo a > a && hg ci -Am a - adding a - $ hg ci --amend -m aa + $ hg debugobsolete --index + 0 cb9a9f314b8b07ba71012fcdbc544b5a4d82ff5b f9bd49731b0b175e42992a3c8fa6c678b2bc11f1 0 \(.*\) {'user': 'test'} (re) + 1 6fdef60fcbabbd3d50e9b9cbc2a240724b91a5e1 d27fb9b066076fd921277a4b9e8b9cb48c95bc6a 0 \(.*\) {'user': 'test'} (re) + 2 1ab51af8f9b41ef8c7f6f3312d4706d870b1fb74 29346082e4a9e27042b62d2da0e2de211c027621 0 \(.*\) {'user': 'test'} (re) + 3 4715cf767440ed891755448016c2b8cf70760c30 7ae79c5d60f049c7b0dd02f5f25b9d60aaf7b36d 0 \(.*\) {'user': 'test'} (re) + $ hg debugobsolete --delete 1 --delete 3 + deleted 2 obsolescense markers $ hg debugobsolete - cb9a9f314b8b07ba71012fcdbc544b5a4d82ff5b f9bd49731b0b175e42992a3c8fa6c678b2bc11f1 0 (.*) {'user': 'test'} (re) - $ hg debugobsolete --delete 0 - Deleted 1 obsolescense markers - $ hg debugobsolete + cb9a9f314b8b07ba71012fcdbc544b5a4d82ff5b f9bd49731b0b175e42992a3c8fa6c678b2bc11f1 0 \(.*\) {'user': 'test'} (re) + 1ab51af8f9b41ef8c7f6f3312d4706d870b1fb74 29346082e4a9e27042b62d2da0e2de211c027621 0 \(.*\) {'user': 'test'} (re) $ cd .. +