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.

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

Export Windows Python on Windows builders #537

Closed
wants to merge 2 commits into from
Closed
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Next

Export Windows Python on Windows builders

  • Loading branch information
UK992 committed Nov 15, 2016
commit 7790a8ce1e7961c92d1bf4923201e3d409021477
@@ -176,9 +176,11 @@ def make_step(self, command):

command = command.split(' ')

# Add `bash -l` before every command on Windows builders
bash_args = ["bash", "-l"] if self.is_windows else []
step_kwargs['command'] = bash_args + command
# Add `bash -l -c` and export Windows Python before every command
# on Windows builders
bash_args = "export PATH=/c/python27:/c/python27/scripts:$PATH; "

This comment has been minimized.

@aneeshusa

aneeshusa Nov 15, 2016

Member

We have an Environment class which makes it easy to add things to environment. Is there some reason you can't update step_env instead (i.e. does the environment not get inherited through bash)?

This comment has been minimized.

@UK992

UK992 Nov 15, 2016

Author Contributor

yes, PATH environment variable does not get inherited through bash.

This comment has been minimized.

@aneeshusa

aneeshusa Nov 15, 2016

Member

Alternately, @larsbergstrom is there a .profile or .bash_profile that would reset the $PATH when we run bash?

command_win = ["bash", "-l", "-c", bash_args + " ".join(command)]
step_kwargs['command'] = command_win if self.is_windows else command

This comment has been minimized.

@metajack

metajack Nov 15, 2016

Contributor

Do you not need command_win = "" above the if? This looks like odd scoping.

This comment has been minimized.

@UK992

UK992 Nov 15, 2016

Author Contributor

i removed above if.

if self.is_windows:
step_env += envs.Environment({
# Set home directory, to avoid adding `cd` command every time
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.