search

tweecode / twee

Twee core
100 stars 30 forks source link

Mac OS 10.6.8 reports CLI error #7

Open taskboy3000 opened 11 years ago

taskboy3000 commented 11 years ago

twee > outfile.html generates this stack trace:

Traceback (most recent call last): File "/Users/jjohnston/src/twee/twee", line 104, in main(sys.argv[1:]) File "/Users/jjohnston/src/twee/twee", line 88, in main print tw.toHtml() TypeError: toHtml() takes at least 2 arguments (1 given)

I resolved this by changing line 87 to print tw.toHtml(None, None)

It seems that even though the method toHtml() sets defaults for its arguments, python would see like to see two parameters passed in. Go figure.

taskboy3000 commented 11 years ago

This bug is not present in the latest git code. It is present in the download from here: http://twee.googlecode.com/files/twee-1.7.1.zip

Feel free to close this.

narFnarF commented 11 years ago

Weird... Your fix is different than the one I found elsewhere on google...

I wrote "print tw.toHtml(scriptPath)" and it worked.

I have no idea why. I just copy paste stuff until it works! :D

I made the edit and forked. You can see my pull request here: https://github.com/tweecode/twee/pull/10

I guess it's irrelevant now??

yuwei commented 10 years ago

This is still an issue with 1.7.1 on Debian 7, I get the same error:

$ ./twee /dev/null  > /dev/null
Traceback (most recent call last):
  File "./twee", line 103, in <module>
    main(sys.argv[1:])
  File "./twee", line 87, in main
    print tw.toHtml()
TypeError: toHtml() takes at least 2 arguments (1 given)

Can you make a new release so we don't keep running into the same issue?