StackStorm / orquestaconvert

Converts Mistral workflows into Orchestra workflows
Apache License 2.0
13 stars 7 forks source link

A few fixups #4

Closed blag closed 5 years ago

blag commented 6 years ago

Add Mac OS X support (the built-in sed is BSD sed, which has a different syntax for in-place editing), but we also support all major BSD userlands just because. This does not explicitly add support for Windows, Haiku, or any other OSes, so sed may still not be found/simply print to stdout/error out/explode on those systems.

Additionally, I pass VIRTUALENV_DIR as an environment variable to make so it creates the virtualenv directory the orquestaconvert.sh (here referred to as oc.sh) script asks for, instead of the default virtualenv directory. If that is not passed in, make still creates the virtualenv directory as before. Previously, this was only working because the default VIRTUALENV_DIR variable in the makefile was the same value as the VIRTUALENV_DIR variable in the oc.sh script, so this decouples those two files a bit more.

That change allows users to specify a different virtualenv by modifying the orquestaconvert.sh script, which should help keep things clean, and makes it easier/correct to add support for that as a command line option to oc.sh in the future.