Edgewall Software
Modify

Opened 12 years ago

Closed 12 years ago

#8685 closed defect (fixed)

Emptied repository results in a browser error

Reported by: mikei@… Owned by: Christian Boos
Priority: normal Milestone: not applicable
Component: plugin/mercurial Version: none
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Steps to recreate:

  1. Start with a new repository
  1. Create a file, add it, commit and push
  1. Remove the file, commit and push
  1. Go to the /browser (click the Browse Source tab)
  1. Observe an error:

Error: No such node No node at revision 1:36b97207408f

========================

The offender is the _init_path function not handling empty repositories properly. (http://trac.edgewall.org/browser/sandbox/mercurial-plugin/tracvc/hg/backend.py)

The attached patch (aganst r8365) adds a check for an empty manifest array.

Attachments (1)

mercurial-plugin-backend-empty-repo.patch (2.2 KB ) - added by anonymous 12 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 by Christian Boos, 12 years ago

Milestone: not applicable

I'll look into this.

comment:2 by Christian Boos, 12 years ago

Resolution: fixed
Status: newclosed

I've committed a simpler patch in r8663. Thanks!

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christian Boos.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Christian Boos to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.