/srv/irclogs.ubuntu.com/2016/11/26/#bzr.txt

jelmervila: I'm hitting more test failures in sid, looks like this is because pycurl is now stricter17:08
vilajelmer: seen that too :-/ but heads down on the ci infra17:09
vilajelmer: I have, I think, a fix for the gettext/lazy_regexp issue though17:09
vilajelmer: I didn't did the pycurl one, stricter for what ?17:09
jelmervila: basically, when you do proxy requests, they now require that you pass in "http://" URLs for the proxy rather than ignoring the scheme as they did before17:12
jelmerhttps://github.com/curl/curl/issues/101517:12
jelmerso it should hopefulyl just be sufficient to replace http+pycurl with just http before calling pycurl17:14
vilajelmer: the wording in that issue is weird... anyway, having a look as I agree s/+pycurl// may be the thing17:15
jelmerfiled https://bugs.launchpad.net/bzr/+bug/164501717:15
ubot5`Ubuntu bug 1645017 in Bazaar "proxy tests fail with newer versions of pycurl" [Undecided,New]17:15
vilathanks !17:16
vilajelmer: hmm, looks like _unqualified_scheme is already used, needs to search if some proxy specific thing is going on17:25
jelmervila: yeah, I think I found it17:42
jelmervila: we set 'all_proxy' in the tests to a URL like http+pycurl://joe:bar@127.0.0.1:35245/17:42
jelmerand then that gets picked up directly by pycurl17:42
jelmeros.environ['all_proxy'] I mean17:43
vilahmm, where ?17:43
jelmervila: bzrlib.tests.test_http17:44
jelmervila: it calls self.overrideEnv('all_proxy', ) in many places17:46
jelmerself.overrideEnv('all_proxy', ...) is all over the place :)17:46
vilarhaa, of course, damn it, was climbing the wrong tree. Hmm, but there are ... several tests failing. But yeah, for setting env vars +pycurl shouldn't be there17:46
vilayeah17:46
* vila tries hard to page-in the whole context17:46
vilait's... surprising that this even works, I'm missing some bits, +pycurl should be used to select the client, I'm confused about why/how it's used for the server... (given that the pycurl client will filter +pycurl)17:48
vilaor was it just all along that pycurl filtered +pycurl and doesn't anymore in which case ... it's overrideEnv calls that needs to be fixed17:49
vilaright, that seems to fix it17:52
viladown to 7 failures17:54
fullermdSee, if you'd just gone with perl in the first place...17:56
vilago f...17:56
vilaerr17:56
vilago for the win ?17:57
vilago fullermd ?17:57
vila;-)17:57
viladown to 1 failure17:57
fullermdO:->17:57
vilapycurl.error: (8, '')17:57
vilawhat on earth is that one ;-)17:57
fullermdIs...   that another emoticon?17:57
vilahaaaa, indeed, so, probably need to skip that test (they no humor ;)17:58
vilathey *have*17:58
vilaTestInvalidStatusServer /me sighs17:59
vilajelmer: patch so far: https://pastebin.canonical.com/171950/18:00
vilajelmer: ftr, this means this is a test-only issue18:01
vila'E_FTP_WEIRD_SERVER_REPLY': 8, seriously ?18:03
fullermdAs FTP goes, that sounds extremely sane.18:04
* fullermd is on a campaign to wipe FTP off the face of the 'net.18:04
vilafullermd: that takes you way too long ! :-p18:04
fullermdMan, you ain't _even_ joking...18:05
fullermdIt's amazing.  The only real fault of telnet is that it's plaintext, and that was sufficient to practically extinctify it in just a couple years, mostly last millennium.18:05
vilawe're dealing with a sabotaged hhtp (ha ha, I'm feeling younger ;) server sending:          self.wfile.write("Invalid status line\r\n")18:05
fullermdFTP is plaintext as well, plus horrific eldritch horror of protocol design, and we can't seem to kill the stupid thing.18:06
vilanah, too damn useful to setup insecure servers, let's put it on IOT gateways18:06
vilaRan 764 tests in 2.206s18:11
vilaOK (known_failures=4)18:11
vila89 tests skipped18:11
vilajelmer: with https://pastebin.canonical.com/171952/18:11
jelmervila: That's a canonical only pastebin :)18:15
vilavila: you idiot18:16
vilahttps://paste.ubuntu.com/23538838/18:16
vilarhaa, didn't press the button on the lp bug...18:18
jelmervila: thanks!18:19
vilajelmer: can I ask you to fix the debian side while I sort out proper branches for bzr itself (which I'll be able to land when I'm done setting up the ci stuff...) ?18:19
jelmervila: yeah18:20
jelmerworking on it now18:20
vilaoh and sorting the mess on the 2.7 and dev branches which still haven't received the post-release cleanups :-/18:20
vilajelmer: thanks a ton !18:20
fullermdvila touched it!  We all saw him touch it!  Now he's on the hook!18:20
=== vila is now known as wasnt-me
wasnt-mewho ?18:22
=== wasnt-me is now known as vila
jelmer:)18:29
jelmervila: new upload on its way18:48
vilajelmer: thanks !19:03
vilaI'll try to followup on the ubuntu side before 2.7.12-7 enter zesty ...19:04
jelmershould be a matter of requesting a sync :)19:18
vilajelmer: right, but that's almost all I know of the process :-/ I still lack practice ... I'm not even sure I know where to ask (but I know who I can ask that question so I should be able to do that ;-)19:24
vilajelmer: upload rejected ?22:29
jelmervila: yeah, with a weird error. I'm going to try again23:12
vilajelmer: ack23:15

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!