[Github-comments] Re: [geany/geany] [Make Custom Target] Always put a space between command and argument (PR #3825)

2024-04-12 Thread Nick Treleaven via Github-comments
At least with #3833 it'll be clearer what command gets run.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3825#issuecomment-2052131066
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] [Make Custom Target] Always put a space between command and argument (PR #3825)

2024-04-12 Thread Nick Treleaven via Github-comments
Closed #3825.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3825#event-12452019970
You are receiving this because you are subscribed to this thread.

Message ID: 

[Github-comments] Re: [geany/geany] [Make Custom Target] Always put a space between command and argument (PR #3825)

2024-04-08 Thread elextr via Github-comments
@elextr requested changes on this pull request.

The space is currently only added to pre-0.19 formats, so adding it to all 
formats is an incompatible change.  It assumes that the user is typing a full 
argument as they are required to do pre-0.19 but post 0.19 formats allow them 
to only have to type part of the argument.

Currently if the user configured "my_build_script f" in the GUI for custom 
command and typed "oo" the command is "my_build_script foo" but this change 
will make it "my_build_script f oo" with no way to recover.



-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3825#pullrequestreview-1987816591
You are receiving this because you are subscribed to this thread.

Message ID: