Modify

Opened 12 years ago

Closed 12 years ago

#10882 closed defect (fixed)

ProjectPlanPlugin does not support Trac 1.0

Reported by: Adam Dorsey - NOAA Affiliate Owned by: Matthias
Priority: normal Component: ProjectPlanPlugin
Severity: normal Keywords: setuptools
Cc: Trac Release: 1.0

Description

2013-02-19 16:45:18,266 Trac[loader] DEBUG: Loading projectplan from /usr/lib/python2.6/site-packages/ProjectPlan-0.93.2-py2.6.egg
2013-02-19 16:45:18,266 Trac[loader] ERROR: Skipping "projectplan = projectplan.projectplan": (version conflict "VersionConflict: (Trac 1.0.1 (/usr/lib/python2.6/site-packages/Trac-1.0.1-py2.6.egg), Requirement.parse('Trac>=0.11,<0.13'))")

I don't know if there are any underlying changes in Trac 1.0 that would prevent the plugin from working as-is. I'll try hacking the version field around and see what breaks horribly.

Attachments (0)

Change History (10)

comment:1 Changed 12 years ago by Adam Dorsey - NOAA Affiliate

OK, so I modified the install_requires line in setup.py to the following:

install_requires = ( u'Trac >=0.11, <=1.0.1' ),

and the plugin loads without crashing Trac.

It seems to work well, the only issue I've found so far is that the popup description for the dependencies links in a ticket is blank. I will keep experimenting and see what else I can break.

comment:2 Changed 12 years ago by Ryan J Ollos

Keywords: setuptools added

comment:3 Changed 12 years ago by Ryan J Ollos

Is this plugin being actively developed? I'd be happy to push the change to the repository if needed.

comment:4 Changed 12 years ago by Adam Dorsey - NOAA Affiliate

Honestly I don't know if it's being actively developed or not, we just use it for an internal Trac that serves as document repository and issue tracker. The dependency hover preview is still broken, but other than that it all seems to work. If you want, go ahead and push this change and I can open another ticket for the dependency preview issue.

comment:5 Changed 12 years ago by Adam Dorsey - NOAA Affiliate

Might be better to do

install_requires = ( u'Trac >=0.11, <1.1' ),

instead

comment:6 Changed 12 years ago by Ryan J Ollos

I'll have to wait for a response from the maintainer, or at least 2 weeks before making the change. Unless we know it is broken in later versions of Trac, I'd suggest we just drop the upper-bound on the version check.

comment:7 in reply to:  4 Changed 12 years ago by Ryan J Ollos

Trac Release: 1.0

Replying to adam.dorsey@noaa.gov:

... I can open another ticket for the dependency preview issue.

Yes, please do that if a ticket doesn't already exist. Even better if you have a patch for the issue!

comment:8 Changed 12 years ago by Adam Dorsey - NOAA Affiliate

See #10888, mouseover popup issue resolved.

comment:9 Changed 12 years ago by Adam Dorsey - NOAA Affiliate

And I agree, just drop the upper bound and we'll see what happens lol.

comment:10 Changed 12 years ago by Ryan J Ollos

Resolution: fixed
Status: newclosed

Issue seems to have been fixed in [12688].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Matthias.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


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

 
Note: See TracTickets for help on using tickets.