<?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>234239</bug_id>
          
          <creation_ts>2021-12-13 05:15:24 -0800</creation_ts>
          <short_desc>[GTK] build fails with undefined symbol in ANGLE if USE_ANGLE_WEBGL is enabled.</short_desc>
          <delta_ts>2021-12-21 05:31:17 -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>ANGLE</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=234293</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Arcady Goldmints-Orlov">crzwdjk</reporter>
          <assigned_to name="Arcady Goldmints-Orlov">crzwdjk</assigned_to>
          <cc>annulen</cc>
    
    <cc>aperez</cc>
    
    <cc>clord</cc>
    
    <cc>dino</cc>
    
    <cc>don.olmstead</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>fujii</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>kbr</cc>
    
    <cc>kkinnunen</cc>
    
    <cc>kondapallykalyan</cc>
    
    <cc>ryuan.choi</cc>
    
    <cc>sergio</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1823485</commentid>
    <comment_count>0</comment_count>
    <who name="Arcady Goldmints-Orlov">crzwdjk</who>
    <bug_when>2021-12-13 05:15:24 -0800</bug_when>
    <thetext>After the recent update of ANGLE, enabling USE_ANGLE_WEBGL on Linux platforms results in the build failing with &quot;undefined reference to &apos;angle::DrmFourCCFormatToGLInternalFormat(int, bool*)&apos;&quot; because the Source/ThirdParty/ANGLE/src/common/linux/ directory doesn&apos;t get built.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1823489</commentid>
    <comment_count>1</comment_count>
      <attachid>446992</attachid>
    <who name="Arcady Goldmints-Orlov">crzwdjk</who>
    <bug_when>2021-12-13 05:20:34 -0800</bug_when>
    <thetext>Created attachment 446992
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1823490</commentid>
    <comment_count>2</comment_count>
    <who name="EWS Watchlist">ews-watchlist</who>
    <bug_when>2021-12-13 05:21:20 -0800</bug_when>
    <thetext>Note that there are important steps to take when updating ANGLE. See https://trac.webkit.org/wiki/UpdatingANGLE</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1823508</commentid>
    <comment_count>3</comment_count>
      <attachid>446992</attachid>
    <who name="Chris Lord">clord</who>
    <bug_when>2021-12-13 06:26:25 -0800</bug_when>
    <thetext>Comment on attachment 446992
Patch

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

LGTM, with one noted change.

&gt; Source/ThirdParty/ANGLE/GL.cmake:237
&gt; +if (is_android OR is_linux OR is_chromeos)
&gt; +    list(APPEND _gl_backend_sources
&gt; +        &quot;src/common/linux/dma_buf_utils.cpp&quot;
&gt; +        &quot;src/common/linux/dma_buf_utils.h&quot;
&gt; +    )
&gt; +endif()

Why not add these to the existing block at line 127?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1823709</commentid>
    <comment_count>4</comment_count>
      <attachid>447047</attachid>
    <who name="Arcady Goldmints-Orlov">crzwdjk</who>
    <bug_when>2021-12-13 13:12:00 -0800</bug_when>
    <thetext>Created attachment 447047
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1823710</commentid>
    <comment_count>5</comment_count>
      <attachid>447048</attachid>
    <who name="Arcady Goldmints-Orlov">crzwdjk</who>
    <bug_when>2021-12-13 13:14:58 -0800</bug_when>
    <thetext>Created attachment 447048
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1823856</commentid>
    <comment_count>6</comment_count>
      <attachid>447048</attachid>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2021-12-13 17:45:30 -0800</bug_when>
    <thetext>Comment on attachment 447048
Patch

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

&gt; Source/ThirdParty/ANGLE/GL.cmake:156
&gt; +        &quot;src/common/linux/dma_buf_utils.h&quot;

