Bug 41830 – [EFL] fix cmake build for EWebLauncher

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

[Joone Hur]

-- Generating done
-- Build files have been written to: /home/joone/git/WebKit/WebKitBuild/Debug_Efl
[  2%] Built target wtf
[  9%] Built target javascriptcore
[  9%] Built target jsc
[100%] Built target webcore
[100%] Built target ewebkit
Linking CXX executable ../Programs/EWebLauncher
libewebkit.so: undefined reference to `soup_message_get_first_party'
libewebkit.so: undefined reference to `soup_message_set_first_party'
libewebkit.so: undefined reference to `soup_session_prepare_for_uri'
libewebkit.so: undefined reference to `soup_cookie_jar_set_accept_policy'
libewebkit.so: undefined reference to `soup_cookie_jar_set_cookie_with_first_party'
collect2: ld returned 1 exit status
make[2]: *** [WebKit/../Programs/EWebLauncher] Error 1
make[1]: *** [WebKit/CMakeFiles/../Programs/EWebLauncher.dir/all] Error 2
make: *** [all] Error 

Building EWebLauncher has a link error.

[Joone Hur]

Created attachment 60833 [details]
Proposed patch

CMakeListsEfl.txt: add ${LIBSOUP24_LDFLAGS} for link flags

[Lucas De Marchi]

(In reply to comment #1)
> Created an attachment (id=60833) [details]
> Proposed patch
> 
> CMakeListsEfl.txt: add ${LIBSOUP24_LDFLAGS} for link flags

This was a recent change to fix cross compile on some environments. I'm CC'ing who did this.

[Rafael Antognolli]

The patch is correct. I didn't notice this because my other patch in bug 41770 also includes this fix. But this one should get in first, then I'll update that one.

[Antonio Gomes]

Comment on attachment 60833 [details]
Proposed patch

Ok.

[WebKit Commit Bot]

Comment on attachment 60833 [details]
Proposed patch

Clearing flags on attachment: 60833

Committed r62895: <http://trac.webkit.org/changeset/62895>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.