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.

Sign up for GitHub

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

Jump to bottom

Solution for Unsupported Devices / Android Users to be able to run code #118

Closed
wants to merge 3 commits into from
Closed

Solution for Unsupported Devices / Android Users to be able to run code #118

wants to merge 3 commits into from

Conversation

KendallDoesCoding
Copy link
Contributor

@KendallDoesCoding KendallDoesCoding commented Dec 6, 2021 •
edited
Loading

Updates pull request from #117

Many people are complaining, that they cannot run the code on Linux and that they cannot run the program at all on Android (which is obvious why for Android), so I added a way for Android users to run this program too, and Linux users to run this program too and edit the code too.

I think Linux is a major concern if .exe doesn't run on the OS and if the code had different variations between Windows and Linux.

@ThioJoe , if you want, you can add another txt file with my instructions instead of it being in the README.md

Fixes #110

This was referenced Dec 6, 2021
Closed
Closed
KendallDoesCoding
KendallDoesCoding commented Dec 6, 2021
View reviewed changes
Copy link
Contributor Author

@KendallDoesCoding KendallDoesCoding left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As per @ThioJoe's comment on #116 , I think instead of calling:
pip install rtfunicode
pip install colorama
pip install google-api-python-client

We just need to call upon the requirements.txt file, although for now as I have tested this I'm leaving it as it is, but if we can cut out these 3 lines of code and replace it with one line, that would be good, but I just need to complete my testing on this.

@KendallDoesCoding
Copy link
Contributor Author

As per @ThioJoe's comment on #116 , I think instead of calling: pip install rtfunicode pip install colorama pip install google-api-python-client

We just need to call upon the requirements.txt file, although for now as I have tested this I'm leaving it as it is, but if we can cut out these 3 lines of code and replace it with one line, that would be good, but I just need to complete my testing on this.

@ThioJoe I just tested it and it works, please request changes for updating this!

@KendallDoesCoding KendallDoesCoding mentioned this pull request Dec 6, 2021
Closed
@KendallDoesCoding
Copy link
Contributor Author

As per @ThioJoe's comment on #116 , I think instead of calling: pip install rtfunicode pip install colorama pip install google-api-python-client
We just need to call upon the requirements.txt file, although for now as I have tested this I'm leaving it as it is, but if we can cut out these 3 lines of code and replace it with one line, that would be good, but I just need to complete my testing on this.

@ThioJoe I just tested it and it works, please request changes for updating this!

I figured out how to commit without you having to request changes. Please merge Removed 3 pips, made it into 1 to main.

@KendallDoesCoding KendallDoesCoding changed the title Unsupported Devices / Android Users Solution Unsupported Devices / Android Users to be able to run code Dec 6, 2021
@TechStudent10
Copy link
Contributor

@KendallDoesCoding maybe edit the description to say:

Updates pull request from #117 

Many people are complaining, that they cannot run the code on Linux and that they cannot run the program at all on Android (which is obvious why for Android), so I added a way for Android users to run this program too, and Linux users to run this program too and edit the code too.

I think Linux is a major concern if .exe doesn't run on the OS and if the code had different variations between Windows and Linux.

@ThioJoe , if you want, you can add another txt file with my instructions instead of it being in the README.md

Fixes #110

@KendallDoesCoding
Copy link
Contributor Author

@KendallDoesCoding maybe edit the description to say:

Updates pull request from #117 

Many people are complaining, that they cannot run the code on Linux and that they cannot run the program at all on Android (which is obvious why for Android), so I added a way for Android users to run this program too, and Linux users to run this program too and edit the code too.

I think Linux is a major concern if .exe doesn't run on the OS and if the code had different variations between Windows and Linux.

@ThioJoe , if you want, you can add another txt file with my instructions instead of it being in the README.md

Fixes #110

Thanks.

TechStudent10
TechStudent10 suggested changes Dec 7, 2021
View reviewed changes
Copy link
Contributor

@TechStudent10 TechStudent10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<p>pip install -r requirements.txt</p> should just be:

```bash
pip install -r requirements.txt

With an ending of ```.

@KendallDoesCoding
Copy link
Contributor Author

<p>pip install -r requirements.txt</p> should just be:

```bash
pip install -r requirements.txt

With an ending of ```.

Well, it worked with what I put, Should I update it to pip install -r requirements.txt```

What does the ``` stand for @TechStudent11 ?

@TechStudent10
Copy link
Contributor

It means "code block".

You know, this:

print("Hi mom!")

@KendallDoesCoding
Copy link
Contributor Author

KendallDoesCoding commented Dec 7, 2021 via email

KendallDoesCoding
KendallDoesCoding commented Dec 7, 2021
View reviewed changes
Copy link
Contributor Author

@KendallDoesCoding KendallDoesCoding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ThioJoe If you want, you can get rid of the pip install -r requirements.txt line, and edit it to be, follow installation mentioned above

above = above on README.md

@KendallDoesCoding
Copy link
Contributor Author

<p>pip install -r requirements.txt</p> should just be:

```bash
pip install -r requirements.txt

With an ending of ```.

With this, Idk if its working or not, but it doesn't print out if its installing or no.

@KendallDoesCoding
Copy link
Contributor Author

@ThioJoe Waiting for your reply... What do you think on this?

@KendallDoesCoding KendallDoesCoding changed the title Solution Unsupported Devices / Android Users to be able to run code Solution for Unsupported Devices / Android Users to be able to run code Dec 7, 2021
@KendallDoesCoding KendallDoesCoding mentioned this pull request Dec 9, 2021
Closed
@KendallDoesCoding
Copy link
Contributor Author

@ThioJoe Can you please review this, put the comment in the wrong PR.

@ThioJoe
Copy link
Owner

ThioJoe commented Dec 9, 2021

I'll have to look into this, I try to minimize recommending third party services when possible. Also need to make sure it works with the GUI portion

@KendallDoesCoding
Copy link
Contributor Author

KendallDoesCoding commented Dec 9, 2021 via email

@KendallDoesCoding
Copy link
Contributor Author

KendallDoesCoding commented Dec 9, 2021 via email

@KendallDoesCoding
Copy link
Contributor Author

KendallDoesCoding commented Dec 9, 2021 via email

@ThioJoe
Copy link
Owner

ThioJoe commented Dec 9, 2021

Frankly if someone is using linux they should be able to figure out how to install python

@KendallDoesCoding
Copy link
Contributor Author

KendallDoesCoding commented Dec 9, 2021 via email

@KendallDoesCoding
Copy link
Contributor Author

#110, #106, #59, #65, #54, are some of them.

@KendallDoesCoding
Copy link
Contributor Author

@ThioJoe I'm getting a error, but I just think it's a code error, I'm creating a issue thread.

@KendallDoesCoding KendallDoesCoding mentioned this pull request Dec 9, 2021
Closed
1 task
@KendallDoesCoding
Copy link
Contributor Author

@TechStudent11 , @ThioJoe Nay, I don't think this would work actually.

Got this error on replit.

Loading...

Log in using the browser window.

^CTraceback (most recent call last): File "YouTubeSpammerPurge.py", line 2289, in main() File "YouTubeSpammerPurge.py", line 1720, in main youtube = get_authenticated_service() # Set easier name for API function File "YouTubeSpammerPurge.py", line 126, in get_authenticated_service creds = flow.run_local_server(port=0, authorization_prompt_message="Log in using the browser window.") File "/opt/virtualenvs/python3/lib/python3.8/site-packages/google_auth_oauthlib/flow.py", line 472, in run_local_server local_server.handle_request() File "/usr/lib/python3.8/socketserver.py", line 294, in handle_request ready = selector.select(timeout) File "/usr/lib/python3.8/selectors.py", line 415, in select fd_event_list = self._selector.poll(timeout) KeyboardInterrupt

@TechStudent11 Tell me if you know how to sort this out, otherwise I will close this PR.

@KendallDoesCoding
Copy link
Contributor Author

I'm trying it out again, I have a weird glitch with my chrome!

@KendallDoesCoding
Copy link
Contributor Author

