search

neurobin / shc

Shell script compiler
https://neurobin.org/projects/softwares/unix/shc/
GNU General Public License v3.0
2.04k stars 348 forks source link

Other PR's merged + code improved + flow improved #168

Open mdeweerd opened 3 months ago

mdeweerd commented 3 months ago

This merges (most) of the other PR's.

I'va also "converted" the travis workflow in a ci.yml that also compiles the script with gcc sanitize options. Those options effectively help detect leaks, and no longer detect leaks with the code in this PR.

mdeweerd commented 3 months ago

@suntong I think this is a good base to work with. I am not going to be the "codeowner" however.

FYI: the effectiveness of the leak detection can currently be seen in this action: https://github.com/mdeweerd/shc/actions/runs/10342670550/job/28625892548#step:4:2148

While it runs without leak reports here: https://github.com/mdeweerd/shc/actions/runs/10342637455/job/28625817171

suntong commented 3 months ago

Wow, thanks a lot for such hard work @mdeweerd, that's indeed what I was hoping for, that someone tries to pick up all the possible enhancements before we go for a new Debian release. Really appreciate it.

Let's give a few days for @neurobin / @ashamedbit to review, and see what they're going to say...

thanks again.