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.

Sign up for GitHub

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

Jump to bottom

Set user parameter to True #9

Merged

bgschiller merged 1 commit into bgschiller:master from nikoskaragiannakis:bugfix/make_pip_installable

Feb 3, 2018

Contributor

nikoskaragiannakis commented Feb 3, 2018 •

edited

Loading

The user parameter in https://github.com/bgschiller/postgres_kernel/blob/master/setup.py#L31 is by default False in the distutils.command.install.install class (https://github.com/python/cpython/blob/master/Lib/distutils/command/install.py#L153). Thus, it tries to use '/usr/local/share/jupyter' and raises a Permission denied error.

To use the local jupyter kernel, you need to pass user=True

This small PR fixes the problem (I tested with python setup.py install before and after the fix).


          set user parameter to True

87c3344

nikoskaragiannakis mentioned this pull request

pip installable #3

Closed

bgschiller merged commit af0b7a3 into bgschiller:master

Owner

bgschiller commented Feb 3, 2018

Awesome, good catch. 👍

nikoskaragiannakis deleted the bugfix/make_pip_installable branch

February 3, 2018 20:23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet