Edgewall Software
Modify

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#6796 closed defect (fixed)

Improve Trac error reporting for misconfigured repositories

Reported by: groovypockets@… Owned by: Christian Boos
Priority: normal Milestone: 0.11.1
Component: version control Version: 0.10.4
Severity: major Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description (last modified by Emmanuel Blot)

Trying to setup Trac for subversion gui

here's the trace

Python Traceback

Traceback (most recent call last):
  File "/var/lib/python-support/python2.5/trac/web/main.py", line 406, in dispatch_request
    dispatcher.dispatch(req)
  File "/var/lib/python-support/python2.5/trac/web/main.py", line 191, in dispatch
    chosen_handler = self._pre_process_request(req, chosen_handler)
  File "/var/lib/python-support/python2.5/trac/web/main.py", line 263, in _pre_process_request
    chosen_handler = f.pre_process_request(req, chosen_handler)
  File "/var/lib/python-support/python2.5/trac/versioncontrol/api.py", line 73, in pre_process_request
    self.get_repository(req.authname).sync()
  File "/var/lib/python-support/python2.5/trac/versioncontrol/api.py", line 94, in get_repository
    ((self.repository_type,)*2))
TracError: Unsupported version control system "/var/svn/svpp". Check that the Python bindings for "/var/svn/svpp" are correctly installed.

Attachments (2)

unsupported_vcs-r7031.diff (3.9 KB ) - added by Christian Boos 16 years ago.
Provide a mechanism by which one can get (slightly) more meaningful messages for this common error
unsupported-vcs-hg-r7031.diff (2.1 KB ) - added by Christian Boos 16 years ago.
Related changes for the TracMercurial plugin (0.11). I had to fight a bit the demandimport stuff …

Download all attachments as: .zip

Change History (10)

comment:1 by Emmanuel Blot, 16 years ago

Description: modified (diff)
Priority: highnormal
Resolution: invalid
Status: newclosed

Installation issue. Please re-read the TracInstall documentation or post to a message to the MailingList if you have trouble installing Trac.

comment:2 by Emmanuel Blot, 16 years ago

Keywords: tobedeleted added

comment:3 by Christian Boos, 16 years ago

Component: generalversion control
Keywords: help added; tobedeleted removed
Milestone: 0.12
Resolution: invalid
Severity: normalmajor
Status: closedreopened
Summary: Trac reports an internal errorImprove Trac error reporting for misconfigured repositories

Here the error was that the repository_type was set to a repository path. It should instead have been something like:

[trac]
repository_dir = /var/svn/svpp
repository_type = svn

I suggest to not delete the ticket as this might help someone having the same error message.

Also in 0.12, I will take a look at making the error messages for the versioncontrol subsystem more helpful, as this is by far the kind of reports we get the most. And it will get even worse with the multirepos support if nothing improves on that topic.

comment:4 by Christian Boos, 16 years ago

Owner: changed from Jonas Borgström to Christian Boos
Status: reopenednew

by Christian Boos, 16 years ago

Attachment: unsupported_vcs-r7031.diff added

Provide a mechanism by which one can get (slightly) more meaningful messages for this common error

comment:5 by Christian Boos, 16 years ago

Keywords: review added; help removed
Milestone: 0.120.11

Tentatively scheduling this for 0.11 if there's positive review.

As said above, this error is by far one of the most common support request, so getting more detailed help message would be very useful.

Still, there's no magic in there and it won't tell exactly how one has managed to mess up the install, only provide useful hints (reporting the actual ImportError, telling to look in the Trac log, etc.).

by Christian Boos, 16 years ago

Related changes for the TracMercurial plugin (0.11). I had to fight a bit the demandimport stuff …

comment:6 by Chris Mulligan <chris.mulligan@…>, 16 years ago

Closed dupe #5150.

comment:7 by Jonas Borgström, 16 years ago

Resolution: fixed
Status: newclosed

I've applied the patches in r7435,7436 but I'm leaving the TracMercurial patch for you Christian since I can't really test it myself.

comment:8 by Christian Boos, 16 years ago

Keywords: review removed

Mercurial related patch applied in [7592] for 0.11 and [7593] for multirepos.

In [7592], I also took care that the plugin still works with 0.11.

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.