Opened 8 years ago

Closed 8 years ago

TracHoursPlugin : syntax error on hours.py

Reported by: Owned by: perez.david@… k0s high TracHoursPlugin critical 0.11

Description

After downloading from source & installing the latest build TracHoursPlugin-0.2.2-py2.3.egg and restarting tracd, this is the error message I get:

trac: Error (invalid syntax (hours.py, line 59))

Is there something I am missing out?

David

comment:1 Changed 8 years ago by k0s

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

need more information to do anything with this ticket. this does not happen to me. using the latest trunk, r5124 , line 59 of source:trachoursplugin/0.11/trachours/hours.py is

                args.extend((k, i) for i in v)


This should be fine syntax. I do python setup.py install to install the plugin. I recommend trying this. I haven't had much luck with eggs....ever.

comment:2 follow-up: ↓ 4 Changed 8 years ago by anonymous

• Resolution wontfix deleted
• Status changed from closed to reopened

Could it be because of the version of Python I am using, 2.3? Tried your install, used a verbose to see installation details, but although there is no error there, it doesn't seem to have install anything. Is there a way to know how if the installation went fine?

comment:3 Changed 8 years ago by k0s

(In [5127]) change the list comprehensions around; refs #4444

comment:4 in reply to: ↑ 2 Changed 8 years ago by k0s

• Resolution set to wontfix
• Status changed from reopened to closed

Could it be because of the version of Python I am using, 2.3? Tried your install, used a verbose to see installation details, but although there is no error there, it doesn't seem to have install anything. Is there a way to know how if the installation went fine?

Yeah, it is unlikely this plugin or any of my plugins will work with 2.3. 2.3 is old and you should upgrade. I made a fix that might help with the immediate issue, but I have no interest in backporting the plugin to legacy python