Using CMD commands with an executable python script

Question

After writing the script below(which works perfectly), I open the cmd.exe windows prompt and type the following

pyinstaller -F --windowed myscript.py

which gives me a file called "myscript.exe".

The problem is when I open the executable and press the button, nothing happens. I think there is a problem with this line:

check_output("shutdown -s -t 60", shell=True)

Even though the script works "as a script", it doesn't work as an executable.
I've tried other syntax like

os.system("shutdown -s -t 60")

but they don't seem to work.

from tkinter import *
from subprocess import check_output,CalledProcessError

class main_gui:
    def __init__(self,master):
        self.master=master
        master.geometry("250x100")
        self.button1=Button(self.master,
                            text="Press me",
                            font="Times 10 bold",
                            command=self.shutdown)
        self.button1.pack()

    def shutdown(self):
        try:
            check_output("shutdown -s -t 60", shell=True)
            print("Computer will shutdown in 60 seconds")
        except CalledProcessError:
            print("Already pressed")

root = Tk()
my_gui = main_gui(root)
root.mainloop()

What can I do?

Fabian · Accepted Answer · 2019-01-03 14:41:30Z

2

What you can do:

using:

import subprocess
subprocess.call(["shutdown", "-f", "-s", "-t", "60"])

this will work. with --windowed

It seems thats there is a problem with check_output with --windowed flag :/

Edit1:

based on eryksun comments. Also was my research results was but now it seems proof.

Use check_call and a creation flag to avoid creating a console window. For example: CREATE_NO_WINDOW = 0x08000000; check_call('shutdown -s -t 60', creationflags=CREATE_NO_WINDOW).

Regarding check_output, since it overrides stdout, Popen has to also duplicate inheritable copies of the existing stdin and stderr handles. This will fail if they're invalid. When run from a console in Windows 7, a GUI can inherit invalid standard handles. A workaround is to override all 3 handles. For example: output = check_output('shutdown -s -t 60', stdin=subprocess.DEVNULL, stderr=subprocess.DEVNULL, creationflags=CREATE_NO_WINDOW

Edit2:

You can also add a icon directly with pyinstaller...
Reference: Windows and Mac OS X specific options

edited Jan 3, 2019 at 14:41

answered Jan 3, 2019 at 1:34

Fabian

1,15011 silver badges26 bronze badges

Sign up to request clarification or add additional context in comments.

7 Comments

Eryk Sun Over a year ago

Use check_call and a creation flag to avoid creating a console window. For example: CREATE_NO_WINDOW = 0x08000000; check_call('shutdown -s -t 60', creationflags=CREATE_NO_WINDOW).

Eryk Sun Over a year ago

Regarding check_output, since it overrides stdout, Popen has to also duplicate inheritable copies of the existing stdin and stderr handles. This will fail if they're invalid. When run from a console in Windows 7, a GUI can inherit invalid standard handles. A workaround is to override all 3 handles. For example: output = check_output('shutdown -s -t 60', stdin=subprocess.DEVNULL, stderr=subprocess.DEVNULL, creationflags=CREATE_NO_WINDOW).

usersina Over a year ago

Using the above method worked on the mentioned script but when I tried it with a longer one, the button somehow opened another instance of the application.

usersina Over a year ago

EDIT: Sometimes it works (the longer script), sometimes it doesn't. Sometimes the button opens another instance of the program without executing the command. I created two executables and then put them in the same directory, none of them works, when I deleted one of them, the other magically started working.

Eryk Sun Over a year ago

@SinaUser, we don't have enough information. Use logging to write errors to a file, instead of depending on printing information to stdout and exceptions to stderr. They aren't available by default in a GUI process.

|

Collectives™ on Stack Overflow

Using CMD commands with an executable python script

1 Answer 1

7 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

7 Comments

Your Answer

Sign up or log in

Post as a guest

Related