Bug 211014

Summary: REGRESSION(210942): [GTK][WPE] EWS build bots fail in compile-webkit step
Product: WebKit Reporter: Diego Pino <dpino>
Component: New BugsAssignee: Diego Pino <dpino>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: annulen, chris.reid, ews-watchlist, gyuyoung.kim, ryuan.choi, sergio, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=210942
https://bugs.webkit.org/show_bug.cgi?id=211035
Attachments:
Description Flags
Patch
none
Patch none

Description Diego Pino 2020-04-25 00:39:05 PDT
See: https://ews-build.webkit.org/#/builders/36/builds/4641

perl Tools/Scripts/build-webkit --release --gtk
 in dir /home/ews/worker/GTK-Build-EWS/build (timeout 1200 secs)
 watching logfiles {}
 argv: ['perl', 'Tools/Scripts/build-webkit', '--release', '--gtk']
 using PTY: False
+ /home/ews/worker/GTK-Build-EWS/build/Tools/jhbuild/jhbuild-wrapper --gtk run cmake --build /home/ews/worker/GTK-Build-EWS/build/WebKitBuild/Release --config Release -- -j8
ninja: error: 'inspector-resources.stamp', needed by 'DerivedSources/webkit2gtk/InspectorGResourceBundle.xml', missing and no known rule to make it
program finished with exit code 1
elapsedTime=0.527474
Comment 1 Diego Pino 2020-04-25 01:34:07 PDT
Created attachment 397551 [details]
Patch
Comment 2 Diego Pino 2020-04-25 01:39:30 PDT
Created attachment 397553 [details]
Patch
Comment 3 Diego Pino 2020-04-25 01:45:54 PDT
r210942 refactored the code that copied WebInspector UI resources from a command to a new target: WebInspectorUI. The result of executing this command was the file inspector-resources.stamp.

InspectorGResources.cmake still had a dependency defined with inspector-resources.stamp. I replaced that dependency with target WebInspectorUI and removed the creation of the inspector-resources.stamp.
Comment 4 Diego Pino 2020-04-25 01:56:17 PDT
Committed r260696: <https://trac.webkit.org/changeset/260696>
Comment 5 Radar WebKit Bug Importer 2020-04-25 01:57:12 PDT
<rdar://problem/62365517>
Comment 6 Diego Pino 2020-04-25 17:16:16 PDT
See: https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Tests%29/builds/13485/steps/layout-test/logs/stdio

Regressions: Unexpected timeouts (49)
  fast/dom/frame-loading-via-document-write.html [ Timeout ]
  fast/dom/set-document-location-host-to-unaccepted-values.html [ Timeout ]
  fast/mediastream/play-newly-added-audio-track.html [ Timeout ]
  http/tests/inspector/dom/cross-domain-inspected-node-access.html [ Timeout ]
  http/tests/inspector/dom/didFireEvent.html [ Timeout ]
  http/tests/inspector/dom/disconnect-dom-tree-after-main-frame-navigation.html [ Timeout ]
  http/tests/inspector/network/beacon-type.html [ Timeout ]
  http/tests/inspector/network/contentextensions/blocked-websocket-crash.html [ Timeout ]
  http/tests/inspector/network/copy-as-curl.html [ Timeout ]
  http/tests/inspector/network/fetch-network-data.html [ Timeout ]
  http/tests/inspector/network/fetch-response-body.html [ Timeout ]
  http/tests/inspector/network/har/har-basic.html [ Timeout ]
  http/tests/inspector/network/har/har-import.html [ Timeout ]
  http/tests/inspector/network/har/har-page-aggressive-gc.html [ Timeout ]
  http/tests/inspector/network/har/har-page.html [ Timeout ]
  http/tests/inspector/network/loadResource-insecure-resource.html [ Timeout ]
  http/tests/inspector/network/local-resource-override-basic.html [ Timeout ]
  http/tests/inspector/network/local-resource-override-main-resource.html [ Timeout ]
  http/tests/inspector/network/local-resource-override-script-tag.html [ Timeout ]
  http/tests/inspector/network/resource-initiatorNode.html [ Timeout ]
  http/tests/inspector/network/resource-metrics.html [ Timeout ]
  http/tests/inspector/network/resource-mime-type.html [ Timeout ]
  http/tests/inspector/network/resource-request-headers.html [ Timeout ]
  http/tests/inspector/network/resource-response-inspector-override.html [ Timeout ]
  http/tests/inspector/network/resource-response-service-worker.html [ Timeout ]
  http/tests/inspector/network/resource-response-source-disk-cache.html [ Timeout ]
  http/tests/inspector/network/resource-response-source-memory-cache-revalidate-expired-only.html [ Timeout ]
  http/tests/inspector/network/resource-response-source-memory-cache.html [ Timeout ]
  http/tests/inspector/network/resource-response-source-network.html [ Timeout ]
  http/tests/inspector/network/resource-sizes-disk-cache.html [ Timeout ]
  http/tests/inspector/network/resource-sizes-memory-cache.html [ Timeout ]
  http/tests/inspector/network/resource-sizes-network.html [ Timeout ]
  http/tests/inspector/network/resource-timing.html [ Timeout ]
  http/tests/inspector/network/set-resource-caching-disabled-disk-cache.html [ Timeout ]
  http/tests/inspector/network/set-resource-caching-disabled-memory-cache.html [ Timeout ]
  http/tests/inspector/network/source-map-loads.html [ Timeout ]
  http/tests/inspector/network/x-frame-options.html [ Timeout ]
  http/tests/inspector/network/xhr-request-data-encoded-correctly.html [ Timeout ]
  http/tests/inspector/network/xhr-response-body.html [ Timeout ]
  http/tests/inspector/target/pause-on-inline-debugger-statement.html [ Timeout ]
  http/tests/inspector/target/provisional-load-cancels-previous-load.html [ Timeout ]
  http/tests/inspector/target/target-events-for-provisional-page.html [ Timeout ]
  http/tests/inspector/worker/blob-script-with-cross-domain-imported-scripts.html [ Timeout ]
  http/tests/resourceLoadStatistics/clear-in-memory-and-persistent-store-one-hour.html [ Timeout ]
  inspector/animation/effectChanged.html [ Timeout ]
  inspector/audit/basic-async.html [ Timeout ]
  inspector/canvas/console-record-2d.html [ Timeout ]
  inspector/controller/runtime-controller-import.html [ Timeout ]
  inspector/cpu-profiler/threads.html [ Timeout ]
Comment 7 Diego Pino 2020-04-27 00:36:07 PDT
Reverted r260696 for reason:

Build fix after r260672, no needed since r260672 is also being reverted

Committed r260741: <https://trac.webkit.org/changeset/260741>
Comment 8 Diego Pino 2020-05-02 03:54:27 PDT

*** This bug has been marked as a duplicate of bug 210942 ***