<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>51708</bug_id>
          
          <creation_ts>2010-12-29 09:06:48 -0800</creation_ts>
          <short_desc>prepare-ChangeLog should support the -g option for specifying the git-commit</short_desc>
          <delta_ts>2010-12-29 11:59:39 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P3</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Martin Robinson">mrobinson</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>aroben</cc>
    
    <cc>eric</cc>
    
    <cc>ojan</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>327581</commentid>
    <comment_count>0</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-12-29 09:06:48 -0800</bug_when>
    <thetext>webkit-patch supports an easy-to-type -g option for specifying the git committish as well as as the longer --git-commit=. I think this should be extended to prepare-ChangeLog as well.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>327585</commentid>
    <comment_count>1</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-12-29 09:24:23 -0800</bug_when>
    <thetext>It seems the semantics of the prepare-ChangeLog option differ from those of the webkit-patch option, but I think that&apos;s a task for another bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>327589</commentid>
    <comment_count>2</comment_count>
      <attachid>77624</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-12-29 09:29:21 -0800</bug_when>
    <thetext>Created attachment 77624
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>327608</commentid>
    <comment_count>3</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-12-29 10:03:43 -0800</bug_when>
    <thetext>Committed r74746: &lt;http://trac.webkit.org/changeset/74746&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>327653</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-12-29 11:59:39 -0800</bug_when>
    <thetext>http://trac.webkit.org/changeset/74746 might have broken Leopard Intel Debug (Tests)</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>77624</attachid>
            <date>2010-12-29 09:29:21 -0800</date>
            <delta_ts>2010-12-29 09:50:33 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-51708-20101229092915.patch</filename>
            <type>text/plain</type>
            <size>4082</size>
            <attacher name="Martin Robinson">mrobinson</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCA3YzE0
ZDk0OTRkMDE1YjQyNzEyMzYzYWU5MWUzMzVlMDY1MDQzN2Q2Li4xYzZkYjQ0OWYwY2E4Yjg3Mjlk
MDdmMTBhMmZlNjQ0YzczYjUwODBmIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIv
VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTAtMTItMjkgIE1hcnRpbiBSb2Jp
bnNvbiAgPG1yb2JpbnNvbkBpZ2FsaWEuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgorCisgICAgICAgIHByZXBhcmUtQ2hhbmdlTG9nIHNob3VsZCBzdXBwb3J0IHRo
ZSAtZyBvcHRpb24gZm9yIHNwZWNpZnlpbmcgdGhlIGdpdC1jb21taXQKKyAgICAgICAgaHR0cHM6
Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTUxNzA4CisKKyAgICAgICAgQWRkIGEg
LWcgc2hvcnRjdXQgZm9yIHRoZSAtLWdpdC1jb21taXQgcHJlcGFyZS1DaGFuZ2VMb2cgb3B0aW9u
LiBUaGlzIG1hdGNoZXMKKyAgICAgICAgd2Via2l0LXBhdGNoLCB0aG91Z2ggdGhlIGV4YWN0IG1l
YW5pbmcgb2YgdGhlIHR3byBvcHRpb25zIHJlbWFpbiBkaWZmZXJlbnQuCisKKyAgICAgICAgKiBT
Y3JpcHRzL3ByZXBhcmUtQ2hhbmdlTG9nOiBBZGQgLWcuCisKIDIwMTAtMTItMjggIFhhbiBMb3Bl
eiAgPHhsb3BlekBpZ2FsaWEuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEVyaWMgU2VpZGVs
LgpkaWZmIC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy9wcmVwYXJlLUNoYW5nZUxvZyBiL1Rvb2xzL1Nj
cmlwdHMvcHJlcGFyZS1DaGFuZ2VMb2cKaW5kZXggMmZjMDNkMmE5MDc3NDA4Y2VkYTEzMzEyNjUz
YjMyYjgxYWQxOTYwNS4uZjdhODJmMzgxMGMyMDE0OTk4YTQxMjIyOWFmZjRkN2VkNDI4NDllMyAx
MDA3NTUKLS0tIGEvVG9vbHMvU2NyaXB0cy9wcmVwYXJlLUNoYW5nZUxvZworKysgYi9Ub29scy9T
Y3JpcHRzL3ByZXBhcmUtQ2hhbmdlTG9nCkBAIC0xMTcsNyArMTE3LDcgQEAgbXkgJHBhcnNlT3B0
aW9uc1Jlc3VsdCA9CiAgICAgICAgICAgICAgICAibmFtZTpzIiA9PiBcJG5hbWUsCiAgICAgICAg
ICAgICAgICAiZW1haWw6cyIgPT4gXCRlbWFpbEFkZHJlc3MsCiAgICAgICAgICAgICAgICAibWVy
Z2UtYmFzZTpzIiA9PiBcJG1lcmdlQmFzZSwKLSAgICAgICAgICAgICAgICJnaXQtY29tbWl0OnMi
ID0+IFwkZ2l0Q29tbWl0LAorICAgICAgICAgICAgICAgImdpdC1jb21taXR8ZzpzIiA9PiBcJGdp
dENvbW1pdCwKICAgICAgICAgICAgICAgICJnaXQtaW5kZXgiID0+IFwkZ2l0SW5kZXgsCiAgICAg
ICAgICAgICAgICAiZ2l0LXJldmlld2VyOnMiID0+IFwkZ2l0UmV2aWV3ZXIsCiAgICAgICAgICAg
ICAgICAiaGVscHxoISIgPT4gXCRzaG93SGVscCwKQEAgLTEyNSwxOCArMTI1LDE4IEBAIG15ICRw
YXJzZU9wdGlvbnNSZXN1bHQgPQogICAgICAgICAgICAgICAgIndyaXRlISIgPT4gXCR3cml0ZUNo
YW5nZUxvZ3MsCiAgICAgICAgICAgICAgICAidXBkYXRlISIgPT4gXCR1cGRhdGVDaGFuZ2VMb2dz
KTsKIGlmICghJHBhcnNlT3B0aW9uc1Jlc3VsdCB8fCAkc2hvd0hlbHApIHsKLSAgICBwcmludCBT
VERFUlIgYmFzZW5hbWUoJDApIC4gIiBbLWJ8LS1idWc9PGJ1Z2lkPl0gWy1kfC0tZGlmZl0gWy1o
fC0taGVscF0gWy1vfC0tb3Blbl0gWy0tZ2l0LWNvbW1pdD08Y29tbWl0dGlzaD5dIFstLWdpdC1y
ZXZpZXdlcj08bmFtZT5dIFtzdm5kaXIxIFtzdm5kaXIyIC4uLl1dXG4iOwotICAgIHByaW50IFNU
REVSUiAiICAtYnwtLWJ1ZyAgICAgICBGaWxsIGluIHRoZSBDaGFuZ2VMb2cgYnVnIGluZm9ybWF0
aW9uIGZyb20gdGhlIGdpdmVuIGJ1Zy5cbiI7Ci0gICAgcHJpbnQgU1RERVJSICIgIC1kfC0tZGlm
ZiAgICAgIFNwZXcgZGlmZiB0byBzdGRvdXQgd2hlbiBydW5uaW5nXG4iOwotICAgIHByaW50IFNU
REVSUiAiICAtLW1lcmdlLWJhc2UgICBQb3B1bGF0ZSB0aGUgQ2hhbmdlTG9ncyB3aXRoIHRoZSBk
aWZmIHRvIHRoaXMgYnJhbmNoXG4iOwotICAgIHByaW50IFNUREVSUiAiICAtLWdpdC1jb21taXQg
ICBQb3B1bGF0ZSB0aGUgQ2hhbmdlTG9ncyBmcm9tIHRoZSBzcGVjaWZpZWQgZ2l0IGNvbW1pdFxu
IjsKLSAgICBwcmludCBTVERFUlIgIiAgLS1naXQtaW5kZXggICAgUG9wdWxhdGUgdGhlIENoYW5n
ZUxvZ3MgZnJvbSB0aGUgZ2l0IGluZGV4IG9ubHlcbiI7Ci0gICAgcHJpbnQgU1RERVJSICIgIC0t
Z2l0LXJldmlld2VyIFdoZW4gcG9wdWxhdGluZyB0aGUgQ2hhbmdlTG9ncyBmcm9tIGEgZ2l0IGNv
bW1pdCBjbGFpbSB0aGF0IHRoZSBzcGNpZmllZCBuYW1lIHJldmlld2VkIHRoZSBjaGFuZ2UuXG4i
OwotICAgIHByaW50IFNUREVSUiAiICAgICAgICAgICAgICAgICBUaGlzIG9wdGlvbiBpcyB1c2Vm
dWwgd2hlbiB0aGUgZ2l0IGNvbW1pdCBsYWNrcyBhIFNpZ25lZC1PZmYtQnk6IGxpbmVcbiI7Ci0g
ICAgcHJpbnQgU1RERVJSICIgIC1ofC0taGVscCAgICAgIFNob3cgdGhpcyBoZWxwIG1lc3NhZ2Vc
biI7Ci0gICAgcHJpbnQgU1RERVJSICIgIC1vfC0tb3BlbiAgICAgIE9wZW4gQ2hhbmdlTG9ncyBp
biBhbiBlZGl0b3Igd2hlbiBkb25lXG4iOwotICAgIHByaW50IFNUREVSUiAiICAtLVtuby1ddXBk
YXRlICBVcGRhdGUgQ2hhbmdlTG9ncyBmcm9tIHN2biBiZWZvcmUgYWRkaW5nIGVudHJ5IChkZWZh
dWx0OiB1cGRhdGUpXG4iOwotICAgIHByaW50IFNUREVSUiAiICAtLVtuby1dd3JpdGUgICBXcml0
ZSBDaGFuZ2VMb2dzIHRvIGRpc2sgKG90aGVyd2lzZSBzZW5kIG5ldyBlbnRyaWVzIHRvIHN0ZG91
dCkgKGRlZmF1bHQ6IHdyaXRlKVxuIjsKKyAgICBwcmludCBTVERFUlIgYmFzZW5hbWUoJDApIC4g
IiBbLWJ8LS1idWc9PGJ1Z2lkPl0gWy1kfC0tZGlmZl0gWy1ofC0taGVscF0gWy1vfC0tb3Blbl0g
Wy1nfC0tZ2l0LWNvbW1pdD08Y29tbWl0dGlzaD5dIFstLWdpdC1yZXZpZXdlcj08bmFtZT5dIFtz
dm5kaXIxIFtzdm5kaXIyIC4uLl1dXG4iOworICAgIHByaW50IFNUREVSUiAiICAtYnwtLWJ1ZyAg
ICAgICAgRmlsbCBpbiB0aGUgQ2hhbmdlTG9nIGJ1ZyBpbmZvcm1hdGlvbiBmcm9tIHRoZSBnaXZl
biBidWcuXG4iOworICAgIHByaW50IFNUREVSUiAiICAtZHwtLWRpZmYgICAgICAgU3BldyBkaWZm
IHRvIHN0ZG91dCB3aGVuIHJ1bm5pbmdcbiI7CisgICAgcHJpbnQgU1RERVJSICIgIC0tbWVyZ2Ut
YmFzZSAgICBQb3B1bGF0ZSB0aGUgQ2hhbmdlTG9ncyB3aXRoIHRoZSBkaWZmIHRvIHRoaXMgYnJh
bmNoXG4iOworICAgIHByaW50IFNUREVSUiAiICAtZ3wtLWdpdC1jb21taXQgUG9wdWxhdGUgdGhl
IENoYW5nZUxvZ3MgZnJvbSB0aGUgc3BlY2lmaWVkIGdpdCBjb21taXRcbiI7CisgICAgcHJpbnQg
U1RERVJSICIgIC0tZ2l0LWluZGV4ICAgICBQb3B1bGF0ZSB0aGUgQ2hhbmdlTG9ncyBmcm9tIHRo
ZSBnaXQgaW5kZXggb25seVxuIjsKKyAgICBwcmludCBTVERFUlIgIiAgLS1naXQtcmV2aWV3ZXIg
IFdoZW4gcG9wdWxhdGluZyB0aGUgQ2hhbmdlTG9ncyBmcm9tIGEgZ2l0IGNvbW1pdCBjbGFpbSB0
aGF0IHRoZSBzcGNpZmllZCBuYW1lIHJldmlld2VkIHRoZSBjaGFuZ2UuXG4iOworICAgIHByaW50
IFNUREVSUiAiICAgICAgICAgICAgICAgICAgVGhpcyBvcHRpb24gaXMgdXNlZnVsIHdoZW4gdGhl
IGdpdCBjb21taXQgbGFja3MgYSBTaWduZWQtT2ZmLUJ5OiBsaW5lXG4iOworICAgIHByaW50IFNU
REVSUiAiICAtaHwtLWhlbHAgICAgICAgU2hvdyB0aGlzIGhlbHAgbWVzc2FnZVxuIjsKKyAgICBw
cmludCBTVERFUlIgIiAgLW98LS1vcGVuICAgICAgIE9wZW4gQ2hhbmdlTG9ncyBpbiBhbiBlZGl0
b3Igd2hlbiBkb25lXG4iOworICAgIHByaW50IFNUREVSUiAiICAtLVtuby1ddXBkYXRlICAgVXBk
YXRlIENoYW5nZUxvZ3MgZnJvbSBzdm4gYmVmb3JlIGFkZGluZyBlbnRyeSAoZGVmYXVsdDogdXBk
YXRlKVxuIjsKKyAgICBwcmludCBTVERFUlIgIiAgLS1bbm8tXXdyaXRlICAgIFdyaXRlIENoYW5n
ZUxvZ3MgdG8gZGlzayAob3RoZXJ3aXNlIHNlbmQgbmV3IGVudHJpZXMgdG8gc3Rkb3V0KSAoZGVm
YXVsdDogd3JpdGUpXG4iOwogICAgIGV4aXQgMTsKIH0KIAo=
</data>
<flag name="review"
          id="68780"
          type_id="1"
          status="+"
          setter="ojan"
    />
          </attachment>
      

    </bug>

</bugzilla>