Re: fix seekdir(3)

2013-11-05 Thread Ingo Schwarze
Ingo Schwarze wrote on Mon, Nov 04, 2013 at 09:51:41AM +0100: I will send a minimal one-line patch to just fix the bug and do nothing else. We should get that one in quickly. Done. [...] Then i will send two cleanup patches to remove useless stuff and put the code into the right place, not

make ftp(1) ignore leading whitespace in URLs

2013-11-05 Thread Stefan Sperling
Before: $ ftp ' http://localhost/snap/INSTALL.amd64' ftp: http: no address associated with name ftp: Can't connect or login to host ` http' After: $ ftp ' http://localhost/snap/INSTALL.amd64' Trying ::1... Trying 127.0.0.1... Requesting http://localhost/snap/INSTALL.amd64 100%

Re: make ftp(1) ignore leading whitespace in URLs

2013-11-05 Thread Alexander Hall
On 11/05/13 13:56, Stefan Sperling wrote: Before: $ ftp ' http://localhost/snap/INSTALL.amd64' ftp: http: no address associated with name ftp: Can't connect or login to host ` http' After: $ ftp ' http://localhost/snap/INSTALL.amd64' Trying ::1... Trying 127.0.0.1... Requesting

Re: make ftp(1) ignore leading whitespace in URLs

2013-11-05 Thread Ingo Schwarze
Hi Stefan, Stefan Sperling wrote on Tue, Nov 05, 2013 at 01:56:33PM +0100: Do others think this useful? I hit this because I made copy/paste errors. Useful? I don't know. Maybe, maybe not. But your patch is NOT OK. Try stuff like $ ftp ' foo' bar I would be surprised if you couldn't

Re: make ftp(1) ignore leading whitespace in URLs

2013-11-05 Thread Janne Johansson
I think this would help the port yt to not crash on such urls, and I think it uses ftp to collect the youtube movies. For some reason, I paste starting spaces a lot, and since youtube urls contain and other non-shelly args, I have to paste into a pair of 's, and if there is a space to begin with,

Re: make ftp(1) ignore leading whitespace in URLs

2013-11-05 Thread Stefan Sperling
On Tue, Nov 05, 2013 at 02:08:21PM +0100, Alexander Hall wrote: On 11/05/13 13:56, Stefan Sperling wrote: Before: $ ftp ' http://localhost/snap/INSTALL.amd64' ftp: http: no address associated with name ftp: Can't connect or login to host ` http' After: $ ftp '

Re: make ftp(1) ignore leading whitespace in URLs

2013-11-05 Thread Stefan Sperling
On Tue, Nov 05, 2013 at 02:18:08PM +0100, Ingo Schwarze wrote: Hi Stefan, Stefan Sperling wrote on Tue, Nov 05, 2013 at 01:56:33PM +0100: Do others think this useful? I hit this because I made copy/paste errors. Useful? I don't know. Maybe, maybe not. But your patch is NOT OK.

Re: make ftp(1) ignore leading whitespace in URLs

2013-11-05 Thread Stuart Henderson
fwiw, neither curl nor wget does this.

Re: make ftp(1) ignore leading whitespace in URLs

2013-11-05 Thread Alexander Hall
On 11/05/13 14:44, Stefan Sperling wrote: On Tue, Nov 05, 2013 at 02:08:21PM +0100, Alexander Hall wrote: On 11/05/13 13:56, Stefan Sperling wrote: Before: $ ftp ' http://localhost/snap/INSTALL.amd64' ftp: http: no address associated with name ftp: Can't connect or login to host ` http'

remove useless #ifdef cruft in sppp(4)

2013-11-05 Thread Stefan Sperling
Neither FreeBSD nor NetBSD have these #ifdef *BSD guards anymore. It doesn't seem worthwhile to keep them in OpenBSD. There was also one typo (__OpenBSD_ vs __OpenBSD__) which kept the code compiling. No binary change. ok? Index: if_spppsubr.c

Re: in6_leavegroup work queue

2013-11-05 Thread Martin Pieuchot
Alexander, I spent quite some time working on this problem and I found some interesting information, see below. On 31/10/13(Thu) 17:20, Alexander Bluhm wrote: On Thu, Oct 31, 2013 at 09:56:11AM +0100, Martin Pieuchot wrote: On 30/10/13(Wed) 16:48, Alexander Bluhm wrote: [...] I'm not

Re: Re : Re: Improve routing functions

2013-11-05 Thread Adam Thompson
On 13-11-04 05:09 PM, Claudio Jeker wrote: On Mon, Nov 04, 2013 at 10:36:39AM -0600, Adam Thompson wrote: The change I think we're both asking for is that in .../usr/sbin/bgpd/kroute.c, on line 505 (5.4-RELEASE), where we see kr-r.priority = RTP_BGP;, we need a way to override that value in

Re: Re : Re: Improve routing functions

2013-11-05 Thread Adam Thompson
On 13-11-05 10:12 AM, Adam Thompson wrote: I can do doc changes, but I think you really, *really* don't want me writing much C code... I can perhaps do some of the initial, trivial, legwork, at best. Now, if bgpd(8) were written in Bourne/Korn shell, I'd be the guy to do this! I recall now

Re: Amd64 relocation R_X86_64_32S in a static lib

2013-11-05 Thread Philip Guenther
On Tue, 5 Nov 2013, Torbjorn Granlund wrote: Philip Guenther guent...@gmail.com writes: Ah, but you are, sorta. In OpenBSD 5.3, platforms where the compiler and toolchain support were for robust for it were switched to build PIE objects and executables by default. So yes, that

Add [-q timeout] option to netcat

2013-11-05 Thread Christopher Zimmermann
Hi, this patch adds this option to nc(1): -q timeout after end-of-file on stdin, wait timeout seconds and then quit. The default is no timeout. This should be compatible with the -q option of the netcat-openbsd package in debian (what the heck ?) I use this to send simple udp

Re: Amd64 relocation R_X86_64_32S in a static lib

2013-11-05 Thread Niels Möller
Torbjorn Granlund t...@gmplib.org writes: Now we have (at least) two OpenBSD ABIs for AMD64, pre 5.3 and now 5.3, 5.4. To make sense of things, I would not be surprised to see R_X86_64_64 banned from 5.5 and on, creating a 3rd OpenBSD AMD64 ABI. I don't understand the fine details of which

Re: Add [-q timeout] option to netcat

2013-11-05 Thread Christopher Zimmermann
I just noticed my patch's bedaviour is actually different to the behaviour of the debian version. for my diff the man page should actually say -q timeout after end-of-file on stdin, timeout the connection after timeout seconds. A timeout of 0 will close the connection after

Re: Amd64 relocation R_X86_64_32S in a static lib

2013-11-05 Thread Mark Kettenis
From: ni...@lysator.liu.se (Niels =?iso-8859-1?Q?M=F6ller?=) Date: Tue, 05 Nov 2013 15:39:35 +0100 Torbjorn Granlund t...@gmplib.org writes: Now we have (at least) two OpenBSD ABIs for AMD64, pre 5.3 and now 5.3, 5.4. To make sense of things, I would not be surprised to see

Re: fix seekdir(3)

2013-11-05 Thread Ingo Schwarze
Ingo Schwarze wrote on Mon, Nov 04, 2013 at 09:51:41AM +0100: Then i will send two cleanup patches to remove useless stuff and put the code into the right place, not changing any functionality. Done committed (thanks to Otto and Todd for checking). Finally, we can work out how to do the

Re: fix seekdir(3)

2013-11-05 Thread Philip Guenther
On Wed, 6 Nov 2013, Ingo Schwarze wrote: Ingo Schwarze wrote on Mon, Nov 04, 2013 at 09:51:41AM +0100: Finally, we can work out how to do the optimization. Probably, that will naturally factorize into two steps: (1) Use the information available in the userland buffer to avoid

Re: make ftp(1) ignore leading whitespace in URLs

2013-11-05 Thread Otto Moerbeek
On Tue, Nov 05, 2013 at 08:54:00PM +0100, Marc Espie wrote: On Tue, Nov 05, 2013 at 02:20:00PM +0100, Janne Johansson wrote: I think this would help the port yt to not crash on such urls, and I think it uses ftp to collect the youtube movies. Fix yt, then. I hate this. Like others say,

Re: make ftp(1) ignore leading whitespace in URLs

2013-11-05 Thread Theo de Raadt
On Tue, Nov 05, 2013 at 08:54:00PM +0100, Marc Espie wrote: On Tue, Nov 05, 2013 at 02:20:00PM +0100, Janne Johansson wrote: I think this would help the port yt to not crash on such urls, and I think it uses ftp to collect the youtube movies. Fix yt, then. I hate this. Like