<?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>20669</bug_id>
          
          <creation_ts>2008-09-05 10:52:32 -0700</creation_ts>
          <short_desc>WebCore/css/makeprop.pl silently fails if gperf fails</short_desc>
          <delta_ts>2008-10-01 18:47:55 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Evan Martin">evan</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>90284</commentid>
    <comment_count>0</comment_count>
    <who name="Evan Martin">evan</who>
    <bug_when>2008-09-05 10:52:32 -0700</bug_when>
    <thetext>If gperf fails (for example, it&apos;s not installed), makeprop.pl goes on its merry way, leading to a confusing compiler error elsewhere.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90285</commentid>
    <comment_count>1</comment_count>
      <attachid>23196</attachid>
    <who name="Evan Martin">evan</who>
    <bug_when>2008-09-05 10:55:54 -0700</bug_when>
    <thetext>Created attachment 23196
trivial patch

I am embarrassed for this patch, since it is not even against your tree.  If I get a chance I will upload a better one.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90324</commentid>
    <comment_count>2</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2008-09-05 15:43:15 -0700</bug_when>
    <thetext>I expect it should still apply though.  We need to get you a copy of the real webkit. :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>90325</commentid>
    <comment_count>3</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2008-09-05 15:48:50 -0700</bug_when>
    <thetext>Needs a ChangeLog too.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>93639</commentid>
    <comment_count>4</comment_count>
      <attachid>24002</attachid>
    <who name="Evan Martin">evan</who>
    <bug_when>2008-10-01 18:41:08 -0700</bug_when>
    <thetext>Created attachment 24002
Die if gperf fails.

 WebCore/ChangeLog       |    7 +++++++
 WebCore/css/makeprop.pl |    2 +-
 2 files changed, 8 insertions(+), 1 deletions(-)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>93642</commentid>
    <comment_count>5</comment_count>
      <attachid>24002</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2008-10-01 18:47:16 -0700</bug_when>
    <thetext>Comment on attachment 24002
Die if gperf fails.

Looks fine.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>93643</commentid>
    <comment_count>6</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2008-10-01 18:47:55 -0700</bug_when>
    <thetext>r37178</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>23196</attachid>
            <date>2008-09-05 10:55:54 -0700</date>
            <delta_ts>2008-10-01 18:41:20 -0700</delta_ts>
            <desc>trivial patch</desc>
            <filename>gperf.patch</filename>
            <type>text/plain</type>
            <size>713</size>
            <attacher name="Evan Martin">evan</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL3dlYmtpdC9wZW5kaW5nL21ha2Vwcm9wLnBsIGIvd2Via2l0L3BlbmRpbmcv
bWFrZXByb3AucGwKaW5kZXggNTE5ZjY3ZC4uMDhlMjBmOSAxMDA2NDQKLS0tIGEvd2Via2l0L3Bl
bmRpbmcvbWFrZXByb3AucGwKKysrIGIvd2Via2l0L3BlbmRpbmcvbWFrZXByb3AucGwKQEAgLTg5
LDcgKzg5LDcgQEAgRU9GCiAKIGNsb3NlIEhFQURFUjsKIAotc3lzdGVtKCJncGVyZiAtYSAtTCBB
TlNJLUMgLUUgLUMgLWMgLW8gLXQgLS1rZXktcG9zaXRpb25zPVwiKlwiIC1OZmluZFByb3AgLUho
YXNoX3Byb3AgLVd3b3JkbGlzdF9wcm9wIC1EIC1zIDIgQ1NTUHJvcGVydHlOYW1lcy5ncGVyZiA+
IENTU1Byb3BlcnR5TmFtZXMuYyIpOworc3lzdGVtKCJncGVyZiAtYSAtTCBBTlNJLUMgLUUgLUMg
LWMgLW8gLXQgLS1rZXktcG9zaXRpb25zPVwiKlwiIC1OZmluZFByb3AgLUhoYXNoX3Byb3AgLVd3
b3JkbGlzdF9wcm9wIC1EIC1zIDIgQ1NTUHJvcGVydHlOYW1lcy5ncGVyZiA+IENTU1Byb3BlcnR5
TmFtZXMuYyIpID09IDAgfHwgZGllICJjYWxsaW5nIGdwZXJmIGZhaWxlZDogJD8iOwogCiBvcGVu
IEMsICI+PkNTU1Byb3BlcnR5TmFtZXMuYyIgfHwgZGllICJDb3VsZCBub3Qgb3BlbiBDU1NQcm9w
ZXJ0eU5hbWVzLmMgZm9yIHdyaXRpbmciOwogcHJpbnQgQyAgInN0YXRpYyBjb25zdCBjaGFyICog
Y29uc3QgcHJvcGVydHlMaXN0W10gPSB7XG4iOwo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>24002</attachid>
            <date>2008-10-01 18:41:08 -0700</date>
            <delta_ts>2008-10-01 18:47:16 -0700</delta_ts>
            <desc>Die if gperf fails.</desc>
            <filename>Die-if-gperf-fails..patch</filename>
            <type>text/plain</type>
            <size>1131</size>
            <attacher name="Evan Martin">evan</attacher>
            
              <data encoding="base64">MmRhMGRkMDhiZDkzZTZhZjJmMjk5YmQ4Y2EzNThiMzU2YTE1YTBmZQpkaWZmIC0tZ2l0IGEvV2Vi
Q29yZS9DaGFuZ2VMb2cgYi9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCA4NGY2NTQxLi4xNTkyODZh
IDEwMDY0NAotLS0gYS9XZWJDb3JlL0NoYW5nZUxvZworKysgYi9XZWJDb3JlL0NoYW5nZUxvZwpA
QCAtMSwzICsxLDEwIEBACisyMDA4LTEwLTAxICBFdmFuIE1hcnRpbiAgPGV2YW5AY2hyb21pdW0u
b3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICog
Y3NzL21ha2Vwcm9wLnBsOgorICAgICAgICBEaWUgaWYgZ3BlcmYgZmFpbHMuCisKIDIwMDgtMTAt
MDEgIERhbiBCZXJuc3RlaW4gIDxtaXR6QGFwcGxlLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBi
eSBNYXJrIFJvd2UuCmRpZmYgLS1naXQgYS9XZWJDb3JlL2Nzcy9tYWtlcHJvcC5wbCBiL1dlYkNv
cmUvY3NzL21ha2Vwcm9wLnBsCmluZGV4IDVkNzRiMDUuLmJjOTc5ZjkgMTAwNjQ0Ci0tLSBhL1dl
YkNvcmUvY3NzL21ha2Vwcm9wLnBsCisrKyBiL1dlYkNvcmUvY3NzL21ha2Vwcm9wLnBsCkBAIC05
Myw3ICs5Myw3IEBAIEVPRgogCiBjbG9zZSBIRUFERVI7CiAKLXN5c3RlbSgiZ3BlcmYgLWEgLUwg
QU5TSS1DIC1FIC1DIC1jIC1vIC10IC0ta2V5LXBvc2l0aW9ucz1cIipcIiAtTmZpbmRQcm9wIC1I
aGFzaF9wcm9wIC1Xd29yZGxpc3RfcHJvcCAtRCAtcyAyIENTU1Byb3BlcnR5TmFtZXMuZ3BlcmYg
PiBDU1NQcm9wZXJ0eU5hbWVzLmNwcCIpOworc3lzdGVtKCJncGVyZiAtYSAtTCBBTlNJLUMgLUUg
LUMgLWMgLW8gLXQgLS1rZXktcG9zaXRpb25zPVwiKlwiIC1OZmluZFByb3AgLUhoYXNoX3Byb3Ag
LVd3b3JkbGlzdF9wcm9wIC1EIC1zIDIgQ1NTUHJvcGVydHlOYW1lcy5ncGVyZiA+IENTU1Byb3Bl
cnR5TmFtZXMuY3BwIikgPT0gMCB8fCBkaWUgImNhbGxpbmcgZ3BlcmYgZmFpbGVkOiAkPyI7CiAK
IG9wZW4gQywgIj4+Q1NTUHJvcGVydHlOYW1lcy5jcHAiIHx8IGRpZSAiQ291bGQgbm90IG9wZW4g
Q1NTUHJvcGVydHlOYW1lcy5jcHAgZm9yIHdyaXRpbmciOwogcHJpbnQgQyAic3RhdGljIGNvbnN0
IGNoYXIgKiBjb25zdCBwcm9wZXJ0eU5hbWVTdHJpbmdzWyRudW1dID0ge1xuIjsK
</data>
<flag name="review"
          id="10849"
          type_id="1"
          status="+"
          setter="eric"
    />
          </attachment>
      

    </bug>

</bugzilla>