These are chat archives for dropbox/pyston

16th
Jun 2015
Kevin Modzelewski
@kmod
Jun 16 2015 00:52
is there a specific type it's not happening for?
I think we only support a couple cases
we only do it for cases where we know __new__ always returns an instance of the class
since otherwise you're supposed to potentially call a different __init__
Chris Toshok
@toshok
Jun 16 2015 00:54
yeah, I found the special case __new__ behavior
we were missing a &PyUnicode_Type check there, so we were never rewriting the float(u”…”) that django does
Chris Toshok
@toshok
Jun 16 2015 01:01
does investigate.py cache builds of pyston_release?
Kevin Modzelewski
@kmod
Jun 16 2015 02:05
yeah
there was a bug in it until today that it didn't work correctly with the new build directory
so you will need to git pull and then rm -rf saved_builds
Chris Toshok
@toshok
Jun 16 2015 02:40
okay cool
Chris Toshok
@toshok
Jun 16 2015 21:25
hm, I added some libraries to the toplevel CMakeLists.txt, and travis-ci doesn’t seem to be using them
failed build here: https://travis-ci.org/dropbox/pyston/jobs/67092327 (notice no -lcurses -lbz2). they’re added here: toshok/pyston@6c770ab#diff-af3b638bc2a3e6c650974192a53c7291R215
things build and run locally just fine
anyone seen anything like that before?
Marius Wachtler
@undingen
Jun 16 2015 22:16
I think you have to add the libraries also to ./test/unittests/CMakeLists.txt
Chris Toshok
@toshok
Jun 16 2015 22:58
ahh, awesome
Chris Toshok
@toshok
Jun 16 2015 23:47
from my latest travis-ci build:
    File "/home/travis/pyston-build/from_cpython/Lib/tarfile.py", line 1527, in __init__
      if hasattr(fileobj, "mode"):
AttributeError: 'file' object has no attribute ‘mode’