Ticket #10282 (closed defect: fixed)

Opened 9 months ago

Last modified 8 months ago

r9290 breaks 0.10 compatibility

Reported by: otaku42 Assigned to: hasienda
Priority: low Component: AccountManagerPlugin
Severity: major Keywords: import
Cc: Trac Release: 0.10

Description

Just as a side note, as I stumbled across this issue while investigating another issue on trac-hacks.org:

r9290 does not work at least with Trac 0.10.6dev that we use on t-h.o. With that change, we get an error like this in the logs when starting Trac: 

2012-09-05 15:40:17,127 Trac[loader] ERROR: Failed to load plugin acct_mgr.admin from /path/to/trac/plugins/TracAccountManager-0.1.3dev_r10314-py2.6.egg
Traceback (most recent call last):
  File "/path/to/pve/lib/python2.6/site-packages/trac/loader.py", line 102, in load_components
    entry_point.load()
  File "/path/to/pve/lib/python2.6/site-packages/distribute-0.6.10-py2.6.egg/pkg_resources.py", line 1948, in load
    entry = __import__(self.module_name, globals(),globals(), ['__name__'])
  File "build/bdist.linux-i686/egg/acct_mgr/admin.py", line 17, in <module>
ImportError: No module named compat

Fixed the issue locally by reverting the change.

Attachments

Change History

09/07/12 17:51:24 changed by hasienda

  • keywords set to import.

I'm testing with trac-0.11 and trac-0.13dev here for a while. So I'm quite sure, that this is a 0.10 issue, and I should have checked this better before back-porting that change from trunk.

09/07/12 20:24:44 changed by hasienda

(In [11950]) AccountManagerPlugin: Revert change in [9290] for 0.10 branch, refs #4830 and #10282.

10/08/12 22:24:51 changed by stoecker

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

Forgotten closing?

10/09/12 00:19:52 changed by hasienda

Rather no. I prefer to receive some feedback, and in general close tickets only after/on release, so pre-release checks against changelog are more efficient even without a sophisticated ticket query.

As an exception for the largely obsolete 0.10 you earn a point here. Let's keep that closed.

Add/Change #10282 (r9290 breaks 0.10 compatibility)




Change Properties
Action