From d3d9e905630f472bb509b8dc98d110060bf03dc0 Mon Sep 17 00:00:00 2001
From: Drew Noakes <git@drewnoakes.com>
Date: Mon, 27 Sep 2021 09:53:39 +1000
Subject: [PATCH 1/8] ProjectReference protocol documentation updates

- Specify that `GetTargetFrameworks` is required for outer builds only
- Fix some markdown formatting so that XML elements appear correctly
- Remove a duplicated word
---
 documentation/ProjectReference-Protocol.md | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/documentation/ProjectReference-Protocol.md b/documentation/ProjectReference-Protocol.md
index 5e2a138e3d5..29800a2e498 100644
--- a/documentation/ProjectReference-Protocol.md
+++ b/documentation/ProjectReference-Protocol.md
@@ -22,7 +22,7 @@ Importing `Microsoft.Common.targets` includes logic that consumes these items an
 
 ## Who this document is for
 
-This document describes that process, including what is required of a project to be referenceable through a `ProjectReference`. It is intended for for MSBuild SDK maintainers, and those who have created a completely custom project type that needs to interoperate with other projects. It may also be of interest if you'd like to see the implementation details of references. Understanding the details should not be necessary to _use_ `ProjectReferences` in your project.
+This document describes that process, including what is required of a project to be referenceable through a `ProjectReference`. It is intended for MSBuild SDK maintainers, and those who have created a completely custom project type that needs to interoperate with other projects. It may also be of interest if you'd like to see the implementation details of references. Understanding the details should not be necessary to _use_ `ProjectReferences` in your project.
 
 ## Targets related to consuming a reference
 
@@ -51,7 +51,7 @@ These targets should exist in a project to be compatible with the common targets
 
 These targets are all defined in `Microsoft.Common.targets` and are defined in Microsoft SDKs. You should only have to implement them yourself if you require custom behavior or are authoring a project that doesn't import the common targets.
 
-If implementing a project with an “outer” (determine what properties to pass to the real build) and “inner” (fully specified) build, only `GetTargetFrameworkProperties` is required in the “outer” build. The other targets listed can be “inner” build only.
+If implementing a project with an “outer” (determine what properties to pass to the real build) and “inner” (fully specified) build, only `GetTargetFrameworkProperties` and `GetTargetFrameworks` are required in the “outer” build. The other targets listed can be “inner” build only.
 
 * `GetTargetFrameworks` tells referencing projects what options are available to the build.
   * It returns an item with the following metadata:
@@ -106,7 +106,7 @@ As of MSBuild 16.10, it is possible to gather additional properties from referen
 
 These properties will then be gathered via the `GetTargetFrameworks` call.  They will be available to the referencing project via the `AdditionalPropertiesFromProject` metadata on the `_MSBuildProjectReferenceExistent` item.  The `AdditionalPropertiesFromProject` value will be an XML string which contains the values of the properties for each `TargetFramework` in the referenced project.  For example:
 
-> :warning: This format is being changed. Soon, the schema will replace <net5.0> with <TargetFramework Name="net5.0">. You can opt into that behavior early by setting the _UseAttributeForTargetFrameworkInfoPropertyNames property to true. This property will have no effect after the transition is complete.
+> :warning: This format is being changed. Soon, the schema will replace `<net5.0>` with `<TargetFramework Name="net5.0">`. You can opt into that behavior early by setting the `_UseAttributeForTargetFrameworkInfoPropertyNames` property to true. This property will have no effect after the transition is complete.
 
 ```xml
 <AdditionalProjectProperties>

From 25176909e5da28fff6b03ec82be653847831a28a Mon Sep 17 00:00:00 2001
From: Ben Villalobos <4691428+BenVillalobos@users.noreply.github.com>
Date: Tue, 5 Oct 2021 10:38:12 -0700
Subject: [PATCH 2/8] Remove unused target

---
 src/Tasks/Microsoft.Common.CurrentVersion.targets | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/src/Tasks/Microsoft.Common.CurrentVersion.targets b/src/Tasks/Microsoft.Common.CurrentVersion.targets
index 8854b0cd156..4387674d46f 100644
--- a/src/Tasks/Microsoft.Common.CurrentVersion.targets
+++ b/src/Tasks/Microsoft.Common.CurrentVersion.targets
@@ -1937,20 +1937,6 @@ Copyright (C) Microsoft Corporation. All rights reserved.
 
   </Target>
 
-  <!--
-    ============================================================
-                                    GetTargetFrameworkProperties
-
-    Overrridden by cross-targeting projects to return the set of
-    properties (in the form "key1=value1;...keyN=valueN") needed
-    to build it with the best target for the referring project's
-    target framework.
-
-    The referring project's $(TargetFrameworkMoniker) is passed
-    in as $(ReferringTargetFramework)
-  -->
-  <Target Name="GetTargetFrameworkProperties" />
-
   <!--
     ============================================================
                                         PrepareProjectReferences

From 4d8af081747d5226e0501878a52f4c563c29a00f Mon Sep 17 00:00:00 2001
From: Ben Villalobos <4691428+BenVillalobos@users.noreply.github.com>
Date: Tue, 5 Oct 2021 10:39:24 -0700
Subject: [PATCH 3/8] Remove PlatformTarget= usage from docs

---
 documentation/ProjectReference-Protocol.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/documentation/ProjectReference-Protocol.md b/documentation/ProjectReference-Protocol.md
index 29800a2e498..e6ed5d55e7f 100644
--- a/documentation/ProjectReference-Protocol.md
+++ b/documentation/ProjectReference-Protocol.md
@@ -176,7 +176,7 @@ Project B: Unmanaged, has `$(Platforms)` constructed from its `Platform` metadat
 Because `AnyCPU` does not map to anything architecture-specific, a custom mapping must be defined. Project A can either:
 1. Define `PlatformLookupTable` in its project or a Directory.Build.props as `AnyCPU=x64` or `AnyCPU=Win32`.
 2. Define `PlatformLookupTable` as metadata on the `ProjectReference` item, which would take priority over a lookup table defined elsewhere.
-     *  When only one mapping is valid, you could also directly define `SetPlatform` metadata as `Platform=foo` (for unmanaged) or `PlatformTarget=bar` (for managed). This would skip over most negotiation logic.
+     *  When only one mapping is valid, you could also directly define `SetPlatform` metadata as `Platform=foo`. This would skip over most negotiation logic.
 
 Example of project A defining a lookup table directly on the `ProjectReference`:
 ```xml

From c84edec11437870c327eebcbdc96fd0135bd209f Mon Sep 17 00:00:00 2001
From: Ben Villalobos <4691428+BenVillalobos@users.noreply.github.com>
Date: Tue, 5 Oct 2021 10:41:17 -0700
Subject: [PATCH 4/8] Remove gettargetframeworkproperties from doc

---
 documentation/ProjectReference-Protocol.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/documentation/ProjectReference-Protocol.md b/documentation/ProjectReference-Protocol.md
index e6ed5d55e7f..9dc2f404fa2 100644
--- a/documentation/ProjectReference-Protocol.md
+++ b/documentation/ProjectReference-Protocol.md
@@ -51,7 +51,7 @@ These targets should exist in a project to be compatible with the common targets
 
 These targets are all defined in `Microsoft.Common.targets` and are defined in Microsoft SDKs. You should only have to implement them yourself if you require custom behavior or are authoring a project that doesn't import the common targets.
 
-If implementing a project with an “outer” (determine what properties to pass to the real build) and “inner” (fully specified) build, only `GetTargetFrameworkProperties` and `GetTargetFrameworks` are required in the “outer” build. The other targets listed can be “inner” build only.
+If implementing a project with an “outer” (determine what properties to pass to the real build) and “inner” (fully specified) build, only `GetTargetFrameworks` is required in the “outer” build. The other targets listed can be “inner” build only.
 
 * `GetTargetFrameworks` tells referencing projects what options are available to the build.
   * It returns an item with the following metadata:

From a5ceebf212e35c83d24a1a682eb910cced7c0c2c Mon Sep 17 00:00:00 2001
From: Ben Villalobos <4691428+BenVillalobos@users.noreply.github.com>
Date: Tue, 5 Oct 2021 13:28:21 -0700
Subject: [PATCH 5/8] Only report MSB4181 when not on warnandcontinue. Always
 set hasloggederrors when logging an error

