wget returning 404 on valid requests in python3 about gdcef HOT 14 CLOSED

All right, it downloads now without errors! If @murt can confirm that too, that would be great.

from gdcef.

@murt Thank to you I just realized that I forgot to copy prerequisite section from the original project:
https://github.com/stigmee/install#step-2-install-python3-packages

from gdcef.

@murt Ok I fixed the doc. Please re-open the ticket if this documentation is still unclear.

from gdcef.

Installed the recommended package "wget" instead of "python3-wget" as stated in the README, but I'm still getting this error while running build.py:

urllib.error.HTTPError: HTTP Error 404: Not Found

File link downloads fine through the webbrowser.

I'm going to look at the version number parser in build.py because it's quite sophisticated for downloading a single file.

from gdcef.

Turns out that URL percent encoding is causing the issue. I'm going to open a PR with the fix, and while I am at it, I'll update the CEF version, too.

from gdcef.

Installed the recommended package "wget" instead of "python3-wget" as stated in the README, but I'm still getting this error while running build.py:

Hi @raphipod install python3-wget ! Do not install wget :) I'm a Python noob: is wget more official than python3-wget ? Except you and @murt people on my team did not get issues with URL encoding with python3-wget

from gdcef.

No worries. It seems that python3-wget is "just" a rewrite for Python 3.
Concerning the URL encoding issues, I really don't know why it fails at my end

from gdcef.

@raphipod I tried quickly: I uninstalled python3-wget and installed back wget and the + seems working but not the %2B (by the way, when you copy the URL from your browser you have %2B). At the opposite, from my memory, python3-wget does not like + but prefer %2B, and both uses import wget and requirements.txt generates wget for python3-wget :( :(

from gdcef.

Concerning the URL encoding issues, I really don't know why it fails at my end

I think this is a mix up between python3_wget and wget while using import wget

from gdcef.

Yeah, we should use python3-wget, already corrected that in #2

from gdcef.

@raphipod @murt I have radically changed of python package for downloading :) no more wget ! I added the requirement file and updated README. I also updated CEF version. I hardly checked. Tell me if this is ok for you. If ok @raphipod you can cancel your PR

from gdcef.

Sure!

from gdcef.

I just noticed that you've commented out the colon in line 346, it exits.

from gdcef.

I just noticed that you've commented out the colon in line 346, it exits.

:( I've git push force. You can do git pull --rebase origin master

from gdcef.