[Webkit-unassigned] [Bug 62260] [CMAKE][WK2] Add an option to build webkit2.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 8 05:48:23 PDT 2011


--- Comment #5 from Patrick R. Gansterer <paroga at paroga.com>  2011-06-08 05:48:22 PST ---
(From update of attachment 96407)
View in context: https://bugs.webkit.org/attachment.cgi?id=96407&action=review

> Source/CMakeLists.txt:16

IMHO a MESSAGE(FATAL_ERROR "config error....") whould be better here.

> Source/CMakeLists.txt:29

I don't understand this line. How it related to WK2?

> Source/CMakeLists.txt:101
> +    SET(WebKit_LIBRARY_NAME WebKit)
> +ENDIF ()
> +
> +    SET(WebKit2_LIBRARY_NAME WebKit2)
> +ENDIF ()

I don't see a problem in setting this variables all the time. This will make the configuration much simpler.
(the same applies to the other similar changes too)

> Source/WebKit2/CMakeLists.txt:94

Please add an empty line when the containig folder changes like we do at the other files: e.g. after "Platform/CoreIPC" to "Plaform", or "Platform" to "PluginProcess".
This makes the long lines a little bit easier to read.

> Source/WebKit2/CMakeLists.txt:410
> +SET (WebProcess_NAME ../Programs/WebProcess)
> +SET (WebProcess_SOURCES "")
> +
> +SET (WebProcess_LIBRARIES

Please removte the space after SET

> Source/WebKit2/CMakeLists.txt:441
> +

Is there an unneded empty line at the end?

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

More information about the webkit-unassigned mailing list