Thomas,<br><br>Patch committed.<br><br><div class="gmail_quote">2009/7/16 Thomas Martitz <span dir="ltr"><<a href="mailto:thomas.martitz@student.htw-berlin.de">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 class="im"><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I started working on the working dir. I have already implemeted the entry boxes, saving and loading it. I'm just mostly finished "%d".<br>
<br>
I'm thinking the basedir checkbox could be removed at the very end?<br>
<br>
Best regards.<br>
</blockquote>
<br></div>
Alright, I've been coding the past few hours and this is what I got so far.<br>
<br>
It works pretty much as I expected. You fill out working dir, then it would 'cd' into that before executing the command. I'm not aware of big any issue, except that there are no default values yet (I'm not sure if they are needed if the default values would be just %d anyway).</blockquote>
<div><br>Yeah, just leave them NULL.<br>One area that needs work is reading old config file [build-settings] to put %p/%d in the directory. Done & improved as well :-)<br><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>
%d and %p also work. I really think the basedir checkbox (and responsible code) could be removed.</blockquote><div><br>Agreed, done<br><br>Couple of notes, <br><br>build_replace_placeholder requires that a document is open, but non-filetype commands don't require a document, obviously you can't substitute %d in that case & should give error like %p if no project,  but you can still substitute %p.  <br>
Use case for make without document, grab a new working directory from VC with included project.geany file, open project and do first build, no documents need be open.  <br>I ran out of time to do it by the time I found it.<br>
<br>Changed the dialog comment to just list the substitutes and refer to manual, it was too long & the dialog is now pretty big (IMHO the comment I had was always too long, its not just the latest additions)<br><br>Dialog fixed regular expression field span & use symbols for column counts.<br>
<br>Fixed stcmp to original code, the strlen is required because a NULL must compare equal to an empty string, that was really the point of stcmp.  Otherwise extraneous entries are written to config files hiding things that should still be visible (took hours to find out why ;-).<br>
<br>In general it seems to be working :-D<br><br>Cheers<br>Lex<br> <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>
Best regards.<br>
<br>_______________________________________________<br>
Geany-devel mailing list<br>
<a href="mailto:Geany-devel@uvena.de">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>