Edgewall Software
Modify

Ticket #6796 (closed defect: fixed)

Opened 4 years ago

Last modified 3 years ago

Improve Trac error reporting for misconfigured repositories

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

Description (last modified by eblot) (diff)

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

unsupported_vcs-r7031.diff (3.9 KB) - added by cboos 4 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 cboos 4 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

comment:1 Changed 4 years ago by eblot

  • Description modified (diff)
  • Priority changed from high to normal
  • Resolution set to invalid
  • Status changed from new to closed

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

comment:2 Changed 4 years ago by eblot

  • Keywords tobedeleted added

comment:3 Changed 4 years ago by cboos

  • Component changed from general to version control
  • Keywords help added; tobedeleted removed
  • Milestone set to 0.12
  • Resolution invalid deleted
  • Severity changed from normal to major
  • Status changed from closed to reopened
  • Summary changed from Trac reports an internal error to Improve 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 Changed 4 years ago by cboos

  • Owner changed from jonas to cboos
  • Status changed from reopened to new

Changed 4 years ago by cboos

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

comment:5 Changed 4 years ago by cboos

  • Keywords review added; help removed
  • Milestone changed from 0.12 to 0.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.).

Changed 4 years ago by cboos

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

comment:6 Changed 4 years ago by Chris Mulligan <chris.mulligan@…>

Closed dupe #5150.

comment:7 Changed 4 years ago by jonas

  • Resolution set to fixed
  • Status changed from new to closed

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 Changed 3 years ago by cboos

  • 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.

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
to The owner will be changed from cboos. Next status will be 'closed'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.