<?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>18485</bug_id>
          
          <creation_ts>2008-04-14 04:06:11 -0700</creation_ts>
          <short_desc>Typo and documentation fix for build-webkit</short_desc>
          <delta_ts>2008-04-24 17:53:35 -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>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</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="Jan Alonzo">jmalonzo</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>jchaffraix</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>77531</commentid>
    <comment_count>0</comment_count>
    <who name="Jan Alonzo">jmalonzo</who>
    <bug_when>2008-04-14 04:06:11 -0700</bug_when>
    <thetext>build-webkit currently has a typo in foreign object and the documentation for --clean is unclear. Also, the doc for --video currently doesn&apos;t show if it&apos;s enabled or not.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77532</commentid>
    <comment_count>1</comment_count>
      <attachid>20527</attachid>
    <who name="Jan Alonzo">jmalonzo</who>
    <bug_when>2008-04-14 04:08:15 -0700</bug_when>
    <thetext>Created attachment 20527
Typo and doc fixes

This patch fixes the typo and FO and --clean doc of build-webkit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77540</commentid>
    <comment_count>2</comment_count>
    <who name="Julien Chaffraix">jchaffraix</who>
    <bug_when>2008-04-14 05:14:32 -0700</bug_when>
    <thetext>&gt; This patch fixes the typo and FO and --clean doc of build-webkit.
&gt; 

The fix for the default video option is weird.
The real issue is that we do not have a platform check in isCygwin() and thus returns an empty string when we are not under cygwin.

I would advise you to make isCygwin() behave as the isOSX() method: check for platforms and add a fallback.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>78617</commentid>
    <comment_count>3</comment_count>
      <attachid>20527</attachid>
    <who name="Maciej Stachowiak">mjs</who>
    <bug_when>2008-04-22 22:35:34 -0700</bug_when>
    <thetext>Comment on attachment 20527
Typo and doc fixes

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>78842</commentid>
    <comment_count>4</comment_count>
    <who name="Mark Rowe (bdash)">mrowe</who>
    <bug_when>2008-04-24 17:53:35 -0700</bug_when>
    <thetext>Landed in r32527.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>20527</attachid>
            <date>2008-04-14 04:08:15 -0700</date>
            <delta_ts>2008-04-22 22:35:34 -0700</delta_ts>
            <desc>Typo and doc fixes</desc>
            <filename>webkit-buildwebkit-fix.patch</filename>
            <type>text/plain</type>
            <size>2579</size>
            <attacher name="Jan Alonzo">jmalonzo</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCBmMDQyNDZkLi4wNzg2MDYyIDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9DaGFuZ2VM
b2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAKKzIwMDgtMDQt
MTQgIEphbiBNaWNoYWVsIEFsb256byAgPGptYWxvbnpvQHVucGx1Z2dhYmxlLmNvbT4KKworICAg
ICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBodHRwOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xODQ4NQorICAgICAgICBUeXBvIGFuZCBkb2N1bWVu
dGF0aW9uIGZpeCBmb3IgYnVpbGQtd2Via2l0CisKKyAgICAgICAgKiBTY3JpcHRzL2J1aWxkLXdl
YmtpdDoKKwogMjAwOC0wNC0wOCAgS2V2aW4gT2xsaXZpZXIgIDxrZXZpbm9AdGhlb2xsaXZpZXJz
LmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBEYXJpbiBBZGxlci4KZGlmZiAtLWdpdCBhL1dl
YktpdFRvb2xzL1NjcmlwdHMvYnVpbGQtd2Via2l0IGIvV2ViS2l0VG9vbHMvU2NyaXB0cy9idWls
ZC13ZWJraXQKaW5kZXggNzI3M2IwOS4uNjRhZmNmMSAxMDA3NTUKLS0tIGEvV2ViS2l0VG9vbHMv
U2NyaXB0cy9idWlsZC13ZWJraXQKKysrIGIvV2ViS2l0VG9vbHMvU2NyaXB0cy9idWlsZC13ZWJr
aXQKQEAgLTU0LDcgKzU0LDcgQEAgbXkgJHN2Z0FzSW1hZ2VTdXBwb3J0ID0gMTsKIG15ICR4cGF0
aFN1cHBvcnQgPSAxOwogbXkgJHhzbHRTdXBwb3J0ID0gMTsKIG15ICRjb3ZlcmFnZVN1cHBvcnQg
PSAwOwotbXkgJHZpZGVvU3VwcG9ydCA9IGlzT1NYKCkgfHwgaXNDeWd3aW4oKTsgIyBFbmFibGUg
YnkgZGVmYXVsdCBvbiBPU1ggYW5kIFdpbmRvd3MKK215ICR2aWRlb1N1cHBvcnQgPSAoaXNPU1go
KSB8fCBpc0N5Z3dpbigpKSA/IDEgOiAwOyAjIEVuYWJsZSBieSBkZWZhdWx0IG9uIE9TWCBhbmQg
V2luZG93cwogbXkgJHNob3dIZWxwID0gMDsKIG15ICRjbGVhbiA9IDA7CiBteSAkYnVpbGRVbml2
ZXJzYWwgPSAwOwpAQCAtNjQsNyArNjQsNyBAQCBteSAkcHJvZ3JhbU5hbWUgPSBiYXNlbmFtZSgk
MCk7CiBteSAkdXNhZ2UgPSA8PEVPRjsKIFVzYWdlOiAkcHJvZ3JhbU5hbWUgW29wdGlvbnNdIFtv
cHRpb25zIHRvIHBhc3MgdG8gYnVpbGQgc3lzdGVtXQogICAtLWhlbHAgICAgICAgICAgICAgICAg
ICAgICAgICBTaG93IHRoaXMgaGVscCBtZXNzYWdlCi0gIC0tY2xlYW4gICAgICAgICAgICAgICAg
ICAgICAgIFBlcmZvcm0gYSBjbGVhbiBidWlsZAorICAtLWNsZWFuICAgICAgICAgICAgICAgICAg
ICAgICBDbGVhbnVwIHRoZSBidWlsZCBkaXJlY3RvcnkKICAgLS11bml2ZXJzYWwgICAgICAgICAg
ICAgICAgICAgQnVpbGQgMi13YXkgdW5pdmVyc2FsIChQUEMgYW5kIEludGVsIDMyLWJpdCkKICAg
LS02NC1iaXQgICAgICAgICAgICAgICAgICAgICAgQnVpbGQgNjQtYml0LCBjb21iaW5lIHdpdGgg
LS11bml2ZXJzYWwgdG8gYnVpbGQgNC13YXkgdW5pdmVyc2FsCiAgIC0tW25vLV1jcm9zcy1kb2N1
bWVudC1tZXNzYWdpbmcgICBUb2dnbGUgY3Jvc3MtZG9jdW1lbnQgbWVzc2FnaW5nIHN1cHBvcnQg
KGRlZmF1bHQ6ICRjcm9zc0RvY3VtZW50TWVzc2FnaW5nU3VwcG9ydCkKQEAgLTc2LDcgKzc2LDcg
QEAgVXNhZ2U6ICRwcm9ncmFtTmFtZSBbb3B0aW9uc10gW29wdGlvbnMgdG8gcGFzcyB0byBidWls
ZCBzeXN0ZW1dCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIGltcGxpZXMgU1ZHIFN1
cHBvcnQpCiAgIC0tW25vLV1zdmctYW5pbWF0aW9uICAgICAgICAgIFRvZ2dsZSBTVkcgYW5pbWF0
aW9uIHN1cHBvcnQgKGRlZmF1bHQ6ICRzdmdBbmltYXRpb25TdXBwb3J0LCBpbXBsaWVzIFNWRyBT
dXBwb3J0KQogICAtLVtuby1dc3ZnLWZpbHRlcnMgICAgICAgICAgICBUb2dnbGUgU1ZHIGZpbHRl
cnMgc3VwcG9ydCAoZGVmYXVsdDogJHN2Z0ZpbHRlcnNTdXBwb3J0LCBpbXBsaWVzIFNWRyBTdXBw
b3J0KQotICAtLVtuby1dc3ZnLWZvcmVpZ24tb2JqZWN0ICAgICBUb2dnbGUgU1ZHIGZvcmdlaW5n
IG9iamVjdCBzdXBwb3J0IChkZWZhdWx0OiAkc3ZnRm9yZWlnbk9iamVjdFN1cHBvcnQsIGltcGxp
ZXMgU1ZHIFN1cHBvcnQpCisgIC0tW25vLV1zdmctZm9yZWlnbi1vYmplY3QgICAgIFRvZ2dsZSBT
VkcgZm9yZWlnbiBvYmplY3Qgc3VwcG9ydCAoZGVmYXVsdDogJHN2Z0ZvcmVpZ25PYmplY3RTdXBw
b3J0LCBpbXBsaWVzIFNWRyBTdXBwb3J0KQogICAtLVtuby1dc3ZnLWZvbnRzICAgICAgICAgICAg
ICBUb2dnbGUgU1ZHIGZvbnRzIHN1cHBvcnQgKGRlZmF1bHQ6ICRzdmdGb250c1N1cHBvcnQsIGlt
cGxpZXMgU1ZHIFN1cHBvcnQpCiAgIC0tW25vLV1zdmctYXMtaW1hZ2UgICAgICAgICAgIFRvZ2ds
ZSBTVkcgYXMgSW1hZ2Ugc3VwcG9ydCAoZGVmYXVsdDogJHN2Z0FzSW1hZ2VTdXBwb3J0LCBpbXBs
aWVzIFNWRyBTdXBwb3J0KQogICAtLVtuby1dc3ZnLXVzZSAgICAgICAgICAgICAgICBUb2dnbGUg
U1ZHIHVzZSBlbGVtZW50IHN1cHBvcnQgKGRlZmF1bHQ6ICRzdmdVc2VTdXBwb3J0LCBpbXBsaWVz
IFNWRyBTdXBwb3J0KQo=
</data>
<flag name="review"
          id="8944"
          type_id="1"
          status="+"
          setter="mjs"
    />
          </attachment>
      

    </bug>

</bugzilla>