Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Script fails to run when spawning lxterminal due to invalid arguments #134

Open
khalim19 opened this issue Jun 1, 2018 · 3 comments
Open

Comments

@khalim19
Copy link

khalim19 commented Jun 1, 2018

On Lubuntu, when executing the makeself-created script in the file explorer GUI, a dialog prompts the user to execute the script using either the option "Execute" or "Execute in Terminal". The latter works fine as the desktop environment (LXDE) seems to spawn a terminal instance on its own, but the former fails to run because the script tries to run the lxterminal command with -title and --xwin arguments which do not exist in lxterminal.

There may be more terminal commands which may not accept the two aforementioned arguments (e.g. gnome-terminal or xfce4-terminal accept --title instead of -title according to their man pages).

@megastep
Copy link
Owner

megastep commented Jun 1, 2018

Yeah, the problem is that right now Makeself searches for various known commands for a terminal, and assumes each of these take the same basic arguments. So it looks like it fails in some cases, and we should probably handle this case better.

@00sapo
Copy link

00sapo commented Aug 13, 2018

Not sure that this is linked:

when you try to run a makeself zip as subprocess, it breaks with

"# Failed to parse arguments: Unknown option -title"

Tested with Python 3.6 and QML.

Reproducible steps:

  • create a makeself zip running something (e.g. a shell Hello World)
  • open python terminal and run:
import os
print(os.popen("./zipapp --").read())

Similar for QML or other languages.

@psy0rz
Copy link
Contributor

psy0rz commented Nov 8, 2019

I can confirm this is also a problem with gnome-terminal, -title is not supported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants