Bug 135555

Summary: [gtk] Include llvm-dev(el) package to satisfy mesa build configuration
Product: WebKit Reporter: Vivek Galatage <vivekg>
Component: Tools / TestsAssignee: Vivek Galatage <vivekg>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, mrobinson, pnormand
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Linux   
Attachments:
Description Flags
Patch
none
Patch none

Description Vivek Galatage 2014-08-03 22:53:48 PDT
Initial setup of gtk on linux requires this pacakge to be installed.
If not installed, during the run of ./Tools/Scripts/update-webkitgtk-libs the following error is thrown:

configure: error: Could not find llvm shared libraries:
	Please make sure you have built llvm with the --enable-shared option
	and that your llvm libraries are installed in /usr/lib/llvm-3.2/lib
	If you have installed your llvm libraries to a different directory you
	can use the --with-llvm-prefix= configure flag to specify this directory.
	NOTE: Mesa is attempting to use llvm shared libraries because you have
	passed one of the following options to configure:
		--with-llvm-shared-libs
		--enable-opencl
	If you do not want to build with llvm shared libraries and instead want to
	use llvm static libraries then remove these options from your configure
	invocation and reconfigure.

Patch follows.
Comment 1 Vivek Galatage 2014-08-03 22:56:28 PDT
Created attachment 235963 [details]
Patch
Comment 2 Vivek Galatage 2014-08-05 08:38:47 PDT
Friendly ping :)
Comment 3 Philippe Normand 2014-08-05 08:42:40 PDT
Comment on attachment 235963 [details]
Patch

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

> Tools/ChangeLog:8
> +        Initial setup of gtk on linux requires this pacakge to be installed.

typo: pacakge
Comment 4 Vivek Galatage 2014-08-05 08:43:58 PDT
Comment on attachment 235963 [details]
Patch

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

>> Tools/ChangeLog:8
>> +        Initial setup of gtk on linux requires this pacakge to be installed.
> 
> typo: pacakge

Sure, will fix it. Thank you.
Comment 5 Vivek Galatage 2014-08-05 08:56:14 PDT
Created attachment 236030 [details]
Patch
Comment 6 WebKit Commit Bot 2014-08-05 09:39:11 PDT
Comment on attachment 236030 [details]
Patch

Clearing flags on attachment: 236030

Committed r172030: <http://trac.webkit.org/changeset/172030>
Comment 7 WebKit Commit Bot 2014-08-05 09:39:15 PDT
All reviewed patches have been landed.  Closing bug.