Bug 45433

Summary: [EFL] Fix duplicate gstreamer macro in CMakeListEfl.txt
Product: WebKit Reporter: Gyuyoung Kim <gyuyoung.kim>
Component: WebKit EFLAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: antognolli+webkit, commit-queue, kenneth, lucas.de.marchi, tonikitoo
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Patch
tonikitoo: review+, tonikitoo: commit-queue-
Patch none

Description Gyuyoung Kim 2010-09-08 18:31:47 PDT 
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}
...
Comment 1 Gyuyoung Kim 2010-09-08 18:35:28 PDT 
Created attachment 66986 [details]
Patch
Comment 2 Antonio Gomes 2010-09-08 18:55:29 PDT 
Comment on attachment 66986 [details]
Patch

Fix the indentation in the changelog before landing.
Comment 3 Gyuyoung Kim 2010-09-08 19:08:05 PDT 
Created attachment 66989 [details]
Patch

Oops, sorry. I fix it. :)

Thanks.
Comment 4 WebKit Commit Bot 2010-09-09 02:06:38 PDT 
Comment on attachment 66989 [details]
Patch

Clearing flags on attachment: 66989

Committed r67071: <http://trac.webkit.org/changeset/67071>
Comment 5 Gyuyoung Kim 2010-09-09 23:02:25 PDT 
This bug's patch was landed to mainline. So, I change this bug's status with Resolved.