Bug 40810 - [EFL] Build break on Debug build
Summary: [EFL] Build break on Debug build
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-17 21:44 PDT by Gyuyoung Kim
Modified: 2010-06-21 06:23 PDT (History)
7 users (show)

See Also:


Attachments
Patch for debug option (2.29 KB, patch)
2010-06-17 21:49 PDT, Gyuyoung Kim
no flags Details | Formatted Diff | Diff
patch for build break (939 bytes, patch)
2010-06-18 00:33 PDT, Gyuyoung Kim
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gyuyoung Kim 2010-06-17 21:44:16 PDT
I cannot found options related to DEBUG. In my opinion, EFLWebKit needs to enable debug option as well.

When I added ENABLE_DEBUG option to cmake/OptionsEfl.cmake, there were "undefined references" as below,

Conservative compression summary:
  Wrote total 47834 bytes (47Kb) from 134514 (131Kb) input data
  Output file is 35.6% the size of the input data
  Saved 86680 bytes (85Kb)
Raw compression summary:
  Wrote total 47834 bytes (47Kb) from 807076 (788Kb) raw input data
  Output file is 5.9% the size of the raw input data
  Saved 759242 bytes (741Kb)
Scanning dependencies of target EWebLauncher
[100%] Building C object WebKit/CMakeFiles/../Programs/EWebLauncher.dir/efl/EWebLauncher/main.c.o
Linking CXX executable ../Programs/EWebLauncher
libewebkit.so: undefined reference to `WebCore::CSSPreloadScanner::CSSPreloadScanner(WebCore::Document*)'
libewebkit.so: undefined reference to `WebCore::CSSPreloadScanner::scan(WebCore::HTML5Token const&, bool)'
libewebkit.so: undefined reference to `WebCore::CSSPreloadScanner::reset()'
collect2: ld returned 1 exit status

There are no "html/CSSPreloadScanner.cpp / h" in WebCore/CMakeList.txt. I added them to WebCore/CMakeList.txt.
Comment 1 Gyuyoung Kim 2010-06-17 21:49:42 PDT
Created attachment 59071 [details]
Patch for debug option
Comment 2 Gyuyoung Kim 2010-06-18 00:30:40 PDT
  Wrote total 47834 bytes (47Kb) from 807076 (788Kb) raw input data
  Output file is 5.9% the size of the raw input data
  Saved 759242 bytes (741Kb)
Scanning dependencies of target EWebLauncher
[100%] Building C object WebKit/CMakeFiles/../Programs/EWebLauncher.dir/efl/EWebLauncher/main.c.o
Linking CXX executable ../Programs/EWebLauncher
libewebkit.so: undefined reference to `WebCore::CSSPreloadScanner::CSSPreloadScanner(WebCore::Document*)'
libewebkit.so: undefined reference to `WebCore::CSSPreloadScanner::scan(WebCore::HTML5Token const&, bool)'
libewebkit.so: undefined reference to `WebCore::CSSPreloadScanner::reset()'
collect2: ld returned 1 exit status
make[2]: *** [WebKit/../Programs/EWebLauncher] error 1
Comment 3 Gyuyoung Kim 2010-06-18 00:33:25 PDT
Created attachment 59075 [details]
patch for build break

It seems that lukas already made a patch for debug build as below,

Bug 40566 -  [EFL] Build system is forcing build type to Release
(https://bugs.webkit.org/show_bug.cgi?id=40566)

However, there are still build breaks when no Release build.

[100%] Building C object WebKit/CMakeFiles/../Programs/EWebLauncher.dir/efl/EWebLauncher/main.c.o
Linking CXX executable ../Programs/EWebLauncher
libewebkit.so: undefined reference to `WebCore::CSSPreloadScanner::CSSPreloadScanner(WebCore::Document*)'
libewebkit.so: undefined reference to `WebCore::CSSPreloadScanner::scan(WebCore::HTML5Token const&, bool)'
libewebkit.so: undefined reference to `WebCore::CSSPreloadScanner::reset()'
collect2: ld returned 1 exit status
make[2]: *** [WebKit/../Programs/EWebLauncher] error 1
Comment 4 Lucas De Marchi 2010-06-18 05:55:23 PDT
The issue is fixed on r61163 and partially on r61323.

Now the only thing that is missing is on #40566. To enable debug it will suffice to do the following in a build directory:

cmake .. -DPORT=Efl -DCMAKE_BUILD_TYPE=Debug


For future updates on CMake, remember that we don't keep track of .h files since they are automatically detected at compilation time.
Comment 5 Lucas De Marchi 2010-06-18 08:37:09 PDT
Ah, sorry about previous comment, I was cheated by the file name. This is another problem.

It has been broken again by r61366. I'm CCing Leandro who is able to fix this without going through commit queue.
Comment 6 Leandro Pereira 2010-06-18 08:55:23 PDT
Looks good to me.

Just change the ChangeLog entry to "Reviewed by NOBODY (OOPS!)" to "Unreviewed build fix" and I'll land it.
Comment 7 Leandro Pereira 2010-06-18 10:19:25 PDT
Comment on attachment 59075 [details]
patch for build break

Committed as r61416.
Comment 8 WebKit Review Bot 2010-06-18 10:21:10 PDT
http://trac.webkit.org/changeset/61416 might have broken Qt Linux Release minimal
The following changes are on the blame list:
http://trac.webkit.org/changeset/61416
http://trac.webkit.org/changeset/61414
http://trac.webkit.org/changeset/61415
Comment 9 Leandro Pereira 2010-06-18 10:25:10 PDT
(In reply to comment #8)
> http://trac.webkit.org/changeset/61416 might have broken Qt Linux Release minimal
> The following changes are on the blame list:
> http://trac.webkit.org/changeset/61416
> http://trac.webkit.org/changeset/61414
> http://trac.webkit.org/changeset/61415


Looks like a false alarm.