Skip to content

add missing newline to easy-install.pth#3582

Closed
lee218llnl wants to merge 1 commit intodevelopfrom
bugfix/python_easy_install2
Closed

add missing newline to easy-install.pth#3582
lee218llnl wants to merge 1 commit intodevelopfrom
bugfix/python_easy_install2

Conversation

@lee218llnl
Copy link
Copy Markdown
Contributor

@lee218llnl lee218llnl commented Mar 28, 2017

Fixes #3575.

I accidentally omitted a newline from the easy-install.pth header. This PR adds the newline.

@tgamblin
Copy link
Copy Markdown
Member

@lee218llnl: the original easy-install.pth had a newline at the end of the file too. See #3583. Do you want to take this one or that one?

@lee218llnl
Copy link
Copy Markdown
Contributor Author

@tgamblin by "original" do you mean the original one in Spack, or the one that easy install would generate outside of spack? From an older, non-spack-installed Python on our systems I see this:

bash-4.1$ cat /usr/apps/python2.7.7/lib/python2.7/site-packages/easy-install.pth
import sys; sys.__plen = len(sys.path)
./setuptools-4.0.1-py2.7.egg
./matplotlib-1.3.1-py2.7-linux-x86_64.egg
./python_dateutil-2.2-py2.7.egg
./six-1.7.2-py2.7.egg
./Cython-0.20.1-py2.7-linux-x86_64.egg
./virtualenv-1.11.6-py2.7.egg
./rpy2-2.4.1-py2.7-linux-x86_64.egg
./pylint-1.2.1-py2.7.egg
./astroid-1.1.1-py2.7.egg
./logilab_common-0.61.0-py2.7.egg
import sys; new=sys.path[sys.__plen:]; del sys.path[sys.__plen:]; p=getattr(sys,'__egginsert',0); sys.path[p:p]=new; sys.__egginsert = p+len(new)
bash-4.1$

@tgamblin
Copy link
Copy Markdown
Member

@lee218llnl original in spack

@tgamblin
Copy link
Copy Markdown
Member

superseded by #3583.

@tgamblin tgamblin closed this Mar 28, 2017
@tgamblin tgamblin deleted the bugfix/python_easy_install2 branch March 28, 2017 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants