SmartFinn / eve-ng-integration

integrates EVE-NG (aka UNetLab) with Linux desktop
http://git.io/eve-ng-integration
MIT License
322 stars 117 forks source link

Fix issue when command is one element in list (lxterminal affected). #10

Closed k0ste closed 7 years ago

k0ste commented 7 years ago

Hello. One arg with 'command' not work with lxterminal, open lxtermianal without telnet session.

Not work:

['lxterminal', '-e', 'telnet 192.168.0.214 32769']

Work as expected:

['lxterminal', '-e', 'telnet', '192.168.0.214', '32769']
[k0ste@WorkStation ~]$ lxterminal --version
lxterminal 0.3.0
[k0ste@WorkStation ~]$ 

This patch make elements separated, should not be a regression with another terminal emulators. But you should check it. Thanks.

SmartFinn commented 7 years ago

Hello.

This patch conflicts with #8. Also, it doesn't resolve the issue on Debian-based systems where used x-terminal-emulator as a default terminal.