Skip to content
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

Ability to import extensions from the sitepath #79

Closed
wants to merge 1 commit into from
Closed

Ability to import extensions from the sitepath #79

wants to merge 1 commit into from

Conversation

theomega
Copy link

@theomega theomega commented Sep 8, 2011

When writing own extensions, the only possibility to import them is to add them to the hyde/ext directory. As this directory will be overwritten during updates and could even be not modifiable for a user at all, there should be a possibility to import extensions from the site-directory. This also enables the user to use site-specific extensions.

This patch enables the functionality without breaking any previously written code.

"""
Loads a python module from string
"""

if (site!=None):
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It wolud look cleaner if you wrote it as if site is not None:, but that's just my opinion :-)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, you must be right, but I am no Python programmer, so please forgive me. Perhaps someone with more knowledge should rewrite this. I will non-the-less change the line and upload a second pull request. Thanks!

@navilan
Copy link
Member

navilan commented Sep 15, 2011

Closing. See above commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants