<?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>38449</bug_id>
          
          <creation_ts>2010-05-02 23:34:41 -0700</creation_ts>
          <short_desc>Web Inspector: build-webkit --inspector-frontend Should Exclude *.re2js</short_desc>
          <delta_ts>2010-05-06 00:05:24 -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>Web Inspector (Deprecated)</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>0</everconfirmed>
          <reporter name="Joseph Pecoraro">joepeck</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bweinstein</cc>
    
    <cc>commit-queue</cc>
    
    <cc>joepeck</cc>
    
    <cc>keishi</cc>
    
    <cc>pfeldman</cc>
    
    <cc>pmuellr</cc>
    
    <cc>rik</cc>
    
    <cc>timothy</cc>
    
    <cc>yurys</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>220027</commentid>
    <comment_count>0</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2010-05-02 23:34:41 -0700</bug_when>
    <thetext>The *.re2js files are preprocessed to produce their *.js counterparts. There is no need for them to be copied when building.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>220029</commentid>
    <comment_count>1</comment_count>
      <attachid>54906</attachid>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2010-05-02 23:36:16 -0700</bug_when>
    <thetext>Created attachment 54906
[PATCH] Proposed Fix

Also, the localizedStrings file might be useful to copy over, seeing as it is used by the Inspector.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>220034</commentid>
    <comment_count>2</comment_count>
      <attachid>54907</attachid>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2010-05-02 23:55:37 -0700</bug_when>
    <thetext>Created attachment 54907
[DRAFT] Copy localizedStrings.js for Mac OS X

This is not up for review that copies over the localizedStrings.js file for OS X systems. Anyone know where it goes for Windows and other platforms? Maybe then it would be worth landing?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221220</commentid>
    <comment_count>3</comment_count>
      <attachid>54906</attachid>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2010-05-05 12:09:12 -0700</bug_when>
    <thetext>Comment on attachment 54906
[PATCH] Proposed Fix

Should we move re2js files out of the folder longer term?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221446</commentid>
    <comment_count>4</comment_count>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2010-05-05 19:39:25 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; (From update of attachment 54906 [details])
&gt; Should we move re2js files out of the folder longer term?

Maybe move them into WebCore/inspector instead of WebCore/inspector/front-end?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221457</commentid>
    <comment_count>5</comment_count>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2010-05-05 20:58:38 -0700</bug_when>
    <thetext>I think staying in WebCore/inspector/front-end is fine, it keeps them next to the generated version.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221532</commentid>
    <comment_count>6</comment_count>
      <attachid>54906</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-05-06 00:05:18 -0700</bug_when>
    <thetext>Comment on attachment 54906
[PATCH] Proposed Fix

Clearing flags on attachment: 54906

Committed r58868: &lt;http://trac.webkit.org/changeset/58868&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221533</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-05-06 00:05:24 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>54906</attachid>
            <date>2010-05-02 23:36:16 -0700</date>
            <delta_ts>2010-05-06 00:05:17 -0700</delta_ts>
            <desc>[PATCH] Proposed Fix</desc>
            <filename>build.patch</filename>
            <type>text/plain</type>
            <size>1258</size>
            <attacher name="Joseph Pecoraro">joepeck</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCBhNGE5MWY4Li5jYmEyNDVkIDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9DaGFuZ2VM
b2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTAtMDUt
MDIgIEpvc2VwaCBQZWNvcmFybyAgPGpvZXBlY2tAd2Via2l0Lm9yZz4KKworICAgICAgICBSZXZp
ZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiBidWlsZC13
ZWJraXQgLS1pbnNwZWN0b3ItZnJvbnRlbmQgU2hvdWxkIEV4Y2x1ZGUgKi5yZTJqcworICAgICAg
ICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mzg0NDkKKworICAgICAg
ICAqIFNjcmlwdHMvd2Via2l0ZGlycy5wbToKKwogMjAxMC0wNC0yOCAgQ2hyaXMgSmVyZG9uZWsg
IDxjamVyZG9uZWtAd2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBEYW5pZWwgQmF0
ZXMuCmRpZmYgLS1naXQgYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdGRpcnMucG0gYi9XZWJL
aXRUb29scy9TY3JpcHRzL3dlYmtpdGRpcnMucG0KaW5kZXggNmYzY2E5Ni4uNGZlY2Y2YiAxMDA2
NDQKLS0tIGEvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRkaXJzLnBtCisrKyBiL1dlYktpdFRv
b2xzL1NjcmlwdHMvd2Via2l0ZGlycy5wbQpAQCAtMTA5OCw3ICsxMDk4LDcgQEAgc3ViIGNvcHlJ
bnNwZWN0b3JGcm9udGVuZEZpbGVzCiAgICAgICAgIHByaW50ICIqKioqKioqKioqKioqKioqKioq
KioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqXG4iOwogICAgICAgICBk
aWU7CiAgICAgfQotICAgIHJldHVybiBzeXN0ZW0gInJzeW5jIiwgIi1hdXQiLCAiLS1leGNsdWRl
PS8uRFNfU3RvcmUiLCAiLS1leGNsdWRlPS5zdm4vIiwgIWlzUXQoKSA/ICItLWV4Y2x1ZGU9L1dl
YktpdC5xcmMiIDogIiIsICRzb3VyY2VJbnNwZWN0b3JQYXRoLCAkaW5zcGVjdG9yUmVzb3VyY2Vz
RGlyUGF0aDsKKyAgICByZXR1cm4gc3lzdGVtICJyc3luYyIsICItYXV0IiwgIi0tZXhjbHVkZT0v
LkRTX1N0b3JlIiwgIi0tZXhjbHVkZT0qLnJlMmpzIiwgIi0tZXhjbHVkZT0uc3ZuLyIsICFpc1F0
KCkgPyAiLS1leGNsdWRlPS9XZWJLaXQucXJjIiA6ICIiLCAkc291cmNlSW5zcGVjdG9yUGF0aCwg
JGluc3BlY3RvclJlc291cmNlc0RpclBhdGg7CiB9CiAKIHN1YiBidWlsZFhDb2RlUHJvamVjdCgk
JEApCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>54907</attachid>
            <date>2010-05-02 23:55:37 -0700</date>
            <delta_ts>2010-05-02 23:55:37 -0700</delta_ts>
            <desc>[DRAFT] Copy localizedStrings.js for Mac OS X</desc>
            <filename>localized.patch</filename>
            <type>text/plain</type>
            <size>2464</size>
            <attacher name="Joseph Pecoraro">joepeck</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCBhNGE5MWY4Li43NWVmY2U5IDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9DaGFuZ2VM
