Opened 6 years ago

Closed 6 years ago

Plugin doesn't load using setuptools develop command

Reported by: Owned by: Ryan J Ollos Olemis Lang normal PyTppThemePlugin normal setuptools develop 0.12

Description

I see this error when starting Trac:

02:28:19 PM Trac[loader] ERROR: Skipping "pytpptheme.theme = pytpptheme.theme": (can't import "ImportError: No module named pytpptheme.theme")


One way to fix the issue is to just rename the pytppthemeplugin/0.11/pydotorgtheme@7223 to pytppthemeplugin/0.11/pytpptheme. Alternatively, the attached patch can be used.

Changed 6 years ago by Ryan J Ollos

Patch against r7223 of the plugin.

comment:1 Changed 6 years ago by Ryan J Ollos

I don't think the attached patch is the correct way to fix this, because it appears that the directory was just misnamed. So probably just a directory rename is all that is needed to fix the issue.

comment:2 Changed 6 years ago by Ryan J Ollos

Resolution: → fixed new → closed

(In [12240]) Fixes #10487: Fixed misnamed directory.

comment:3 Changed 6 years ago by Olemis Lang

I appreciate your feedback . Initially I was thinking of reverting them because since the beginning I had in mind not to change that folder name . Notice that even if your changes make it work from fresh svn checkout + setuptools develop command (that's why /me not reverting [12240] ) , in the end it will break distutils sdist and bdist* commands . Hence I've committed [12242] .

In the future it'd be nice if you please could mention proposed changes in the form of a patch and wait for me to review and give green light before committing changes .

comment:4 Changed 6 years ago by Olemis Lang

or better ...are you ok with creating a patch queue at bitbucket so that you (and others) can hack the code and submit patches in an organized manner ?

Modify Ticket

Change Properties