[pw-ci] [PATCH] post_pw: Allow for whitespace on input
Michael Santana
msantana at redhat.com
Fri Mar 4 14:23:21 UTC 2022
I am not too familiar with xargs but a quick look at the man page
tells me that it defaults to just doing echo without any arguments.
>From my limited testing it did get rid of starting/trailing white
spaces. If this is the intended use then this patch LGTM
On Thu, Mar 3, 2022 at 2:36 PM Aaron Conole <aconole at redhat.com> wrote:
>
> The input parsing side is a bit too rigid, which prevents adding reports
> from all different robots. Allow the variables to have extra whitespace
> and trim it out when it arrives.
>
> Signed-off-by: Aaron Conole <aconole at redhat.com>
> ---
> post_pw.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/post_pw.sh b/post_pw.sh
> index 2998644..e98e8ab 100755
> --- a/post_pw.sh
> +++ b/post_pw.sh
> @@ -60,7 +60,7 @@ send_post() {
> fi
>
> context="$(echo "$report" | sed -n 's/.*Test-Label: //p' | tr ' ' '_' | tr ':' '-')"
> - state="$(echo "$report" | sed -n 's/.*Test-Status: //p')"
> + state="$(echo "$report" | sed -n 's/.*Test-Status: //p' | xargs)"
> description="$(echo "$report" | sed -ne 's/^_\(.*\)_$/\1/p')"
> patch_id="$(echo "$report" | sed -ne 's at .*href.*/patch[es]*/\(.*\)/\?".*@\1 at ip' | sed 's@/@@')"
> target_url="$link"
> --
> 2.31.1
>
More information about the Pwci
mailing list