Bug 115375

Summary: [Windows, WinCairo] Remove Link And Includes for Pthreads from JavaScriptCore
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: JavaScriptCoreAssignee: Brent Fulgham <bfulgham>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Unspecified   
Bug Depends on: 114864    
Bug Blocks:    
Attachments:
Description Flags
Patch thorton: review+

Brent Fulgham
Reported 2013-04-29 13:56:43 PDT
Update the build rules for the Windows/WinCairo builds to no longer reference or link to pthreadsVC2.dll. Nothing in JavaScriptCore uses the pthreads library on the Windows port, so linking to the library serves no purpose.
Attachments
Patch (7.70 KB, patch)
2013-04-29 14:54 PDT, Brent Fulgham 		thorton: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Brent Fulgham
Comment 1 2013-04-29 14:39:26 PDT
There are a handful of remaining references. This update attempts to clear out that last couple of them.
Brent Fulgham
Comment 2 2013-04-29 14:54:09 PDT
Created attachment 200049 [details] Patch
Brent Fulgham
Comment 3 2013-04-29 14:54:55 PDT
I don't know why Visual Studio does weird things like change the last line of the file; I can't easily prevent that when generating the patch. :-(
Brent Fulgham
Comment 4 2013-04-29 15:07:41 PDT
Committed r149320: <http://trac.webkit.org/changeset/149320>
Note You need to log in before you can comment on or make changes to this bug.