<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/101853/">http://git.reviewboard.kde.org/r/101853/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On July 6th, 2011, 9:41 a.m., <b>Milian Wolff</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="http://git.reviewboard.kde.org/r/101853/diff/1/?file=26108#file26108line329" style="color: black; font-weight: bold; text-decoration: underline;">plugins/appwizard/appwizardplugin.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">QString AppWizardPlugin::createProject(const ApplicationInfo& info)</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">329</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">QDirIterator</span> <span class="n">it</span><span class="p">(</span><span class="n">QDir</span><span class="o">::</span><span class="n">cleanPath</span><span class="p">(</span> <span class="n">dest</span><span class="p">.</span><span class="n">toLocalFile</span><span class="p">()),</span> <span class="n">QStringList</span><span class="p">()</span> <span class="o"><<</span> <span class="s">"*.kdev4"</span><span class="p">,</span> <span class="n">QDir</span><span class="o">::</span><span class="n">NoFilter</span><span class="p">,</span> <span class="n">QDirIterator</span><span class="o">::</span><span class="n">Subdirectories</span><span class="p">);</span></pre></td>
</tr>
</tbody>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">please add a comment here and explain why this is is required for your case, i.e. copy'n'paste from your description above</pre>
</blockquote>
<p>On July 6th, 2011, 11:40 a.m., <b>Bastian Schmidt</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Loops through the new project directory and tries to detect the first .kdev4 file.
If one is found this file will be used. So .kdev4 file can be stored in any subdirectory and the
project templates can be more complex.</pre>
</blockquote>
<p>On July 6th, 2011, 11:42 a.m., <b>Milian Wolff</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">hehe yes that but I mean you should add it inside the code ;-)</pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">lol,
sorry I'm new at this review board. Now I have clicked the appwizardplugin.cpp link and added the comment.
Was this correct?</pre>
<br />
<p>- Bastian</p>
<br />
<p>On July 5th, 2011, 6:58 a.m., Bastian Schmidt wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDevelop.</div>
<div>By Bastian Schmidt.</div>
<p style="color: grey;"><i>Updated July 5, 2011, 6:58 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hi,
we are using project layouts like this:
project/
- branches/
- 0.0.1/
- bla.kdev4
- src/
- ...
- trunk/ <- development version
- bla.kdev4
- src/
- ...
As I created an application template for this layout, I recognized that
kdevelop expect the .kdev4 file in the project root directory. For new
projects, with this layout, the root directory is project/trunk/. So I need the
possibility to store the template .kdev4 file in the trunk directory.
I apply this feature to kdevplatform/plugins/appwizard/appwizardplugin.cpp. The
attached patch adds 6 lines to appwizardplugin.cpp.
Patch Description:
The patch loops through the new project directory and tries to detect the first
.kdev4 file. If one is found this file will be used. That's all.
So I can store a template .kdev4 file in any subdirectory.
Bastian</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=276744">276744</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>plugins/appwizard/appwizardplugin.cpp <span style="color: grey">(c3ac864)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101853/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>