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

Any chance of Linux support? #43

Open
EmeraldSnorlax opened this issue Jul 20, 2019 · 10 comments · May be fixed by #216
Open

Any chance of Linux support? #43

EmeraldSnorlax opened this issue Jul 20, 2019 · 10 comments · May be fixed by #216
Labels
help wanted Extra attention is needed

Comments

@EmeraldSnorlax
Copy link

Seeing as its all python it should be relatively easy to port -- trying to run build.py or main.py just crashes on me -- perhaps because of using \ for paths instead of / ?
Output of tests.py is here

@Kataiser
Copy link
Owner

Kataiser commented Jul 20, 2019

Yeah, tests and build use backslashes all over the place, but the bulk of the main code should be close to compatible with Linux/MacOS (using os.path functions). As I don't have either OS, I haven't tested them for real. Also note that most of the code has been made specifically for the built releases, which means that I have little to no idea how well it works as it's organized in the repo.

To answer the main question, yes, I would like Linux support at some point. While you're at it, may I see the crash for main.py on Linux?

@EmeraldSnorlax
Copy link
Author

Sure! Here: https://pastebin.com/c394VmL0

@Kataiser
Copy link
Owner

Ah yes, forward slashes and backslashes in the same path lol

@EmeraldSnorlax
Copy link
Author

EmeraldSnorlax commented Jul 21, 2019

Going down the line and fixing the errors it complains about eventually leaves me off here after running main.py: https://pastebin.com/Xmvwdez7

(ps by fix i mean using a hacky solution and just replacing the \ with a / :^), as well as other file related errors )

It starts, but doesn't want to do much else

Crash log

unrelated but found your bandcamp -- holy frick your music is amazing

@Kataiser
Copy link
Owner

tasklist is a Windows only command apparently. Honestly, getting the file stuff working is probably the tip of the iceberg considering how much process scanning is going on in this project. At least psutil is cross platform...

@Kataiser
Copy link
Owner

yo thanks :)

@EmeraldSnorlax
Copy link
Author

if its too much of a hassle, dont worry about it, i can live without telling the whole world that im a friendly murderer on 2Fort

@Kataiser Kataiser added good first issue Good for newcomers help wanted Extra attention is needed and removed good first issue Good for newcomers labels Oct 9, 2019
@Jan200101
Copy link
Contributor

Jan200101 commented Oct 10, 2019

Manged to get it working while working on #53
#53 could be completed once I know the names of the processes

@Kataiser
Copy link
Owner

Nice!

@EmeraldSnorlax
Copy link
Author

actually, dont worry about it, you can close this if you like. i ended up making my own script for it instead.

@Jan200101 Jan200101 linked a pull request Nov 12, 2021 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants