Bug 135620 – more cmake progress

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Misc.

[Alex Christensen]

JSC works on Windows and Mac with this, and I'm starting to get things working in WebCore, but that's still WIP.

[David Farler]

Neat. What are your ideas around `-G Xcode`?

[Alex Christensen]

Created attachment 236046 [details]
Patch

[Alex Christensen]

(In reply to comment #1)
> Neat. What are your ideas around `-G Xcode`?
I'm pretending Xcode doesn't exist to get a working proof of concept as fast as possible.  I can't read the Xcode files anyways.  Using Xcode with CMake is my end goal, though.

[Alex Christensen]

Created attachment 236049 [details]
Patch

[Laszlo Gombos]

Comment on attachment 236049 [details]
Patch

lgtm.

[Alex Christensen]

http://trac.webkit.org/changeset/172093

[Gyuyoung Kim]

Comment on attachment 236049 [details]
Patch

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

> Source/WebCore/PlatformMac.cmake:21
> +    "/usr/include/libxslt"

If user installs it to other place, can't this be problem ? EFL and GTK port use macro for this inclusion.

In OptionEfl.cmake,
 find_package(LibXml2 2.8.0 REQUIRED)
 find_package(LibXslt 1.1.7 REQUIRED)

Source/WebCore/PlatformEfl.cmake
 ${LIBXSLT_INCLUDE_DIR}
 ${LIBXML2_INCLUDE_DIR}

[Mark Rowe (bdash)]

(In reply to comment #7)
> (From update of attachment 236049 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=236049&action=review
> 
> > Source/WebCore/PlatformMac.cmake:21
> > +    "/usr/include/libxslt"
> 
> If user installs it to other place, can't this be problem ? EFL and GTK port use macro for this inclusion.
> 
> In OptionEfl.cmake,
>  find_package(LibXml2 2.8.0 REQUIRED)
>  find_package(LibXslt 1.1.7 REQUIRED)
> 
> Source/WebCore/PlatformEfl.cmake
>  ${LIBXSLT_INCLUDE_DIR}
>  ${LIBXML2_INCLUDE_DIR}

The Mac port only ever builds against frameworks in the OS X SDKs. Packages that the user may have installed aren't interesting.

[Gyuyoung Kim]

(In reply to comment #8)
> (In reply to comment #7)
> > (From update of attachment 236049 [details] [details])
> > View in context: https://bugs.webkit.org/attachment.cgi?id=236049&action=review
> > 
> > > Source/WebCore/PlatformMac.cmake:21
> > > +    "/usr/include/libxslt"
> > 
> > If user installs it to other place, can't this be problem ? EFL and GTK port use macro for this inclusion.
> > 
> > In OptionEfl.cmake,
> >  find_package(LibXml2 2.8.0 REQUIRED)
> >  find_package(LibXslt 1.1.7 REQUIRED)
> > 
> > Source/WebCore/PlatformEfl.cmake
> >  ${LIBXSLT_INCLUDE_DIR}
> >  ${LIBXML2_INCLUDE_DIR}
> 
> The Mac port only ever builds against frameworks in the OS X SDKs. Packages that the user may have installed aren't interesting.

I see. However, I would like to use cmake macro for including third party library.

[Mark Rowe (bdash)]

> I see. However, I would like to use cmake macro for including third party library.

I'm not quite what you're suggesting or why you're suggesting it. Can you please elaborate?

[Gyuyoung Kim]

(In reply to comment #10)
> > I see. However, I would like to use cmake macro for including third party library.
> 
> I'm not quite what you're suggesting or why you're suggesting it. Can you please elaborate?

EFL and GTK ports have tried to avoid hardcoding in CMake. That's why I suggested to use macro. However, as you said, hardcoded inclusion looks fine for mac. If you guys don't want to use it, I do not say it anymore. I hope this hardcoded is only used for mac port only.

[David Farler]

http://vtk.org/Wiki/CMake:How_To_Find_Libraries