@TechStudent11 , @ThioJoe Nay, doesn't work

https://replit.com/join/bhxcmqnbjn-kendalldoescodi

@TechStudent10
Copy link
Contributor

It's a KeyboardInterupt. You pressed Ctrl+C.

@KendallDoesCoding
Copy link
Contributor Author

KendallDoesCoding commented Dec 10, 2021 via email

@TechStudent10
Copy link
Contributor

But, I didnt see anything in the window that opened too, which said login to the window.

@KendallDoesCoding ? Anyways, I'm going to bed now. It's about 10 PM. Actually, it's to 11.

@KendallDoesCoding
Copy link
Contributor Author

KendallDoesCoding commented Dec 10, 2021 via email

@ThioJoe
Copy link
Owner

ThioJoe commented Dec 10, 2021

I don't think I want to officially support a method that requires people to upload or store their authentication credentials on a third party site

@TechStudent10
Copy link
Contributor

I guess you're right, yeah.

@KendallDoesCoding
Copy link
Contributor Author

I don't think I want to officially support a method that requires people to upload or store their authentication credentials on a third party site

Yup, I get what you mean. I didn't think of that until you said this. Also, it anyway doesn't seem to be working so I'm closing this ticket. I just thought it would be a good idea, because we are getting lots of reports saying its not running for Linux.

@KendallDoesCoding KendallDoesCoding deleted the patch-3 branch December 11, 2021 04:06
@KendallDoesCoding KendallDoesCoding restored the patch-3 branch December 11, 2021 17:04
@KendallDoesCoding
Copy link
Contributor Author

@ThioJoe I think the online version of VS Code would work, I don't think of it as a third party service, because if I couldn't run .exe files, I would just run it in VS Code, although, I can't figure out how to run python there.

@TechStudent11 , @ThioJoe Let me know if you can figure out how to, the url for the vscode online is here - https://vscode.dev/github/ThioJoe/YouTube-Spammer-Purge

It should have the github repo already opened, it worked for me.

@KendallDoesCoding
Copy link
Contributor Author

I changed the README.md to this for the time-being, obviously it wont stay like this if we figure out how we can run the code in vs code's online website.

@TechStudent10
Copy link
Contributor

You can't. vscode.dev can't run terminals. If you want to do that, you need a Codespace, which costs money.

@KendallDoesCoding
Copy link
Contributor Author

KendallDoesCoding commented Dec 11, 2021 via email

@TechStudent10
Copy link
Contributor

@KendallDoesCoding Not it isn't. github.dev might be what you're thinking about. That's the online GitHub editor. Codespaces on the other hand, makes you pay for VMs to write your code on.

@KendallDoesCoding
Copy link
Contributor Author

Got it!

@KendallDoesCoding
Copy link
Contributor Author

@ThioJoe Do you know any IDE software that let's you run code online?

@KendallDoesCoding
Copy link
Contributor Author

It seems like no...

Closing this PR, again!

@ArjixWasTaken
Copy link

Uhhh....android users can just use termux, a linux terminal emulator that can run and install actual linux software...
so if it works in linux, it should work in termux

https://termux.com/

@KendallDoesCoding KendallDoesCoding deleted the patch-3 branch December 20, 2021 17:50
@KendallDoesCoding KendallDoesCoding restored the patch-3 branch December 20, 2021 17:50
@KendallDoesCoding
Copy link
Contributor Author

Uhhh....android users can just use termux, a linux terminal emulator that can run and install actual linux software... so if it works in linux, it should work in termux

https://termux.com/

Oh wow, interesting. I would mention this to the README file. But, idk how it works and @ThioJoe mentioned he doesn't like inculding third party services. So yeah. But good to know!

@KendallDoesCoding
Copy link
Contributor Author

Actually, let me delete branch.

@KendallDoesCoding KendallDoesCoding deleted the patch-3 branch December 20, 2021 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TechStudent10 TechStudent10 TechStudent10 requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

on my linux device it says the file type is not suported even though it's for linux
4 participants
@KendallDoesCoding @TechStudent10 @ThioJoe @ArjixWasTaken