I think you shouldn&apos;t edit GL.cmake because it is genereated by gni-to-cmake.py.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1823957</commentid>
    <comment_count>7</comment_count>
    <who name="Chris Lord">clord</who>
    <bug_when>2021-12-14 03:49:54 -0800</bug_when>
    <thetext>(In reply to Fujii Hironori from comment #6)
&gt; Comment on attachment 447048 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=447048&amp;action=review
&gt; 
&gt; &gt; Source/ThirdParty/ANGLE/GL.cmake:156
&gt; &gt; +        &quot;src/common/linux/dma_buf_utils.h&quot;
&gt; 
&gt; I think you shouldn&apos;t edit GL.cmake because it is genereated by
&gt; gni-to-cmake.py.

It&apos;s hand-edited after the generation, I think given the coarseness of this tool and that this file is already hand-edited (re-running the listed command at the top of the file generates a different file), it&apos;s reasonable to add this manually.

The current script isn&apos;t sophisticated enough to interpret the build file in src/common/linux (in fact, it currently just strips out that definition entirely, so using the script generates an empty file).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1823985</commentid>
    <comment_count>8</comment_count>
    <who name="Arcady Goldmints-Orlov">crzwdjk</who>
    <bug_when>2021-12-14 07:03:54 -0800</bug_when>
    <thetext>I guess what I had in the back of my mind when I added the stuff in a separate block is that the other blocks were auto-generated, and this one is manually added and thus easier to merge with the auto-generated stuff. Maybe I should put a comment in to that effect as well? I honestly am not that familiar with the process of importing ANGLE and converting the build scripts so I am not sure what the right approach is and I am glad we are having this discussion.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1823993</commentid>
    <comment_count>9</comment_count>
    <who name="Chris Lord">clord</who>
    <bug_when>2021-12-14 07:31:40 -0800</bug_when>
    <thetext>(In reply to Arcady Goldmints-Orlov from comment #8)
&gt; I guess what I had in the back of my mind when I added the stuff in a
&gt; separate block is that the other blocks were auto-generated, and this one is
&gt; manually added and thus easier to merge with the auto-generated stuff. Maybe
&gt; I should put a comment in to that effect as well? I honestly am not that
&gt; familiar with the process of importing ANGLE and converting the build
&gt; scripts so I am not sure what the right approach is and I am glad we are
&gt; having this discussion.

That sounds reasonable to me - I guess if we really don&apos;t want to touch the auto-generated files, it could be a separate file or included in PlatformGTK. These build files and the process of generating them doesn&apos;t really seem to be so involved that we should be worrying too much about this, imho.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1824063</commentid>
    <comment_count>10</comment_count>
      <attachid>447048</attachid>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2021-12-14 11:56:26 -0800</bug_when>
    <thetext>Comment on attachment 447048
Patch

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

&gt;&gt;&gt; Source/ThirdParty/ANGLE/GL.cmake:156
&gt;&gt;&gt; +        &quot;src/common/linux/dma_buf_utils.h&quot;
&gt;&gt; 
&gt;&gt; I think you shouldn&apos;t edit GL.cmake because it is genereated by gni-to-cmake.py.
&gt; 
&gt; It&apos;s hand-edited after the generation, I think given the coarseness of this tool and that this file is already hand-edited (re-running the listed command at the top of the file generates a different file), it&apos;s reasonable to add this manually.
&gt; 
&gt; The current script isn&apos;t sophisticated enough to interpret the build file in src/common/linux (in fact, it currently just strips out that definition entirely, so using the script generates an empty file).

I looked at those dma_buf_util files and in the GN its actually its own library which is probably why it isn&apos;t being picked up. I didn&apos;t write that gni-to-cmake.py script and I don&apos;t think it ran successfully on Windows if I&apos;m remembering correctly.

Ideally this should be generated in the GLESv2.cmake file NOT GL.cmake. You&apos;ll notice that the backend CMake files only append sources in `src/libANGLE/renderer/${name}`.

I&apos;m okay with this change happening in GLESv2.cmake with a big ol FIXME with an actual bug associated with it so next time there&apos;s a roll someone hopefully notices it. I&apos;m also okay with just adding it in the PlatformGTK.cmake in the root but I do think there should be a bug about it either way.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1824069</commentid>
    <comment_count>11</comment_count>
      <attachid>447048</attachid>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2021-12-14 12:07:56 -0800</bug_when>
    <thetext>Comment on attachment 447048
Patch

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

&gt;&gt;&gt;&gt; Source/ThirdParty/ANGLE/GL.cmake:156
&gt;&gt;&gt;&gt; +        &quot;src/common/linux/dma_buf_utils.h&quot;
&gt;&gt;&gt; 
&gt;&gt;&gt; I think you shouldn&apos;t edit GL.cmake because it is genereated by gni-to-cmake.py.
&gt;&gt; 
&gt;&gt; It&apos;s hand-edited after the generation, I think given the coarseness of this tool and that this file is already hand-edited (re-running the listed command at the top of the file generates a different file), it&apos;s reasonable to add this manually.
&gt;&gt; 
&gt;&gt; The current script isn&apos;t sophisticated enough to interpret the build file in src/common/linux (in fact, it currently just strips out that definition entirely, so using the script generates an empty file).
&gt; 
&gt; I looked at those dma_buf_util files and in the GN its actually its own library which is probably why it isn&apos;t being picked up. I didn&apos;t write that gni-to-cmake.py script and I don&apos;t think it ran successfully on Windows if I&apos;m remembering correctly.
&gt; 
&gt; Ideally this should be generated in the GLESv2.cmake file NOT GL.cmake. You&apos;ll notice that the backend CMake files only append sources in `src/libANGLE/renderer/${name}`.
&gt; 
&gt; I&apos;m okay with this change happening in GLESv2.cmake with a big ol FIXME with an actual bug associated with it so next time there&apos;s a roll someone hopefully notices it. I&apos;m also okay with just adding it in the PlatformGTK.cmake in the root but I do think there should be a bug about it either way.

It&apos;s not a library. It&apos;s a source set that is used by GL and Valkan backends.
I think we should add linux.cmake, and include it from PlatformGTK.cmake and PlatformWPE.cmake, or from CMakeLists.txt.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1824071</commentid>
    <comment_count>12</comment_count>
      <attachid>447048</attachid>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2021-12-14 12:18:57 -0800</bug_when>
    <thetext>Comment on attachment 447048
Patch

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

&gt;&gt;&gt;&gt;&gt; Source/ThirdParty/ANGLE/GL.cmake:156
&gt;&gt;&gt;&gt;&gt; +        &quot;src/common/linux/dma_buf_utils.h&quot;
&gt;&gt;&gt;&gt; 
&gt;&gt;&gt;&gt; I think you shouldn&apos;t edit GL.cmake because it is genereated by gni-to-cmake.py.
&gt;&gt;&gt; 
&gt;&gt;&gt; It&apos;s hand-edited after the generation, I think given the coarseness of this tool and that this file is already hand-edited (re-running the listed command at the top of the file generates a different file), it&apos;s reasonable to add this manually.
&gt;&gt;&gt; 
&gt;&gt;&gt; The current script isn&apos;t sophisticated enough to interpret the build file in src/common/linux (in fact, it currently just strips out that definition entirely, so using the script generates an empty file).
&gt;&gt; 
&gt;&gt; I looked at those dma_buf_util files and in the GN its actually its own library which is probably why it isn&apos;t being picked up. I didn&apos;t write that gni-to-cmake.py script and I don&apos;t think it ran successfully on Windows if I&apos;m remembering correctly.
&gt;&gt; 
&gt;&gt; Ideally this should be generated in the GLESv2.cmake file NOT GL.cmake. You&apos;ll notice that the backend CMake files only append sources in `src/libANGLE/renderer/${name}`.
&gt;&gt; 
&gt;&gt; I&apos;m okay with this change happening in GLESv2.cmake with a big ol FIXME with an actual bug associated with it so next time there&apos;s a roll someone hopefully notices it. I&apos;m also okay with just adding it in the PlatformGTK.cmake in the root but I do think there should be a bug about it either way.
&gt; 
&gt; It&apos;s not a library. It&apos;s a source set that is used by GL and Valkan backends.
&gt; I think we should add linux.cmake, and include it from PlatformGTK.cmake and PlatformWPE.cmake, or from CMakeLists.txt.

Ok with that route I guess we would just list the sources in a variable and then the GTK/WPE folks can just append those values to it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1825715</commentid>
    <comment_count>13</comment_count>
      <attachid>447628</attachid>
    <who name="Arcady Goldmints-Orlov">crzwdjk</who>
    <bug_when>2021-12-20 13:33:18 -0800</bug_when>
    <thetext>Created attachment 447628
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1825732</commentid>
    <comment_count>14</comment_count>
      <attachid>447628</attachid>
    <who name="Kenneth Russell">kbr</who>
    <bug_when>2021-12-20 14:04:38 -0800</bug_when>
    <thetext>Comment on attachment 447628
Patch

Looks fine! r+.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1825979</commentid>
    <comment_count>15</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2021-12-21 05:30:52 -0800</bug_when>
    <thetext>Committed r287312 (245464@main): &lt;https://commits.webkit.org/245464@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 447628.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1825980</commentid>
    <comment_count>16</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-12-21 05:31:17 -0800</bug_when>
    <thetext>&lt;rdar://problem/86765194&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>446992</attachid>
            <date>2021-12-13 05:20:34 -0800</date>
            <delta_ts>2021-12-13 13:11:57 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-234239-20211213082033.patch</filename>
            <type>text/plain</type>
            <size>1238</size>
            <attacher name="Arcady Goldmints-Orlov">crzwdjk</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjg2NjU0CmRpZmYgLS1naXQgYS9Tb3VyY2UvVGhpcmRQYXJ0
eS9BTkdMRS9DaGFuZ2VMb2cgYi9Tb3VyY2UvVGhpcmRQYXJ0eS9BTkdMRS9DaGFuZ2VMb2cKaW5k
ZXggMTcyN2IwZmNjYTI0YTA2NWM5YjM0ZjdkMzIyNTA4NzE0ZDc2MzFjZC4uYzYwMzQ1ZGU0ZjUy
NzdjYWNmNDE0ZWJhMjc5YWJkN2JiNmNiMTI1ZiAxMDA2NDQKLS0tIGEvU291cmNlL1RoaXJkUGFy
dHkvQU5HTEUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9UaGlyZFBhcnR5L0FOR0xFL0NoYW5nZUxv
ZwpAQCAtMSwzICsxLDEyIEBACisyMDIxLTEyLTEzICBBcmNhZHkgR29sZG1pbnRzLU9ybG92ICA8
YWdvbGRtaW50c0BpZ2FsaWEuY29tPgorCisgICAgICAgIFtHVEtdIGJ1aWxkIGZhaWxzIHdpdGgg
dW5kZWZpbmVkIHN5bWJvbCBpbiBBTkdMRSBpZiBVU0VfQU5HTEVfV0VCR0wgaXMgZW5hYmxlZC4K
KyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIzNDIzOQor
CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogR0wuY21h
a2U6IEFkZCBhIG1hbnVhbCB0cmFuc2xhdGlvbiBvZiBzcmMvY29tbW9uL2xpbnV4L0JVSUxELmdu
CisKIDIwMjEtMTItMDcgIE15bGVzIEMuIE1heGZpZWxkICA8bW1heGZpZWxkQGFwcGxlLmNvbT4K
IAogICAgICAgICBbTWFjQ2F0YWx5c3RdIEZpeCBidWlsZCBhZnRlciByMjg2NjAzCmRpZmYgLS1n
aXQgYS9Tb3VyY2UvVGhpcmRQYXJ0eS9BTkdMRS9HTC5jbWFrZSBiL1NvdXJjZS9UaGlyZFBhcnR5
L0FOR0xFL0dMLmNtYWtlCmluZGV4IDMwMzMyMzYwYjkyNWU0YjM4NDI5MDU1YjJkZTI3ZWM3NjRl
YmRmYjMuLmVmYTU3MDJkZmExZmZiYThhZTNmOWU0ZWNlY2MxN2VlZmNiYTA0MTYgMTAwNjQ0Ci0t
LSBhL1NvdXJjZS9UaGlyZFBhcnR5L0FOR0xFL0dMLmNtYWtlCisrKyBiL1NvdXJjZS9UaGlyZFBh
cnR5L0FOR0xFL0dMLmNtYWtlCkBAIC0yMjksMyArMjI5LDkgQEAgaWYoYW5nbGVfZW5hYmxlX2ds
X251bGwpCiAgICAgKQogZW5kaWYoKQogCitpZiAoaXNfYW5kcm9pZCBPUiBpc19saW51eCBPUiBp
c19jaHJvbWVvcykKKyAgICBsaXN0KEFQUEVORCBfZ2xfYmFja2VuZF9zb3VyY2VzCisgICAgICAg
ICJzcmMvY29tbW9uL2xpbnV4L2RtYV9idWZfdXRpbHMuY3BwIgorICAgICAgICAic3JjL2NvbW1v
bi9saW51eC9kbWFfYnVmX3V0aWxzLmgiCisgICAgKQorZW5kaWYoKQo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>447047</attachid>
            <date>2021-12-13 13:12:00 -0800</date>
            <delta_ts>2021-12-13 13:14:55 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-234239-20211213161159.patch</filename>
            <type>text/plain</type>
            <size>1238</size>
            <attacher name="Arcady Goldmints-Orlov">crzwdjk</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjg2NjU0CmRpZmYgLS1naXQgYS9Tb3VyY2UvVGhpcmRQYXJ0
eS9BTkdMRS9DaGFuZ2VMb2cgYi9Tb3VyY2UvVGhpcmRQYXJ0eS9BTkdMRS9DaGFuZ2VMb2cKaW5k
ZXggMTcyN2IwZmNjYTI0YTA2NWM5YjM0ZjdkMzIyNTA4NzE0ZDc2MzFjZC4uYzYwMzQ1ZGU0ZjUy
NzdjYWNmNDE0ZWJhMjc5YWJkN2JiNmNiMTI1ZiAxMDA2NDQKLS0tIGEvU291cmNlL1RoaXJkUGFy
dHkvQU5HTEUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9UaGlyZFBhcnR5L0FOR0xFL0NoYW5nZUxv
ZwpAQCAtMSwzICsxLDEyIEBACisyMDIxLTEyLTEzICBBcmNhZHkgR29sZG1pbnRzLU9ybG92ICA8
YWdvbGRtaW50c0BpZ2FsaWEuY29tPgorCisgICAgICAgIFtHVEtdIGJ1aWxkIGZhaWxzIHdpdGgg
dW5kZWZpbmVkIHN5bWJvbCBpbiBBTkdMRSBpZiBVU0VfQU5HTEVfV0VCR0wgaXMgZW5hYmxlZC4K
KyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIzNDIzOQor
CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogR0wuY21h
a2U6IEFkZCBhIG1hbnVhbCB0cmFuc2xhdGlvbiBvZiBzcmMvY29tbW9uL2xpbnV4L0JVSUxELmdu
CisKIDIwMjEtMTItMDcgIE15bGVzIEMuIE1heGZpZWxkICA8bW1heGZpZWxkQGFwcGxlLmNvbT4K
IAogICAgICAgICBbTWFjQ2F0YWx5c3RdIEZpeCBidWlsZCBhZnRlciByMjg2NjAzCmRpZmYgLS1n
aXQgYS9Tb3VyY2UvVGhpcmRQYXJ0eS9BTkdMRS9HTC5jbWFrZSBiL1NvdXJjZS9UaGlyZFBhcnR5
L0FOR0xFL0dMLmNtYWtlCmluZGV4IDMwMzMyMzYwYjkyNWU0YjM4NDI5MDU1YjJkZTI3ZWM3NjRl
YmRmYjMuLmVmYTU3MDJkZmExZmZiYThhZTNmOWU0ZWNlY2MxN2VlZmNiYTA0MTYgMTAwNjQ0Ci0t
LSBhL1NvdXJjZS9UaGlyZFBhcnR5L0FOR0xFL0dMLmNtYWtlCisrKyBiL1NvdXJjZS9UaGlyZFBh
cnR5L0FOR0xFL0dMLmNtYWtlCkBAIC0yMjksMyArMjI5LDkgQEAgaWYoYW5nbGVfZW5hYmxlX2ds
X251bGwpCiAgICAgKQogZW5kaWYoKQogCitpZiAoaXNfYW5kcm9pZCBPUiBpc19saW51eCBPUiBp
c19jaHJvbWVvcykKKyAgICBsaXN0KEFQUEVORCBfZ2xfYmFja2VuZF9zb3VyY2VzCisgICAgICAg
ICJzcmMvY29tbW9uL2xpbnV4L2RtYV9idWZfdXRpbHMuY3BwIgorICAgICAgICAic3JjL2NvbW1v
bi9saW51eC9kbWFfYnVmX3V0aWxzLmgiCisgICAgKQorZW5kaWYoKQo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>447048</attachid>
            <date>2021-12-13 13:14:58 -0800</date>
            <delta_ts>2021-12-20 13:33:14 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-234239-20211213161458.patch</filename>
            <type>text/plain</type>
            <size>1328</size>
            <attacher name="Arcady Goldmints-Orlov">crzwdjk</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjg2NjU0CmRpZmYgLS1naXQgYS9Tb3VyY2UvVGhpcmRQYXJ0
eS9BTkdMRS9DaGFuZ2VMb2cgYi9Tb3VyY2UvVGhpcmRQYXJ0eS9BTkdMRS9DaGFuZ2VMb2cKaW5k
ZXggMTcyN2IwZmNjYTI0YTA2NWM5YjM0ZjdkMzIyNTA4NzE0ZDc2MzFjZC4uYzYwMzQ1ZGU0ZjUy
NzdjYWNmNDE0ZWJhMjc5YWJkN2JiNmNiMTI1ZiAxMDA2NDQKLS0tIGEvU291cmNlL1RoaXJkUGFy
dHkvQU5HTEUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9UaGlyZFBhcnR5L0FOR0xFL0NoYW5nZUxv
ZwpAQCAtMSwzICsxLDEyIEBACisyMDIxLTEyLTEzICBBcmNhZHkgR29sZG1pbnRzLU9ybG92ICA8
YWdvbGRtaW50c0BpZ2FsaWEuY29tPgorCisgICAgICAgIFtHVEtdIGJ1aWxkIGZhaWxzIHdpdGgg
dW5kZWZpbmVkIHN5bWJvbCBpbiBBTkdMRSBpZiBVU0VfQU5HTEVfV0VCR0wgaXMgZW5hYmxlZC4K
KyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIzNDIzOQor
CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogR0wuY21h
a2U6IEFkZCBhIG1hbnVhbCB0cmFuc2xhdGlvbiBvZiBzcmMvY29tbW9uL2xpbnV4L0JVSUxELmdu
CisKIDIwMjEtMTItMDcgIE15bGVzIEMuIE1heGZpZWxkICA8bW1heGZpZWxkQGFwcGxlLmNvbT4K
IAogICAgICAgICBbTWFjQ2F0YWx5c3RdIEZpeCBidWlsZCBhZnRlciByMjg2NjAzCmRpZmYgLS1n
aXQgYS9Tb3VyY2UvVGhpcmRQYXJ0eS9BTkdMRS9HTC5jbWFrZSBiL1NvdXJjZS9UaGlyZFBhcnR5
L0FOR0xFL0dMLmNtYWtlCmluZGV4IDMwMzMyMzYwYjkyNWU0YjM4NDI5MDU1YjJkZTI3ZWM3NjRl
YmRmYjMuLjFlZTljYmY5ZDc0YzgzMjg3YzkwZjg3ZDQ1MzVhMjJmNDVhYmY2MDAgMTAwNjQ0Ci0t
LSBhL1NvdXJjZS9UaGlyZFBhcnR5L0FOR0xFL0dMLmNtYWtlCisrKyBiL1NvdXJjZS9UaGlyZFBh
cnR5L0FOR0xFL0dMLmNtYWtlCkBAIC0xNTIsNiArMTUyLDggQEAgaWYoaXNfYW5kcm9pZCBPUiBp
c19saW51eCBPUiBpc19jaHJvbWVvcykKICAgICAgICAgInNyYy9saWJBTkdMRS9yZW5kZXJlci9n
bC9lZ2wvZWdsX3V0aWxzLmNwcCIKICAgICAgICAgInNyYy9saWJBTkdMRS9yZW5kZXJlci9nbC9l
Z2wvZWdsX3V0aWxzLmgiCiAgICAgICAgICJzcmMvbGliQU5HTEUvcmVuZGVyZXIvZ2wvZWdsL2Z1
bmN0aW9uc2VnbF90eXBlZGVmcy5oIgorICAgICAgICAic3JjL2NvbW1vbi9saW51eC9kbWFfYnVm
X3V0aWxzLmNwcCIKKyAgICAgICAgInNyYy9jb21tb24vbGludXgvZG1hX2J1Zl91dGlscy5oIgog
ICAgICkKIGVuZGlmKCkKIAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>447628</attachid>
            <date>2021-12-20 13:33:18 -0800</date>
            <delta_ts>2021-12-21 05:30:55 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-234239-20211220163317.patch</filename>
            <type>text/plain</type>
            <size>2173</size>
            <attacher name="Arcady Goldmints-Orlov">crzwdjk</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjg3MjYzCmRpZmYgLS1naXQgYS9Tb3VyY2UvVGhpcmRQYXJ0
eS9BTkdMRS9DaGFuZ2VMb2cgYi9Tb3VyY2UvVGhpcmRQYXJ0eS9BTkdMRS9DaGFuZ2VMb2cKaW5k
ZXggNTRjYzMzNTg2MGVmMWVmZTRhN2FkN2I0OGQ3YzBlOTdkNmIxMTRmMi4uOGFlM2YwNThjYmYw
OTczNDkyNWQ5Mzg4ODhjMzhjNjY2ZmU5ZDI1MSAxMDA2NDQKLS0tIGEvU291cmNlL1RoaXJkUGFy
dHkvQU5HTEUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9UaGlyZFBhcnR5L0FOR0xFL0NoYW5nZUxv
ZwpAQCAtMSwzICsxLDE2IEBACisyMDIxLTEyLTEzICBBcmNhZHkgR29sZG1pbnRzLU9ybG92ICA8
YWdvbGRtaW50c0BpZ2FsaWEuY29tPgorCisgICAgICAgIFtHVEtdIGJ1aWxkIGZhaWxzIHdpdGgg
dW5kZWZpbmVkIHN5bWJvbCBpbiBBTkdMRSBpZiBVU0VfQU5HTEVfV0VCR0wgaXMgZW5hYmxlZC4K
KyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIzNDIzOQor
CisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEFkZCBhIG5l
dyBsaW51eC5jbWFrZSBmaWxlIHRoYXQgZGVmaW5lcyBhbmdsZV9kbWFfYnVmX3NvdXJjZXMuCisg
ICAgICAgIFRoaXMgbGlzdCBnZXRzIGFwcGVuZGVkIGl0IGludG8gQU5HTEVfU09VUkNFUyBpbiBQ
bGF0Zm9ybUdUSy5jbWFrZS4KKworICAgICAgICAqIGxpbnV4LmNtYWtlOiBBZGRlZC4gTWFudWFs
IHRyYW5zbGF0aW9uIG9mIHNyYy9jb21tb24vbGludXgvQlVJTEQuZ24uCisgICAgICAgICogUGxh
dGZvcm1HVEsuY21ha2U6CisKIDIwMjEtMTItMTcgIEZ1amlpIEhpcm9ub3JpICA8SGlyb25vcmku
RnVqaWlAc29ueS5jb20+CiAKICAgICAgICAgW0FOR0xFXSAiV0FSTjogZHJpdmVyX3V0aWxzLmNw
cDoxNjkgKHJ4OjpHZXRWZW5kb3JTdHJpbmcpOiEgVW5pbXBsZW1lbnRlZDogcng6OkdldFZlbmRv
clN0cmluZyIgb24gV2luQ2Fpcm8gZGVidWcgdGVzdGluZyBib3QKZGlmZiAtLWdpdCBhL1NvdXJj
ZS9UaGlyZFBhcnR5L0FOR0xFL1BsYXRmb3JtR1RLLmNtYWtlIGIvU291cmNlL1RoaXJkUGFydHkv
QU5HTEUvUGxhdGZvcm1HVEsuY21ha2UKaW5kZXggM2NjOWRkZGE2ZGY4ZDJkMTgzMWVjY2YxZjBl
YTk1NDllYmFkNTJjYi4uODQzMmQxMTYyM2I2NmVkODdhY2ZjNGIwYjI3MWQ1NGRhOGFlZDQ2NyAx
MDA2NDQKLS0tIGEvU291cmNlL1RoaXJkUGFydHkvQU5HTEUvUGxhdGZvcm1HVEsuY21ha2UKKysr
IGIvU291cmNlL1RoaXJkUGFydHkvQU5HTEUvUGxhdGZvcm1HVEsuY21ha2UKQEAgLTEsNCArMSw1
IEBACiBsaXN0KEFQUEVORCBBTkdMRV9ERUZJTklUSU9OUyBBTkdMRV9QTEFURk9STV9MSU5VWCkK
K2luY2x1ZGUobGludXguY21ha2UpCiAKIGlmIChVU0VfT1BFTkdMKQogICAgICMgRW5hYmxlIEdM
U0wgY29tcGlsZXIgb3V0cHV0LgpAQCAtMTIsNiArMTMsOCBAQCBpZiAoVVNFX0FOR0xFX0VHTCBP
UiBVU0VfQU5HTEVfV0VCR0wpCiAgICAgICAgICR7YW5nbGVfc3lzdGVtX3V0aWxzX3NvdXJjZXNf
bGludXh9CiAgICAgICAgICR7YW5nbGVfc3lzdGVtX3V0aWxzX3NvdXJjZXNfcG9zaXh9CiAKKyAg
ICAgICAgJHthbmdsZV9kbWFfYnVmX3NvdXJjZXN9CisKICAgICAgICAgJHtsaWJhbmdsZV9nbF9l
Z2xfZGxfc291cmNlc30KICAgICAgICAgJHtsaWJhbmdsZV9nbF9lZ2xfc291cmNlc30KICAgICAg
ICAgJHtsaWJhbmdsZV9nbF9zb3VyY2VzfQpkaWZmIC0tZ2l0IGEvU291cmNlL1RoaXJkUGFydHkv
QU5HTEUvbGludXguY21ha2UgYi9Tb3VyY2UvVGhpcmRQYXJ0eS9BTkdMRS9saW51eC5jbWFrZQpu
ZXcgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAw
MDAwMDAwMDAwLi5mM2Q5Y2MwNmUwM2ViNTU5MTM2NjYxMjk5ZGJmMmEzZDViYzU0ZGZlCi0tLSAv
ZGV2L251bGwKKysrIGIvU291cmNlL1RoaXJkUGFydHkvQU5HTEUvbGludXguY21ha2UKQEAgLTAs
MCArMSw2IEBACitpZihpc19hbmRyb2lkIE9SIGlzX2xpbnV4IE9SIGlzX2Nocm9tZW9zKQorICBz
ZXQoYW5nbGVfZG1hX2J1Zl9zb3VyY2VzCisgICAgInNyYy9jb21tb24vbGludXgvZG1hX2J1Zl91
dGlscy5jcHAiCisgICAgInNyYy9jb21tb24vbGludXgvZG1hX2J1Zl91dGlscy5oIgorICApCitl
bmRpZigpCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>