Hi Daniel,<br><br>Welcome back, responding inline again.<br><br><div class="gmail_quote">On Wed, Aug 31, 2011 at 3:25 PM, Daniel Teske <span dir="ltr"><<a href="mailto:daniel.teske@nokia.com">daniel.teske@nokia.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi,<br>
<br>
I'm back from vacation and had another look at the diff between master and 2.3-<br>
staging.[1] For that I merged master into 2.3-staging, since a few changes<br>
that I did in master aren't yet in the 2.3-staging branch.<br>
<br>
There are currently 5 merge conflicts, none particular complicated to figure<br>
out. Unfourtanetly that didn't compile, due to a header cleanup patch by<br>
Friedemann. For reference, the 0001-Compile-fixes is what was required to make<br>
it compile again.<br>
Also the exporting FolderNavigationWidget is no longer needed, since the<br>
showInGraphicalShell method is moved in master, see 0001-Compile-fixes<br>
<br>
Also I have prepared two patches for the android branch:<br>
- 0002-Remove-no-longer-used-ressource.h<br>
The file is a left-over from the fix Ray did to the qtcreator.rc file. Since it's<br>
now fixed in a different way, the file is no longer needed.<br>
<br></blockquote><div>Great.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
- 0003-Code-stylistics-to-RemoteGdbServerAdapter-changes.patch<br>
I have adjusted the code a little bit to fit the creator code sytle a little<br>
better. That is using !isEmpty() instead of size() and adding spaces around<br>
operators. Andre thinks the patch is now fine.<br>
<br></blockquote><div>Ditto.<br> <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Also<br>
> n) lowerCasing of paths in qt4buildconfiguration.cpp<br>
That turns out to be somewhat more involved, I still have it on my todo list.<br>
<div class="im"><br></div></blockquote><div>Ok.<br> <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">
> j) bin/necessitas and bin/necessitas.bat<br>
</div>Ossi reintroduced the LD_LIBRARY_SCRIPT, I still don't really understand the<br>
topic. <br></blockquote><div><br>Ossi? Do you mean Oswald? Is he on holiday at the minute? I've got some MRs for Qt 4.8 that Oswald had been looking at that've been inactive for a week now, once these are merged, I've got a few more to send... BogDan said this was so that the local Qt libs are used instead of any others. necessitas.bat can be killed as Windows looks in the same path as the .exe for dlls.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">> s) main.cpp #ifdef<br>
> There is a change in main.cpp changing the plugin search paths to be only<br>
> searched on the "matching" platform.<br>
</div>I cherry-picked that change to master.<br>
<br>
> p) debugger plugin<br>
<div class="im">> Disabling setting PYTHONPATH. Why is that needed? Note that the code that<br>
> is disabled was rewritten a lot since you disabled it.<br>
</div>We removed the same code that was #ifdefed out in master now, so on merging,<br>
just remove the code.<br>
<br>
So going over my intial todo list, I still have a few open questions. As far<br>
as I can tell they are all for Ray. :)<br>
<div class="im"><br>
> m) qmakeStep::moreArguments()<br>
> Adding the "-win32" to the qmake command line. As far as I understand you<br>
> want to have qmake run in HOST_WIN_MODE on windows. But that is the<br>
> default if qmake is compiled for windows.<br></div></blockquote><div><br>I'll look into this. I think it's because I compile our qmake for MinGW on an MSYS environment so HOST_WIN_MODE is probably not defined.<br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">
<br>
</div>> p) debugger plugin<br>
<div class="im">> Always offering all toolchains in DebuggerToolChainComboBox::init<br>
> The commit message already classifies that as a hack, I just don't<br>
> understand why it was needed. The check for hostAbi.os() == abi.os()<br>
> should be true for msys toolchains too. Or is .os() broken for msys<br>
> toolchains?<br>
<br>
</div>and<br>
<div class="im">> t) main.cpp myMessageOutput<br>
> The main motivation for that seems to be:<br>
> "qInstallMsgHandler so that asserts are not fatal"<br>
> We like Q_ASSERTS to be fatal, we do have a separate macro QTC_ASSERT for<br>
> cases where a non fatal handling is possible/desired.<br></div></blockquote><div><br>I was hitting various Q_ASSERTS for things like widgets not being created in the main thread and I just wanted them ignored so this was my work around. Obviously these asserts are coming from Qt, so QTC_ASSERT can't be used for this. Hopefully all these instances are now fixed, but I'll need to check that too.<br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">
<br>
</div>Ray, any comments on them would be nice.<br>
<br></blockquote><div>I'm currently very busy at work but I'll try to make it a priority to investigate these remaining few issues. <br><br>Cheers,<br><br>Ray.<br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<br>
daniel<br>
<br>
[1] That seemed to be the most recent branch<br>
<br>_______________________________________________<br>
Necessitas-devel mailing list<br>
<a href="mailto:Necessitas-devel@kde.org">Necessitas-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/necessitas-devel" target="_blank">https://mail.kde.org/mailman/listinfo/necessitas-devel</a><br>
<br></blockquote></div><br>