b2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTAtMDUt
MDIgIEpvc2VwaCBQZWNvcmFybyAgPGpvZXBlY2tAd2Via2l0Lm9yZz4KKworICAgICAgICBSZXZp
ZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiBidWlsZC13
ZWJraXQgLS1pbnNwZWN0b3ItZnJvbnRlbmQgU2hvdWxkIEV4Y2x1ZGUgKi5yZTJqcworICAgICAg
ICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mzg0NDkKKworICAgICAg
ICBUaGlzIGFsc28gYXR0ZW1wdHMgdG8gY29weSBvdmVyIHRoZSBsb2NhbGl6ZWRTdHJpbmdzLmpz
IGZpbGUgZm9yCisgICAgICAgIE1hYyBPUyBYIHBsYXRmb3Jtcy4KKworICAgICAgICAqIFNjcmlw
dHMvd2Via2l0ZGlycy5wbToKKwogMjAxMC0wNC0yOCAgQ2hyaXMgSmVyZG9uZWsgIDxjamVyZG9u
ZWtAd2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBEYW5pZWwgQmF0ZXMuCmRpZmYg
LS1naXQgYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdGRpcnMucG0gYi9XZWJLaXRUb29scy9T
Y3JpcHRzL3dlYmtpdGRpcnMucG0KaW5kZXggNmYzY2E5Ni4uMTlhNjE4OSAxMDA2NDQKLS0tIGEv
V2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRkaXJzLnBtCisrKyBiL1dlYktpdFRvb2xzL1Njcmlw
dHMvd2Via2l0ZGlycy5wbQpAQCAtMTA3MCw3ICsxMDcwLDkgQEAgc3ViIGNvcHlJbnNwZWN0b3JG
cm9udGVuZEZpbGVzCiB7CiAgICAgbXkgJHByb2R1Y3REaXIgPSBwcm9kdWN0RGlyKCk7CiAgICAg
bXkgJHNvdXJjZUluc3BlY3RvclBhdGggPSBzb3VyY2VEaXIoKSAuICIvV2ViQ29yZS9pbnNwZWN0
b3IvZnJvbnQtZW5kLyI7CisgICAgbXkgJGxvY2FsaXplZFN0cmluZ3NQYXRoID0gc291cmNlRGly
KCkgLiAiL1dlYkNvcmUvRW5nbGlzaC5scHJvai9sb2NhbGl6ZWRTdHJpbmdzLmpzIjsKICAgICBt
eSAkaW5zcGVjdG9yUmVzb3VyY2VzRGlyUGF0aCA9ICRFTlZ7IldFQktJVElOU1BFQ1RPUlJFU09V
UkNFU0RJUiJ9OworICAgIG15ICRsb2NhbGl6ZWRTdHJpbmdzRGVzdGluYXRpb24gPSAiIjsKIAog
ICAgIGlmICghZGVmaW5lZCgkaW5zcGVjdG9yUmVzb3VyY2VzRGlyUGF0aCkpIHsKICAgICAgICAg
JGluc3BlY3RvclJlc291cmNlc0RpclBhdGggPSAiIjsKQEAgLTEwNzgsNiArMTA4MCw3IEBAIHN1
YiBjb3B5SW5zcGVjdG9yRnJvbnRlbmRGaWxlcwogCiAgICAgaWYgKGlzQXBwbGVNYWNXZWJLaXQo
KSkgewogICAgICAgICAkaW5zcGVjdG9yUmVzb3VyY2VzRGlyUGF0aCA9ICRwcm9kdWN0RGlyIC4g
Ii9XZWJDb3JlLmZyYW1ld29yay9SZXNvdXJjZXMvaW5zcGVjdG9yIjsKKyAgICAgICAgJGxvY2Fs
aXplZFN0cmluZ3NEZXN0aW5hdGlvbiA9ICRwcm9kdWN0RGlyIC4gIi9XZWJDb3JlLmZyYW1ld29y
ay9SZXNvdXJjZXMvRW5nbGlzaC5scHJvaiI7CiAgICAgfSBlbHNpZiAoaXNBcHBsZVdpbldlYktp
dCgpKSB7CiAgICAgICAgICRpbnNwZWN0b3JSZXNvdXJjZXNEaXJQYXRoID0gJHByb2R1Y3REaXIg
LiAiL1dlYktpdC5yZXNvdXJjZXMvaW5zcGVjdG9yIjsKICAgICB9IGVsc2lmIChpc1F0KCkgfHwg
aXNHdGsoKSkgewpAQCAtMTA5OCw3ICsxMTAxLDEzIEBAIHN1YiBjb3B5SW5zcGVjdG9yRnJvbnRl
bmRGaWxlcwogICAgICAgICBwcmludCAiKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioq
KioqKioqKioqKioqKioqKioqKioqKioqKioqKlxuIjsKICAgICAgICAgZGllOwogICAgIH0KLSAg
ICByZXR1cm4gc3lzdGVtICJyc3luYyIsICItYXV0IiwgIi0tZXhjbHVkZT0vLkRTX1N0b3JlIiwg
Ii0tZXhjbHVkZT0uc3ZuLyIsICFpc1F0KCkgPyAiLS1leGNsdWRlPS9XZWJLaXQucXJjIiA6ICIi
LCAkc291cmNlSW5zcGVjdG9yUGF0aCwgJGluc3BlY3RvclJlc291cmNlc0RpclBhdGg7CisKKyAg
ICBpZiAoLWQgJGxvY2FsaXplZFN0cmluZ3NEZXN0aW5hdGlvbikgeworICAgICAgICBzeXN0ZW0g
ImNwIiwgJGxvY2FsaXplZFN0cmluZ3NQYXRoLCAkbG9jYWxpemVkU3RyaW5nc0Rlc3RpbmF0aW9u
OworICAgICAgICBwcmludCAiRmFpbGVkIHRvIGNvcHkgb3ZlciBsb2NhbGl6ZWRTdHJpbmdzLmpz
LiIgaWYgJD8KKyAgICB9CisKKyAgICByZXR1cm4gc3lzdGVtICJyc3luYyIsICItYXV0IiwgIi0t
ZXhjbHVkZT0vLkRTX1N0b3JlIiwgIi0tZXhjbHVkZT0qLnJlMmpzIiwgIi0tZXhjbHVkZT0uc3Zu
LyIsICFpc1F0KCkgPyAiLS1leGNsdWRlPS9XZWJLaXQucXJjIiA6ICIiLCAkc291cmNlSW5zcGVj
dG9yUGF0aCwgJGluc3BlY3RvclJlc291cmNlc0RpclBhdGg7CiB9CiAKIHN1YiBidWlsZFhDb2Rl
UHJvamVjdCgkJEApCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>