search

gforsyth / xonda

DEPRECATED: (previously a thin wrapper around conda for xonsh)
BSD 3-Clause "New" or "Revised" License
11 stars 9 forks source link

Use xonsh to call 'real' conda, not subprocess #12

Closed gforsyth closed 7 years ago

gforsyth commented 7 years ago

Swapping in the @$ operator to force resolution of the actual conda entry point means that subprocess doesn't swallow stdout and stderr. And given that this is already a xsh-based xontrib, it seems reasonable that it would use xonsh syntax for this.

Should address #11

@scopatz, @asmeurer -- if either of you can try this out and confirm that it fixes your issue that would be great. Thanks!

asmeurer commented 7 years ago

Looks like it fixes it. I removed the which stuff from our code and it works with this branch.

scopatz commented 7 years ago

Thanks @gforsyth!