---
 Sleep.1.0.0/.signature.p7s                       | Bin 0 -> 9468 bytes
 Sleep.1.0.0/sleep.png                            | Bin 0 -> 1629 bytes
 .../Components/RequestBuilder/TaskBuilder.cs     |  12 ++----------
 .../Components/RequestBuilder/TaskHost.cs        |   2 +-
 4 files changed, 3 insertions(+), 11 deletions(-)
 create mode 100644 Sleep.1.0.0/.signature.p7s
 create mode 100644 Sleep.1.0.0/sleep.png

diff --git a/Sleep.1.0.0/.signature.p7s b/Sleep.1.0.0/.signature.p7s
new file mode 100644
index 0000000000000000000000000000000000000000..6621c412ef8197f875c90faffb8255fa1684f82e
GIT binary patch
literal 9468
zcmds-cT`i^x5r5_^p5lrn)Gm!&_xlYGa!N#u|TK>0)ZrefQkta6bnTV?0^-dBQl7B
zN>wR>Ac`nbm7*XZ&BD8(4LH7;dF!q9oAn<5gmX{Hx#yhy*?WIa0Giwkf>oJrMxW+{
zFhkR5a)SU(t{)14p}7H01gk6E4B`ocLm}&!0FOT!Led=J_70vTGM?zKhh}5LpwL<<
z9Ss15Mxg;T3X9S}fsfS8aOC}Zrbc*kUBXTqqA}Xv%FWM)V&UbmQ^z_Kv&+L8&~^y)
zv$eLs7;8JLlXh%kV@soPZ3Sqs5`YH1oCbqJpimx;_wV|JUYh)f>v#PoHU{W2=Cw3q
z$U1Odo@jP}6#=t{Blw~AcIfp0Kcm3L&tZi3#&7c^QIO{T?r0G}i1C=6pC=?F1myu%
zxr0?GccLF!4v=9~!T7~idKi+)K_pKPq`48&mPn)^w;2MGLY#mW8jS`P|D1(5v0!PX
z_@9~v4I;C80l<xc(IDJR01d(hr9mJ}Ib7=-`?ej~QxG#z;yN41cP+PbL(<W#$;K~8
zT}3BXj+rAJnd_4=IQ`-R#i-cW&dY2ZkwRf{@!7HOL+$pUqm-w(Q!!e>^>5$k-^>{G
z`6@kbi|e|KY?taOc=bcrqlIs`%dC^I<~`Gfbgfz6h}L|a%@4&YPhelYR2TlC$X#?J
z@<pY;(F2_38NnaBE*uZ`YyX^K$*D5WPU^Vi`s`G^X3YWn7-V`=zaLwj`9X9lmLM}2
z^9<vhahIRQ`%zcu;#oaX`oP@PS8YP*pxo#~GPwp26Z({5Xk9@^pQBLKVR5ef+lkj5
zitZPte9XErR#JV_AX)I)eEOMT78l`=?NAtm33A09zyVy~rb=)_AoFlIc#!A-H>1qY
zD6a!x;0GYVc)$f0gbNrCXXH&LKBb}6b{xNPL;B)`UmZ6f#i*8r3j=~je%)a0IRv)>
z!j!>vudC<dSTBhj=8;q}P4aq)>DwCQmK!rX-uCLPL(%2>MghfA&J@=@e7`yMJ>L{*
zZtRqzi5I<TI^<9jdW-L+H#Mp&f>l3h;P(xx$ord$6pu~2%**toL`e;Lj>f#}dgyIl
zw0F3WRe5;B#{=bu3{)mcHfZ{}9E033-+dXf?)<Gy@(=6pL_GRTq}R-O>>sA8B@J&a
znL8o*#_e;tjduy*{={b~Uk)v1!3gbxnY;bMKJ%HlMz$-!%PkvI$}7E<_w#lgdC!6M
zs^I4o@Sli~a_5npJ1HjbM~J&tf`b&aT-lJ$6L>MFr*gLP?2Zx@W|l_OfKB0zIW}YU
zjInWmwhjPjaN#A>g|iBaxwMse1=(#YCMMgktUkHy|54LLW6=NxgT^q77c^ar&Z;6n
z`ri@+4XA^M1tf^#3PJ453^B-67}-Yj@I>0-z5Vh2-VB`logL3SFA|U7;?468VSdhz
zHWz7n_SN`of!=GQFA@DWFTTdK2Mz^aT!$)&n@imG;M(l5R&9~?J6U5F=+|D#XLE%r
z>Kz_z_I_UTRY{@tdAMv+UxMtJ_z>D`dX%ge-%hLzr9g2*nH>JnBinHS@7QC;t!n94
z29K&DlXJx_VpzI3zVukR3YTY973Kz@y|96svjZ}2HAV~`e43cvzY#6oG&IiE=U8S@
zJV+T(`9XMkn4+u289&-rY<)Fz$cOD#)I_$ob#c}7iDsm{>@bVRX4^9l$GQ0?t50!1
z%=n<XW14*LesqCbB-xP0+^2Kghm3s2?LpSRbon|q^Selw{ORUJJElPzz?FIo@G)$0
z!FE{y1ZcZZ2-BbJn}K}^c{m5ahG0=afUbcV1_MHu|A#=}et<6^w_FAwbm9FJN`Ria
zI?<gRu*M?*+g4gZ>Do)J+;I$tL0;2PeR;c=Hrm{X;VrfqqTERofJ$XdND3YX902sv
zbkelw8p}hG2pa!%9KxUbV*h#H?TjfP;bwpdpo3szcys!90uvMtC<BVi?+AqmiZdog
z2AASz->AOL#*VR`OFdz$nCV<q8m`QC<=Rjn?1Ag7+LUJym(%}<nIHfbn7Mg`(^;-)
zLZ*3ro$5h(rQn+qd&SlCM5{&tT>D{m*YA~^xU2Gxe;-xuLfUKRcTY7Z266AW?SJi*
zbI(3Gsx{BmPX>Xf+`0HIbJ|^uLMOBY5V6x2Osx0J#}|k(b0w_*dN;3l$D@;SxpFnN
z3d-q9ilTafico!2!j%W|yge5LOw+^m@6S7bV-IrAtH1(|xU!ow)qMLeH*I;k#poLU
z*nkIzJj+&0p;oLM^oKg>uz0(<@tF_1BnIaw9=?UcUfVi~@=vr4a=w{@AREu3-8Zp_
z3Kvn&Z?8GcgZRcwojcw!OEaH#^G~;%<4Y2bMkOW;?I##v3cb?b3tv6wv11fu=5~;o
z+DpuY6x|BnWai))ty=NJjg36@Gc*4USb?z*1_UNrYbEx<XaJgMO$=63S5wy+pq^Mu
zP=0_HJa}*xel|xi1O$mkA^M|v7AzLd%+GA=>ETE8_dqiaA=r$KU)nmz)RTfDlDv_&
zo&iKMo<byrBHcofR(N+3kxcZWEVqUIgSLOm)=v?4>FL$V>yk@Gy^Mws*99KJLkR~7
zzOq~sbD<in@!r(HigS-Gk%Qi0LX)SXe01OS6IeqM!d^xk3e->T5v|s`*WZ0Urs;DB
zC$3cCHPk?~(0ZSFPFK&@<Z!I_r51}DUf)~VoPX1y&?7&po_Zobapti{w@Y4Wc1y#<
z^Mdw4xtnH69<cS<n6C3S_~2KezHKAR_UHYY=OpjW>t-D!^OK%pc<o`fgx;X6t7p}0
z3pP@N`IB~Nx6twf+&^+i)+QX5p9VfX&O0do#-YVQtyB7lNukRyb4*tLX8GK*?&13A
z6XT=rWjEPrMgq+{LM-jNVJfZk%HV;VJK9*A6}DD^EA?QF&HmD;|A>=jY^sua8r7Th
z+_>nebz~!oltRf`4l^HF2*3Zt;gCfZ1_1tmW*RmPO;`Veg+K9(ydQ^A!z~8jYVKq;
zZ#0Ng!k@eSgwX${o8=+_jexCy)?z*b5)cLflmLZQ02EuA%`f@Kf<SmvIUsNdK#f}N
z1G<v~K=40wC)Ufo04ROy@_YZ4J7EM5D=vi*IV=SZOP+;;@lFfg5DGB;#D@i>zJ&EY
z32UmBmYCJ+3=wtjv544{ZQ?3P4t*lYo=-FRgqW{6Dx^Fz`bEqk?p1D2mQJjGIW`=9
zRvxZ%yPF@OE&DYwTiPhY%Qhm>VKgbOY5j2lgb!WR;!b<+)Zn=XXHuRiT^MS!L5kyF
zC%un&s?bw**Q-ple(X&qCcj&Nne!AOS0FFKg-6&aRToe$j=!=EmAk)9xa`B5-Id(G
z_ZbwLLeHQ{ymN&O9j}jx-u(9D&hJrdfpHs%rO5(4*L-WLdE|Gq*z2B=>-4yZ6wIFd
zRC24lK*e)PgLR|%CDF_BH9ncKjqf;|Zl*Y>((-TBsgJfZ9WH1sM|@N+&8__Q3{4Z1
z255p3U><u!;FpJXZEl-JgOmZnD`_Pw49&h$V1@wVD+ShdXzm3TY5+QDU5pN(<qSw7
z*q72#RvswNLf-tx0|XS1MsR@P2pmGw-U)?3plDuzn}gFAPr?%jM6##9FEJR59+xv(
zRybM)j=Z{o&c&?i9ir{pRhBb<!RHOF&d9r!(sMC%s<^aR&KJ;JGbRKrFDSe0!f^q3
zlz))-B2LwVvFdn#56=TAA2QJ&pth}^t2&xm4NxBe)Cz!l4*=hICEJH1os}Rja;`Mf
zTuW=ZdieE|+bu7R`n>D!#Qqkwv)S~}BtVS?g0E8kjnEm&^H(ksOo-t$2vg%c1!4oh
z_G~t8!Oo&HM0`w70@LXOaZW}7cl>3=pI7p4M#lDR!ESV@fA&zxG31eEcF&<rB5m`L
zv)o5R8#hAi6ym?X(sbSXHOA1Q7b#LaeQ}av!a09x`a@byRZ>N<Y{Q5yoD`xQ5yv<7
zaPVW}SXjqouAj=oq=>!dBiws!-|-sW5Ni-rf;~;m-lyfle$Zy4jxtxNL=%eYG-{Mr
zo!5ESq&H!zrqt+U*iq$6&XXC#+rpTkQyr&M@<Z+@suW*(nU-4VVcYk>fFS%K!Sbl8
z>@n%|>gvcu<5_(Dosv?L&(`g*1f_B5cUMic4n0Oj%!is*^c*rLiqQKIa_KaFDS*Z=
zwtBz0&@_HF@F8O5v@sMPzG;BQH^ESRIM<v3&^KUL^@hMfBas1@RuZfb1rIWUeIY2E
zF@`V;!2mP^Lo4S8+ykWPt@r5u+QL}@k4*_~q<xBTVUM@udx*?;>RTKm%C|c}1>`?E
zA27>2_~YHQT<yccYAgDst;WLK7%<KNR+$%=yGufm>TdoC)5N}bMTrRlnn&wkWIC4l
zuvUdzx7>&I(%O>`(x~*S)JTAOWNjcK0f;R|7y@>oemH-Mr#o^RkrY5A;V7VVg_c@~
zXF*X&aHV=DDTqu#T6<DLh$LT7-dRrURvIwatf__EiTCrgqu~4k82KCn&z>Zt8qyS-
zM&mXEqq$O0#7bX^=(dPfc}N%WOT)Oz`uIevC4M3{K)t_q(n7%c#YuC45*wZ|wkkuy
zLamzaas_hjboIa}9}KA2`O%8*N=Pu-)j=+|TpR+Nq6v|N#063I6G`|mP~i$9dm@Sc
zgitidGDcq*zxr|)q`f~r7!>(%ge4IS%zyA+c=v_1LmDoQK?z+@#&p0fTS%@K(rc_X
zm}#R|qQn2=@Ar49dM!?SUjanpwOjl8^Yc#5CuC(`xSmRAk57#Z8*mcQIZ-{;?-6}F
zBk=3T`=uQXR!q0+)X3Lr@qx)wy84Ai%ojdz%~@GgNp~sX;xIBeT6K(+|A~Vxg2)|4
z%>mmThkC_(>q9>|z2tgSWPJL8>ics-D&b7pVTRZP-PoQ99?^&x)$*w?(_4G`D`MP^
zxiDFFRln&AGQt?@Ne24kpRxy;urxnuQAvK*^tE)0ug3{Tt8e_gp-~_A?VQrM%mSI<
zkw*wwbbOWw^YTq~^}70Mv;{@`JT}TYWb}BwPY=u=`#k%yJyY!zi%(LiOXoZ6=P1|N
z-3q+7zX^_)IrtorKe?EyQwsrV-k(LbwJIC}@LP_BARKTU;F7*KZ8xwBa0KiHS@C{2
zZ%=h^ycZx1NCC`<bt+sis3=?@uK&3xJ^SP--^!*VKCNcLs^KCiV9DcwIV*waO+<m`
z6i)^%g(2vw2a^|`F-H0$cD)lahc`35Y{%F3XowjT%-o#9Eb@~jF2qh2Gc)<H0afqt
zzN8TMdjH>M4{>pJ)|G0O25fhqlba(%o-n<*(S73FY%nG{p&;a5_PO$cKD2ZVHQB+L
ztKd;<!S!9)pWUDS{+q<Ec#F|^K}9L&T%NB~8yyZp{pU@UU)+0(dZi!B=?;Eh?`1M6
z#{0~QeC3eI?52-5UEEqf8_c|;II0L=q~5Ml>_|$v7BHMGQO?Xe*7o{M|B10SOn&5m
zV^3T~d3sjGCW+!!ouT&pJ$l=H4B{Sm5!|x|UtJZPq&6oDa2;w$)_Bzt7eJ<lUpRa}
z75CCOCN;i#m=4rvR*cP8+8>Ry99@@)2ps=C^>oQd0}c0CXJC6us<QJ*Fdfwa0ft<H
zSNOVuR`dNkjHVwSYq=nDk?*G^`~LthFs(%|Duavg0<SQ@@=|IWvX;&NTZf_uY*?hM
z)L%N3e;^Q+?Yg+je*47}q`jiljA8qUd`OOdU^XUhoiF{I0uv_1$Lg_%fTdE|qgP6$
zs1f&~?j{3vhxi#cvBSgHnS0(w?DN~A)0%*IZ+I|N{OQviv5J?;2@_+V2s1fnbWsrx
z6@EXUTOD=n<Hx+1Q;jEU{f1gni}j==buU&<p1V02)m_zmFt7%pbladWl5<04ILl_M
zS+)|a`TUcOtzQbyTXr8kEBfWZAXeZ-SM8D437>x4OLQcKOg7xC|H1m@w03#&QJ?D0
z$pXvP#t*CsF2zxynz0A6MKE!)4>5*fySOqp7u3(bP#G<G+1DoF$E%k(b^7szUb9bS
zc3iDmDf9!?>DtQzVF(zHkk=-m5X3J9LoaaUK{3n)*bD4l8@>Dn$Sz|ERKtXpLun5b
zIn>YHZGkE70c1dar4bYm5&dN|QUDG>EjI&`L>T}HCJ0M<B&a#^Ev0|tg`qtdLofJU
zP$yIe)Bu$<r8EV){F+9K-ygK>i`7V=#JG?V|3dI&Sx{6Yc;O&HuYzZF`PvJ{&$AM+
zs9^vN9X0GKH0HU#cyh%$n)L3*J;Mp544sH0L2hegfq%-*prL%VHVu(I7~>yY-L2cN
zA9zS?A9~ZdeImV%dUX0m-1>nnN;w02Co^6-ctiyP%5NlbLS9G*M5@Tiy@BI1Dfd{v
zizk>q?)h*M_O(7<{RJDr8awjh_UXFc;@!=3w@6pLdEtL7Rlhhj<wGW-g!OTpougsA
zdbgyNb>|bF1B9*qV|S`&aC>aOSWo$$oVExnpJ@0FYkxHbyUE>aH_a9_QWJ#A@`{F^
zEtx?$cGqxCeaPz=IWH`gRobh^RM8ll^c3>?QmsxIx9zRw-XxeSn^oEm1wR4IH-nTd
z-4$n#z3$T$xZlzDymCmzxhUDm9)76y+=wjfWVCY2d~PO=3bNA@WG8l+ovr<Ock7uA
z(jl^N8AGc-vGZSJ3#jLz!4M6D{X@^w1+W<K!IGZ$uZPi~QUCQlXC7B<)=XywK76N8
zpegJ9lls&d%K8xR_zU0PT0<{c22b^CZT4)FL;3-;ZzLiI4HPV})_#JqeJ8h^l*uZ;
z!7kRqd`G`nR8Db547_-*V?DZoMQ?U)n{1G207sF_?l=|+l9{8YRUTA-%k<<MjLvC;
ze%0#}yJVjpdCPA#%kwykO_*Shb8P+$Rjn$%HBOLixy;EeLdc1B!+*Dqei&<uzP0=1
zV~TCBBSEIZK7#lJam&D>L?+mCWJGKHOfq$Y>i4qLEQrqYJozo>S?$J@9HX7~4Mm$7
zO2_oSz5RScVfYq1r_fWjXoOV~%($viA2w3LG_EI8eVf=nnjd?98Gp6lN>u`D6`}tW
z=(aF`Dh1aCw0~0PC}cHIvO|R|P;MdZT3{$cqSMq;qmaRB+ycS`86(Iu?qpYA+pw&~
zIjAXyEF%;|sxN>Cl$XmOQYDvY34ZCqP0ml>3T#|y3tdqi!L~R&?(Zh(N&3e))mnwq
zYy6PR%hB~>{(_P&8_`*zT1?>*YjFBY-mbb$UcMPIxs*5ddxlHKS+k~_3YO9hg{9X?
z0q*L(_DbQpCf4r{dm@~U-*b#j<Yaltb5~1ljxGG_#^-Jo7D?*|Xs=^;`pql;@NNn;
z7{Ic=%Gkv-DQWS;v;J+XZ?COV5R-+<wfd*$KV(`>-xk`HXjoZdKp&WT5MiWmvPa0d
z_v*$r{DwB^L)}Nb54&ZTSfkdT>%SnIx0`n;Fw=0BGgK!towMn7LC5BEVYM>Vl!h)-
zR(jR91o=X-GlS7AAKna6T>V|IIfQZ6898+h%$f;Y>aTdqbU&mZRe&TQ*Hz1_D#c__
zX(>?SEnY8)xDxNOXXo+Fz1ZC}?&=ac)H?9;pqH^hFq#H+27d_R0ICfD^*Hcv;S$Od
z{xir|i)CrltAOZ=qylP(oNHv0zfli=mi}@LBg>mA@{#VfrE!9kyGFUu?Hc8`r^kr9
z*nM}NkGTbCuNobyjs^ot0Q><999cza3}!k6)wSll*7x?HK+Tca9vVdd$CTaZ-V&)U
z+zdn9S^n7!gdvfITOoXsb5Cjd7TVl7lvR-lJ6n0GO@bZy$n9vvcoThh&eoKCjsl5q
zlM{lM6vX%4s5=`KBmBMdP9ayvO#Op)_6KzhdlER0k3?!ClEtF!^DkzmJBNgHX!N8W
zdr?BPQ6yP4G|Y~dywHv%|31{p?9`gz9UYrc?!Z4%Cp0W`o#%W$eUC<`Y%x}0qn7pU
z!*&(>`;8O(*FExY+}xxj{B0-dOKo^=Wwdtm=JF98@hiE`p7(Ry@1KYEWEAF{4jr_W
zts=ccTlzsx?b5!wm*rOT6i@Dp0WlHM<-<WAauejr49;aW+p(k+e|(#o8;9N7E(=c_
RcIHu4M9B%*;SV!${ucvlL23X1

literal 0
HcmV?d00001

diff --git a/Sleep.1.0.0/sleep.png b/Sleep.1.0.0/sleep.png
new file mode 100644
index 0000000000000000000000000000000000000000..30670ebb5e7f0d25b79b129a77426d613b858193
GIT binary patch
literal 1629
zcmbuAc`(}v6vv}+D<VpkS}F~-bg5(1QIQCOZBkd+9<-{&S^ce}De7EDXzQxh5k#w7
zrKrYHBx^<uRS|Km#?^|VG|{>xu1%+Z?(WRabmsHkcfK?4uQ%@?IpeXi(yG!R5J=Y1
z!PZr%mBJ{H6c^^}h7VPQA`<S3wE@*K)fR+Clz_&eL7@69nH?W7p_jVua4sAK`Z4Wm
zh-A$!`+-1WD~`5kccM3M@>05z-w)l(Pq-akxHopv2cxvKpzIWza-1ZlA{TMM04hov
z`?xQqtIUPVc$qY^T2JI0zpcfCyJbq_$}>>_EI*yD(`F>VBt}1t^xjn9JSP;J6T0|x
z<9G7u+XspDf=GYncByg9q;;ftdh1lY1VTZl;`^~%eV?j$4F5Jo1{PaY%Vgp&Gz8?F
znT6pfbUHmsuohTW6+A_|X=-XZ=f(#}X(k6!{*{Im9)iv%v906<keqr2Dq(50+&n55
zq$w&L_<tCnP1e@c_2Ww{W`_`SjFZI!vlzKQCN6JqJgDsE?Zn$$c0f}MUA`|eBI2H)
z2Mz{<n}SAk%ZM)=EJn{5#_al1_tJJy_Q-LcjU_##`#6i0Eh{G{=+r`*bB>E9YNN?1
znyh^cIR(z@fe@x88wzQhiMBgtZbw+84=VH9b^HW`WP<|WB&>qt%{`SDu~=-4M>Ss)
zH?6V`Gw@#f>DZ^#z9p|EjD2cz0q^`gM%=&8^zg(2#<l~`er>2x6nwF=9r|(L5jM0Z
zS1YbE>54`8l<8>5rlSV@#U4NnvtIw&5>?>b0}mdG<mMd-?mNnaXaoLLMTv7t1j#%z
z<Fj<s>TSisY7a`MO39+A6WOK8o^rF3KL7!|!3)ZZgsCe{8(@vv`o8|6l5KX{y%$rO
z(v}#Xv`Wcos<X!3El7?sY;i~|I=Go#nuLUzu);QmR5YM(W*9BLHS9lm$U_Mw6}dXT
zWF>bdibOSe1JeQSJ4N0vggjoMJ47%2OvRS2XoBaNMYoK(n7eyBBV6YsrCb$b>h7NQ
zpy6c5>?-nVMH7R|*hHFm?s+b2?K_6I6kGI#^rESjOB>+O)`*@E-(BnB9D3a4-)I_#
z`jph>Z8?%_ZD4;w{1=gyk2Dv3Cd!)i{N;965ViNgF31SRnGig-C0kip?Tx&Q*rW`F
z8O5vxYCxm6q^QLNe)izBVD|$n4~gis3a}XRF9;RT)aSk8j({=i@vE;EW<qHD2M5K9
zJS^)fUlkgMWIeW{^mti?GkNRa(4ZNHxeFN`_h2iEvlIQY(JbepC?L)k9lJ&@KEFeC
z(9ShW=7MhpUibv4kKxESGj*6p#!(n@@90*xYx*h=7K_q{XBxlOcjgQ$`;gJ5iA{<G
zVu2nor#EPpyaB$|sQ#-ZuySHF)=rJU@}_7q{j&JkP+-nr#Vq+R*y5$Qes1CjvSO{<
zoNBPRSrVxB)^B4p40)Qfmp@ZBPQnc~k83GKr8lWnb+hLsj945wmm4>lYtH3yb=x8q
zTrbdw^=E3t$C!oGvo)~+wN)MW*XEL;1ZIP3f5U>ovRx<XgAMs}tdbvD(GeCvd}RbI
zwyF~POX4Uvv2K;CRL1X|{1eOsR4=6h5EoT#gLbPx0vJx}1Qeyd3Pl!4?5czV;=<WF
zO5EWjiOD>xuI_z8BwNQ%;Y&|{8E$=A;y<2M&{z0b!U!^ji@I7SZ_b~^QrqJ<%Uilq
z1hLk6?g_iwo8uWuda)VUEyrjXnEqNI^i-Sz3r)$k{|8Zi<#7=a!$FNj8X4oMvWK6w
zx4(NreC}+0t)m37uPO#-0bGQp?zf{ck(}WWD{m3p#D0$#ZV}<pw?(5#G(vPTZtQF=
zUw>G+Qx{FyJ85+Hms9_F59>#y0?BC2JE-r4!M&7^_Al@(u9_FV1dyX0-nQ1}a?;-!
CAo2kK

literal 0
HcmV?d00001

diff --git a/src/Build/BackEnd/Components/RequestBuilder/TaskBuilder.cs b/src/Build/BackEnd/Components/RequestBuilder/TaskBuilder.cs
index 85e71b2c2ba..a4261264274 100644
--- a/src/Build/BackEnd/Components/RequestBuilder/TaskBuilder.cs
+++ b/src/Build/BackEnd/Components/RequestBuilder/TaskBuilder.cs
@@ -940,16 +940,8 @@ private async Task<WorkUnitResult> ExecuteInstantiatedTask(ITaskExecutionHost ta
                 IBuildEngine be = host.TaskInstance.BuildEngine;
                 if (taskReturned && !taskResult && !taskLoggingContext.HasLoggedErrors && (be is TaskHost th ? th.BuildRequestsSucceeded : false) && (be is IBuildEngine7 be7 ? !be7.AllowFailureWithoutError : true))
                 {
-                    if (_continueOnError == ContinueOnError.WarnAndContinue)
-                    {
-                        taskLoggingContext.LogWarning(null,
-                            new BuildEventFileInfo(_targetChildInstance.Location),
-                            "TaskReturnedFalseButDidNotLogError",
-                            _taskNode.Name);
-
-                        taskLoggingContext.LogComment(MessageImportance.Normal, "ErrorConvertedIntoWarning");
-                    }
-                    else
+                    // HasLoggedErrors will not be true if WarnAndContinue is set, avoid logging an error
+                    if (_continueOnError != ContinueOnError.WarnAndContinue)
                     {
                         taskLoggingContext.LogError(new BuildEventFileInfo(_targetChildInstance.Location),
                             "TaskReturnedFalseButDidNotLogError",
diff --git a/src/Build/BackEnd/Components/RequestBuilder/TaskHost.cs b/src/Build/BackEnd/Components/RequestBuilder/TaskHost.cs
index 154ace42f96..90abe787f10 100644
--- a/src/Build/BackEnd/Components/RequestBuilder/TaskHost.cs
+++ b/src/Build/BackEnd/Components/RequestBuilder/TaskHost.cs
@@ -455,8 +455,8 @@ public void LogErrorEvent(Microsoft.Build.Framework.BuildErrorEventArgs e)
                 {
                     e.BuildEventContext = _taskLoggingContext.BuildEventContext;
                     _taskLoggingContext.LoggingService.LogBuildEvent(e);
-                    _taskLoggingContext.HasLoggedErrors = true;
                 }
+                    _taskLoggingContext.HasLoggedErrors = true;
             }
         }
 

From 4f7381e572b5deab38cf269ac38211043feb3f8d Mon Sep 17 00:00:00 2001
From: Ben Villalobos <4691428+BenVillalobos@users.noreply.github.com>
Date: Wed, 6 Oct 2021 09:53:35 -0700
Subject: [PATCH 6/8] Revert "Remove unused target"

This reverts commit 25176909e5da28fff6b03ec82be653847831a28a.
---
 src/Tasks/Microsoft.Common.CurrentVersion.targets | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/src/Tasks/Microsoft.Common.CurrentVersion.targets b/src/Tasks/Microsoft.Common.CurrentVersion.targets
index 4387674d46f..8854b0cd156 100644
--- a/src/Tasks/Microsoft.Common.CurrentVersion.targets
+++ b/src/Tasks/Microsoft.Common.CurrentVersion.targets
@@ -1937,6 +1937,20 @@ Copyright (C) Microsoft Corporation. All rights reserved.
 
   </Target>
 
+  <!--
+    ============================================================
+                                    GetTargetFrameworkProperties
+
+    Overrridden by cross-targeting projects to return the set of
+    properties (in the form "key1=value1;...keyN=valueN") needed
+    to build it with the best target for the referring project's
+    target framework.
+
+    The referring project's $(TargetFrameworkMoniker) is passed
+    in as $(ReferringTargetFramework)
+  -->
+  <Target Name="GetTargetFrameworkProperties" />
+
   <!--
     ============================================================
                                         PrepareProjectReferences

From 8dac1baf8435f000845c17ec25a57121308b7ba8 Mon Sep 17 00:00:00 2001
From: Ben Villalobos <4691428+BenVillalobos@users.noreply.github.com>
Date: Wed, 6 Oct 2021 10:36:35 -0700
Subject: [PATCH 7/8] Mark gettargetframeworkproperties obsolete

---
 src/Tasks/Microsoft.Common.CurrentVersion.targets | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/Tasks/Microsoft.Common.CurrentVersion.targets b/src/Tasks/Microsoft.Common.CurrentVersion.targets
index 8854b0cd156..783465d0ddf 100644
--- a/src/Tasks/Microsoft.Common.CurrentVersion.targets
+++ b/src/Tasks/Microsoft.Common.CurrentVersion.targets
@@ -1941,13 +1941,7 @@ Copyright (C) Microsoft Corporation. All rights reserved.
     ============================================================
                                     GetTargetFrameworkProperties
 
-    Overrridden by cross-targeting projects to return the set of
-    properties (in the form "key1=value1;...keyN=valueN") needed
-    to build it with the best target for the referring project's
-    target framework.
-
-    The referring project's $(TargetFrameworkMoniker) is passed
-    in as $(ReferringTargetFramework)
+    OBSOLETE: present only for theoretical backward compatibility.
   -->
   <Target Name="GetTargetFrameworkProperties" />
 

From 2717192fa20e8a139fac92b766fcbe0447e11eb5 Mon Sep 17 00:00:00 2001
From: Ben Villalobos <4691428+BenVillalobos@users.noreply.github.com>
Date: Mon, 11 Oct 2021 09:29:58 -0700
Subject: [PATCH 8/8] Revert "Only report MSB4181 when not on warnandcontinue.
 Always set hasloggederrors when logging an error"

This reverts commit a5ceebf212e35c83d24a1a682eb910cced7c0c2c.
---
 Sleep.1.0.0/.signature.p7s                       | Bin 9468 -> 0 bytes
 Sleep.1.0.0/sleep.png                            | Bin 1629 -> 0 bytes
 .../Components/RequestBuilder/TaskBuilder.cs     |  12 ++++++++++--
 .../Components/RequestBuilder/TaskHost.cs        |   2 +-
 4 files changed, 11 insertions(+), 3 deletions(-)
 delete mode 100644 Sleep.1.0.0/.signature.p7s
 delete mode 100644 Sleep.1.0.0/sleep.png

diff --git a/Sleep.1.0.0/.signature.p7s b/Sleep.1.0.0/.signature.p7s
deleted file mode 100644
index 6621c412ef8197f875c90faffb8255fa1684f82e..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 9468
zcmds-cT`i^x5r5_^p5lrn)Gm!&_xlYGa!N#u|TK>0)ZrefQkta6bnTV?0^-dBQl7B
zN>wR>Ac`nbm7*XZ&BD8(4LH7;dF!q9oAn<5gmX{Hx#yhy*?WIa0Giwkf>oJrMxW+{
zFhkR5a)SU(t{)14p}7H01gk6E4B`ocLm}&!0FOT!Led=J_70vTGM?zKhh}5LpwL<<
z9Ss15Mxg;T3X9S}fsfS8aOC}Zrbc*kUBXTqqA}Xv%FWM)V&UbmQ^z_Kv&+L8&~^y)
zv$eLs7;8JLlXh%kV@soPZ3Sqs5`YH1oCbqJpimx;_wV|JUYh)f>v#PoHU{W2=Cw3q
z$U1Odo@jP}6#=t{Blw~AcIfp0Kcm3L&tZi3#&7c^QIO{T?r0G}i1C=6pC=?F1myu%
zxr0?GccLF!4v=9~!T7~idKi+)K_pKPq`48&mPn)^w;2MGLY#mW8jS`P|D1(5v0!PX
z_@9~v4I;C80l<xc(IDJR01d(hr9mJ}Ib7=-`?ej~QxG#z;yN41cP+PbL(<W#$;K~8
zT}3BXj+rAJnd_4=IQ`-R#i-cW&dY2ZkwRf{@!7HOL+$pUqm-w(Q!!e>^>5$k-^>{G
z`6@kbi|e|KY?taOc=bcrqlIs`%dC^I<~`Gfbgfz6h}L|a%@4&YPhelYR2TlC$X#?J
z@<pY;(F2_38NnaBE*uZ`YyX^K$*D5WPU^Vi`s`G^X3YWn7-V`=zaLwj`9X9lmLM}2
z^9<vhahIRQ`%zcu;#oaX`oP@PS8YP*pxo#~GPwp26Z({5Xk9@^pQBLKVR5ef+lkj5
zitZPte9XErR#JV_AX)I)eEOMT78l`=?NAtm33A09zyVy~rb=)_AoFlIc#!A-H>1qY
zD6a!x;0GYVc)$f0gbNrCXXH&LKBb}6b{xNPL;B)`UmZ6f#i*8r3j=~je%)a0IRv)>
z!j!>vudC<dSTBhj=8;q}P4aq)>DwCQmK!rX-uCLPL(%2>MghfA&J@=@e7`yMJ>L{*
zZtRqzi5I<TI^<9jdW-L+H#Mp&f>l3h;P(xx$ord$6pu~2%**toL`e;Lj>f#}dgyIl
zw0F3WRe5;B#{=bu3{)mcHfZ{}9E033-+dXf?)<Gy@(=6pL_GRTq}R-O>>sA8B@J&a
znL8o*#_e;tjduy*{={b~Uk)v1!3gbxnY;bMKJ%HlMz$-!%PkvI$}7E<_w#lgdC!6M
zs^I4o@Sli~a_5npJ1HjbM~J&tf`b&aT-lJ$6L>MFr*gLP?2Zx@W|l_OfKB0zIW}YU
zjInWmwhjPjaN#A>g|iBaxwMse1=(#YCMMgktUkHy|54LLW6=NxgT^q77c^ar&Z;6n
z`ri@+4XA^M1tf^#3PJ453^B-67}-Yj@I>0-z5Vh2-VB`logL3SFA|U7;?468VSdhz
zHWz7n_SN`of!=GQFA@DWFTTdK2Mz^aT!$)&n@imG;M(l5R&9~?J6U5F=+|D#XLE%r
z>Kz_z_I_UTRY{@tdAMv+UxMtJ_z>D`dX%ge-%hLzr9g2*nH>JnBinHS@7QC;t!n94
z29K&DlXJx_VpzI3zVukR3YTY973Kz@y|96svjZ}2HAV~`e43cvzY#6oG&IiE=U8S@
zJV+T(`9XMkn4+u289&-rY<)Fz$cOD#)I_$ob#c}7iDsm{>@bVRX4^9l$GQ0?t50!1
z%=n<XW14*LesqCbB-xP0+^2Kghm3s2?LpSRbon|q^Selw{ORUJJElPzz?FIo@G)$0
z!FE{y1ZcZZ2-BbJn}K}^c{m5ahG0=afUbcV1_MHu|A#=}et<6^w_FAwbm9FJN`Ria
zI?<gRu*M?*+g4gZ>Do)J+;I$tL0;2PeR;c=Hrm{X;VrfqqTERofJ$XdND3YX902sv
zbkelw8p}hG2pa!%9KxUbV*h#H?TjfP;bwpdpo3szcys!90uvMtC<BVi?+AqmiZdog
z2AASz->AOL#*VR`OFdz$nCV<q8m`QC<=Rjn?1Ag7+LUJym(%}<nIHfbn7Mg`(^;-)
zLZ*3ro$5h(rQn+qd&SlCM5{&tT>D{m*YA~^xU2Gxe;-xuLfUKRcTY7Z266AW?SJi*
zbI(3Gsx{BmPX>Xf+`0HIbJ|^uLMOBY5V6x2Osx0J#}|k(b0w_*dN;3l$D@;SxpFnN
z3d-q9ilTafico!2!j%W|yge5LOw+^m@6S7bV-IrAtH1(|xU!ow)qMLeH*I;k#poLU
z*nkIzJj+&0p;oLM^oKg>uz0(<@tF_1BnIaw9=?UcUfVi~@=vr4a=w{@AREu3-8Zp_
z3Kvn&Z?8GcgZRcwojcw!OEaH#^G~;%<4Y2bMkOW;?I##v3cb?b3tv6wv11fu=5~;o
z+DpuY6x|BnWai))ty=NJjg36@Gc*4USb?z*1_UNrYbEx<XaJgMO$=63S5wy+pq^Mu
zP=0_HJa}*xel|xi1O$mkA^M|v7AzLd%+GA=>ETE8_dqiaA=r$KU)nmz)RTfDlDv_&
zo&iKMo<byrBHcofR(N+3kxcZWEVqUIgSLOm)=v?4>FL$V>yk@Gy^Mws*99KJLkR~7
zzOq~sbD<in@!r(HigS-Gk%Qi0LX)SXe01OS6IeqM!d^xk3e->T5v|s`*WZ0Urs;DB
zC$3cCHPk?~(0ZSFPFK&@<Z!I_r51}DUf)~VoPX1y&?7&po_Zobapti{w@Y4Wc1y#<
z^Mdw4xtnH69<cS<n6C3S_~2KezHKAR_UHYY=OpjW>t-D!^OK%pc<o`fgx;X6t7p}0
z3pP@N`IB~Nx6twf+&^+i)+QX5p9VfX&O0do#-YVQtyB7lNukRyb4*tLX8GK*?&13A
z6XT=rWjEPrMgq+{LM-jNVJfZk%HV;VJK9*A6}DD^EA?QF&HmD;|A>=jY^sua8r7Th
z+_>nebz~!oltRf`4l^HF2*3Zt;gCfZ1_1tmW*RmPO;`Veg+K9(ydQ^A!z~8jYVKq;
zZ#0Ng!k@eSgwX${o8=+_jexCy)?z*b5)cLflmLZQ02EuA%`f@Kf<SmvIUsNdK#f}N
z1G<v~K=40wC)Ufo04ROy@_YZ4J7EM5D=vi*IV=SZOP+;;@lFfg5DGB;#D@i>zJ&EY
z32UmBmYCJ+3=wtjv544{ZQ?3P4t*lYo=-FRgqW{6Dx^Fz`bEqk?p1D2mQJjGIW`=9
zRvxZ%yPF@OE&DYwTiPhY%Qhm>VKgbOY5j2lgb!WR;!b<+)Zn=XXHuRiT^MS!L5kyF
zC%un&s?bw**Q-ple(X&qCcj&Nne!AOS0FFKg-6&aRToe$j=!=EmAk)9xa`B5-Id(G
z_ZbwLLeHQ{ymN&O9j}jx-u(9D&hJrdfpHs%rO5(4*L-WLdE|Gq*z2B=>-4yZ6wIFd
zRC24lK*e)PgLR|%CDF_BH9ncKjqf;|Zl*Y>((-TBsgJfZ9WH1sM|@N+&8__Q3{4Z1
z255p3U><u!;FpJXZEl-JgOmZnD`_Pw49&h$V1@wVD+ShdXzm3TY5+QDU5pN(<qSw7
z*q72#RvswNLf-tx0|XS1MsR@P2pmGw-U)?3plDuzn}gFAPr?%jM6##9FEJR59+xv(
zRybM)j=Z{o&c&?i9ir{pRhBb<!RHOF&d9r!(sMC%s<^aR&KJ;JGbRKrFDSe0!f^q3
zlz))-B2LwVvFdn#56=TAA2QJ&pth}^t2&xm4NxBe)Cz!l4*=hICEJH1os}Rja;`Mf
zTuW=ZdieE|+bu7R`n>D!#Qqkwv)S~}BtVS?g0E8kjnEm&^H(ksOo-t$2vg%c1!4oh
z_G~t8!Oo&HM0`w70@LXOaZW}7cl>3=pI7p4M#lDR!ESV@fA&zxG31eEcF&<rB5m`L
zv)o5R8#hAi6ym?X(sbSXHOA1Q7b#LaeQ}av!a09x`a@byRZ>N<Y{Q5yoD`xQ5yv<7
zaPVW}SXjqouAj=oq=>!dBiws!-|-sW5Ni-rf;~;m-lyfle$Zy4jxtxNL=%eYG-{Mr
zo!5ESq&H!zrqt+U*iq$6&XXC#+rpTkQyr&M@<Z+@suW*(nU-4VVcYk>fFS%K!Sbl8
z>@n%|>gvcu<5_(Dosv?L&(`g*1f_B5cUMic4n0Oj%!is*^c*rLiqQKIa_KaFDS*Z=
zwtBz0&@_HF@F8O5v@sMPzG;BQH^ESRIM<v3&^KUL^@hMfBas1@RuZfb1rIWUeIY2E
zF@`V;!2mP^Lo4S8+ykWPt@r5u+QL}@k4*_~q<xBTVUM@udx*?;>RTKm%C|c}1>`?E
zA27>2_~YHQT<yccYAgDst;WLK7%<KNR+$%=yGufm>TdoC)5N}bMTrRlnn&wkWIC4l
zuvUdzx7>&I(%O>`(x~*S)JTAOWNjcK0f;R|7y@>oemH-Mr#o^RkrY5A;V7VVg_c@~
zXF*X&aHV=DDTqu#T6<DLh$LT7-dRrURvIwatf__EiTCrgqu~4k82KCn&z>Zt8qyS-
zM&mXEqq$O0#7bX^=(dPfc}N%WOT)Oz`uIevC4M3{K)t_q(n7%c#YuC45*wZ|wkkuy
zLamzaas_hjboIa}9}KA2`O%8*N=Pu-)j=+|TpR+Nq6v|N#063I6G`|mP~i$9dm@Sc
zgitidGDcq*zxr|)q`f~r7!>(%ge4IS%zyA+c=v_1LmDoQK?z+@#&p0fTS%@K(rc_X
zm}#R|qQn2=@Ar49dM!?SUjanpwOjl8^Yc#5CuC(`xSmRAk57#Z8*mcQIZ-{;?-6}F
zBk=3T`=uQXR!q0+)X3Lr@qx)wy84Ai%ojdz%~@GgNp~sX;xIBeT6K(+|A~Vxg2)|4
z%>mmThkC_(>q9>|z2tgSWPJL8>ics-D&b7pVTRZP-PoQ99?^&x)$*w?(_4G`D`MP^
zxiDFFRln&AGQt?@Ne24kpRxy;urxnuQAvK*^tE)0ug3{Tt8e_gp-~_A?VQrM%mSI<
zkw*wwbbOWw^YTq~^}70Mv;{@`JT}TYWb}BwPY=u=`#k%yJyY!zi%(LiOXoZ6=P1|N
z-3q+7zX^_)IrtorKe?EyQwsrV-k(LbwJIC}@LP_BARKTU;F7*KZ8xwBa0KiHS@C{2
zZ%=h^ycZx1NCC`<bt+sis3=?@uK&3xJ^SP--^!*VKCNcLs^KCiV9DcwIV*waO+<m`
z6i)^%g(2vw2a^|`F-H0$cD)lahc`35Y{%F3XowjT%-o#9Eb@~jF2qh2Gc)<H0afqt
zzN8TMdjH>M4{>pJ)|G0O25fhqlba(%o-n<*(S73FY%nG{p&;a5_PO$cKD2ZVHQB+L
ztKd;<!S!9)pWUDS{+q<Ec#F|^K}9L&T%NB~8yyZp{pU@UU)+0(dZi!B=?;Eh?`1M6
z#{0~QeC3eI?52-5UEEqf8_c|;II0L=q~5Ml>_|$v7BHMGQO?Xe*7o{M|B10SOn&5m
zV^3T~d3sjGCW+!!ouT&pJ$l=H4B{Sm5!|x|UtJZPq&6oDa2;w$)_Bzt7eJ<lUpRa}
z75CCOCN;i#m=4rvR*cP8+8>Ry99@@)2ps=C^>oQd0}c0CXJC6us<QJ*Fdfwa0ft<H
zSNOVuR`dNkjHVwSYq=nDk?*G^`~LthFs(%|Duavg0<SQ@@=|IWvX;&NTZf_uY*?hM
z)L%N3e;^Q+?Yg+je*47}q`jiljA8qUd`OOdU^XUhoiF{I0uv_1$Lg_%fTdE|qgP6$
zs1f&~?j{3vhxi#cvBSgHnS0(w?DN~A)0%*IZ+I|N{OQviv5J?;2@_+V2s1fnbWsrx
z6@EXUTOD=n<Hx+1Q;jEU{f1gni}j==buU&<p1V02)m_zmFt7%pbladWl5<04ILl_M
zS+)|a`TUcOtzQbyTXr8kEBfWZAXeZ-SM8D437>x4OLQcKOg7xC|H1m@w03#&QJ?D0
z$pXvP#t*CsF2zxynz0A6MKE!)4>5*fySOqp7u3(bP#G<G+1DoF$E%k(b^7szUb9bS
zc3iDmDf9!?>DtQzVF(zHkk=-m5X3J9LoaaUK{3n)*bD4l8@>Dn$Sz|ERKtXpLun5b
zIn>YHZGkE70c1dar4bYm5&dN|QUDG>EjI&`L>T}HCJ0M<B&a#^Ev0|tg`qtdLofJU
zP$yIe)Bu$<r8EV){F+9K-ygK>i`7V=#JG?V|3dI&Sx{6Yc;O&HuYzZF`PvJ{&$AM+
zs9^vN9X0GKH0HU#cyh%$n)L3*J;Mp544sH0L2hegfq%-*prL%VHVu(I7~>yY-L2cN
zA9zS?A9~ZdeImV%dUX0m-1>nnN;w02Co^6-ctiyP%5NlbLS9G*M5@Tiy@BI1Dfd{v
zizk>q?)h*M_O(7<{RJDr8awjh_UXFc;@!=3w@6pLdEtL7Rlhhj<wGW-g!OTpougsA
zdbgyNb>|bF1B9*qV|S`&aC>aOSWo$$oVExnpJ@0FYkxHbyUE>aH_a9_QWJ#A@`{F^
zEtx?$cGqxCeaPz=IWH`gRobh^RM8ll^c3>?QmsxIx9zRw-XxeSn^oEm1wR4IH-nTd
z-4$n#z3$T$xZlzDymCmzxhUDm9)76y+=wjfWVCY2d~PO=3bNA@WG8l+ovr<Ock7uA
z(jl^N8AGc-vGZSJ3#jLz!4M6D{X@^w1+W<K!IGZ$uZPi~QUCQlXC7B<)=XywK76N8
zpegJ9lls&d%K8xR_zU0PT0<{c22b^CZT4)FL;3-;ZzLiI4HPV})_#JqeJ8h^l*uZ;
z!7kRqd`G`nR8Db547_-*V?DZoMQ?U)n{1G207sF_?l=|+l9{8YRUTA-%k<<MjLvC;
ze%0#}yJVjpdCPA#%kwykO_*Shb8P+$Rjn$%HBOLixy;EeLdc1B!+*Dqei&<uzP0=1
zV~TCBBSEIZK7#lJam&D>L?+mCWJGKHOfq$Y>i4qLEQrqYJozo>S?$J@9HX7~4Mm$7
zO2_oSz5RScVfYq1r_fWjXoOV~%($viA2w3LG_EI8eVf=nnjd?98Gp6lN>u`D6`}tW
z=(aF`Dh1aCw0~0PC}cHIvO|R|P;MdZT3{$cqSMq;qmaRB+ycS`86(Iu?qpYA+pw&~
zIjAXyEF%;|sxN>Cl$XmOQYDvY34ZCqP0ml>3T#|y3tdqi!L~R&?(Zh(N&3e))mnwq
zYy6PR%hB~>{(_P&8_`*zT1?>*YjFBY-mbb$UcMPIxs*5ddxlHKS+k~_3YO9hg{9X?
z0q*L(_DbQpCf4r{dm@~U-*b#j<Yaltb5~1ljxGG_#^-Jo7D?*|Xs=^;`pql;@NNn;
z7{Ic=%Gkv-DQWS;v;J+XZ?COV5R-+<wfd*$KV(`>-xk`HXjoZdKp&WT5MiWmvPa0d
z_v*$r{DwB^L)}Nb54&ZTSfkdT>%SnIx0`n;Fw=0BGgK!towMn7LC5BEVYM>Vl!h)-
zR(jR91o=X-GlS7AAKna6T>V|IIfQZ6898+h%$f;Y>aTdqbU&mZRe&TQ*Hz1_D#c__
zX(>?SEnY8)xDxNOXXo+Fz1ZC}?&=ac)H?9;pqH^hFq#H+27d_R0ICfD^*Hcv;S$Od
z{xir|i)CrltAOZ=qylP(oNHv0zfli=mi}@LBg>mA@{#VfrE!9kyGFUu?Hc8`r^kr9
z*nM}NkGTbCuNobyjs^ot0Q><999cza3}!k6)wSll*7x?HK+Tca9vVdd$CTaZ-V&)U
z+zdn9S^n7!gdvfITOoXsb5Cjd7TVl7lvR-lJ6n0GO@bZy$n9vvcoThh&eoKCjsl5q
zlM{lM6vX%4s5=`KBmBMdP9ayvO#Op)_6KzhdlER0k3?!ClEtF!^DkzmJBNgHX!N8W
zdr?BPQ6yP4G|Y~dywHv%|31{p?9`gz9UYrc?!Z4%Cp0W`o#%W$eUC<`Y%x}0qn7pU
z!*&(>`;8O(*FExY+}xxj{B0-dOKo^=Wwdtm=JF98@hiE`p7(Ry@1KYEWEAF{4jr_W
zts=ccTlzsx?b5!wm*rOT6i@Dp0WlHM<-<WAauejr49;aW+p(k+e|(#o8;9N7E(=c_
RcIHu4M9B%*;SV!${ucvlL23X1

diff --git a/Sleep.1.0.0/sleep.png b/Sleep.1.0.0/sleep.png
deleted file mode 100644
index 30670ebb5e7f0d25b79b129a77426d613b858193..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 1629
zcmbuAc`(}v6vv}+D<VpkS}F~-bg5(1QIQCOZBkd+9<-{&S^ce}De7EDXzQxh5k#w7
zrKrYHBx^<uRS|Km#?^|VG|{>xu1%+Z?(WRabmsHkcfK?4uQ%@?IpeXi(yG!R5J=Y1
z!PZr%mBJ{H6c^^}h7VPQA`<S3wE@*K)fR+Clz_&eL7@69nH?W7p_jVua4sAK`Z4Wm
zh-A$!`+-1WD~`5kccM3M@>05z-w)l(Pq-akxHopv2cxvKpzIWza-1ZlA{TMM04hov
z`?xQqtIUPVc$qY^T2JI0zpcfCyJbq_$}>>_EI*yD(`F>VBt}1t^xjn9JSP;J6T0|x
z<9G7u+XspDf=GYncByg9q;;ftdh1lY1VTZl;`^~%eV?j$4F5Jo1{PaY%Vgp&Gz8?F
znT6pfbUHmsuohTW6+A_|X=-XZ=f(#}X(k6!{*{Im9)iv%v906<keqr2Dq(50+&n55
zq$w&L_<tCnP1e@c_2Ww{W`_`SjFZI!vlzKQCN6JqJgDsE?Zn$$c0f}MUA`|eBI2H)
z2Mz{<n}SAk%ZM)=EJn{5#_al1_tJJy_Q-LcjU_##`#6i0Eh{G{=+r`*bB>E9YNN?1
znyh^cIR(z@fe@x88wzQhiMBgtZbw+84=VH9b^HW`WP<|WB&>qt%{`SDu~=-4M>Ss)
zH?6V`Gw@#f>DZ^#z9p|EjD2cz0q^`gM%=&8^zg(2#<l~`er>2x6nwF=9r|(L5jM0Z
zS1YbE>54`8l<8>5rlSV@#U4NnvtIw&5>?>b0}mdG<mMd-?mNnaXaoLLMTv7t1j#%z
z<Fj<s>TSisY7a`MO39+A6WOK8o^rF3KL7!|!3)ZZgsCe{8(@vv`o8|6l5KX{y%$rO
z(v}#Xv`Wcos<X!3El7?sY;i~|I=Go#nuLUzu);QmR5YM(W*9BLHS9lm$U_Mw6}dXT
zWF>bdibOSe1JeQSJ4N0vggjoMJ47%2OvRS2XoBaNMYoK(n7eyBBV6YsrCb$b>h7NQ
zpy6c5>?-nVMH7R|*hHFm?s+b2?K_6I6kGI#^rESjOB>+O)`*@E-(BnB9D3a4-)I_#
z`jph>Z8?%_ZD4;w{1=gyk2Dv3Cd!)i{N;965ViNgF31SRnGig-C0kip?Tx&Q*rW`F
z8O5vxYCxm6q^QLNe)izBVD|$n4~gis3a}XRF9;RT)aSk8j({=i@vE;EW<qHD2M5K9
zJS^)fUlkgMWIeW{^mti?GkNRa(4ZNHxeFN`_h2iEvlIQY(JbepC?L)k9lJ&@KEFeC
z(9ShW=7MhpUibv4kKxESGj*6p#!(n@@90*xYx*h=7K_q{XBxlOcjgQ$`;gJ5iA{<G
zVu2nor#EPpyaB$|sQ#-ZuySHF)=rJU@}_7q{j&JkP+-nr#Vq+R*y5$Qes1CjvSO{<
zoNBPRSrVxB)^B4p40)Qfmp@ZBPQnc~k83GKr8lWnb+hLsj945wmm4>lYtH3yb=x8q
zTrbdw^=E3t$C!oGvo)~+wN)MW*XEL;1ZIP3f5U>ovRx<XgAMs}tdbvD(GeCvd}RbI
zwyF~POX4Uvv2K;CRL1X|{1eOsR4=6h5EoT#gLbPx0vJx}1Qeyd3Pl!4?5czV;=<WF
zO5EWjiOD>xuI_z8BwNQ%;Y&|{8E$=A;y<2M&{z0b!U!^ji@I7SZ_b~^QrqJ<%Uilq
z1hLk6?g_iwo8uWuda)VUEyrjXnEqNI^i-Sz3r)$k{|8Zi<#7=a!$FNj8X4oMvWK6w
zx4(NreC}+0t)m37uPO#-0bGQp?zf{ck(}WWD{m3p#D0$#ZV}<pw?(5#G(vPTZtQF=
zUw>G+Qx{FyJ85+Hms9_F59>#y0?BC2JE-r4!M&7^_Al@(u9_FV1dyX0-nQ1}a?;-!
CAo2kK

diff --git a/src/Build/BackEnd/Components/RequestBuilder/TaskBuilder.cs b/src/Build/BackEnd/Components/RequestBuilder/TaskBuilder.cs
index a4261264274..85e71b2c2ba 100644
--- a/src/Build/BackEnd/Components/RequestBuilder/TaskBuilder.cs
+++ b/src/Build/BackEnd/Components/RequestBuilder/TaskBuilder.cs
@@ -940,8 +940,16 @@ private async Task<WorkUnitResult> ExecuteInstantiatedTask(ITaskExecutionHost ta
                 IBuildEngine be = host.TaskInstance.BuildEngine;
                 if (taskReturned && !taskResult && !taskLoggingContext.HasLoggedErrors && (be is TaskHost th ? th.BuildRequestsSucceeded : false) && (be is IBuildEngine7 be7 ? !be7.AllowFailureWithoutError : true))
                 {
-                    // HasLoggedErrors will not be true if WarnAndContinue is set, avoid logging an error
-                    if (_continueOnError != ContinueOnError.WarnAndContinue)
+                    if (_continueOnError == ContinueOnError.WarnAndContinue)
+                    {
+                        taskLoggingContext.LogWarning(null,
+                            new BuildEventFileInfo(_targetChildInstance.Location),
+                            "TaskReturnedFalseButDidNotLogError",
+                            _taskNode.Name);
+
+                        taskLoggingContext.LogComment(MessageImportance.Normal, "ErrorConvertedIntoWarning");
+                    }
+                    else
                     {
                         taskLoggingContext.LogError(new BuildEventFileInfo(_targetChildInstance.Location),
                             "TaskReturnedFalseButDidNotLogError",
diff --git a/src/Build/BackEnd/Components/RequestBuilder/TaskHost.cs b/src/Build/BackEnd/Components/RequestBuilder/TaskHost.cs
index 90abe787f10..154ace42f96 100644
--- a/src/Build/BackEnd/Components/RequestBuilder/TaskHost.cs
+++ b/src/Build/BackEnd/Components/RequestBuilder/TaskHost.cs
@@ -455,8 +455,8 @@ public void LogErrorEvent(Microsoft.Build.Framework.BuildErrorEventArgs e)
                 {
                     e.BuildEventContext = _taskLoggingContext.BuildEventContext;
                     _taskLoggingContext.LoggingService.LogBuildEvent(e);
-                }
                     _taskLoggingContext.HasLoggedErrors = true;
+                }
             }
         }