Bug 208757 - generate-bindings-all.pl: "my" variable $fh masks earlier declaration in same scope
Summary: generate-bindings-all.pl: "my" variable $fh masks earlier declaration in same...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Fujii Hironori
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-03-06 20:56 PST by David Kilzer (:ddkilzer)
Modified: 2020-03-08 13:33 PDT (History)
7 users (show)

See Also:


Attachments
Patch (1.40 KB, patch)
2020-03-07 23:48 PST, Fujii Hironori
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2020-03-06 20:56:55 PST
Noticed here:  <https://build.webkit.org/builders/WinCairo%2064-bit%20WKL%20Release%20%28Build%29/builds/16615/steps/compile-webkit/logs/stdio>

[...]
Updating WinCairoRequirements.zip...
Found existing release: v2020.01.21
Seeking latest release from WebKitForWindows/WinCairoRequirements...
Found release to download: v2020.01.21
Already up-to-date!
[0/460] Generate bindings (WebCoreBindings)
"my" variable $fh masks earlier declaration in same scope at C:/WebKit-BuildWorker/wincairo-wkl-release/build/Source/WebCore/bindings/scripts/generate-bindings-all.pl line 75.
Preprocess IDL
[2/219] cmd.exe /C "cd /D C:\WebKit-BuildWorker\wincairo-wkl-release\build\WebKitBuild\Release && C:\tools\cmake\bin\cmake.exe -E copy_directory C:/WebKit-BuildWorker/wincairo-wkl-release/build/Source/WebInspectorUI/UserInterface C:/WebKit-BuildWorker/wincairo-wkl-release/build/WebKitBuild/Release/bin64/WebKit.resources/WebInspectorUI && C:\tools\cmake\bin\cmake.exe -E copy C:/WebKit-BuildWorker/wincairo-wkl-release/build/WebKitBuild/Release/JavaScriptCore/DerivedSources/inspector/InspectorBackendCommands.js C:/WebKit-BuildWorker/wincairo-wkl-release/build/WebKitBuild/Release/bin64/WebKit.resources/WebInspectorUI/Protocol && C:\tools\cmake\bin\cmake.exe -E copy C:/WebKit-BuildWorker/wincairo-wkl-release/build/Source/WebInspectorUI/Localizations/en.lproj/localizedStrings.js C:/WebKit-BuildWorker/wincairo-wkl-release/build/WebKitBuild/Release/bin64/WebKit.resources/WebInspectorUI"
[3/219] Building CXX object Source\WebKitLegacy\CMakeFiles\WebKitLegacy.dir\win\WebHistoryItem.cpp.obj
[4/219] Building CXX object Source\WebKitLegacy\CMakeFiles\WebKitLegacy.dir\win\WebKitDLL.cpp.obj
[5/219] Building CXX object Source\WebKit\CMakeFiles\WebKit.dir\__\__\WebKit\DerivedSources\DownloadProxyMessageReceiver.cpp.obj
[...]
Comment 1 Fujii Hironori 2020-03-07 17:16:26 PST
introduced by r257975 (Bug 205107).
Comment 2 Fujii Hironori 2020-03-07 17:17:59 PST
Not only WinCairo port, but all CMake ports.

https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Build%29/builds/30850/steps/compile-webkit/logs/stdio

> "my" variable $fh masks earlier declaration in same scope at /home/slave/webkitgtk/gtk-linux-64-release/build/Source/WebCore/bindings/scripts/generate-bindings-all.pl line 75.
Comment 3 Fujii Hironori 2020-03-07 23:48:38 PST
Created attachment 392943 [details]
Patch
Comment 4 Yusuke Suzuki 2020-03-08 01:16:12 PST
Comment on attachment 392943 [details]
Patch

r=me
Comment 5 WebKit Commit Bot 2020-03-08 13:32:17 PDT
The commit-queue encountered the following flaky tests while processing attachment 392943 [details]:

editing/spelling/spellcheck-async-remove-frame.html bug 160571 (authors: morrita@google.com, rniwa@webkit.org, and tony@chromium.org)
The commit-queue is continuing to process your patch.
Comment 6 WebKit Commit Bot 2020-03-08 13:32:55 PDT
Comment on attachment 392943 [details]
Patch

Clearing flags on attachment: 392943

Committed r258112: <https://trac.webkit.org/changeset/258112>
Comment 7 WebKit Commit Bot 2020-03-08 13:32:57 PDT
All reviewed patches have been landed.  Closing bug.
Comment 8 Radar WebKit Bug Importer 2020-03-08 13:33:12 PDT
<rdar://problem/60203979>