techee/geany-patches — comment on commit 7951b695f797c2fbda5db02506e0b9125c9d42ac
File:  line 0
By techee:

OK, I see - forget about this patch. However, in order to make the dialog a bit clearer, I'd suggest some more changes:

* rename "Independent commands" to "Filetype independent commands" (for someone not familiar with the build system it's not clear independent of what the commands are)

* add filetype to execute commands. As I understand it, execute commands are per-filetype and the current label makes impression that they are filetype-independent. So you'd get labels like "C execute commands".

* is the documentation in the "Build menu configuration" section in the table at the "System filetype" x "Filetype independent" position? It says it gets the configuration from filetypes.xxx but filetype-independent entries shouldn't be in the filetypes.xxx files (I haven't checked the sources for this - it's just how I understand it)

* I remember in the past we were talking about replacing the hardcoded dialog for "make custom target" with something like %c{enter custom text here} inside the command string. This would eliminate a bit the special cases in the dialog. Is something like that wanted? I could have a look at that.