[OE-core] [PATCH 1/1] create-pull-request: fix sed command for SUBJECT

Burton, Ross ross.burton at intel.com
Thu Dec 20 11:48:58 UTC 2018


So if I sent a patch with the subject "Fix #1234" it fails?

Sounds like a good reason to rewrite create-pull-request in Python to
avoid this nonsense.

Ross
On Thu, 20 Dec 2018 at 08:05, Robert Yang <liezhi.yang at windriver.com> wrote:
>
> Fixed when SUBJECT contains slash(/):
> sed: -e expression #1, char 35: unknown option to `s'
>
> Use '#' as separator to fix the problem.
>
> Signed-off-by: Robert Yang <liezhi.yang at windriver.com>
> ---
>  scripts/create-pull-request | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/create-pull-request b/scripts/create-pull-request
> index 280880b..ff0f7e6 100755
> --- a/scripts/create-pull-request
> +++ b/scripts/create-pull-request
> @@ -278,7 +278,7 @@ fi
>
>  # Replace the SUBJECT token with it.
>  if [ -n "$SUBJECT" ]; then
> -       sed -i -e "s/\*\*\* SUBJECT HERE \*\*\*/$SUBJECT/" "$CL"
> +       sed -i -e "s#\*\*\* SUBJECT HERE \*\*\*#$SUBJECT#" "$CL"
>  fi
>
>
> --
> 2.10.2
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core


More information about the Openembedded-core mailing list