-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document nginx plugin #57
Conversation
@@ -16,7 +16,7 @@ | |||
|
|||
|
|||
def resource_path(filename): | |||
return resource_filename('junebug.plugins.nginx', filename) | |||
return path.join(path.dirname(__file__), filename) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Building the docs wasn't working with pkg_resources.resource_filename
, was getting:
File "/.../junebug/ve/lib/python2.6/site-packages/pkg_resources.py", line 1547, in _extract_resource
timestamp, size = self._get_date_and_size(self.zipinfo[zip_path])
KeyError: 'junebug/plugins/nginx/vhost.template'
Guessing resource_filename
wasn't happy with junebug being installed as an egg, but really not too sure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably also want to remove the import for resource_filename
then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. I thought the reason we used pkg_resources.resource_filename
was because the __file__
approach wouldn't work in a zip, though.
Ready for review. |
Ready for re-review. |
👍 from me |
👍 |
No description provided.