search

LeagueOfPoro / EsportsCapsuleFarmer

Automatically watch all live matches on lolesports and farm Esports Capsules
https://www.youtube.com/c/LeagueOfPoro
Other
95 stars 23 forks source link

Typo in README.md #10

Closed Tj717 closed 2 years ago

Tj717 commented 2 years ago

Unless I'm mistaken, in Installation (advanced), it should be ./main.py instead of .\main.py right?

alepouna commented 2 years ago

The \ for file paths is used in Windows (but they support both) image Only Linux (and mac probably) will probably care.

Good habit to use / though.

Tj717 commented 2 years ago

My problem was that when I ran the two commands as is, they gave the error: [path for my python.exe]: can't open file 'D:\\EsportsCapsuleFarmer\\.main.py': [Errno 2] No such file or directory and 

605 INFO: PyInstaller: 5.1
606 INFO: Python: 3.10.1
621 INFO: Platform: Windows-10-10.0.19044-SP0
622 INFO: wrote D:\EsportsCapsuleFarmer\.main.spec
628 INFO: UPX is not available.
script 'D:\EsportsCapsuleFarmer\.main.py' not found

respectively. When I changed the \ to / they were able to execute and returned no errors and I don't even know why. (I'm on Windows)

LeagueOfPoro commented 2 years ago

D:\EsportsCapsuleFarmer.main.py

The dot is your problem. the correct path would be D:\EsportsCapsuleFarmer\main.py or even D:\EsportsCapsuleFarmer\.\main.py should work.