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

Dial back Pythonfinder integration effort #2582

Merged
merged 57 commits into from Jul 30, 2018

Conversation

Projects
2 participants
@uranusjr
Member

uranusjr commented Jul 14, 2018

Let’s try this a little at a time…

This only uses Pythonfinder in find_a_system_python, and leaves everything else alone. I hope this approach will make rewrites easier to tackle.

@uranusjr uranusjr force-pushed the pythonfinder-lite branch 2 times, most recently from 4d54c61 to 893fb8f Jul 14, 2018

@techalchemy techalchemy force-pushed the pythonfinder-lite branch 9 times, most recently from 9c68041 to ff985cc Jul 15, 2018

'{} -c "import sys; print(sys.executable)"'.format(line),
)
if not isinstance(output, str):
output = output.decode(sys.getdefaultencoding())

This comment has been minimized.

@techalchemy

techalchemy Jul 17, 2018

Member

fs_str(output)

But we don't need to handle this anyway

c = delegator.run(
cmd, block=False, timeout=PIPENV_TIMEOUT, env=pip_config,
)
c.block()

This comment has been minimized.

@techalchemy

techalchemy Jul 17, 2018

Member

If we don't block here and we pass block=False we can't read the output...

This comment has been minimized.

@uranusjr

uranusjr Jul 17, 2018

Member

I think it should be indented inside the with block?

This comment has been minimized.

@techalchemy

techalchemy Jul 17, 2018

Member

makes sense to me, I just put it there because of no reason in particular

@uranusjr uranusjr added this to In progress in 2018.7.x Release via automation Jul 18, 2018

@techalchemy techalchemy force-pushed the pythonfinder-lite branch from 8143f40 to de8ddb0 Jul 18, 2018

@techalchemy

This comment has been minimized.

Member

techalchemy commented Jul 18, 2018

just rebased this, I think I'm probably good with it if you are (I can cut 1.0 of pythonfinder if we are satisfied)

uranusjr and others added some commits Jul 14, 2018

Update pythonfinder and make full use of it in support calls
Signed-off-by: Dan Ryan <dan@danryan.co>
Fully integrate pythonfinder for system pythons
Signed-off-by: Dan Ryan <dan@danryan.co>
Fix search path
Signed-off-by: Dan Ryan <dan@danryan.co>
Updated buildkite to echo out the path
Signed-off-by: Dan Ryan <dan@danryan.co>
Add some debugging
Signed-off-by: Dan Ryan <dan@danryan.co>
Updated buildkite to set home properly
Signed-off-by: Dan Ryan <dan@danryan.co>
More debug info for buildkite
Signed-off-by: Dan Ryan <dan@danryan.co>

techalchemy and others added some commits Jul 17, 2018

Better algorithm for find_all_versions
Signed-off-by: Dan Ryan <dan@danryan.co>
Strip dashes from `py -n` commands
Signed-off-by: Dan Ryan <dan@danryan.co>
Fix pythonfinder bug unnesting python versions
Signed-off-by: Dan Ryan <dan@danryan.co>
Fix click encoding for terminal outputs
Signed-off-by: Dan Ryan <dan@danryan.co>
Block before getting outputs
Signed-off-by: Dan Ryan <dan@danryan.co>
Add quotes
Signed-off-by: Dan Ryan <dan@danryan.co>
Fix aggregation for support
Signed-off-by: Dan Ryan <dan@danryan.co>
Pass arch into Windows finder
This is still not working though.
Update pythonfinder for better windows support
Signed-off-by: Dan Ryan <dan@danryan.co>
Update pythonfinder to default patch versions to 0
Signed-off-by: Dan Ryan <dan@danryan.co>
Update pythonfinder
Signed-off-by: Dan Ryan <dan@danryan.co>
Cached properties!
Signed-off-by: Dan Ryan <dan@danryan.co>
Add cached property to vendored deps
Signed-off-by: Dan Ryan <dan@danryan.co>
Update vendoring tasks for updating single vendored deps
Signed-off-by: Dan Ryan <dan@danryan.co>
Update vendoring instructions
Signed-off-by: Dan Ryan <dan@danryan.co>
Update pythonfinder
Signed-off-by: Dan Ryan <dan@danryan.co>

@techalchemy techalchemy force-pushed the pythonfinder-lite branch from 9b8b999 to ef060c4 Jul 25, 2018

techalchemy and others added some commits Jul 25, 2018

Update vendor.txt
Signed-off-by: Dan Ryan <dan@danryan.co>

@uranusjr uranusjr merged commit a853814 into master Jul 30, 2018

2 checks passed

VSTS: pipenv-Python Package-CI 20180730.3 succeeded
Details
buildkite/pipenv Build #1050 passed (11 minutes, 5 seconds)
Details

2018.7.x Release automation moved this from In progress to Done Jul 30, 2018

@uranusjr uranusjr deleted the pythonfinder-lite branch Jul 30, 2018

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