Skip to content

include path to cmd.sh script in output generated by run_shell_cmd when a command fails + use colors: red for ERROR line, yellow for path to output files + cmd.sh script#4666

Merged
branfosj merged 9 commits intoeasybuilders:5.0.xfrom
boegel:run_shell_cmd_cmd.sh_script
Oct 15, 2024
Merged

include path to cmd.sh script in output generated by run_shell_cmd when a command fails + use colors: red for ERROR line, yellow for path to output files + cmd.sh script#4666
branfosj merged 9 commits intoeasybuilders:5.0.xfrom
boegel:run_shell_cmd_cmd.sh_script

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Oct 3, 2024

(WIP because tests will need some love)

…en a command fails + use colors: red for ERROR line, yellow for path to output files + cmd.sh script
@boegel boegel added this to the 5.0 milestone Oct 3, 2024
@boegel boegel marked this pull request as ready for review October 3, 2024 13:20
Comment thread easybuild/tools/run.py
Copy link
Copy Markdown
Contributor

@lexming lexming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but I'm not seeing any colors although I have rich installed. I see the color progress bar during installation, but then I got the following output on error:

[bold red]ERROR: Shell command failed![/bold red]
    full command              ->  potato &&  ./configure --prefix=/home/lexming/easybuild/install/software/cURL/8.0.1-GCCcore-12.3.0  --build=x86_64-pc-linux-gnu
--host=x86_64-pc-linux-gnu --with-zlib --with-ssl=$EBROOTOPENSSL
    exit code                 ->  127
    called from               ->  'configure_step' function in /home/lexming/src/easybuild-easyblocks/easybuild/easyblocks/generic/configuremake.py (line 326)
    working directory         ->  /home/lexming/.local/easybuild/build/cURL/8.0.1/GCCcore-12.3.0/curl-8.0.1
[bold yellow]    output (stdout + stderr)  ->  /tmp/eb-5zxm3a9y/run-shell-cmd-output/potato-t2b3f6fz/out.txt[/bold yellow]
[bold yellow]    interactive shell script  ->  /tmp/eb-5zxm3a9y/run-shell-cmd-output/potato-t2b3f6fz/cmd.sh[/bold yellow]

Comment thread easybuild/tools/run.py
Comment thread easybuild/tools/run.py Outdated
@boegel boegel force-pushed the run_shell_cmd_cmd.sh_script branch 2 times, most recently from e0e9dfe to 4998e4a Compare October 14, 2024 09:03
@boegel boegel force-pushed the run_shell_cmd_cmd.sh_script branch from 4998e4a to afd3527 Compare October 14, 2024 09:41
Comment thread easybuild/tools/run.py Outdated
@branfosj branfosj merged commit bb9f41f into easybuilders:5.0.x Oct 15, 2024
@boegel boegel deleted the run_shell_cmd_cmd.sh_script branch October 15, 2024 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants