Missing `stderr` redirection of `Updating ...` likely breaks pulp-actions in version 0.29.0

[ricted98]

I believe there is some issue with the printing of some strings during the checkout. Namely, if you try bender script -n flist | tee bender.txt and inspect the file content, you will see not only the file list as expected, but also:

   Updating axi:                      -> 0.31.1
    Updating common_cells:             -> 1.39.0
    Updating common_verification:      -> 0.2.5
    Updating fpnew:                    -> 2c7947726a0859c08cc27cbbd45662424dedb41e
    Updating fpu_div_sqrt_mvp:         -> 1.0.4
    Updating tech_cells_generic:       -> 0.2.13

This effectively breaks bender-up-to-date action parsing here: https://github.com/pulp-platform/pulp-actions/blob/9694ff5ec2c69dfbbb49f9b56c1e83b691a6b010/bender-up-to-date/bender-up-to-date.sh#L12.

I noticed this issue since the CI of CVA6 is systematically failing.

[micprog]

This is fixed in #231

[ricted98]

Ah I see, sorry for not noticing it in the recent commit history after v0.29.0. Would it be possible to have another release then? The other option would be to keep all GitHub CIs to the previous bender version until the new one is released or manually putting the latest commit hash?

[micprog]

working on a release 👍

[ricted98]

Great, thanks for the help! Closing this issue as it is completed for me.