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

Harald Sitter sitter at kde.org
Mon Dec 19 08:09:59 UTC 2016


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

(Updated Dec. 19, 2016, 8:09 a.m.)


Status
------

This change has been marked as submitted.


Review request for Konsole.


Changes
-------

Submitted with commit c6ffcaff557760f46ba5813823868e656b45da0f by Harald Sitter to branch Applications/16.12.


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/20161219/c7428243/attachment.html>


More information about the konsole-devel mailing list