OpenMS / THIRDPARTY

External binary dependencies (e.g. search engine binaries) packaged in installers
2 stars 12 forks source link

[ADD] Add Trap to sirius script to kill child processes #29

Closed lukaszimmermann closed 7 years ago

lukaszimmermann commented 7 years ago

Also fixes issue with dirname (did not produce expected result when used with sh). Now, "${0%/*}" is used for the script directory, which is more portable.

The save function was commented out, because it did not work in KNIME and caused Sirius to be invoked with no parameters in KNIME.

oliveralka commented 7 years ago

looks good and works for mac as well as linux (Thanks Lukas!).