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

Update documentation to re: finding reposadolib #1

Closed
timsutton opened this issue Mar 26, 2012 · 4 comments
Closed

Update documentation to re: finding reposadolib #1

timsutton opened this issue Mar 26, 2012 · 4 comments

Comments

@timsutton
Copy link

This has already come up on munki-dev, but margarita.py assumes that it can read the reposadolib module.

Traceback (most recent call last):
File "margarita.py", line 15, in
from reposadolib import reposadocommon
ImportError: No module named reposadolib

If I add a symlink to my reposadolib module directory at the root of the margarita installation, it works correctly. I imagine there might be a more elegant / proper way to handle this (via a config file, etc.).

In the end, my reposado user's apps folder structure looks like this:

.
├── margarita
│   ├── margarita.py
│   ├── preferences.plist -> /home/reposado/git/reposado/code/preferences.plist
│   ├── README.md
│   ├── reposadolib -> /home/reposado/git/reposado/code/reposadolib
│   ├── static
│   └── templates
└── reposado
├── code
├── docs
├── LICENSE.txt
├── other
├── README.md
└── setup.py

@timsutton
Copy link
Author

I guess I should've used Markdown bullets rather than just copy/paste the default output from the tree command for the above, but you get the idea.

@jessepeterson
Copy link
Owner

Thanks for the feedback. Yes, it is assumed that reposedo is installed into the same Python site-packages as Margarita and Flask are executing from. I haven't taken into account reposado running directly from code. I'll update the docs for that. Thanks again!

@timsutton
Copy link
Author

I think a lot of users are actually running reposado directly from code, as that's how the documentation suggests running it. Thanks for creating this project, it looks great!

@jessepeterson
Copy link
Owner

I updated the docs; hopefully they're clear enough for folks to follow along. THanks for your report!

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

No branches or pull requests

2 participants