Summary: | [webkitperl] Stop using LoadAsModule.pm | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Basuke Suzuki <Basuke.Suzuki> | ||||||||||||
Component: | Tools / Tests | Assignee: | Basuke Suzuki <Basuke.Suzuki> | ||||||||||||
Status: | REOPENED --- | ||||||||||||||
Severity: | Normal | CC: | ap, Basuke.Suzuki, ews-watchlist, jbedard, kkinnunen, stephan.szabo, 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=74836 https://bugs.webkit.org/show_bug.cgi?id=206901 https://bugs.webkit.org/show_bug.cgi?id=239762 |
||||||||||||||
Bug Depends on: | |||||||||||||||
Bug Blocks: | 231869 | ||||||||||||||
Attachments: |
|
Description
Basuke Suzuki
2021-10-16 14:54:34 PDT
Created attachment 441543 [details]
PATCH
Created attachment 441545 [details]
PATCH
With this change, most of the warnings are removed when test webkitperl. › Tools/Scripts/test-webkitperl Tools/Scripts/webkitperl/auto-version_unittest/autoVersionTests.pl ................................. ok Tools/Scripts/webkitperl/auto-version_unittest/versionStampTests.pl ................................ ok Tools/Scripts/webkitperl/filter-build-webkit_unittest/shouldIgnoreLine_unittests.pl ................ ok Tools/Scripts/webkitperl/prepare-ChangeLog_unittest/extractLineRangeBeforeAndAfterChange.pl ........ ok Tools/Scripts/webkitperl/prepare-ChangeLog_unittest/fetchRadarURLFromBugXMLData.pl ................. ok Tools/Scripts/webkitperl/prepare-ChangeLog_unittest/filenameWithParentheses.pl ..................... ok Tools/Scripts/webkitperl/prepare-ChangeLog_unittest/generateFunctionLists.pl ....................... ok Tools/Scripts/webkitperl/prepare-ChangeLog_unittest/parser_unittests.pl ............................ ok Tools/Scripts/webkitperl/run-leaks_unittest/run-leaks-report-v1.0.pl ............................... ok Tools/Scripts/webkitperl/run-leaks_unittest/run-leaks-report-v2.0-new.pl ........................... ok Tools/Scripts/webkitperl/run-leaks_unittest/run-leaks-report-v2.0-old.pl ........................... ok Tools/Scripts/webkitperl/test262_unittest/test262-runner-tests.pl .................................. ok Tools/Scripts/webkitperl/VCSUtils_unittest/decodeGitBinaryPatch.pl ................................. ok Tools/Scripts/webkitperl/VCSUtils_unittest/fixChangeLogPatch.pl .................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/fixChangeLogPatchThenSetChangeLogDateAndReviewer.pl ..... ok Tools/Scripts/webkitperl/VCSUtils_unittest/fixSVNPatchForAdditionWithHistory.pl .................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/generatePatchCommand.pl ................................. ok Tools/Scripts/webkitperl/VCSUtils_unittest/mergeChangeLogs.pl ...................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseChunkRange.pl ...................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl ............................................ ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseDiffHeader.pl ...................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseDiffStartLine.pl ................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseDiffWithMockFiles.pl ............................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseFirstEOL.pl ........................................ ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseGitDiffHeader.pl ................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/parsePatch.pl ........................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffFooter.pl ................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl ................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseSvnProperty.pl ..................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseSvnPropertyValue.pl ................................ ok Tools/Scripts/webkitperl/VCSUtils_unittest/parseUnifiedDiffHeader.pl ............................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/prepareParsedPatch.pl ................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/removeEOL.pl ............................................ ok Tools/Scripts/webkitperl/VCSUtils_unittest/runCommand.pl ........................................... 1/3 Can't exec "/usr/bin/non-existent-command": No such file or directory at /Users/basuke/work/WebKit/Tools/Scripts/VCSUtils.pm line 2433. Tools/Scripts/webkitperl/VCSUtils_unittest/runCommand.pl ........................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/runPatchCommand.pl ...................................... ok Tools/Scripts/webkitperl/VCSUtils_unittest/setChangeLogDateAndReviewer.pl .......................... ok Tools/Scripts/webkitperl/webkitdirs_unittest/appendToEnvironmentVariableList.pl .................... ok Tools/Scripts/webkitperl/webkitdirs_unittest/availableXcodeSDKS.pl ................................. ok Tools/Scripts/webkitperl/webkitdirs_unittest/checkForArgumentAndRemoveFromArrayRef.pl .............. ok Tools/Scripts/webkitperl/webkitdirs_unittest/checkForArgumentAndRemoveFromArrayRefGettingValue.pl .. ok Tools/Scripts/webkitperl/webkitdirs_unittest/extractNonMacOSHostConfiguration.pl ................... ok Tools/Scripts/webkitperl/webkitdirs_unittest/prependToEnvironmentVariableList.pl ................... ok Tools/Scripts/webkitperl/webkitdirs_unittest/splitVersion.pl ....................................... ok All tests successful. Files=43, Tests=710, 11 wallclock secs ( 0.06 usr 0.05 sys + 2.45 cusr 0.74 csys = 3.30 CPU) Result: PASS Created attachment 441546 [details]
PATCH
Added missing copyright header
Created attachment 441617 [details]
Stop using keyword-like usage of subroutine
Created attachment 441622 [details]
Fix
Comment on attachment 441622 [details]
Fix
Should be fine on Apple Internal configurations too, these scripts already had webkitperl dependencies.
Committed r284399 (243175@main): <https://commits.webkit.org/243175@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 441622 [details]. Reverted r284399 for reason: Breaks commit message generation Committed r284419 (243187@main): <https://commits.webkit.org/243187@main> |