Re: [Bug-wget] [PATCH 1/3] Added tests for HTTP authentication using credentials from .netrc

2017-05-16 Thread Tomas Hozza
Hello Tim. I'm pleased to hear that you figured it out before I had the chance to look at it thoroughly. Thanks for merging my changes. Regards, Tomas On 16.05.2017 10:32, Tim Rühsen wrote: > Hi Tomas, > > fixed the problem by skipping the processing of 'pubring.kbx'. This file > is created

Re: [Bug-wget] [PATCH 1/3] Added tests for HTTP authentication using credentials from .netrc

2017-05-16 Thread Tim Rühsen
Hi Tomas, fixed the problem by skipping the processing of 'pubring.kbx'. This file is created by libgpgme if we change $HOME. Your changes have been pushed now. Thanks for your work. The charset/locale issue is a general problem when the test suite reads and compares binary data. Currently, we

Re: [Bug-wget] [PATCH 1/3] Added tests for HTTP authentication using credentials from .netrc

2017-05-15 Thread Tim Rühsen
Hi Tomas, my locale looks similar: $ locale LANG=en_US.UTF-8 LANGUAGE=en_US.UTF-8 LC_CTYPE="en_US.UTF-8" LC_NUMERIC="en_US.UTF-8" LC_TIME="en_US.UTF-8" LC_COLLATE="en_US.UTF-8" LC_MONETARY="en_US.UTF-8" LC_MESSAGES="en_US.UTF-8" LC_PAPER="en_US.UTF-8" LC_NAME="en_US.UTF-8"

Re: [Bug-wget] [PATCH 1/3] Added tests for HTTP authentication using credentials from .netrc

2017-05-15 Thread Tim Rühsen
P.S.: Just saw that the byte seems a bit random, on a second run I got: 'utf-8' codec can't decode byte 0xc3 in position 18: invalid continuation byte With Best Regards, Tim On 05/15/2017 05:04 PM, Tim Rühsen wrote: > Hi Tomas, > > my locale looks similar: > > $ locale > LANG=en_US.UTF-8

Re: [Bug-wget] [PATCH 1/3] Added tests for HTTP authentication using credentials from .netrc

2017-05-15 Thread Tomas Hozza
Hello Tim. I run all the tests before sending patches and they all passed. If you can send me more info about the failures, I could have a look at it. My locale is: $ locale LANG=en_US.UTF-8 LC_CTYPE="en_US.UTF-8" LC_NUMERIC=sk_SK.UTF-8 LC_TIME=sk_SK.UTF-8 LC_COLLATE="en_US.UTF-8"

Re: [Bug-wget] [PATCH 1/3] Added tests for HTTP authentication using credentials from .netrc

2017-05-15 Thread Tim Rühsen
Hi Tomas, thank you very much for the fixes and the new tests ! Currently, patch 1/3 breaks two Metalink tests: FAIL: Test-metalink-http.py FAIL: Test-metalink-http-quoted.py I am still investigating, but it has something to do with the environment that you reset in testenv/test/base_test.py: