<div>Heyo, interesting topic!</div><div><br></div>In my opinion all QT_NO_* defines should be dropped from WebKit. I believe QtWebKit is sufficiently complex that you shouldn't use it without a fully-featured Qt.<div><br>

<div>For every additional build-time flag, the complexity matrix grows larger, and we're not testing any of it beyond building with all those flags enabled together. If we do need to support that, I'd rather we had a monolithic USE(LESS_QTWEBKIT) flag rather than separate ones.</div>

<div><br></div><div>(inb4 embedded people yelling at me)</div><div><br></div><div>Hugs,</div><div>Kling</div><div><br></div><div><br><div class="gmail_quote">On Tue, Sep 27, 2011 at 3:52 PM, Osztrogonac Csaba <span dir="ltr"><<a href="mailto:oszi@inf.u-szeged.hu" target="_blank">oszi@inf.u-szeged.hu</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi QtWebKit developers,<br>
<br>
The "Qt Linux Release minimal" buildbot on <a href="http://build.webkit.org" target="_blank">http://build.webkit.org</a> is<br>
our ifdef guard protector buildbot. It builds WebKit with this command:<br>
Tools/Scripts/build-webkit --release --qt --minimal CONFIG+=qt_minimal<br>
( details: <a href="http://build.webkit.org/buildslaves/szeged-linux-2" target="_blank">http://build.webkit.org/<u></u>buildslaves/szeged-linux-2</a> )<br>
<br>
My question is if QtWebKit community still need supporting QT_NO_* guards in WebKit<br>
trunk or not. If yes, which ones are still necessary and which ones can we drop?<br>
<br>
br,<br>
Ossy<br>
<br>
<br>
Minimal Qt means that Qt was built the following QT_NO_* defines:<br>
------------------------------<u></u>------------------------------<u></u>------<br>
#define QT_NO_ANIMATION<br>
#define QT_NO_BEARERMANAGEMENT<br>
#define QT_NO_CLIPBOARD<br>
#define QT_NO_COMBOBOX<br>
#define QT_NO_CONCURRENT<br>
#define QT_NO_CRASHHANDLER<br>
#define QT_NO_CURSOR<br>
#define QT_NO_DESKTOPSERVICES<br>
#define QT_NO_FILEDIALOG<br>
#define QT_NO_GRAPHICSEFFECT<br>
#define QT_NO_IM<br>
#define QT_NO_INPUTDIALOG<br>
#define QT_NO_LINEEDIT<br>
#define QT_NO_MESSAGEBOX<br>
#define QT_NO_OPENSSL<br>
#define QT_NO_PRINTER<br>
#define QT_NO_QUUID_STRING<br>
#define QT_NO_SHORTCUT<br>
#define QT_NO_STYLE_STYLESHEET<br>
#define QT_NO_SYSTEMTRAYICON<br>
#define QT_NO_TEMPORARYFILE<br>
#define QT_NO_TOOLTIP<br>
#define QT_NO_UITOOLS<br>
#define QT_NO_UNDOCOMMAND<br>
#define QT_NO_UNDOSTACK<br>
#define QT_NO_XRENDER<br>
<br>
And the configure options:<br>
---------------------------<br>
-qconfig webkit-minimal<br>
-platform linux-g++-32<br>
-embedded<br>
-nomake demos<br>
-nomake docs<br>
-nomake examples<br>
-no-webkit<br>
-no-openssl<br>
-no-declarative<br>
-no-scripttools<br>
-opensource<br>
______________________________<u></u>_________________<br>
webkit-qt mailing list<br>
<a href="mailto:webkit-qt@lists.webkit.org" target="_blank">webkit-qt@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-qt" target="_blank">http://lists.webkit.org/<u></u>mailman/listinfo.cgi/webkit-qt</a><br>
</blockquote></div><br></div></div>