Bug 126779

Summary: FreeBSD build fixes
Product: WebKit Reporter: Koop Mast <kwm>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ajacoutot, andersca, commit-queue, dbates, landry, sam
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 128598    
Attachments:
Description Flags
build fixes
none
Build fixes for FreeBSD none

Description Koop Mast 2014-01-10 13:45:12 PST
Add sys/wait.h for WIFEXITED and WEXITSTATUS defines.
FreeBSD doesn't have SIGCLD, wrap the code in #if defined(SIGCLD).
Comment 1 Koop Mast 2014-01-10 13:45:58 PST
Created attachment 220882 [details]
build fixes
Comment 2 Koop Mast 2014-01-15 10:53:20 PST
Comment on attachment 220882 [details]
build fixes

set review flag
Comment 3 Sergio Correia (qrwteyrutiyoup) 2014-01-15 11:49:58 PST
Comment on attachment 220882 [details]
build fixes

View in context: https://bugs.webkit.org/attachment.cgi?id=220882&action=review

> ChangeLog:10
> +

I believe you shouldn't update this file, only the one in Source/WebKit2/Changelog.
Comment 4 Koop Mast 2014-01-18 06:42:38 PST
Created attachment 221543 [details]
Build fixes for FreeBSD

Same patch as before but used Tools/Scripts/prepare-ChangeLog to generate the Changelog.
Comment 5 WebKit Commit Bot 2014-01-22 03:11:03 PST
Comment on attachment 221543 [details]
Build fixes for FreeBSD

Clearing flags on attachment: 221543

Committed r162513: <http://trac.webkit.org/changeset/162513>
Comment 6 WebKit Commit Bot 2014-01-22 03:11:05 PST
All reviewed patches have been landed.  Closing bug.
Comment 7 Antoine Jacoutot 2014-01-27 13:42:31 PST
Hi.

I don't understand this patch. Isn't SIGCLD the same as SIGCHLD?
If so then why not s/SIGCLD/SIGCHLD which will work everywhere. That is what we've been using on OpenBSD.
Unless I am missing something obvious...
Comment 8 Landry Breuil 2014-02-14 13:51:11 PST
(In reply to comment #7)
> Hi.
> 
> I don't understand this patch. Isn't SIGCLD the same as SIGCHLD?
> If so then why not s/SIGCLD/SIGCHLD which will work everywhere. That is what we've been using on OpenBSD.
> Unless I am missing something obvious...

Yes... Koop, can you elaborate on why #if'ing out those parts ? We've used this for 2.2:
http://www.openbsd.org/cgi-bin/cvsweb/ports/www/webkit/patches/patch-Source_WebKit2_UIProcess_Plugins_unix_PluginProcessProxyUnix_cpp?rev=1.4;content-type=text%2Fplain