Closed perillo closed 5 months ago
Is seems that this change is also incorrect: https://github.com/SimonKagstrom/kcov/actions/runs/8208545224.
I was sure that the behavior was the same as the original code.
Let's merge this anyway, I'll resolve the test errors later. Thanks a lot!
Thanks to you.
Add the
look_path
function for searching executable in directories named by thePATH
environment variable in theutils
source file.Refactorize the
parse
function in configuration.cc, using the newlook_path
function. This greatly simplify the code.This PR implements an idea discussed #415.
The new change should behave the same as the current implementation, but unfortunately I can not verify this, because many tests fails on my system, probably due to incorrectly building the test suite.