[Konsole-devel] Review Request: Patch for command-line option to read tab titles and tab commands from file

[lists at toell.net]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1079/#review1696
-----------------------------------------------------------


Why don't use a XML based file? 
This would be probably more useful and expandable for future use. For now just a few tags from a future DTD/Schema would be used, but this may change in future. We could even share a xml-based file through konsole and other applications, using konsole's remote interface through DBus. 

- arno


On 2009-07-21 09:55:49, Thomas Dreibholz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1079/
> -----------------------------------------------------------
> 
> (Updated 2009-07-21 09:55:49)
> 
> 
> Review request for Konsole.
> 
> 
> Summary
> -------
> 
> The attached patch introduces the new command line option "-tabsfromfile <filename>" to konsole. On startup, konsole reads the file which contains one line for each tab to be created. Syntax is "<tab title>: <command> <args> ..."; comment lines start with '#'. This new option can e.g. be used to conveniently open ssh connections to a pool of PCs; with one tab for each PC. Such a configuration file could look like this:
> # SSH sessions to all Network Testbed PCs
> Control: ssh -CX user at control.my-domain.org
> West:    ssh -CX user at west.my-domain.org
> North:   ssh -CX user at north.my-domain.org
> South:   ssh -CX user at south.my-domain.org
> East:    ssh -CX user at east.my-domain.org
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/apps/konsole/src/Application.h 999292 
>   /trunk/KDE/kdebase/apps/konsole/src/Application.cpp 999292 
>   /trunk/KDE/kdebase/apps/konsole/src/main.cpp 999292 
> 
> Diff: http://reviewboard.kde.org/r/1079/diff
> 
> 
> Testing
> -------
> 
> Tested with SVN revision 999263.
> 
> 
> Thanks,
> 
> Thomas
> 
>