Bug 43460

Summary: prepare-ChangeLog fails mysteriously if curl doesn't support ssl
Product: WebKit Reporter: Aaron Boodman <aa>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: aroben, cjerdonek, darin, dbates, ddkilzer, eric, joepeck
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Patch eric: review+

Description Aaron Boodman 2010-08-03 19:42:19 PDT
It ends up printing "Bug $bugNumber has no bug description. Maybe you set wrong bug ID?", which isn't really the problem.
Comment 1 Aaron Boodman 2010-08-03 20:05:20 PDT
Created attachment 63408 [details]
Patch
Comment 2 Joseph Pecoraro 2010-08-03 20:17:28 PDT
This patch looks good to me, however the output might be confusing to
users. I think it would be fine with an added line of output like:

  "Could not get the bug description for bug $bugNumber"

Other than that I'm good with it. I'm CCing Eric, who has some more
experience with Scripts (and he originally wrote the code above using
curl) to see if he can sanity check it as well.

Thanks for the quick improvement Aaron!
Comment 3 Eric Seidel (no email) 2010-08-03 20:20:22 PDT
Comment on attachment 63408 [details]
Patch

Looks FANTASTIC.  Thank you.

Maybe we should move to using some built in perl way of fetching the bug instead?  I don't know what built-in perl SSL solutions there are.
Comment 4 Eric Seidel (no email) 2010-08-03 20:21:28 PDT
Comment on attachment 63408 [details]
Patch

I CC'd other folks who have perl experience in case they have bright ideas.  But looks good to me.
Comment 5 Joseph Pecoraro 2010-08-03 20:24:39 PDT
(In reply to comment #4)
> (From update of attachment 63408 [details])
> I CC'd other folks who have perl experience in case they have
> bright ideas.  But looks good to me.

Thanks for the quick reply! I CC'd someone as well.

@Aaron, I still suggest adding the additional line out output to clarify
what went wrong. It also hints at why curl was used in the first place.
Comment 6 Aaron Boodman 2010-08-04 10:13:04 PDT
Committed r64654: <http://trac.webkit.org/changeset/64654>