Bug 53630 – WebCore.gyp webcore_prerequisites needs some things moved into direct_dependent_settings

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Mark Mentovai]

http://crbug.com/71537

Looks like this broke in r66364.

[Mark Mentovai]

Created attachment 80976 [details]
Fix v1

Chrome try runs in progress:

http://build.chromium.org/p/tryserver.chromium/builders/mac/builds/11309
http://build.chromium.org/p/tryserver.chromium/builders/linux/builds/10901
http://build.chromium.org/p/tryserver.chromium/builders/win/builds/13703

Note for checkin: WebCore/WebCore.gyp/mac/check_objc_rename.sh needs to have the executable bit set.
bash$ svn propset svn:executable '*' WebCore/WebCore.gyp/mac/check_objc_rename.sh

[WebKit Review Bot]

Attachment 80976 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1

Source/WebCore/WebCore.gyp/WebCore.gyp:1025:  Line contains tab character.  [whitespace/tab] [5]
Total errors found: 1 in 3 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Mark Mentovai]

Created attachment 80977 [details]
Fix v2 (no tabs)

[Mark Mentovai]

Created attachment 81026 [details]
Fix v3 (no nm -U)

Apparently, nm -U (don’t display undefined symbols) is new in cctools 782 (Xcode 3.2.3). Chrome still needs to support 3.2.2 (and, for some real stragglers, 3.1.5), so the -U needs to go. This has no impact on the script, it’ll work just as well without -U.

[Dimitri Glazkov (Google)]

Comment on attachment 81026 [details]
Fix v3 (no nm -U)

ok.

[WebKit Commit Bot]

Comment on attachment 81026 [details]
Fix v3 (no nm -U)

Clearing flags on attachment: 81026

Committed r77562: <http://trac.webkit.org/changeset/77562>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 81026 [details]:

http/tests/websocket/tests/handshake-challenge-randomness.html bug 53738 (author: [e-mail])
The commit-queue is continuing to process your patch.

[Dirk Pranke]

I think this may have broken the mac build ...

[Dirk Pranke]

Committed r77589: <http://trac.webkit.org/changeset/77589>

[Mark Mentovai]

Dirk, can you please show me the Mac build break you encountered?

[Mark Mentovai]

OK, I found http://build.webkit.org/builders/Chromium%20Mac%20Release/builds/18787. The errors in the build log:

PhaseScriptExecution "Postbuild \"Check Objective-C Rename\"" ../../WebKit/chromium/xcodebuild/WebCore.build/Release/webcore_html.build/Script-C80125C94888F34350BF52AA.sh
    cd /Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp
    /bin/sh -c /Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp/../../WebKit/chromium/xcodebuild/WebCore.build/Release/webcore_html.build/Script-C80125C94888F34350BF52AA.sh

/Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp/../../WebKit/chromium/xcodebuild/WebCore.build/Release/webcore_html.build/Script-C80125C94888F34350BF52AA.sh: line 2: /Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp/mac/check_objc_rename.sh: Permission denied
/Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp/../../WebKit/chromium/xcodebuild/WebCore.build/Release/webcore_html.build/Script-C80125C94888F34350BF52AA.sh: line 2: exec: /Users/chrome-bot/Desktop/WebKit-BuildSlave/chromium-mac-release/build/Source/WebCore/WebCore.gyp/mac/check_objc_rename.sh: cannot execute: Unknown error: 0

and then

mark@rj bash$ svn proplist -v http://svn.webkit.org/repository/webkit/trunk/Source/WebCore/WebCore.gyp/mac/check_objc_rename.sh@77562
mark@rj bash$ 

(as in “no properties set.”)

This looks like check_objc_rename.sh was checked in without the svn:executable property set as I recommended in comment 1. When I asked Dimitri to commit-queue this, I also asked whether the commit-queue supported setting svn:executable, and he said he didn’t know but there was only one way to find out. :)

The v3 patch from this patch set should be checked in manually, with the svn:executable property set properly on check_objc_rename.sh.

[Dirk Pranke]

Yup, that sounds like the problem. Thanks Mark! I imagine either Dimitri or I can land it manually in the morning.

[WebKit Review Bot]

http://trac.webkit.org/changeset/77562 might have broken GTK Linux 32-bit Debug and GTK Linux 64-bit Debug
The following tests are not passing:
fast/events/pagehide-timeout.html

[Dimitri Glazkov (Google)]

(In reply to comment #13)
> Yup, that sounds like the problem. Thanks Mark! I imagine either Dimitri or I can land it manually in the morning.

I don't have an svn checkout.

[Mark Mentovai]

It ought to be easy to pull an svn tree to check this in. I guess there’s no way to set svn properties from git?

Dirk, do you have a WebKit svn checkout handy?

[Dirk Pranke]

Yes, I do. I'll land it when I get into work in a little while.

[Dirk Pranke]

Committed r77663. <http://trac.webkit.org/changeset/77663>

[WebKit Review Bot]

http://trac.webkit.org/changeset/77663 might have broken GTK Linux 32-bit Debug and GTK Linux 64-bit Debug
The following tests are not passing:
fast/blockflow/Kusa-Makura-background-canvas.html
fast/blockflow/japanese-ruby-horizontal-bt.html
fast/blockflow/japanese-ruby-vertical-lr.html
fast/blockflow/japanese-ruby-vertical-rl.html
fast/ruby/nested-ruby.html
fast/text/emphasis-avoid-ruby.html