Bug 43856 - Perl warnings when running commit-log-editor
Summary: Perl warnings when running commit-log-editor
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Daniel Bates
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-11 09:16 PDT by Daniel Bates
Modified: 2010-08-11 13:01 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.21 KB, patch)
2010-08-11 09:22 PDT, Daniel Bates
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Bates 2010-08-11 09:16:53 PDT
Following the patch for Bug #40548 (committed in changeset 65148 <http://trac.webkit.org/changeset/65148>), when running the commit-log-editor Perl complains with the following warnings:

Global symbol "$builtEditorApplication" requires explicit package name at ./commit-log-editor line 85.
Global symbol "$builtEditorApplication" requires explicit package name at ./commit-log-editor line 86.
Bareword "true" not allowed while "strict subs" in use at ./commit-log-editor line 68.
Bareword "false" not allowed while "strict subs" in use at ./commit-log-editor line 73.
Bareword "false" not allowed while "strict subs" in use at ./commit-log-editor line 80.
Bareword "false" not allowed while "strict subs" in use at ./commit-log-editor line 87.
Execution of ./commit-log-editor aborted due to compilation errors.
Comment 1 Daniel Bates 2010-08-11 09:22:22 PDT
Created attachment 64123 [details]
Patch

Perl doesn't have symbolic names for True/False. Instead, we should use boolean values. Moreover, the variable installedEditorApplication is unused. It looks like it should be renamed builtEditorApplication so that it uses the existing machinery to set the editor.
Comment 2 Dave Hyatt 2010-08-11 12:08:32 PDT
This patch addresses some of the errors, but not these:

"Global symbol "$builtEditorApplication" requires explicit package name at ./commit-log-editor line 85.
Global symbol "$builtEditorApplication" requires explicit package name at ./commit-log-editor line 86."

I still can't commit. Help!
Comment 3 Daniel Bates 2010-08-11 12:57:46 PDT
Committed in changeset 65179 <http://trac.webkit.org/changeset/65179>.
Comment 4 Daniel Bates 2010-08-11 12:58:42 PDT
Comment on attachment 64123 [details]
Patch

Clearing commit-queue flag since I committed this by hand because the queue was slow today :-(
Comment 5 Daniel Bates 2010-08-11 13:01:11 PDT
(In reply to comment #2)
> This patch addresses some of the errors, but not these:
> 
> "Global symbol "$builtEditorApplication" requires explicit package name at ./commit-log-editor line 85.
> Global symbol "$builtEditorApplication" requires explicit package name at ./commit-log-editor line 86."
> 
> I still can't commit. Help!

Spoke with David Hyatt on IRC. He inadvertently did not copy over the change to rename the variable installedEditorApplication to builtEditorApplication (included in this patch). He confirmed that his change resolves the "Global symbol" warnings.