flathub / com.valvesoftware.Steam

https://flathub.org/apps/details/com.valvesoftware.Steam
337 stars 70 forks source link

steamcmd_wrapper still broken #1132

Closed lucius-martius closed 8 months ago

lucius-martius commented 11 months ago

I had the same issue in #1028 and just read that it was fixed in #1096, but I'm now getting a different error:

$ flatpak run --command=steamcmd com.valvesoftware.Steam +login foo bar
Traceback (most recent call last):
  File "/app/bin/steamcmd", line 8, in <module>
    sys.exit(main())
  File "/app/lib/python3.10/site-packages/steamcmd_wrapper.py", line 21, in main
    file_ignorer = FileIgnorer(STEAMCMD_BOOTSTRAP_DIR,
NameError: name 'FileIgnorer' is not defined. Did you mean: 'file_ignorer'?

OS: Gentoo Linux Flatpak version: 1.14.4 Steam Version: 1.0.0.78

rabidlogic commented 10 months ago

I get a slightly different error, but same general broken.

(deck@steamdeck site-packages)$ flatpak run --user --command=steamcmd com.valvesoftware.Steam//beta +login test
Traceback (most recent call last):
  File "/app/bin/steamcmd", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/app/lib/python3.11/site-packages/steamcmd_wrapper.py", line 24, in main
    shutil.copytree.copytree(
    ^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'function' object has no attribute 'copytree'

OS: SteamOS Flatpak Version: 1.12.4 Steam Version: 1.0.0.78-beta

doraskayo commented 10 months ago

Can you check again?

rabidlogic commented 10 months ago

Unfortunately, I'm getting the same error.

This is on a completely fresh install.

doraskayo commented 10 months ago

@nanonyme, FYI

rabidlogic commented 10 months ago

@doraskayo I forgot to mention that I tried both beta and main. I could not get beta to install because the Platform dependency wasn't available. I'm still getting the same error on main.

lucius-martius commented 10 months ago

The FileIgnorer error seems gone now but I still get the copytree error @rabidlogic was getting.

rabidlogic commented 8 months ago

@lucius-martius I think I got this resolved in #1186 - Mind testing that build out?

lucius-martius commented 8 months ago

Looks good! For me it works without issue now. 👍