[Webkit-unassigned] [Bug 57535] Fix compilation on Solaris 10 with Sun Studio 12, missing definition for time_t

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=57535





--- Comment #12 from David Levin <levin at chromium.org>  2011-04-04 13:56:56 PST ---
(In reply to comment #11)
> (In reply to comment #10)
> > (From update of attachment 87961 [details] [details])
> > This broke the build on several platforms so r-.
> 
> What's the recommended way to isolate the header for a specific OS?
> 
> I thought the second solution, based on previous patches that I've submitted would have been the right way to solve this problem.  I was very surprised to see the Linux/GTK systems fail the way they did.

It looks like the failures for the QT and cr-linux builds are because the files which include the header must not have the define for OS(). I believe this is in config.h, so I would make sure those files include it.  (The may involve modified a script which generates some files.)

For Windows, I don't quite get what went wrong there (maybe something similar?).

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.