Skip to content

Conversation

@nmaludy
Copy link
Member

@nmaludy nmaludy commented Apr 29, 2020

Sync dist_utils.py with latest from st2 repo

Fix is related to: StackStorm/st2#4929

Fixes error in CI: https://travis-ci.org/github/StackStorm/st2/jobs/681137227

    ERROR: Command errored out with exit status 1:
     command: /home/travis/build/StackStorm/st2/virtualenv/bin/python2.7 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-BVwhVy/st2-auth-backend-flat-file/setup.py'"'"'; __file__='"'"'/tmp/pip-install-BVwhVy/st2-auth-backend-flat-file/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-mhz4SU
         cwd: /tmp/pip-install-BVwhVy/st2-auth-backend-flat-file/
    Complete output (7 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-BVwhVy/st2-auth-backend-flat-file/setup.py", line 32, in <module>
        install_reqs, dep_links = fetch_requirements(REQUIREMENTS_FILE)
      File "dist_utils.py", line 74, in fetch_requirements
        if req.link:
    AttributeError: 'ParsedRequirement' object has no attribute 'link'
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

@nmaludy nmaludy added the bug label Apr 29, 2020
@nmaludy nmaludy requested review from Kami, arm4b and blag April 29, 2020 19:08
@nmaludy nmaludy self-assigned this Apr 29, 2020
Copy link
Member

@punkrokk punkrokk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I would like @armab approval first though

@punkrokk punkrokk added this to the 3.3.0 milestone Jun 30, 2020
@punkrokk punkrokk merged commit dbc3f59 into master Aug 4, 2020
@arm4b arm4b deleted the bugfix/sync-distutils branch August 4, 2020 18:42
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.

5 participants