<?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>150849</bug_id>
          
          <creation_ts>2015-11-03 12:43:24 -0800</creation_ts>
          <short_desc>Teach Makefile to build LayoutTestRelay when building for iOS Simulator</short_desc>
          <delta_ts>2015-11-10 21:19:58 -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>WebKit Local Build</version>
          <rep_platform>iPhone / iPad</rep_platform>
          <op_sys>Unspecified</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>
          <dependson>150962</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Daniel Bates">dbates</reporter>
          <assigned_to name="Daniel Bates">dbates</assigned_to>
          <cc>ap</cc>
    
    <cc>commit-queue</cc>
    
    <cc>ddkilzer</cc>
    
    <cc>jmarcell</cc>
    
    <cc>lforschler</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1139031</commentid>
    <comment_count>0</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2015-11-03 12:43:24 -0800</bug_when>
    <thetext>For convenience, we should support building LayoutTestRelay by default when building WebKit using `make SDKROOT=iphonesimulator`.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139032</commentid>
    <comment_count>1</comment_count>
      <attachid>264714</attachid>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2015-11-03 12:47:08 -0800</bug_when>
    <thetext>Created attachment 264714
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139057</commentid>
    <comment_count>2</comment_count>
      <attachid>264714</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2015-11-03 13:32:55 -0800</bug_when>
    <thetext>Comment on attachment 264714
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=264714&amp;action=review

&gt; Tools/LayoutTestRelay/Makefile:14
&gt; +ifneq (,$(SAVED_SDK_ROOT))
&gt; +	override SDKROOT = $(SAVED_SDK_ROOT)
&gt; +endif

Should we also unset SAVED_SDK_ROOT here?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139742</commentid>
    <comment_count>3</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2015-11-05 13:57:38 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; Comment on attachment 264714 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=264714&amp;action=review
&gt; 
&gt; &gt; Tools/LayoutTestRelay/Makefile:14
&gt; &gt; +ifneq (,$(SAVED_SDK_ROOT))
&gt; &gt; +	override SDKROOT = $(SAVED_SDK_ROOT)
&gt; &gt; +endif
&gt; 
&gt; Should we also unset SAVED_SDK_ROOT here?

Yes, will modify patch to undefine SAVED_SDK_ROOT inside the if block above before landing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139744</commentid>
    <comment_count>4</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2015-11-05 14:01:11 -0800</bug_when>
    <thetext>Committed r192073: &lt;http://trac.webkit.org/changeset/192073&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139801</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2015-11-05 15:14:47 -0800</bug_when>
    <thetext>Re-opened since this is blocked by bug 150962</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139845</commentid>
    <comment_count>6</comment_count>
      <attachid>264909</attachid>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2015-11-05 17:38:43 -0800</bug_when>
    <thetext>Created attachment 264909
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1139977</commentid>
    <comment_count>7</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2015-11-06 08:59:27 -0800</bug_when>
    <thetext>Committed r192106: &lt;http://trac.webkit.org/changeset/192106&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1140056</commentid>
    <comment_count>8</comment_count>
      <attachid>264909</attachid>
    <who name="Jason Marcell">jmarcell</who>
    <bug_when>2015-11-06 14:43:45 -0800</bug_when>
    <thetext>Comment on attachment 264909
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=264909&amp;action=review

&gt; Tools/ChangeLog:11
&gt; +        * LayoutTestRelay/Makefile: Define OVERWRITE_SDKROOT and OVERWRITE_ARCHS.

Nit: OVERWRITE_{ARCHS, SDKROOT} -&gt; OVERRIDE_{ARCHS, SDKROOT}

&gt; Makefile.shared:7
&gt; +	ifneq (,$(OVERRIDE_SDKROOT))

Why is this nested inside the first if block? Don&apos;t you want to use the OVERRIDE_SDKROOT regardless of whether SDKROOT was set?

&gt; Makefile.shared:9
&gt; +			XCODE_OPTIONS := $(XCODE_OPTIONS) SDKROOT=$(OVERRIDE_SDKROOT)

Why did you use this syntax instead of the += syntax that was previously used?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1140057</commentid>
    <comment_count>9</comment_count>
      <attachid>264909</attachid>
    <who name="Jason Marcell">jmarcell</who>
    <bug_when>2015-11-06 14:43:54 -0800</bug_when>
    <thetext>Comment on attachment 264909
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=264909&amp;action=review

&gt; Tools/ChangeLog:11
&gt; +        * LayoutTestRelay/Makefile: Define OVERWRITE_SDKROOT and OVERWRITE_ARCHS.

Nit: OVERWRITE_{ARCHS, SDKROOT} -&gt; OVERRIDE_{ARCHS, SDKROOT}

&gt; Makefile.shared:7
&gt; +	ifneq (,$(OVERRIDE_SDKROOT))

Why is this nested inside the first if block? Don&apos;t you want to use the OVERRIDE_SDKROOT regardless of whether SDKROOT was set?

&gt; Makefile.shared:9
&gt; +			XCODE_OPTIONS := $(XCODE_OPTIONS) SDKROOT=$(OVERRIDE_SDKROOT)

Why did you use this syntax instead of the += syntax that was previously used?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1141051</commentid>
    <comment_count>10</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2015-11-10 20:40:45 -0800</bug_when>
    <thetext>(In reply to comment #8)
&gt; Comment on attachment 264909 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=264909&amp;action=review
&gt; 
&gt; &gt; Tools/ChangeLog:11
&gt; &gt; +        * LayoutTestRelay/Makefile: Define OVERWRITE_SDKROOT and OVERWRITE_ARCHS.
&gt; 
&gt; Nit: OVERWRITE_{ARCHS, SDKROOT} -&gt; OVERRIDE_{ARCHS, SDKROOT}
&gt; 

Fixed in &lt;http://trac.webkit.org/changeset/192289&gt;.

&gt; &gt; Makefile.shared:7
&gt; &gt; +	ifneq (,$(OVERRIDE_SDKROOT))
&gt; 
&gt; Why is this nested inside the first if block? Don&apos;t you want to use the
&gt; OVERRIDE_SDKROOT regardless of whether SDKROOT was set?
&gt; 

Yes, we should respect OVERRIDE_SDKROOT regardless of whether SDKROOT is set. Filed bug #151127 to address this issue.

&gt; &gt; Makefile.shared:9
&gt; &gt; +			XCODE_OPTIONS := $(XCODE_OPTIONS) SDKROOT=$(OVERRIDE_SDKROOT)
&gt; 
&gt; Why did you use this syntax instead of the += syntax that was previously
&gt; used?

Notice that XCODE_OPTIONS is a recursively expanded variable by &lt;https://ftp.gnu.org/old-gnu/Manuals/make-3.79.1/html_chapter/make_6.html#SEC59&gt;. The += operator preserves the flavor of the variable by &lt;https://ftp.gnu.org/old-gnu/Manuals/make-3.79.1/html_chapter/make_6.html#SEC65&gt;. Due to the way Makefiles are executed, we want to expand $OVERRIDE_SDKROOT in the definition of XCODE_OPTIONS immediately (i.e. make XCODE_OPTIONS a simply expanded variable) so that we can clear it and allow subsequent Makefiles to define it again. Therefore, I used the := operator to define XCODE_OPTIONS as a simply-expanded variable. For some reason, I was concerned about unconditionally defining XCODE_OPTIONS as a simply expanded variable and chose to only convert XCODE_OPTIONS to a simply-expanded variable when $(OVERRIDE_SDKROOT) is defined. We should look to unconditionally define XCODE_OPTIONS as a simply expanded variable and then we can use the += operator when appending the expression &quot;SDKROOT=$(OVERRIDE_SDKROOT)&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1141052</commentid>
    <comment_count>11</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2015-11-10 20:45:01 -0800</bug_when>
    <thetext>Re-opening this bug since I inadvertently did not append LayoutTestRelay to the end of the list of modules to build in file Tools/Makefile. So, `make SDKROOT=iphonesimulator` only works when executed inside directory Tools/LayoutTestRelay. We should make `make SDKROOT=iphonesimulator` work when executed from directory Tools and the top-level WebKit checkout.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1141054</commentid>
    <comment_count>12</comment_count>
      <attachid>265266</attachid>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2015-11-10 20:55:24 -0800</bug_when>
    <thetext>Created attachment 265266
[Patch] Addendum - Build LayoutTestRelay when make invoked from Tools or top-level checkout</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1141058</commentid>
    <comment_count>13</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2015-11-10 21:19:58 -0800</bug_when>
    <thetext>Committed r192293: &lt;http://trac.webkit.org/changeset/192293&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>264714</attachid>
            <date>2015-11-03 12:47:08 -0800</date>
            <delta_ts>2015-11-05 17:38:40 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-150849-20151103124614.patch</filename>
            <type>text/plain</type>
            <size>1929</size>
            <attacher name="Daniel Bates">dbates</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTkwNzc1CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggY2I5YTRiNmE2YTRjYzI4YjU4NTc5ZjliNDU0N2VkYzgy
MTg5Nzc3ZC4uMTQ3OTAyMTZhZDk0ZGIyNTFkNjA1ODJhYTg0ZWEwYWIzZWY2YjQyZiAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1
IEBACisyMDE1LTExLTAzICBEYW5pZWwgQmF0ZXMgIDxkYWJhdGVzQGFwcGxlLmNvbT4KKworICAg
ICAgICBUZWFjaCBNYWtlZmlsZSB0byBidWlsZCBMYXlvdXRUZXN0UmVsYXkgd2hlbiBidWlsZGlu
ZyBmb3IgaU9TIFNpbXVsYXRvcgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9MTUwODQ5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgKiBMYXlvdXRUZXN0UmVsYXkvTWFrZWZpbGU6IFRlbXBvcmFyaWx5IG92ZXJy
aWRlIFNES1JPT1Qgd2hlbiBpdCBpcyBpcGhvbmVzaW11bGF0b3Igc28KKyAgICAgICAgdGhhdCB3
ZSBidWlsZCB3aXRoIHRoZSBkZWZhdWx0IFNESy4gV2UgYXNzdW1lIHRoYXQgdGhlIGRlZmF1bHQg
U0RLIGlzIHRoZSBPUyBYIFNESy4KKyAgICAgICAgKiBNYWtlZmlsZTogQXBwZW5kIExheW91dFRl
c3RSZWxheSB0byB0aGUgbGlzdCBvZiBtb2R1bGVzIHRvIGJ1aWxkIHdoZW4gYnVpbGRpbmcgd2l0
aAorICAgICAgICBTREsgaXBob25lc2ltdWxhdG9yLgorCiAyMDE1LTEwLTA4ICBEYW5pZWwgQmF0
ZXMgIDxkYWJhdGVzQGFwcGxlLmNvbT4KIAogICAgICAgICBBZGQgaU9TIDkgZGV2aWNlIGJ1aWxk
ZXIgdG8gV2ViS2l0IEJvdCBXYXRjaGVyJ3MgRGFzaGJvYXJkCmRpZmYgLS1naXQgYS9Ub29scy9M
YXlvdXRUZXN0UmVsYXkvTWFrZWZpbGUgYi9Ub29scy9MYXlvdXRUZXN0UmVsYXkvTWFrZWZpbGUK
aW5kZXggZTQ4OWZjNGJmODkxMGMwYmNhYzlhMDUyYTAyNTg1MDM1MDM4NmUxYi4uYjQ3ZjE1ZWRi
M2Y5MjJjYmY4M2U4M2Q5Y2I0NGQ2ZTM5ZWMyMzlkMiAxMDA2NDQKLS0tIGEvVG9vbHMvTGF5b3V0
VGVzdFJlbGF5L01ha2VmaWxlCisrKyBiL1Rvb2xzL0xheW91dFRlc3RSZWxheS9NYWtlZmlsZQpA
QCAtMSwzICsxLDE0IEBACiBTQ1JJUFRTX1BBVEggPSAuLi9TY3JpcHRzCiAKK2lmbmVxICgsJChT
REtST09UKSkKKwlpZmVxICgkKGZpbmRzdHJpbmcgaXBob25lc2ltdWxhdG9yLCQoU0RLUk9PVCkp
LGlwaG9uZXNpbXVsYXRvcikKKwkJU0FWRURfU0RLX1JPT1QgPSAkKFNES1JPT1QpCisJCW92ZXJy
aWRlIFNES1JPT1QgPQorCWVuZGlmCitlbmRpZgorCiBpbmNsdWRlIC4uLy4uL01ha2VmaWxlLnNo
YXJlZAorCitpZm5lcSAoLCQoU0FWRURfU0RLX1JPT1QpKQorCW92ZXJyaWRlIFNES1JPT1QgPSAk
KFNBVkVEX1NES19ST09UKQorZW5kaWYKZGlmZiAtLWdpdCBhL1Rvb2xzL01ha2VmaWxlIGIvVG9v
bHMvTWFrZWZpbGUKaW5kZXggZGNkYjRkOGE0YjAzY2E3NmJkODQ2ZTFjNjY4ZWIyYjQ4ZGEyZjgx
MC4uMTU0MGQxNjJiNmQ0YjVkOWE4M2E1MDc2YjI5NTkxZTQ2NWIxYWM0OCAxMDA2NDQKLS0tIGEv
VG9vbHMvTWFrZWZpbGUKKysrIGIvVG9vbHMvTWFrZWZpbGUKQEAgLTQsNiArNCw5IEBAIGlmbmVx
ICgsJChTREtST09UKSkKIAlpZmVxICgsJChmaW5kc3RyaW5nIG1hY29zeCwkKFNES1JPT1QpKSkK
IAkJTU9EVUxFUyA9IER1bXBSZW5kZXJUcmVlIFdlYktpdFRlc3RSdW5uZXIgLi4vU291cmNlL1Ro
aXJkUGFydHkvZ3Rlc3QveGNvZGUgVGVzdFdlYktpdEFQSQogCWVuZGlmCisJaWZlcSAoJChmaW5k
c3RyaW5nIGlwaG9uZXNpbXVsYXRvciwkKFNES1JPT1QpKSxpcGhvbmVzaW11bGF0b3IpCisJCU1P
RFVMRVMgKz0gTGF5b3V0VGVzdFJlbGF5CisJZW5kaWYKIGVuZGlmCiAKIGFsbDoK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>264909</attachid>
            <date>2015-11-05 17:38:43 -0800</date>
            <delta_ts>2015-11-05 21:12:35 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-150849-20151105173744.patch</filename>
            <type>text/plain</type>
            <size>3142</size>
            <attacher name="Daniel Bates">dbates</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTkyMDg4CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggZDhhZTE3YjgyZTM4NjdiMzBiMWQ0ZDFmNDA1OWRkYjUw
MjNiYjhiNy4uNThhYTJjMzY3M2NmNzk4ODVmMzRiYTIzYzhiMGJjMzEyOWZjMGNjZiAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1
IEBACisyMDE1LTExLTA1ICBEYW5pZWwgQmF0ZXMgIDxkYWJhdGVzQGFwcGxlLmNvbT4KKworICAg
ICAgICBUZWFjaCBNYWtlZmlsZSB0byBidWlsZCBMYXlvdXRUZXN0UmVsYXkgd2hlbiBidWlsZGlu
ZyBmb3IgaU9TIFNpbXVsYXRvcgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9MTUwODQ5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgT3ZlcnJpZGUgdGhlIHVzZXItc3BlY2lmaWVkIGFyZ3VtZW50cyBTREtST09U
IGFuZCBBUkNIUyB0byB1c2UgdGhlIGRlZmF1bHQgU0RLCisgICAgICAgIGFuZCBkZWZhdWx0IEFS
Q0hTIHNpbmNlIExheW91dFRlc3RSZWxheSBpcyBhIE1hYyBjb21tYW5kIGxpbmUgdG9vbC4KKwor
ICAgICAgICAqIExheW91dFRlc3RSZWxheS9NYWtlZmlsZTogRGVmaW5lIE9WRVJXUklURV9TREtS
T09UIGFuZCBPVkVSV1JJVEVfQVJDSFMuCisKIDIwMTUtMTEtMDUgIEphc29uIE1hcmNlbGwgIDxq
bWFyY2VsbEBhcHBsZS5jb20+CiAKICAgICAgICAgcnVuLXdlYmtpdC10ZXN0IHNob3VsZCBsb29r
IGluIC0tcm9vdCBkaXJlY3RvcnkgZm9yIExheW91dFRlc3RSZWxheQpkaWZmIC0tZ2l0IGEvVG9v
bHMvTGF5b3V0VGVzdFJlbGF5L01ha2VmaWxlIGIvVG9vbHMvTGF5b3V0VGVzdFJlbGF5L01ha2Vm
aWxlCmluZGV4IGU0ODlmYzRiZjg5MTBjMGJjYWM5YTA1MmEwMjU4NTAzNTAzODZlMWIuLmZkOWQw
YjY4NmY4ZjE1MmRlYTIyNTkyNGEwNmM4MTU1NWU4NjQ4ZWMgMTAwNjQ0Ci0tLSBhL1Rvb2xzL0xh
eW91dFRlc3RSZWxheS9NYWtlZmlsZQorKysgYi9Ub29scy9MYXlvdXRUZXN0UmVsYXkvTWFrZWZp
bGUKQEAgLTEsMyArMSw2IEBACiBTQ1JJUFRTX1BBVEggPSAuLi9TY3JpcHRzCiAKK09WRVJSSURF
X0FSQ0hTID0gZGVmYXVsdAorT1ZFUlJJREVfU0RLUk9PVCA9IGRlZmF1bHQKKwogaW5jbHVkZSAu
Li8uLi9NYWtlZmlsZS5zaGFyZWQKZGlmZiAtLWdpdCBhL0NoYW5nZUxvZyBiL0NoYW5nZUxvZwpp
bmRleCBkMDIzYTgwZGUyMDBmODQyMDMxMDIxYmNlZGFmYzNlZjM4YTk3NDM3Li5hYTNhNWY3YjBk
MTBkMjU5OThhNDBkZTY4YzEzZmI0ZmVhNzUwZDk0IDEwMDY0NAotLS0gYS9DaGFuZ2VMb2cKKysr
IGIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTUtMTEtMDUgIERhbmllbCBCYXRlcyAg
PGRhYmF0ZXNAYXBwbGUuY29tPgorCisgICAgICAgIFRlYWNoIE1ha2VmaWxlIHRvIGJ1aWxkIExh
eW91dFRlc3RSZWxheSB3aGVuIGJ1aWxkaW5nIGZvciBpT1MgU2ltdWxhdG9yCisgICAgICAgIGh0
dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNTA4NDkKKworICAgICAgICBS
ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBBZGQgc3VwcG9ydCBmb3Igb3Zl
cnJpZGluZyB0aGUgdXNlci1wcm92aWRlZCBhcmd1bWVudHMgU0RLUk9PVCBhbmQgQVJDSFMKKyAg
ICAgICAgb24gYSBwZXIgTWFrZWZpbGUgYmFzaXMuCisKKyAgICAgICAgKiBNYWtlZmlsZS5zaGFy
ZWQ6CisKIDIwMTUtMTEtMDMgIEJyZW50IEZ1bGdoYW0gIDxiZnVsZ2hhbUBhcHBsZS5jb20+CiAK
ICAgICAgICAgW1dpbl0gQ01ha2UgYnVpbGQgdXBkYXRlLgpkaWZmIC0tZ2l0IGEvTWFrZWZpbGUu
c2hhcmVkIGIvTWFrZWZpbGUuc2hhcmVkCmluZGV4IGE1Y2IyMTA4MzM3NDRkYzk4NGEwZWEzOGIy
YTE0NzU4MTMyNmVmMzguLjVkYWU4MTgyODM3YWIwZDZmNTA1NTdhMzlhMTAwNzZiZjk2OWMxZDgg
MTAwNjQ0Ci0tLSBhL01ha2VmaWxlLnNoYXJlZAorKysgYi9NYWtlZmlsZS5zaGFyZWQKQEAgLTQs
MTIgKzQsMjcgQEAgU0VUX0NPTE9SX0RJQUdOT1NUSUNTX0FSRyA9IGlmIFtbIC10IDEgXV07IHRo
ZW4gQ09MT1JfRElBR05PU1RJQ1NfQVJHPSJDT0xPUl9ESUEKIFhDT0RFX09QVElPTlMgPSBgcGVy
bCAtSSQoU0NSSVBUU19QQVRIKSAtTXdlYmtpdGRpcnMgLWUgJ3ByaW50IFhjb2RlT3B0aW9uU3Ry
aW5nKCknIC0tICQoQlVJTERfV0VCS0lUX09QVElPTlMpYCAkJHtDT0xPUl9ESUFHTk9TVElDU19B
Ukd9ICQoQVJHUykKIAogaWZuZXEgKCwkKFNES1JPT1QpKQotCVhDT0RFX09QVElPTlMgKz0gU0RL
Uk9PVD0kKFNES1JPT1QpCisJaWZuZXEgKCwkKE9WRVJSSURFX1NES1JPT1QpKQorCQlpZm5lcSAo
ZGVmYXVsdCwkKE9WRVJSSURFX1NES1JPT1QpKQorCQkJWENPREVfT1BUSU9OUyA6PSAkKFhDT0RF
X09QVElPTlMpIFNES1JPT1Q9JChPVkVSUklERV9TREtST09UKQorCQllbmRpZgorCQlPVkVSUklE
RV9TREtST09UID0KKwllbHNlCisJCVhDT0RFX09QVElPTlMgOj0gJChYQ09ERV9PUFRJT05TKSBT
REtST09UPSQoU0RLUk9PVCkKKwllbmRpZgogZW5kaWYKIAogaWZuZXEgKCwkKEFSQ0hTKSkKLQlY
Q09ERV9PUFRJT05TICs9IEFSQ0hTPSIkKEFSQ0hTKSIKLQlYQ09ERV9PUFRJT05TICs9IE9OTFlf
QUNUSVZFX0FSQ0g9Tk8KKwlpZm5lcSAoLCQoT1ZFUlJJREVfQVJDSFMpKQorCQlpZm5lcSAoZGVm
YXVsdCwkKE9WRVJSSURFX0FSQ0hTKSkKKwkJCVhDT0RFX09QVElPTlMgOj0gJChYQ09ERV9PUFRJ
T05TKSBBUkNIUz0iJChPVkVSUklERV9BUkNIUykiCisJCQlYQ09ERV9PUFRJT05TICs9IE9OTFlf
QUNUSVZFX0FSQ0g9Tk8KKwkJZW5kaWYKKwkJT1ZFUlJJREVfQVJDSFMgPQorCWVsc2UKKwkJWENP
REVfT1BUSU9OUyA6PSAkKFhDT0RFX09QVElPTlMpIEFSQ0hTPSIkKEFSQ0hTKSIKKwkJWENPREVf
T1BUSU9OUyArPSBPTkxZX0FDVElWRV9BUkNIPU5PCisJZW5kaWYKIGVuZGlmCiAKIERFRkFVTFRf
VkVSQk9TSVRZIDo9ICQoc2hlbGwgZGVmYXVsdHMgcmVhZCBvcmcud2Via2l0LkJ1aWxkQ29uZmln
dXJhdGlvbiBCdWlsZFRyYW5zY3JpcHRWZXJib3NpdHkgMj4vZGV2L251bGwgfHwgZWNobyAiZGVm
YXVsdCIpCg==
</data>
<flag name="review"
          id="290024"
          type_id="1"
          status="+"
          setter="ap"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>265266</attachid>
            <date>2015-11-10 20:55:24 -0800</date>
            <delta_ts>2015-11-10 21:14:56 -0800</delta_ts>
            <desc>[Patch] Addendum - Build LayoutTestRelay when make invoked from Tools or top-level checkout</desc>
            <filename>Bug150849Addendum.patch</filename>
            <type>text/plain</type>
            <size>1624</size>
            <attacher name="Daniel Bates">dbates</attacher>
            
              <data encoding="base64">RnJvbSA3ZjgxZDFjZTBmNzgyMzE1MTczYTVmOWE5MWJjMmNlNWI3ZDk3YzQyIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBEYW5pZWwgQmF0ZXMgPGRiYXRlc0B3ZWJraXQub3JnPgpEYXRl
OiBUdWUsIDEwIE5vdiAyMDE1IDIwOjQ4OjI2IC0wODAwClN1YmplY3Q6IFtQQVRDSF0gVGVhY2gg
TWFrZWZpbGUgdG8gYnVpbGQgTGF5b3V0VGVzdFJlbGF5IHdoZW4gYnVpbGRpbmcgZm9yIGlPUwog
U2ltdWxhdG9yIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNTA4NDkK
ClJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgoKVW5sZXNzIHZhcmlhYmxlIERPX05PVF9CVUlM
RF9MQVlPVVRfVEVTVF9SRUxBWSBpcyBub24tZW1wdHksIGJ1aWxkIExheW91dFRlc3RSZWxheSB3
aGVuCmJ1aWxkaW5nIFdlYktpdCB0b29scy4KCiogTWFrZWZpbGU6Ci0tLQogVG9vbHMvQ2hhbmdl
TG9nIHwgMTIgKysrKysrKysrKysrCiBUb29scy9NYWtlZmlsZSAgfCAgMyArKysKIDIgZmlsZXMg
Y2hhbmdlZCwgMTUgaW5zZXJ0aW9ucygrKQoKZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBi
L1Rvb2xzL0NoYW5nZUxvZwppbmRleCBiMWI3M2FkLi5mODI1ODI1IDEwMDY0NAotLS0gYS9Ub29s
cy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTUt
MTEtMTAgIERhbmllbCBCYXRlcyAgPGRhYmF0ZXNAYXBwbGUuY29tPgorCisgICAgICAgIFRlYWNo
IE1ha2VmaWxlIHRvIGJ1aWxkIExheW91dFRlc3RSZWxheSB3aGVuIGJ1aWxkaW5nIGZvciBpT1Mg
U2ltdWxhdG9yCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p
ZD0xNTA4NDkKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg
ICBVbmxlc3MgdmFyaWFibGUgRE9fTk9UX0JVSUxEX0xBWU9VVF9URVNUX1JFTEFZIGlzIG5vbi1l
bXB0eSwgYnVpbGQgTGF5b3V0VGVzdFJlbGF5IHdoZW4KKyAgICAgICAgYnVpbGRpbmcgV2ViS2l0
IHRvb2xzLgorCisgICAgICAgICogTWFrZWZpbGU6CisKIDIwMTUtMTEtMTAgIERhdmlkIEtpbHpl
ciAgPGRka2lsemVyQGFwcGxlLmNvbT4KIAogICAgICAgICBbY2hlY2std2Via2l0LXN0eWxlXSBG
YWxzZSBwb3NpdGl2ZSAocmVhZGFiaWxpdHkvZW51bV9jYXNpbmcpIG9uIENGLXN0eWxlZCBlbnVt
IG1lbWJlcnMKZGlmZiAtLWdpdCBhL1Rvb2xzL01ha2VmaWxlIGIvVG9vbHMvTWFrZWZpbGUKaW5k
ZXggZGNkYjRkOC4uNTVlODllYSAxMDA2NDQKLS0tIGEvVG9vbHMvTWFrZWZpbGUKKysrIGIvVG9v
bHMvTWFrZWZpbGUKQEAgLTQsNiArNCw5IEBAIGlmbmVxICgsJChTREtST09UKSkKIAlpZmVxICgs
JChmaW5kc3RyaW5nIG1hY29zeCwkKFNES1JPT1QpKSkKIAkJTU9EVUxFUyA9IER1bXBSZW5kZXJU
cmVlIFdlYktpdFRlc3RSdW5uZXIgLi4vU291cmNlL1RoaXJkUGFydHkvZ3Rlc3QveGNvZGUgVGVz
dFdlYktpdEFQSQogCWVuZGlmCisJaWZlcSAoLCQoRE9fTk9UX0JVSUxEX0xBWU9VVF9URVNUX1JF
TEFZKSkKKwkJTU9EVUxFUyArPSBMYXlvdXRUZXN0UmVsYXkKKwllbmRpZgogZW5kaWYKIAogYWxs
OgotLSAKMi4zLjggKEFwcGxlIEdpdC01OCkKCg==
</data>
<flag name="review"
          id="290326"
          type_id="1"
          status="+"
          setter="ap"
    />
          </attachment>
      

    </bug>

</bugzilla>