Bright-Computing / bic

Bright-Illumina collaboration
GNU General Public License v2.0
4 stars 5 forks source link

Expand work on commit 4c92836, because it breaks bash tracing, as it has now #64

Closed fgeorgatos closed 7 years ago

fgeorgatos commented 7 years ago

Case:

[fgeorgatos@node060 ~]$ bash -x -c 'echo hello'
+ set +x
hello
[fgeorgatos@node060 ~]$ unset BASH_ENV
[fgeorgatos@node060 ~]$ bash -x -c 'echo hello'
+ echo hello
hello
[fgeorgatos@node060 ~]$

Example how to save+restore xtrace (-x) option, which would be the better way to handle this:

xtrace_saved="$(set -o | grep xtrace | grep 'on')";
set +x

echo hello

[ -n "$xtrace_saved" ] && set -x 
unset xtrace_saved
fgeorgatos commented 7 years ago

ETA?!

fgeorgatos commented 7 years ago

@johnnydevaprasad : we need some progress on this one...

fgeorgatos commented 7 years ago

calling this fixed via #69

fgeorgatos commented 7 years ago

and #67