Bug 232224 - [FlatpakSDK] Import ccache 4.4.2 over fdo-sdk 4.4.0
Summary: [FlatpakSDK] Import ccache 4.4.2 over fdo-sdk 4.4.0
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Lauro Moura
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-10-24 19:35 PDT by Lauro Moura
Modified: 2021-10-28 08:19 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.07 KB, patch)
2021-10-26 20:14 PDT, Lauro Moura
no flags Details | Formatted Diff | Diff
Updated patch adding patch to junction (2.55 KB, patch)
2021-10-27 19:37 PDT, Lauro Moura
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lauro Moura 2021-10-24 19:35:14 PDT
The ccache version shipped with the current FlatpakSDK is 4.4.0 (through fdo's flatpak sdk), which is affected by a bug[1] that could trigger false positive direct hits by failing to track modifications in some header files in some special cases.

This has led to some strange issues in the GTK/WPE ports last few weeks, with erratic builds that produced green built-products but triggered errors when running the tests, failed builds with linking errors and other mysterious problems.

The issue is fixed in ccache 4.4.2[2], already imported in the main fdo-sdk branch.

[1] https://github.com/ccache/ccache/issues/935
[2] https://ccache.dev/releasenotes.html#_ccache_4_4_2
Comment 1 Lauro Moura 2021-10-26 20:14:51 PDT
Created attachment 442563 [details]
Patch
Comment 2 Philippe Normand 2021-10-27 01:07:58 PDT
Comment on attachment 442563 [details]
Patch

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

> Tools/buildstream/ChangeLog:3
> +        [Flatpak SDK] Import ccache 4.4.2 over fdo-sdk 4.4.0

What I'd do usually is patch the FDO junction. It's less readable but better for future maintenance I think.
Comment 3 Philippe Normand 2021-10-27 02:01:39 PDT
Actually it was updated upstream already: https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/blob/release/21.08/elements/components/ccache.bst

If you update the junction ref I think you should get the new version, no need to import a recipe here. The new version is not in a tagged release yet though, so you'd need to change the source kind I think... https://github.com/WebKit/WebKit/blob/main/Tools/buildstream/elements/freedesktop-sdk.bst#L3
Comment 4 Lauro Moura 2021-10-27 19:37:24 PDT
Created attachment 442667 [details]
Updated patch adding patch to junction
Comment 5 EWS 2021-10-28 00:31:06 PDT
Downloading beautifulsoup4-4.9.3...
Installing beautifulsoup4-4.9.3...
Installed beautifulsoup4-4.9.3!
Downloading soupsieve-1.9.6...
Installing soupsieve-1.9.6...
Installed soupsieve-1.9.6!
/Volumes/Data/worker/Commit-Queue/build/Tools/Scripts/libraries/autoinstalled/python-3-x86_64/bs4/element.py:16: UserWarning: The soupsieve package is not installed. CSS selectors cannot be used.
  'The soupsieve package is not installed. CSS selectors cannot be used.'
Downloading attrs-18.1.0...
Installing attrs-18.1.0...
Installed attrs-18.1.0!
Downloading mechanize-0.4.5...
Installing mechanize-0.4.5...
Installed mechanize-0.4.5!
Traceback (most recent call last):
  File "Tools/Scripts/webkit-patch", line 42, in <module>
    from webkitpy.tool.main import WebKitPatch
  File "/Volumes/Data/worker/Commit-Queue/build/Tools/Scripts/webkitpy/tool/main.py", line 39, in <module>
    from webkitpy.tool import commands
  File "/Volumes/Data/worker/Commit-Queue/build/Tools/Scripts/webkitpy/tool/commands/__init__.py", line 7, in <module>
    from webkitpy.tool.commands.queries import *
  File "/Volumes/Data/worker/Commit-Queue/build/Tools/Scripts/webkitpy/tool/commands/queries.py", line 48, in <module>
    from webkitpy.common.net.buildbot import BuildBot
  File "/Volumes/Data/worker/Commit-Queue/build/Tools/Scripts/webkitpy/common/net/buildbot/__init__.py", line 5, in <module>
    from .buildbot import BuildBot, Builder, Build
  File "/Volumes/Data/worker/Commit-Queue/build/Tools/Scripts/webkitpy/common/net/buildbot/buildbot.py", line 45, in <module>
    from mechanize import Browser
ModuleNotFoundError: No module named 'mechanize'
Comment 6 Lauro Moura 2021-10-28 08:06:09 PDT
Comment on attachment 442667 [details]
Updated patch adding patch to junction

Retrying cq+ after webkitpy hiccup in the previous attempt.
Comment 7 EWS 2021-10-28 08:18:26 PDT
Committed r284978 (243626@main): <https://commits.webkit.org/243626@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 442667 [details].
Comment 8 Radar WebKit Bug Importer 2021-10-28 08:19:30 PDT
<rdar://problem/84758248>