Hi Thomas,<br><br>Thanks for looking at it and working on improvements already.<br><br>(below copied from your previous email so I can reply to both at once)<br><br>

> Great work! That's a thing I'm really missing for Geany.<br>
<br>
> I'm digging in a bit. The dialog looks good already, but I think it lacks a important field: <br>> Working directory for the command.<br><br>To run a command in a specific directory just make the command field "cd <the directory>; <the rest of the command>"<br>

<br>That seems to be what your code is trying to do through a field for the directory, if I have misunderstood please explain.<br><br>If that is the case then I don't feel there is a need for another field in the dialog or any express support in the code, anyway for now my concern is getting the basic functionality working reliably and getting it used.<br>


<br>
> It should be obvious, that for many projects, the source files are
structured,<br>> so that having the working dir default to the dir where the
source file is located doesn't work. <br>> And the base directory of a
project also doesn't always work.<br><br>True, this is just replicating the current way Geany performs so that it can load old config files.<br>
<br>
> I'm looking at the code in the hope I can add it. It's a bit hard for
me to read the code, <br>> it lacks comments and structure here and there,
but I hope I'll find my way.<br><br>Yeah, thats the way of code thats evolved  from an experiment through several iterations (15 according to my local version control)  I intend to improve things when its all working.<br>


_<br><br><div class="gmail_quote">2009/7/12 Thomas Martitz <span dir="ltr"><<a href="mailto:thomas.martitz@student.htw-berlin.de" target="_blank">thomas.martitz@student.htw-berlin.de</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Thomas Martitz schrieb:<div><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
I'm looking at the code in the hope I can add it. It's a bit hard for me to read the code, it lacks comments and structure here and there, but I hope I'll find my way. <br>
</blockquote>
<br>
<br></div>
Alright, I've got to the point where I can add it.<br>
<br>
BUT I needed to rewrite much parts (such as doing separate (even if it's the same) handling of GeanyBuildCommand.label and .command in a loop instead). The code duplication and hardcodedness was so irritating that I couldn't get it to work without.</blockquote>

<div><br>I don't understand why it needs to be a loop like that, just add another field to GeanyBuildCommand and use it where you need to.  Much easier to understand for simple folk like me ;-)  Then all other places addressing label and command don't need to change.<br>

</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
<br>
I attached the patch. In case it's accepted, I can continue to work in really integrating the working dir field (I didn't add it yet). Adding the working dir entry field is mostly adding a item to the enum GeanyBuildCmdEntries now.</blockquote>

<div><br>I tried to apply your patch to my working copy but 25 out of 27 chunks were rejected, thats the problem with patches on a rapidly changing source.  So the patch can't be used directly, and for now I can't spare the time to work out the problems since there are still important bits of functionality to fix/add, but if the method of setting directory suggested above isn't sufficient let me know.<br>

<br>Cheers<br>Lex<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
<br>
<br>
Best regards.<br>
<br>_______________________________________________<br>
Geany-devel mailing list<br>
<a href="mailto:Geany-devel@uvena.de" target="_blank">Geany-devel@uvena.de</a><br>
<a href="http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel" target="_blank">http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel</a><br>
<br></blockquote></div><br>