[Konsole-devel] Review Request 128299: add a debian compat option

[Kurt Hindenburg]

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



Someone from Debian had asked about providing a new patch which I did for them.  However, I've been reluctant to add distro specific code in Konsole.

- Kurt Hindenburg


On June 28, 2016, 8:14 a.m., Harald Sitter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128299/
> -----------------------------------------------------------
> 
> (Updated June 28, 2016, 8:14 a.m.)
> 
> 
> Review request for Konsole and Kurt Hindenburg.
> 
> 
> Repository: konsole
> 
> 
> Description
> -------
> 
> debian policy for reasons beyond my apprehension wants a -T option to
> set the window title (like in xterm).
> to make debian's (and derivate's) life easier let's support a no-op
> -T option but hide it. this option will not appear in --help output
> but be accepted by the parser all the same.
> 
> this enables Debian and friends to drop a long standing useless patch
> and at the same time prevents us from introducing a conflicting -T in
> the future.
> 
> 
> Diffs
> -----
> 
>   src/main.cpp d8b4b2964471a8edc1b711bbc8220624d33690bb 
> 
> Diff: https://git.reviewboard.kde.org/r/128299/diff/
> 
> 
> Testing
> -------
> 
> - builds
> - not listed in --help
> - konsole -T yolo works
> 
> 
> Thanks,
> 
> Harald Sitter
> 
>

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