jankatins / knitpy

knitpy: Elegant, flexible and fast dynamic report generation with python
https://github.com/jankatins/knitpy
Other
368 stars 29 forks source link

Full or relative path required on render #12

Closed thatcher closed 9 years ago

thatcher commented 9 years ago

The following raises an exception:

>>> report.render('test.pymd', output='html')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "build\bdist.win-amd64\egg\knitpy\knitpy.py", line 649, in render
WindowsError: [Error 123] The filename, directory name, or volume label syntax is incorrect: ''

While this does not:

>>> report.render('./test.pymd', output='html')

Thinking render should probably gracefully check current working directory even in the absence of ./

Set up code and environment below:

C:\work>python
Python 2.7.8 |Anaconda 2.1.0 (64-bit)| (default, Jul  2 2014, 15:12:11) [MSC v.1500 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
Anaconda is brought to you by Continuum Analytics.
Please check out: http://continuum.io/thanks and https://binstar.org
>>> from knitpy.knitpy import Knitpy
>>> report = Knitpy()
jankatins commented 9 years ago

I can reproduce it here...

jankatins commented 9 years ago

There is a fix in #13 (hopefully...), if you want to install from github...

jankatins commented 9 years ago

Merged #13, so this here should be done

jankatins commented 9 years ago

Thanks for the bugreport!