Review Request 129647: Add --nofork as compatibility alias for --separate

Martin Tobias Holmedahl Sandsmark martin.sandsmark at kde.org
Sat Dec 17 12:08:27 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129647/#review101479
-----------------------------------------------------------



looks good to me.

but reminds me that I don't like string literals being hardcoded in different places.

- Martin Tobias Holmedahl Sandsmark


On Dec. 13, 2016, 8:21 a.m., Harald Sitter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129647/
> -----------------------------------------------------------
> 
> (Updated Dec. 13, 2016, 8:21 a.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Repository: konsole
> 
> 
> Description
> -------
> 
> In previous incarnations of kuniqueapplication it used to inject a common
> command option --nofork which is meant to bypass single-instance behavior.
> Given that konsole can and is being invoked from scripts they may well want
> to ensure that the fork they created is the actual instance of konsole.
> i.e. to monitor return values and life time
> 
> Presently, since the options are divergent between konsole4 and konsole5,
> scripts are either incompatible with older konsoles or with newer konsoles.
> To make life easier for everyone add an compat alias --nofork, which
> behaves exactly like separate.
> 
> (this unbreaks steam, which is a notable recent offender of falling into
>  this particular trap)
> 
> 
> Diffs
> -----
> 
>   src/Application.cpp 5b352ec9b10899b44abbd6ee1609422ef1434620 
>   src/main.cpp 26d3da990b6aee564c0c9a237a56a65b1d372508 
> 
> Diff: https://git.reviewboard.kde.org/r/129647/diff/
> 
> 
> Testing
> -------
> 
> builds + noforks + separates
> 
> poor steam also is happily opening a konsole now
> 
> 
> Thanks,
> 
> Harald Sitter
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20161217/c9ede2a7/attachment.html>


More information about the konsole-devel mailing list