Skip to content
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

BF: Don't modify PATH if not necessary #5035

Merged
merged 2 commits into from Oct 14, 2020
Merged

BF: Don't modify PATH if not necessary #5035

merged 2 commits into from Oct 14, 2020

Commits on Oct 14, 2020

  1. BF: Don't modify PATH if not necessary 

    PATH modification in Runner classes is meant to make sure we use
bundled w/ annex git executable (except when instructed otherwise
by DATALAD_USE_DEFAULT_GIT). However, don't mess with PATH when
what we find to be bundled is the same thing that is found with
unmodified PATH anyway. This is particularly important when the
'bundeled' location is /usr/bin and we are running in a virtual
environment, leading to that PATH overruling the venv.

(Closes datalad#5030)
    @bpoldrack
    bpoldrack committed Oct 14, 2020
    Configuration menu
    Copy the full SHA
    c76490d View commit details
    Browse the repository at this point in the history

  2. BF: Compare realpath to realpath

    @bpoldrack
    bpoldrack committed Oct 14, 2020
    Configuration menu
    Copy the full SHA
    908d3e8 View commit details
    Browse the repository at this point in the history