erikrose / peep

A "pip install" that is cryptographically guaranteed repeatable
MIT License
221 stars 28 forks source link

support pip 7, where 'use_wheel' doesn't exist #94

Closed kouk closed 8 years ago

edmorley commented 9 years ago

Ah raced the PR update :-)

kouk commented 9 years ago

;-)

erikrose commented 9 years ago

Doesn't look like the tests pass under pip 7.0.1. I added 7.0 and 7.0.1 to the tox setup, and this happens:

tox -e py27-pip701

======================================================================
ERROR: If a hash matches, peep should do its work and exit happily.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 291, in test_success
    useless==1.0""")
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 267, in install_from_string
    return cls.install_from_path(reqs_path)
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 243, in install_from_path
    local=cls.index_url())
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 106, in run
    shell=True)
  File "/usr/local/Cellar/python/2.7.3/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 511, in check_call
    raise CalledProcessError(retcode, cmd)
CalledProcessError: Command 'python /Users/erose/Checkouts/peep/peep.py install -r /var/folders/39/cttgwlvj0rs_73t1z11pltrm0000gn/T/peep-WrGLGa/reqs.txt --index-url http://localhost:8001/' returned non-zero exit status 1

======================================================================
ERROR: Like test_success but with index argument in requirements.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 301, in test_success_req_index
    useless==1.0""")
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 279, in install_from_string_with_index
    return cls.install_from_path_no_index(reqs_path)
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 256, in install_from_path_no_index
    reqs=reqs_path)
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 106, in run
    shell=True)
  File "/usr/local/Cellar/python/2.7.3/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 511, in check_call
    raise CalledProcessError(retcode, cmd)
CalledProcessError: Command 'python /Users/erose/Checkouts/peep/peep.py install -r /var/folders/39/cttgwlvj0rs_73t1z11pltrm0000gn/T/peep-fv7p22/reqs.txt --no-index' returned non-zero exit status 1

======================================================================
ERROR: Make sure peep installing a GitHub-sourced tarball installs it,
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 341, in test_upgrade
    index_url=self.index_url()))
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 267, in install_from_string
    return cls.install_from_path(reqs_path)
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 243, in install_from_path
    local=cls.index_url())
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 106, in run
    shell=True)
  File "/usr/local/Cellar/python/2.7.3/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 511, in check_call
    raise CalledProcessError(retcode, cmd)
CalledProcessError: Command 'python /Users/erose/Checkouts/peep/peep.py install -r /var/folders/39/cttgwlvj0rs_73t1z11pltrm0000gn/T/peep-199BxH/reqs.txt --index-url http://localhost:8001/' returned non-zero exit status 1

======================================================================
FAIL: Make sure various permutations of inline comments are parsed
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 385, in test_inline_comments
    ['t9XWiL3TRb-ol3d9KXdWaIzwLhs3QsVoheLlwrmW_4I'])
AssertionError: [] != ['t9XWiL3TRb-ol3d9KXdWaIzwLhs3QsVoheLlwrmW_4I']

======================================================================
FAIL: Non-hash or malformed hash comments should be ignored.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 407, in test_non_hash_comments
    eq_(reqs[1]._expected_hashes(), ['abc'])
AssertionError: [] != ['abc']

======================================================================
FAIL: Make sure trailing whitespace is stripped from hashes.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/erose/Checkouts/peep/tests/__init__.py", line 419, in test_whitespace_stripping
    eq_(reqs[0]._expected_hashes(), ['trailing_space_should_be_stripped'])
AssertionError: [] != ['trailing_space_should_be_stripped']

----------------------------------------------------------------------
Ran 9 tests in 3.369s

FAILED (errors=3, failures=3)
kouk commented 9 years ago

Yes, as @glogiotatidis explained in https://github.com/erikrose/peep/pull/97 there were further incompatibilities. He proposed a fix in that PR, however I went with a different implementation in my last commit here.

edmorley commented 9 years ago

@erikrose do you have an opinion re this vs #97? Would be good to support pip 7.x soon :-)

indygreg commented 9 years ago

Some of my servers are starting to pick up pip 7 via system packaging and this bug is causing more and more headaches to work around. Is there any way I can help move this forward?

erikrose commented 8 years ago

Belated thanks for this nifty patch, kouk! Peep 2.5 is out, with your fix.