Bug 26383 – prepare-ChangeLog should have a --bug= argument and use it for url autofill

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Eric Seidel (no email)]

prepare-ChangeLog should have a --bug= argument and use it for url autofill

It could use this to auto-fill in the bugzilla URL as well as the title of the bugzilla bug.

[Eric Seidel (no email)]

Created attachment 32163 [details]
First attempt

[Maciej Stachowiak]

Comment on attachment 32163 [details]
First attempt

r=me

I'm not sure about having quite so many "GOES HERE" fields, but let's see how it works out.

[Eric Seidel (no email)]

(In reply to comment #2)
> (From update of attachment 32163 [details] [review])
> r=me
> 
> I'm not sure about having quite so many "GOES HERE" fields, but let's see how
> it works out.

Yeah. I have similar concerns about "GOES HERE".  Especially since now on every ChangeLog you'll get a note about a missing description which you'll have to add.

I chose "GOES HERE" over WARNING: because I've seen people post patches with WARNING: in them, as though they did not understand that it was a template string they were supposed to remove.

I will send out a note to webkit-dev announcing this change and inviting further input.

[Eric Seidel (no email)]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebKitTools/ChangeLog
	M	WebKitTools/Scripts/prepare-ChangeLog
Committed r45464
http://trac.webkit.org/changeset/45464

[David Kilzer (:ddkilzer)]

Comment on attachment 32163 [details]
First attempt

>+if ($bugNumber) {
>+    $bugURL = "https://bugs.webkit.org/show_bug.cgi?id=$bugNumber";
>+    my $bugXMLURL = "$bugURL&ctype=xml";
>+    # Perl has no built in XML processing, so we'll fetch and parse with curl and grep
>+    my $descriptionLine = `curl --silent "$bugXMLURL" | grep short_desc`;
>+    $descriptionLine =~ /<short_desc>(.*)<\/short_desc>/;
>+    $bugDescription = $1;
>+    print STDERR "  Description from bug $bugNumber:\n    \"$bugDescription\".\n";
>+}

It would be nice if you had used libwww-perl modules to fetch the URL instead of shelling out to curl (to keep the script more portable), but I'm afraid using libwww-perl would require much more than one line of code here.

[Eric Seidel (no email)]

Yeah.  :( Once I realized there was no way to parse XML with the modules I had installed (I listed them all and looked through the first!) I kinda gave up on perl actually doing what I wanted and resorted to shell scripting.  My thought was that this may eventually convert to another language anyway...