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

Add 'cssmin' to requirements.txt #57

Open
srid opened this issue Feb 6, 2011 · 4 comments
Open

Add 'cssmin' to requirements.txt #57

srid opened this issue Feb 6, 2011 · 4 comments

Comments

@srid
Copy link

srid commented Feb 6, 2011

$ python hyde.py  -g -s ~/blog -d ~/blog-deploy 
INFO:Pre processing /Users/user/blog[2011-02-05 16:51:40,922]
INFO:Processing /index.html[2011-02-05 16:51:40,923]
INFO:Processing /about/about.html[2011-02-05 16:51:41,385]
INFO:Processing /archives/firstpost.html[2011-02-05 16:51:41,413]
INFO:Processing /archives/archives.html[2011-02-05 16:51:41,426]
INFO:Processing /media/css/base.css[2011-02-05 16:51:41,438]
Generation Failed
Traceback (most recent call last):
  File "/Users/sridharr/code/hyde/hydeengine/__init__.py", line 417, in process_all
    self.process(resource)
  File "/Users/sridharr/code/hyde/hydeengine/__init__.py", line 377, in process
    return self.processor.process(item)
  File "/Users/sridharr/code/hyde/hydeengine/processor.py", line 111, in process
    processor.process(resource)
  File "/Users/sridharr/code/hyde/hydeengine/media_processors.py", line 102, in process
    import cssmin
ImportError: No module named cssmin
@valpackett
Copy link

cssmin is optional. Not everybody needs it.

Maybe create "recommended.txt"?

@srid
Copy link
Author

srid commented Feb 7, 2011

cssmin is optional. Not everybody needs it.

If it is optional, then why does the default example mentioned in README fail to run? To explain, I followed the "Running with Hyde" section, specifically step 1 (Initializer) and 2 (Generator).

Maybe create "recommended.txt"?

I suggest that the optional dependencies be put in setup.py as extras_require. See the documentation or SQLObject's setup.py for an example.

@navilan
Copy link
Member

navilan commented Feb 7, 2011

yeah - recommended seems like a good idea. Will do.

@zenbum
Copy link

zenbum commented Sep 11, 2012

What @srid said. The README example failing first-hand really gives a bad first impression...

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

4 participants