Edgewall Software

Ticket #6796 (closed defect: fixed)

Opened 7 months ago

Last modified 5 weeks 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: review
Cc:

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 months 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 months ago.
Related changes for the TracMercurial plugin (0.11). I had to fight a bit the demandimport stuff …

Change History

Changed 7 months ago by eblot

  • priority changed from high to normal
  • status changed from new to closed
  • resolution set to invalid
  • description modified (diff)

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

Changed 7 months ago by eblot

  • keywords tobedeleted added

Changed 7 months ago by cboos

  • status changed from closed to reopened
  • severity changed from normal to major
  • component changed from general to version control
  • summary changed from Trac reports an internal error to Improve Trac error reporting for misconfigured repositories
  • milestone set to 0.12
  • keywords help added; tobedeleted removed
  • resolution invalid deleted

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.

Changed 7 months ago by cboos

  • owner changed from jonas to cboos
  • status changed from reopened to new

Changed 4 months ago by cboos

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

Changed 4 months 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 months ago by cboos

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

Changed 3 months ago by Chris Mulligan <chris.mulligan@…>

Closed dupe #5150.

Changed 5 weeks ago by jonas

  • status changed from new to closed
  • resolution set to fixed

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.

Add/Change #6796 (Improve Trac error reporting for misconfigured repositories)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.