<?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>279660</bug_id>
          
          <creation_ts>2024-09-13 02:19:45 -0700</creation_ts>
          <short_desc>REGRESSION(283334@main): [GLIB] Script for updating third-party libs does not work anymore for flatpak</short_desc>
          <delta_ts>2024-09-13 08:27:15 -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>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=279261</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="Carlos Alberto Lopez Perez">clopez</reporter>
          <assigned_to name="Carlos Alberto Lopez Perez">clopez</assigned_to>
          <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2059943</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2024-09-13 02:19:45 -0700</bug_when>
    <thetext>Since 283334@main the scripts update-webkitgtk-libs and update-webkitwpe-libs does not work anymore.

This is because the call to the script update-webkit-flatpak is now inside now a condition shouldUseFlatpak() that is always false at the invocation of that script due to this two reasons:

1. The subroutine shouldUseFlatpak() returns false if the ports is not specified, and this two scripts are usually called without passed the flag --gtk or --wpe
2. Even if you call the script passing the port as in &quot;update-webkitgtk-libs --gtk&quot; shouldUseFlatpak() will only return true if the flatpak is already installed, but it will not return true on the first run (no WebKitBuild dir) and we want to default to using flatpak unless the environment tells us to do something different.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2059945</commentid>
    <comment_count>1</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2024-09-13 02:28:11 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/33609</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2060011</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-09-13 08:26:25 -0700</bug_when>
    <thetext>Committed 283621@main (6ddc6a2f2f12): &lt;https://commits.webkit.org/283621@main&gt;

Reviewed commits have been landed. Closing PR #33609 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2060012</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2024-09-13 08:27:15 -0700</bug_when>
    <thetext>&lt;rdar://problem/135955438&gt;</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>