Pkg should not be shell-ing out

[vtjnash]

Why is the Julia Pkg manager trying to shell out? Window's doesn't have a shell.

julia> Pkg.add("Cairo")
C:\Documents and Settings\Administrator\My Documents\Downloads\julia-05c6461b55\
Git\bin\sh.exe: *** fork: can't reserve memory for stack 0x490000 - 0x690000, Wi
n32 error 0
      0 [main] sh 2604 sync_with_child: child 2848(0x32C) died before initializa
tion with status code 0x1
    238 [main] sh 2604 sync_with_child: *** child state waiting for longjmp
sh: fork: Resource temporarily unavailable
ERROR: failed process: Process(`sh -c "cd 'C:\Documents and Settings\Administrat
or\.julia\.cache\BinDeps' && git rev-parse --git-dir"`, ProcessExited(128)) [128
]
 in pipeline_error at process.jl:476
 in success at process.jl:468
 in readbytes at process.jl:430

[StefanKarpinski]

Provide me a way to cd after forking a subprocess and I can change it. cc: @loladiro

[Keno]

Pretty easy to do. The only question is syntax. I'd propose something similar to what we do with setenv(sh,{"A"=>"B"}) right now.

[vtjnash]

Mildly acceptable workaround is to use cmd

julia> ;cmd /c "cd C:\\ && ls"
AUTOEXEC.BAT            MSDOS.SYS     Program Files              WINDOWS
CONFIG.SYS              MinGW         Program Files (x86)        boot.ini
Documents and Settings  MinGW64       RECYCLER                   ntldr
IO.SYS                  NTDETECT.COM  System Volume Information  pagefile.sys

julia> ;sh -c "cd / && ls"
C:\Documents and Settings\Administrator\My Documents\Downloads\julia-05c6461b55\
Git\bin\sh.exe: *** fork: can't reserve memory for stack 0x490000 - 0x690000, Wi
n32 error 0
      0 [main] sh 600 sync_with_child: child 2372(0x32C) died before initializat
ion with status code 0x1
    607 [main] sh 600 sync_with_child: *** child state waiting for longjmp
sh: fork: Resource temporarily unavailable
ERROR: failed process: Process(`sh -c 'cd / && ls'`, ProcessExited(128)) [128]
 in pipeline_error at process.jl:476
 in success at process.jl:468
 in run at process.jl:453
 in repl_cmd at client.jl:62

@loladiro's suggestion is better

[StefanKarpinski]

How about a keyword argument that defaults to "" and does nothing when the empty string is supplied? For any other directory it does the cd first.

[StefanKarpinski]

Call it dir?

[Keno]

To clarify, you're proposing a dir keyword argument to setenv?

[Keno]

Ok, proposed syntax is

run(`ls`; dir="/")

,will implement that.

[Keno]

Implemented in dee7f6c. @StefanKarpinski you should have all the tools to tackle this now.