Bug 57602

Summary:

[Qt] DumpRenderTree breaks compilation in some uClibc environments

Product:

WebKit

Reporter:

Keith Kyzivat <kamaji>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Enhancement

CC:

benjamin, commit-queue, kamaji, kling, laszlo.gombos, ossy, yael

Priority:

Keywords:

Qt, QtTriaged

Version:

528+ (Nightly build)

Hardware:

Other

OS:

Linux

Attachments:

Description	Flags
Patch	none
Patch	none

Keith Kyzivat

Reported 2011-03-31 19:48:07 PDT

When using uClibc, depending on how it was compiled, it can pretend to be glibc, by defining __GLIBC__. DumpRenderTree uses __GLIBC__ guards around a backtrace debugging helper that is defined in glibc, but not defined in uClibc. kit/Tools/DumpRenderTree/qt -I. -o obj/release/main.o ../../../../webkit/Tools/DumpRenderTree/qt/main.cpp ../../../../webkit/Tools/DumpRenderTree/qt/main.cpp:59:22: error: execinfo.h: No such file or directory ../../../../webkit/Tools/DumpRenderTree/qt/main.cpp: In function 'QString get_backtrace()': ../../../../webkit/Tools/DumpRenderTree/qt/main.cpp:79: error: 'backtrace' was not declared in this scope ../../../../webkit/Tools/DumpRenderTree/qt/main.cpp:84: error: 'backtrace_symbols' was not declared in this scope make: *** [obj/release/main.o] Error 1

Attachments
Patch (1.23 KB, patch) 2011-03-31 19:53 PDT, Keith Kyzivat	no flags	Details Formatted Diff Diff
Patch (1.23 KB, patch) 2011-04-01 11:38 PDT, Keith Kyzivat	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Keith Kyzivat

Comment 1 2011-03-31 19:53:21 PDT

Created attachment 87820 [details] Patch

Keith Kyzivat

Comment 2 2011-03-31 19:54:00 PDT

add myself to CC

Laszlo Gombos

Comment 3 2011-04-01 01:41:16 PDT

Comment on attachment 87820 [details] Patch r=me. Please fix the typo in the title (uClbc) before committing.

Keith Kyzivat

Comment 4 2011-04-01 08:42:27 PDT

Comment on attachment 87820 [details] Patch Title fixed.

WebKit Review Bot

Comment 5 2011-04-01 08:42:41 PDT

Comment on attachment 87820 [details] Patch Rejecting attachment 87820 [details] from commit-queue. keith.kyzivat@nokia.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py. - If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags. - If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed). The commit-queue restarts itself every 2 hours. After restart the commit-queue will correctly respect your committer rights.

Benjamin Poulain

Comment 6 2011-04-01 09:15:23 PDT

Comment on attachment 87820 [details] Patch WTF

Benjamin Poulain

Comment 7 2011-04-01 09:15:59 PDT

Why would you cq+ this? The title has not been fixed in the patch.

Keith Kyzivat

Comment 8 2011-04-01 11:27:46 PDT

Doh, my fault -- I didn't realize (or closely check) that the typo was in the commit message... Fixing shortly.

Keith Kyzivat

Comment 9 2011-04-01 11:38:04 PDT

Created attachment 87887 [details] Patch

WebKit Commit Bot

Comment 10 2011-04-04 10:30:14 PDT

Comment on attachment 87887 [details] Patch Clearing flags on attachment: 87887 Committed r82844: <http://trac.webkit.org/changeset/82844>

WebKit Commit Bot

Comment 11 2011-04-04 10:30:20 PDT

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.