Bug 131980 - Buildfix after r167650 broke non-Cocoa builds
Summary: Buildfix after r167650 broke non-Cocoa builds
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Peter Molnar
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-22 02:39 PDT by Peter Molnar
Modified: 2014-04-22 04:06 PDT (History)
3 users (show)

See Also:


Attachments
patch (735 bytes, patch)
2014-04-22 02:42 PDT, Peter Molnar
pnormand: review-
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Molnar 2014-04-22 02:39:32 PDT
Buildfix after r167650 broke non-Cocoa builds
Comment 1 Peter Molnar 2014-04-22 02:42:56 PDT
Created attachment 229873 [details]
patch

#ifdef-ed out the Mac specific line to fix the non-Cocoa builds.
Comment 2 Mario Sanchez Prada 2014-04-22 02:47:43 PDT
Comment on attachment 229873 [details]
patch

Sounds good to me, although we should let bfulgham now about it anyway
Comment 3 WebKit Commit Bot 2014-04-22 03:17:23 PDT
Comment on attachment 229873 [details]
patch

Rejecting attachment 229873 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-01', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 229873, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
131980.
Found no modified ChangeLogs, cannot create a commit message.
All changes require a ChangeLog.  See:
 http://webkit.org/coding/contributing.html
Found no modified ChangeLogs, cannot create a commit message.
All changes require a ChangeLog.  See:
 http://webkit.org/coding/contributing.html
Found no modified ChangeLogs, cannot create a commit message.
All changes require a ChangeLog.  See:
 http://webkit.org/coding/contributing.html
Updating OpenSource
Current branch master is up to date.

Full output: http://webkit-queues.appspot.com/results/4802375017562112
Comment 4 Philippe Normand 2014-04-22 03:45:21 PDT
Comment on attachment 229873 [details]
patch

Missing ChangeLog, prepare-ChangeLog can help with that.
Comment 5 Mario Sanchez Prada 2014-04-22 03:56:56 PDT
(In reply to comment #4)
> (From update of attachment 229873 [details])
> Missing ChangeLog, prepare-ChangeLog can help with that.

Opps, that's right. Sorry for my quick review
Comment 6 Peter Molnar 2014-04-22 04:05:40 PDT
(In reply to comment #5)
> (In reply to comment #4)
> > (From update of attachment 229873 [details] [details])
> > Missing ChangeLog, prepare-ChangeLog can help with that.
> 
> Opps, that's right. Sorry for my quick review

Sorry, added the ChangeLog and landed patch in r167654.