Edouard Choinière

Before merging this PR, it might need a new title, since it isn't a new workflow. Also, is the existing workflow file name still appropriate for what it now does?...

I'm not sure if it's totally related, but with the previous PR that changed the grass script env loading, what I observed in my debugging in the last two-three months,...

> Unrelated, but it probably shouldn't use os.linesep. From https://docs.python.org/3.12/library/os.html#os.linesep os.linesep¶ The string used to separate (or, rather, terminate) lines on the current platform. This may be a single character,...

I'll take a closer look later, but I'm really favorable on the concept. Thanks!

So now that #3435 is done, what is the status of this PR?

Does this PR need to be updated following the recent changes in r.horizon? Yes, but they are not all merged yet.

@petrasovaa I didn't see your reply last week as you directly edited my comment instead of replying.

I have a doubt about the validity of this, since I just learned the existence of `make libsclean`, and I changed my repro to ```bash set -x export LOG_NUMBER=${LOG_NUMBER:-27} &&...

I think the report isn't valid, since the clean between the steps wasn't enough. But I worked hard on the second part of the pyc errors last week, I'm writing...

I also have concerns about passing around an object that has all env vars populated. I know in CI, we need to be careful to avoid leaking them to software...