<?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>270776</bug_id>
          
          <creation_ts>2024-03-10 19:03:55 -0700</creation_ts>
          <short_desc>[Skia] libSkia.a is always considered outdated</short_desc>
          <delta_ts>2024-04-26 04:55:58 -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>WPE WebKit</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=273315</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>268972</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Patrick Griffis">pgriffis</reporter>
          <assigned_to name="Adrian Perez">aperez</assigned_to>
          <cc>aperez</cc>
    
    <cc>bugs-noreply</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2019990</commentid>
    <comment_count>0</comment_count>
    <who name="Patrick Griffis">pgriffis</who>
    <bug_when>2024-03-10 19:03:55 -0700</bug_when>
    <thetext>If you run: `ninja -d explain -C WebKitBuild/WPE/Debug libSkia.a`

It outputs:

    ninja explain: output Source/ThirdParty/skia/CMakeFiles/Skia_headers_symlink doesn&apos;t exist
    ninja explain: Source/ThirdParty/skia/CMakeFiles/Skia_headers_symlink is dirty
    ninja explain: Skia/Headers/skia is dirty
    [1/1] Link Skia headers

This causes every single build to relink Skia and everything that depends on it.

Currently this is defined in CMake as:

    add_custom_target(Skia_headers_symlink
        COMMAND ${CMAKE_COMMAND} -E create_symlink
            &quot;${CMAKE_CURRENT_SOURCE_DIR}/include&quot;
            &quot;${Skia_FRAMEWORK_HEADERS_DIR}/skia&quot;
        BYPRODUCTS
            &quot;${Skia_FRAMEWORK_HEADERS_DIR}/skia&quot;
        COMMENT &quot;Link Skia headers&quot;
    )
    add_dependencies(Skia Skia_headers_symlink)

I&apos;m not sure what the proper solution to this is.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2030487</commentid>
    <comment_count>1</comment_count>
    <who name="Adrian Perez">aperez</who>
    <bug_when>2024-04-23 01:42:04 -0700</bug_when>
    <thetext>I think we may be able to use a POST_BUILD custom command to generate the
symlink after libSkia.a is built *without* having the build system consider
that the library is always outdated. Let me give it a try.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2030488</commentid>
    <comment_count>2</comment_count>
    <who name="Adrian Perez">aperez</who>
    <bug_when>2024-04-23 01:48:45 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/27620</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2030490</commentid>
    <comment_count>3</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-04-23 02:01:25 -0700</bug_when>
    <thetext>Committed 277860@main (c48d5a8a6a38): &lt;https://commits.webkit.org/277860@main&gt;

Reviewed commits have been landed. Closing PR #27620 and removing active labels.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>