RESOLVED FIXED 45433
[EFL] Fix duplicate gstreamer macro in CMakeListEfl.txt 
https://bugs.webkit.org/show_bug.cgi?id=45433
Summary [EFL] Fix duplicate gstreamer macro in CMakeListEfl.txt
Gyuyoung Kim
Reported Thursday, September 9, 2010 2:31:47 AM UTC
In WebKit/efl/CMakeListEfl.txt, duplicated ENABLE_VIDEO macros were added. So, I remove unnecessary thing. IF (ENABLE_VIDEO) LIST(APPEND WebKit_INCLUDE_DIRECTORIES "${WEBCORE_DIR}/platform/graphics/gstreamer" ${Gstreamer-App_INCLUDE_DIRS} ${Gstreamer-Interfaces_INCLUDE_DIRS} ${Gstreamer-Pbutils_INCLUDE_DIRS} ${Gstreamer-Video_INCLUDE_DIRS} ) ENDIF() IF (ENABLE_VIDEO) LIST(APPEND WebKit_INCLUDE_DIRECTORIES "${WEBCORE_DIR}/platform/graphics/gstreamer" ${GStreamer-App_INCLUDE_DIRS} ${GStreamer-Interfaces_INCLUDE_DIRS} ${GStreamer-Pbutils_INCLUDE_DIRS} ...
Attachments
Patch (1.32 KB, patch)
2010-09-08 18:35 PDT, Gyuyoung Kim 		tonikitoo: review+
tonikitoo: commit-queue-
DetailsFormatted DiffDiff
Patch (1.33 KB, patch)
2010-09-08 19:08 PDT, Gyuyoung Kim 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gyuyoung Kim
Comment 1 Thursday, September 9, 2010 2:35:28 AM UTC
Created attachment 66986 [details] Patch
Antonio Gomes
Comment 2 Thursday, September 9, 2010 2:55:29 AM UTC
Comment on attachment 66986 [details] Patch Fix the indentation in the changelog before landing.
Gyuyoung Kim
Comment 3 Thursday, September 9, 2010 3:08:05 AM UTC
Created attachment 66989 [details] Patch Oops, sorry. I fix it. :) Thanks.
WebKit Commit Bot
Comment 4 Thursday, September 9, 2010 10:06:38 AM UTC
Comment on attachment 66989 [details] Patch Clearing flags on attachment: 66989 Committed r67071: <http://trac.webkit.org/changeset/67071>
Gyuyoung Kim
Comment 5 Friday, September 10, 2010 7:02:25 AM UTC
This bug's patch was landed to mainline. So, I change this bug's status with Resolved.
Note You need to log in before you can comment on or make changes to this bug.