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

openApp() stucks on Linux after 2.0.5 upgrade #438

Closed
Alexious-sh opened this issue Mar 23, 2021 · 5 comments
Closed

openApp() stucks on Linux after 2.0.5 upgrade #438

Alexious-sh opened this issue Mar 23, 2021 · 5 comments
Assignees
Labels
aApp/Window problems with class App bug possible might be a bug - is currently evaluated fixed the problem has been fixed
Milestone

Comments

@Alexious-sh
Copy link

Hi!
Version 2.0.5 is unable to start any app, which is blocking the shell. E.g.:

openApp("vlc")
print("Done!")

It stuck on the first line until you manually close the app. Print will be fired only after that. I guess, it completely breaks app running behavior.

Found related issue #366 - looks like the change there causes the issue. That's strange because I can run such a command without blocking the terminal:

sh -c "vlc" &

PS: I'm running Ubuntu 20.04 if it makes sense.

@RaiMan
Copy link
Owner

RaiMan commented Mar 24, 2021

Could you please try, wether

run('sh -c "vlc" &')

does the job?

@Miksser
Copy link

Miksser commented Mar 24, 2021

Hi. I tried use simple code

run('sh -c "vlc"') # with & i tried too 
topHalfScreen = Region(0,0,1200,800)
topHalfScreen.wait("1.png").click()

but the process hangs forever

@Alexious-sh
Copy link
Author

Yep. The same for me. The process just hangs until I close the app.

@RaiMan
Copy link
Owner

RaiMan commented Mar 24, 2021

Ok, thanks.

I will to get an Ubuntu 20 to running the next days.

So I can test and evaluate myself.

@RaiMan RaiMan self-assigned this Mar 24, 2021
@RaiMan RaiMan added the bug possible might be a bug - is currently evaluated label Mar 24, 2021
@RaiMan RaiMan added this to the 2.0.5 milestone Mar 24, 2021
@balmma
Copy link
Contributor

balmma commented Apr 17, 2021

This is most probably all fixed with PR #418 where all this argument splitting is done properly and the sh wrapper is removed entirely. Probably worth porting this back to the 2.0.x branch or pushing 2.1 further?

@RaiMan RaiMan modified the milestones: 2.0.5, 2.0.6 Jul 24, 2021
@RaiMan RaiMan added the aApp/Window problems with class App label Jan 23, 2022
@RaiMan RaiMan added the fixed the problem has been fixed label Apr 10, 2022
@RaiMan RaiMan closed this as completed Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aApp/Window problems with class App bug possible might be a bug - is currently evaluated fixed the problem has been fixed
Projects
None yet
Development

No branches or pull requests

4 participants