RFR: CODETOOLS-7901505 webrev.ksh extract of bug title fails after Jira 6 Upgrade

Mon Oct 26 21:15:38 UTC 2015

I'll take care of this.

-- Jon

On 10/23/2015 08:39 AM, Magnus Ihse Bursie wrote:
>  From the bug report:
> ---
> Since the Jira 6 upgrade, webrev.ksh does not correctly extract the bug id/title.
>
> It produces:
>    Bug id: <title>[JDK-8132541] (process) ProcessBuilder support for redirection to discard output - Java Bug System</title>
>
> previously the <title> and </title> would have been removed by line 2749:
>              msg=`$WGET --timeout=10 --tries=1 -q $url -O - | grep '<title>' | sed 's/<title>\[#\(.*\)\] \(.*\) - Java Bug System<\/title>/\1 : \2/' | html_dequote | html_quote`
>
> The '#' in the pattern seems to be no longer necessary.
> ---
>
> This patch applies the suggestion from the bug report:
>
> diff --git a/webrev.ksh b/webrev.ksh
> --- a/webrev.ksh
> +++ b/webrev.ksh
> @@ -27,7 +27,7 @@
>   # Documentation is available via 'webrev -h'.
>   #
>   
> -WEBREV_UPDATED=25.12-hg+openjdk.java.net
> +WEBREV_UPDATED=25.13-hg+openjdk.java.net
>   
>   HTML='<?xml version="1.0"?>
>   <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
> @@ -2746,7 +2746,7 @@
>           url="${BUGURL}${id}"
>   
>           if [[ -n $WGET ]]; then
> -            msg=`$WGET --timeout=10 --tries=1 -q $url -O - | grep '<title>' | sed 's/<title>\[#\(.*\)\] \(.*\) - Java Bug System<\/title>/\1 : \2/' | html_dequote | html_quote`
> +            msg=`$WGET --timeout=10 --tries=1 -q $url -O - | grep '<title>' | sed 's/<title>\[\(.*\)\] \(.*\) - Java Bug System<\/title>/\1 : \2/' | html_dequote | html_quote`
>           fi
>           if [[ -z $msg ]]; then
>               msg="${id}"
>
>
> Example webrev without patch:
> http://cr.openjdk.java.net/~ihse/JDK-8140282-remove-test-directories/webrev.02/
>
> Example webrev with patch:
> http://cr.openjdk.java.net/~ihse/JDK-8140282-remove-test-directories/webrev.03/
>
> /Magnus
>