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

Nonfatal error during Neuron make install #60

Closed
wvangeit opened this issue Mar 6, 2018 · 3 comments

Comments

@wvangeit
Copy link

commented Mar 6, 2018

When I build the latest commit in master (5524b18)

I'm getting the following error on Mac OS X High Sierra (Python 2.7.14 provided by pyenv), when compiling Neuron:

byte-compiling /Users/werner/local/nrnnogui/lib/python2.7/site-packages/neuron/hclass3.py to hclass3.pyc
File "/Users/werner/local/nrnnogui/lib/python2.7/site-packages/neuron/hclass3.py", line 30
class hc(hoc.HocObject, metaclass=MetaHocObject):
^
SyntaxError: invalid syntax

It seems it's not a fatal error, because the build process finishes 'normally'

configure options: ../configure --with-nrnpython --without-x --disable-rx3d

@nrnhines

This comment has been minimized.

Copy link
Collaborator

commented Mar 6, 2018

Thanks. That is ok because hclass3.py is not imported by neuron/init.py if the python major
version is not 3. (hclass2.py and hclass3.py exist soley because of the different syntax required by
python major versons 2 and 3.). If anyone knows how to avoid in your situation trying to compile if not imported, I'd like to know as that
could avoid the error. I wonder if the contents of hclass2.py and hclass3.py could become a string
that is executed based on the value of sys.version_info[0].

@wvangeit

This comment has been minimized.

Copy link
Author

commented Mar 7, 2018

Ok, sure. If it doesn't affect the installation, it's not a problem. (Might only be confusing for people installing Neuron).

@nrnhines

This comment has been minimized.

Copy link
Collaborator

commented Apr 27, 2018

this is fixed in d2f4dd2

@nrnhines nrnhines closed this Apr 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.