Re: apmd hangs

2014-09-09 Thread Ingo Schwarze
Hi Mark, Mark Kettenis wrote on Mon, Sep 08, 2014 at 11:35:36PM +0200: The more code documentation I read, the more I'm convinced that coordinating state changes between logical processors isn't necessary and actually is responsible for the hangs people have been seeing. So here is a diff

Re: apmd hangs

2014-09-09 Thread Ingo Schwarze
Hi David, David Coppa wrote on Tue, Sep 09, 2014 at 07:44:47PM +0200: On Tue, Sep 9, 2014 at 7:27 PM, Ingo Schwarze schwa...@usta.de wrote: i'm sorry to say it makes no difference for me (i'm not opposed to the diff, though). On my laptop, building ports works fine, running firefox works

Re: apmd hangs

2014-09-09 Thread Ingo Schwarze
Hi Amit, Amit Kulkarni wrote on Tue, Sep 09, 2014 at 08:47:22PM -0500: A basic rule of thumb when building ports: raise your /etc/login.conf limits...especially datasize-cur needs to be 2G and datasize-max needs to be 3G. The reason being there are some ports where the linker blows up to 2G

new rc.conf(8) manual

2014-08-19 Thread Ingo Schwarze
Hi, while working on rcctl(8), i noticed that the rc.conf(8) manual is of... err, how can i express it politely... somewhat doubtful quality. Here is a stab at it. As a first step, i moved all the examples from the DESCRIPTION to the EXAMPLES esction. After that, the DESCRIPTION section was

Re: new rc.conf(8) manual

2014-08-19 Thread Ingo Schwarze
Hi Robert, Robert Peichaer wrote on Wed, Aug 20, 2014 at 12:39:51AM +0200: In the Base system daemon configuration variables section, in the last paragraph this is incorrect. ... including a string containing only a single blank character, ... Looking at the _rc_parse_conf() code in

Re: improve ldomctl(8) man page

2014-08-12 Thread Ingo Schwarze
Hi Stefan, Stefan Sperling wrote on Fri, Aug 08, 2014 at 08:56:30PM +0200: I can never remember the exact steps of configuring logical domains on sparc64 without referencing the undeadly article at http://undeadly.org/cgi?action=articlesid=20121214153413 It would be nice if this

Re: Is there a repo for the latest LibreSSL portable?

2014-08-10 Thread Ingo Schwarze
Hi Nick, i'm not speaking for LibreSSL, but about OpenBSD subprojects in general; i'm running src/usr.bin/mandoc (portable version: mdocml.bsd.lv). Nicholas Wilson wrote on Sun, Aug 10, 2014 at 02:05:16PM +0100: Certainly from my point of view it would make things simpler if LibreSSL were run

Re: [DIFF] sftp-server.8, sshd_config.5 after syslog_r change

2014-07-28 Thread Ingo Schwarze
Theo de Raadt wrote on Fri, Jul 18, 2014 at 03:04:28PM -0600: Unfortunately, no. The ssh manual pages are also used by the -portable effort. We do not bother documenting these divergences; there is little harm. Actually you could submit a new diff which suggest that logging might need a

Re: [DIFF] sftp-server.8, sshd_config.5 after syslog_r change

2014-07-28 Thread Ingo Schwarze
Theo de Raadt wrote on Mon, Jul 28, 2014 at 09:20:36AM -0600: The mention of sendsyslog is not acceptable. When this man page shows up on some other system, it will be an Xr pointing to nowhere. The information is too specific. Frankly, noone will care. Old systems will continue doing

Re: cvs log -r1.x vs. -r 1.x

2014-07-28 Thread Ingo Schwarze
Hi Stefan, Stefan Sperling wrote on Tue, Jul 15, 2014 at 12:17:08PM +0200: cvs log -r1.x shows the log entry for HEAD, instead of the log entry for 1.x as cvs -r 1.x does. The typos in this sentence confused me so thoroughly that when first looking at your mail, i didn't get your intent at

Re: man.cgi: clean exit when absent or empty manpath.conf

2014-07-18 Thread Ingo Schwarze
Salut Sebastien, Sébastien Marie wrote on Fri, Jul 18, 2014 at 08:50:17AM +0200: Starting to play with man.cgi (src/usr.bin/mandoc/cgi.c), Thank you for testing. it seems that man.cgi will segfault if configuration file is absent or empty. Here a patch that display error message and 505,

Re: usr.bin/man: use getlist(char *) instead of hand-rolled equivalent in config(char *)

2014-07-17 Thread Ingo Schwarze
Hi, Kent R. Spillner wrote on Wed, Jun 04, 2014 at 10:01:12AM -0500: config(char *) contains a hand-rolled version of getlist(char *). Indeed. The only difference is that the hand-rolled version includes a NULL check before the strcmp. You misread the code. There is no NULL check for the

Re: Patch for caesar(6)

2014-07-03 Thread Ingo Schwarze
Hi David, David Crosby wrote on Thu, Jul 03, 2014 at 07:44:12AM -0600: This little patch fixes a compiler warning (found with WARNINGS turned on), A correct fix for that is below. OK to commit it? unambiguates the printit function, and removes an unnecessary comment. Even though we don't

Re: Patch for caesar(6)

2014-07-03 Thread Ingo Schwarze
Hi, David Crosby wrote on Thu, Jul 03, 2014 at 07:44:12AM -0600: This little patch fixes a compiler warning i just committed my version of the fix. Thanks to David for reporting the issue and to Miod for checking my patch. Yours, Ingo

Re: daily(8) scratch and junk files removal

2014-07-03 Thread Ingo Schwarze
Hi Rafael, Rafael Zalamena wrote on Wed, Jul 02, 2014 at 02:21:25PM -0300: I've made a quick test and there is no problem in 'tmux-*' existing in /var/tmp after a reboot. Running 'tmux attach' in a socket with no tmux process controlling just spits out: 'no sessions' and if you run 'tmux' a

Re: daily(8) scratch and junk files removal

2014-07-02 Thread Ingo Schwarze
Hi Stuart, Stuart Henderson wrote on Wed, Jul 02, 2014 at 01:36:21PM +0100: I think this is correct. Any nay-sayers? ok schwarze@ On 2014/07/01 19:07, Rafael Zalamena wrote: I noticed a problem in one of my OpenBSD installation where tmux(1) would lose its session socket after a few

Re: Rename MAP_ANON to MAP_ANONYMOUS

2014-06-30 Thread Ingo Schwarze
Hi Matthew, so in a nutshell, mmap(2) was originally a BSD idea and first implemented in SunOS? And there is no doubt that *BSD always had MAP_ANON and never MAP_ANONYMOUS and that SunOS primarily defines MAP_ANON and MAP_ANONYMOUS only for /* (source compatibility) */, right? And that the

Re: infnan.3

2014-06-24 Thread Ingo Schwarze
Hi, Theo de Raadt wrote on Tue, Jun 24, 2014 at 09:09:49AM -0600: Ingo Schwarze wrote: It's obvious that whole page needs a content update by a VAX expert. that is correct. All the rest of the discussion is moot. Only miod and martynas can swing this the right way. I disagree with Ted

Re: missing .Pa in spamdb.8

2014-06-22 Thread Ingo Schwarze
Hi Jan, committed, thanks. Ingo Jan Stary wrote on Sun, Jun 22, 2014 at 02:06:51PM +0200: Index: usr.sbin/spamdb/spamdb.8 === RCS file: /cvs/src/usr.sbin/spamdb/spamdb.8,v retrieving revision 1.17 diff -u -p -u -p -r1.17

Re: infnan.3

2014-06-22 Thread Ingo Schwarze
Hi Jan, i don't like this particular patch. It's obvious that whole page needs a content update by a VAX expert. Quibbling over formatting details is not going to fix it. Having old-fashioned formatting actually makes it even more obvious how old-fashioned the content is. Regarding the macros

Re: infnan.3

2014-06-22 Thread Ingo Schwarze
Hi Ted, Ted Unangst wrote on Sun, Jun 22, 2014 at 12:57:18PM -0400: On Sun, Jun 22, 2014 at 17:39, Ingo Schwarze wrote: i don't like this particular patch. It's obvious that whole page needs a content update by a VAX expert. Quibbling over formatting details is not going to fix it. Having

Re: compare memcmp with 0

2014-06-20 Thread Ingo Schwarze
Hi Theo, Theo de Raadt wrote on Thu, Jun 19, 2014 at 09:58:01PM -0600: It could be argued that the bcmp manual page does a poor job documenting this. It should use the mandoc blink tag. OK? It's not perfect yet because when you nest blink tags, it already switches back to non-blinking mode

Re: Support for LC_TIME

2014-05-08 Thread Ingo Schwarze
Hi, Marc Espie wrote on Thu, May 08, 2014 at 07:20:52PM +0200: On Thu, May 08, 2014 at 12:07:30PM +0200, Stefan Sperling wrote: On Wed, May 07, 2014 at 07:44:51PM +0200, Ingo Schwarze wrote: While LC_CTYPE and LC_COLLATE make some sense, LC_MONETARY, LC_NUMERIC, and LC_TIME are badly

Re: Remove ln -F

2014-05-07 Thread Ingo Schwarze
Hi, Philip Guenther wrote on Tue, May 06, 2014 at 09:35:18AM -0700: On Tue, May 6, 2014 at 9:18 AM, Kenneth Westerback wrote: On 6 May 2014 11:45, Ingo Schwarze schwa...@usta.de wrote: if somebody wants to commit, ok schwarze@. Otherwise, i'm accepting OKs. See inline for a note

Re: Support for LC_TIME

2014-05-07 Thread Ingo Schwarze
Hi, POSIX doesn't require support for any locales except POSIX and C. While LC_CTYPE and LC_COLLATE make some sense, LC_MONETARY, LC_NUMERIC, and LC_TIME are badly overengineered, pointless bloat, causing nothing but surprising, erratic behaviour and portability problems when trying to parse

Re: zless is less

2014-05-07 Thread Ingo Schwarze
Hi Vadim, Vadim Zhukov wrote on Sat, May 03, 2014 at 05:44:26PM +0400: Just run into the following issue with zmore/zless: if you have PAGER envvar set, it's no matter whatever you'll call, zmore or zless - it'll always run $PAGER. The problem is that I have PAGER=/usr/bin/more but

Re: [PATCH] rcs void casts

2014-05-07 Thread Ingo Schwarze
Hi Fritjof, Fritjof Bornebusch wrote on Wed, May 07, 2014 at 10:32:05PM +0200: there are a few void casts in rcs. But I have a question about that. Are these casts really necessary? No, they are not necessary. I've read that the compiler warns, because of unused variables. But no compiler

Re: Fix loop test in vipw

2014-05-07 Thread Ingo Schwarze
, Ingo On Sat, Apr 26, 2014 at 10:52:50PM +0200, Ingo Schwarze wrote: On Tue, Apr 22, 2014 at 01:29:03AM +, Ben Cornett wrote: The following corrects the termination condition on the write loop in copyfile. From inspection, i say the patch is clearly correct. I was unable to find any

Re: Remove ln -F

2014-05-06 Thread Ingo Schwarze
Hi, if somebody wants to commit, ok schwarze@. Otherwise, i'm accepting OKs. See inline for a note on the history. Yours, Ingo Tristan Le Guern wrote on Mon, May 05, 2014 at 03:06:12PM +0200: This patch removes the undocumented ln flag -F, which forces the creation of a hard link for a

Re: Fix loop test in vipw

2014-04-26 Thread Ingo Schwarze
Hi, On Tue, Apr 22, 2014 at 01:29:03AM +, Ben Cornett wrote: The following corrects the termination condition on the write loop in copyfile. From inspection, i say the patch is clearly correct. I was unable to find any condition where the bug might actually hit: The file descriptor `to'

Re: LibreSSL OPENSSL_malloc... removal

2014-04-22 Thread Ingo Schwarze
Hi, Vadim Lebedev wrote on Tue, Apr 22, 2014 at 01:17:16PM +: Theo de Raadt deraadt at cvs.openbsd.org writes: Vadim Lebedev wrote: The removal of OPENSSL_malloc/OPENSSL_free ... etc will cause a LOT of pain Which is why they are not removed. I was alarmed by this:

Switch OpenBSD manuals to DocBook

2014-04-01 Thread Ingo Schwarze
Hi, given that Kristaps Dzonsons has recently written docbook2mdoc, http://mdocml.bsd.lv/docbook2mdoc/ which is a DocBook-XML to mdoc(7) converter, and that Eric S. Raymond's mdoc(7) to DocBook-XML converter http://www.catb.org/~esr/doclifter/ is readily available, i am going to switch

Re: security(8): check for nobody in /etc/ftpusers

2014-03-23 Thread Ingo Schwarze
Hi Arto, Arto Jonsson wrote on Sun, Mar 23, 2014 at 08:09:02PM +0200: Or should this check removed completely? Yes, and Gleydson Soares has Andrew's and my OK for removing check_ftpusers() completely. I don't think checking for nobody is needed. Hopefully, nobody will have no password set up,

Re: lock(1) timeout message deduplication

2014-03-14 Thread Ingo Schwarze
Hi Jean-Philippe, Jean-Philippe Ouellet wrote on Wed, Mar 12, 2014 at 07:11:05PM -0400: On Wed, Mar 12, 2014 at 11:09:14PM +0100, Ingo Schwarze wrote: I don't really like the warnx(3) call from the bye() ALRM handler either, but that's a separate matter. Me neither. Maybe something like

Re: lock(1) timeout message deduplication

2014-03-12 Thread Ingo Schwarze
Hi, the analysis and the patch are correct. Besides, even though hi() does not contain anything that isn't signal safe on OpenBSD, it is still nice to have that lengthy function no longer be a signal handler. I don't really like the warnx(3) call from the bye() ALRM handler either, but that's a

Re: fix seekdir(3)

2014-03-06 Thread Ingo Schwarze
Hi, back in November 2013, following an idea by guenther@, i cooked up another optimization for seekdir(3), then failed to send out the patch. So here it is. Philip Guenther wrote on Tue, Nov 05, 2013 at 07:57:43PM -0800: On Wed, 6 Nov 2013, Ingo Schwarze wrote: * Worst case: opendir

make mandoc -Tlocale the default

2014-03-06 Thread Ingo Schwarze
Hi, having fixed the accent issues with mandoc -Tutf8, i have checked the base system for build system issues with switching the default, the resulting fixes are included in the following patch. Anybody wants to check with ports before commit, such that we can do any necessary cleanup

Re: make mandoc -Tlocale the default

2014-03-06 Thread Ingo Schwarze
Hi Ted, Ted Unangst wrote on Thu, Mar 06, 2014 at 04:46:50PM -0500: Seems reasonable to me. I went the man.conf route because it's easier for users to revert, Oh, if somebody really insists in setting LC_CTYPE but wants to see manuals in US-ASCII anyway (huh?), adding -Tascii to mandoc in

Re: rm reference to obsolete functions

2014-02-21 Thread Ingo Schwarze
Hi, Jan Klemkow wrote on Wed, Jan 29, 2014 at 08:44:05PM +0100: I've made a little diff to clean up some manpages with references to the obsoleted functions usleep(1), alarm(3). Actually, i like this diff and would like to commit it, for the following reason: We include pointers in SEE ALSO

Re: man.conf mandoc -Tlocale

2014-02-16 Thread Ingo Schwarze
Hi Ted, Ted Unangst wrote on Fri, Feb 14, 2014 at 12:42:20PM -0500: On Fri, Feb 14, 2014 at 14:02, Ingo Schwarze wrote: I even considered switching the mandoc(1) default from -Tascii to -Tlocale in general, but forgot about it again. If you like the idea, that would be something to do after

zap man.template

2014-02-16 Thread Ingo Schwarze
Hi, the file /usr/share/misc/man.template is horribly outdated and incomplete, compare it to mdoc.template in the same directory. I think deleting it completely is better than updating it, because nobody is supposed to manually write new man(7) manuals, certainly not in OpenBSD, but nowehere

Re: man.conf mandoc -Tlocale

2014-02-14 Thread Ingo Schwarze
Hi Ted, Ted Unangst wrote on Thu, Feb 13, 2014 at 09:22:04PM -0500: About 20 years after the invention of utf-8, I've decided to see what all the fuss is about and experiment with uxterm and whatnot. Naturally, this means I want to see sweet fancy quotes in all my man pages instead of the

Re: upd(4) proposal

2014-02-14 Thread Ingo Schwarze
Hi, a few comments regarding the manual: Andre de Oliveira wrote on Fri, Feb 14, 2014 at 02:07:29PM +0100: diff --git a/share/man/man4/upd.4 b/share/man/man4/upd.4 new file mode 100644 index 000..5a9054d --- /dev/null +++ b/share/man/man4/upd.4 @@ -0,0 +1,96 @@ +.\ $OpenBSD$ +.\

Re: fix ntpd.8 rfc authors

2014-02-12 Thread Ingo Schwarze
Hi Arto, Arto Jonsson wrote on Wed, Feb 12, 2014 at 04:39:39PM +0200: University of Delaware mixup. Done, thanks. Ingo Index: ntpd.8 === RCS file: /cvs/src/usr.sbin/ntpd/ntpd.8,v retrieving revision 1.36 diff -u -p -r1.36

Re: [PATCH] nginx.conf(5): Less EXAMPLES, more HISTORY

2014-01-20 Thread Ingo Schwarze
Hi Donovan, Donovan Watteau wrote on Sat, Jan 11, 2014 at 04:18:59PM +0100: OpenBSD now provides an nginx.conf(5) manual page and I love this, thanks! But it ends with an EXAMPLES section which only deals with some horrible use of whitespace, which is not recommended. I don't get it.

Re: [PATCH] nginx.conf(5): Less EXAMPLES, more HISTORY

2014-01-11 Thread Ingo Schwarze
. +.Sh AUTHORS +.An -nosplit +The bulk of the text was semi-automatically converted +from the XML sources of +.Lk http://nginx.org/en/docs/#development +to +.Xr mdoc 7 +by +.An Florian Obser Aq Mt flor...@openbsd.org . +The description of the grammar was written by +.An Ingo Schwarze Aq Mt schwa

Re: Alter daemon scheduling priority with renice for rc.d

2013-12-28 Thread Ingo Schwarze
Hi, Craig R. Skinner wrote on Sat, Dec 28, 2013 at 09:16:23PM +: On 2013-12-19 Thu 13:43 PM |, Craig R. Skinner wrote: Enhance rc.d/rc.subr with lowered/raised daemon running priority. Replace /etc/rc.d/daemon rc_renice=X with /etc/rc.conf.local daemon_nice=X I'd hate this, or anything

Re: Add HISTORY section to locale.1

2013-12-26 Thread Ingo Schwarze
Hi Brian, Brian Callahan wrote on Wed, Dec 25, 2013 at 11:55:43PM -0700: A user asked me earlier today when our locale(1) appeared. Do we want to put that in the man page? Definitely. However, i'd suggest to reserve the wording (first) appeared in for original inventions in the operating

Re: Weard security report

2013-11-16 Thread Ingo Schwarze
Hi Sven, sven falempin wrote on Wed, Nov 06, 2013 at 10:24:53AM -0500: Running security(8): == /var/db/cloud.json diffs (-OLD +NEW) == --- /dev/null Fri Oct 25 01:30:33 2013 +++ /var/db/cloud.json Thu Oct 17 17:21:15 2013 Previous security: == /var/db/cloud.json

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

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: 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

Re: fix seekdir(3)

2013-11-04 Thread Ingo Schwarze
04, 2013 at 12:26:34AM -0800: On Sat, Nov 2, 2013 at 5:48 PM, Ingo Schwarze schwa...@usta.de wrote: Here is an updated patch which now works correctly with Otto's regression test, with the new test i just committed, and with the test from the Perl test suite Andrew pointed out, even

Re: fix seekdir(3)

2013-11-04 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. That would also be a candidate for -stable, i think. I hope to come round to that tonight. Here it is. This fixes

Re: fix seekdir(3)

2013-11-02 Thread Ingo Schwarze
1.9 +++ seekdir.c 3 Nov 2013 00:23:09 - @@ -1,46 +1,71 @@ /* $OpenBSD: seekdir.c,v 1.9 2007/06/05 18:11:48 kurt Exp $ */ /* - * Copyright (c) 1983, 1993 - * The Regents of the University of California. All rights reserved. + * Copyright (c) 2013 Ingo Schwarze schwa

Re: update perl Module::Build in base

2013-11-01 Thread Ingo Schwarze
Hi Andrew, Andrew Fresh wrote on Wed, Oct 30, 2013 at 01:50:56PM -0700: On Wed, Oct 30, 2013 at 08:31:58PM +0100, Ingo Schwarze wrote: -'DISTRIBUTION' = 'DAGOLDEN/Module-Build-0.39_01.tar.gz', +'DISTRIBUTION' = 'LEONT/Module-Build-0.4007.tar.gz', It's only 4.003 in 5.18

fix seekdir(3), was: update perl Module::Build

2013-11-01 Thread Ingo Schwarze
kurt Exp $ */ /* - * Copyright (c) 1983, 1993 - * The Regents of the University of California. All rights reserved. + * Copyright (c) 2103 Ingo Schwarze schwa...@openbsd.org * - * Redistribution and use in source and binary forms, with or without - * modification, are permitted provided

man.conf(5) _subdir search order

2013-07-12 Thread Ingo Schwarze
Hi, i'm moving this thread from misc@ to tech@ because i propose a patch. When replying, please make sure you do not cross-post. Jérémie Courrèges-Anglas wrote on Fri, Jul 12, 2013 at 02:05:48PM +0200: Jan Stary h...@stare.cz writes: The mdoc(7) manpage says about .Bl that The -width

Re: correct includes in man(3) pages

2013-06-03 Thread Ingo Schwarze
Hi, Jan Klemkow wrote on Mon, Jun 03, 2013 at 11:21:39PM +0200: This diff converts all obsolete '.Fd' tags into '.In' tags of manpages of category three. I hope this diff is ok so. If something is wrong with is, just wrote me and I will fix it. Even though .Fd is not wrong, .In is nicer,

Re: Patch for games/trek

2013-06-01 Thread Ingo Schwarze
Hi Ted, Ted Roby wrote on Sat, Jun 01, 2013 at 02:25:50PM -0700: In /usr/src/games/trek When computer calculates warpcost, labels are incorrectly transposed for stardates and cost. This is affirmed by the fact that time should go down as warp speed goes up, and energy consumed should also

warn that security(8) is not an IDS

2012-08-12 Thread Ingo Schwarze
Hi, a recent inquiry by a user reminded me that the security(8) manual is trying to explain the tool's limitations, but rather muddling the point. So here is an attempt to better explain what's wrong with it. OK? Ingo Index: security.8

Re: Use .Lk instead of .Pa in URLs

2012-08-12 Thread Ingo Schwarze
Hi Arto, i have now committed most of your .Pa - .Lk cleanup patch: Arto Jonsson wrote on Fri, Jul 27, 2012 at 12:02:00PM +0300: Index: gnu/usr.sbin/sendmail/sendmail/sendmail.8 Index: kerberosV/src/lib/krb5/kerberos.8 Index: usr.sbin/nginx/man/nginx.8 Index: usr.sbin/openssl/openssl.1 Not

Re: vmt ref vmwh

2012-08-12 Thread Ingo Schwarze
Hi Ted, Ted Unangst wrote on Tue, Jul 24, 2012 at 08:57:59PM -0400: The vmwh package is very handy, but somewhat hard to discover. Can we add a little hint to the man page for vmt? I know very little about VMWare, but from the mdoc(7) perspective, the are no issues, except that i think the

Re: Mention RFC 6106 in rtadvd.conf(5) SEE ALSO section

2012-07-23 Thread Ingo Schwarze
Hi Lawrence, hi Brad, Lawrence Teo wrote on Mon, Jul 23, 2012 at 09:36:35AM -0400: On Sun, Jul 22, 2012 at 09:42:03PM -0400, Brad Smith wrote: Mention RFC 6106 in the rtadvd.conf(5) man pages SEE ALSO section. OK Thanks for providing and checking the content, i don't know IPv6. However, this

add the touch(1) -h flag

2012-07-11 Thread Ingo Schwarze
Hi, right now, i had to do some tests that required symlinks with specifically crafted access and modification times. For that purpose, NetBSD, FreeBSD and Linux agree on having the non-POSIX touch(1) -h flag, in analogy to chown(8) and chgrp(1). Here is a simple patch to add it to our touch(1).

Re: mtree(8): add sha256digest support

2012-07-07 Thread Ingo Schwarze
@@ -2,7 +2,7 @@ # $OpenBSD: security,v 1.18 2012/05/17 16:06:03 pascal Exp $ # -# Copyright (c) 2011 Ingo Schwarze schwa...@openbsd.org +# Copyright (c) 2011, 2012 Ingo Schwarze schwa...@openbsd.org # Copyright (c) 2011 Andrew Fresh and...@afresh1.com # # Permission to use, copy, modify

Re: mtree(8): add sha256digest support

2012-07-07 Thread Ingo Schwarze
Ingo Schwarze wrote on Sat, Jul 07, 2012 at 05:22:05PM +0200: Christian Weisgerber wrote on Sat, Jul 07, 2012 at 03:40:00PM +0200: This adds support for the sha256digest keyword to create/compare SHA2-256 digests of files. In the man page, also replace SHA-1 with SHA2-256 in the examples

Re: inet(3) man page: remove extra verbiage

2012-05-31 Thread Ingo Schwarze
Hi, Lawrence Teo wrote on Thu, May 31, 2012 at 12:14:54AM -0400: .Pp -The .Fn inet_aton -routine interprets the specified character string as an Internet address, +interprets the specified character string as an Internet address, placing the address into the structure provided. I

Re: Use a default device for eject(1)

2012-05-20 Thread Ingo Schwarze
Hi Martin, Martin Pieuchot wrote on Sun, May 20, 2012 at 04:46:40PM +0200: Diff below makes eject(1) use cd0 as default device like cdio(1) does. I'm aware this is an arbitrary choice but I see no drawback in having a default device to eject and this behavior is coherent with cdio(1)'s. I

Re: typo in sysctl.8

2012-03-11 Thread Ingo Schwarze
Hi Ville, Ville Valkonen wrote on Sun, Mar 11, 2012 at 01:36:56PM +0200: Fix typo in sysctl.8. Sysctl machdep prints osfxsr but I couldn't find this from the man page. Also grepping through /usr/include found osfxsr matches but no osxsfr. Committed, thanks. Ingo --- sysctl.8.orig

Re: request for the inclusion of the pcap-filter manpage

2012-03-04 Thread Ingo Schwarze
Hi Han, Han Boetes wrote on Sat, Mar 03, 2012 at 11:08:53PM +0059: The current libpcap distribution contains the pcap-filter manpage which in my opinion is a very useful manpage for the use of tcpdump. Sure. On OpenBSD, the same information is included in the tcpdump(8) manual.

Re: -not for find(1)

2012-02-27 Thread Ingo Schwarze
Hi, Steffen Daode Nurpmeso wrote on Mon, Feb 27, 2012 at 07:24:00PM +0100: this patch adds the -not operator to find(1). I personally always found -not easier to use due to shell escaping, but today may laziness has bitten back. And it's just one more non-POSIX-compliant option. Please

Re: fix incorrect document titles in several man pages

2012-02-11 Thread Ingo Schwarze
Hi Christiano, Christiano F. Haesbaert wrote on Sat, Feb 11, 2012 at 10:26:18AM -0200: Reads fine to me, any objections ? That's OK schwarze@, but don't forget that the tree is locked and you need an OK deraadt@ as well. Yours, Ingo Index: src/bin/md5/sha256.1

Re: man segfault diff

2012-02-05 Thread Ingo Schwarze
Hi Tobias, Tobias Ulmer wrote on Tue, Jan 31, 2012 at 04:37:23AM +0100: - Remove confusing unused len fields from TAG and ENTRY - Remove a couple of internal unused variables - Prevent parse_path() from inserting empty ENTRYs into the list, leading to a crash due to negative array access

man(1) bus error on amd64

2012-01-04 Thread Ingo Schwarze
diff -u -p -r1.43 man.c --- man.c 7 Jul 2011 04:24:35 - 1.43 +++ man.c 5 Jan 2012 02:05:39 - @@ -2,7 +2,7 @@ /* $NetBSD: man.c,v 1.7 1995/09/28 06:05:34 tls Exp $ */ /* - * Copyright (c) 2010, 2011 Ingo Schwarze schwa...@openbsd.org + * Copyright (c) 2010, 2011

Re: du close handle before exiting

2011-12-31 Thread Ingo Schwarze
Hi, Loganaden Velvindron wrote on Sat, Dec 31, 2011 at 11:39:23AM -0500: In du.c, fts_close() is missing before exit. fts_close() may also return -1 on hitting an error. Looking at fts.c, fts_close(3) does three things: 1) lots of free(3) 2) close(2) 3) fchdir(2) back to the directory it

Re: du close handle before exiting

2011-12-31 Thread Ingo Schwarze
Hi, Loganaden Velvindron wrote on Sat, Dec 31, 2011 at 12:30:44PM -0500: the diff is pointless... Well, not quite; looking at it, i found a few issues with the fts(3) manual: * Use the standard sentence regarding the return value; it applies to this case. * The return value of close(2)

font translation woes in Pod::Man::switchquotes

2011-12-04 Thread Ingo Schwarze
Hi, while working on the p5-Moose port, Andreas Voegele reported misformatting by mandoc(1) that can be reproduced with the following minimal example provided by Andreas: $ cat example.pod =over =item Two arg method B $obj-($x, $y) bla. =back $ pod2man example.pod example.man $ mandoc

typo in Pod::Man special character table

2011-12-04 Thread Ingo Schwarze
Hi, while looking at Pod::Man, here is another, simpler bug in there. The preamble defines an \*(Ae string, but no \*(AE, and then the character table uses \*(AE, but not \*(Ae. The reason for the confusion probably is that (at least newer) groffs define an \(AE built-in special character.

Re: escape man(1) arguments from glob(3)

2011-11-20 Thread Ingo Schwarze
Hi Joerg, Joerg Sonnenberger wrote on Sun, Nov 20, 2011 at 06:00:16AM +0100: On Sun, Nov 20, 2011 at 02:27:39AM +0100, Ingo Schwarze wrote: Or we need the patch below. It looks a bit messy, but i think it is safe. Invert the logic. Check with strcspn first, if there is a character

escape man(1) arguments from glob(3)

2011-11-19 Thread Ingo Schwarze
Hi, i'm moving this thread here from misc@ because i'm proposing a patch to fix the issue. Jason McIntyre wrote on Sat, Nov 19, 2011 at 08:45:33AM +: On Fri, Nov 18, 2011 at 11:04:12PM -0700, Barry Grumbine wrote: man [ used to give me the test(1) manpage, doesn't anymore. Is that

Re: [panc...@nopcode.org: License]

2011-10-27 Thread Ingo Schwarze
Hi Pancake, pancake wrote on Thu, Oct 27, 2011 at 06:03:13PM +0200: Sorry for the lag and the silence. I've been pretty busy and was unable to reply in this thread. No problem with the delay, your time is your own to spend. However, i'm not sure this discussion is apropriate on tech@. You

Re: [panc...@nopcode.org: License]

2011-10-15 Thread Ingo Schwarze
Hi, Ted Unangst wrote on Fri, Oct 14, 2011 at 08:29:40PM -0400: On Fri, Oct 14, 2011, Christiano F. Haesbaert wrote: On 14 October 2011 17:19, Ted Unangst t...@tedunangst.com wrote: On Fri, Oct 14, 2011, Edd Barrett wrote: A port I am maintaining uses our file(1) implementation. A debian

Re: enable aucat by default

2011-10-06 Thread Ingo Schwarze
Hi Alexandre, Alexandre Ratchov wrote on Thu, Oct 06, 2011 at 08:29:26PM +0200: On the one hand, we expect audio to work by default. On the other hand format conversions, channel mapping, resampling and alike belong to the audio sub-system; until 2009, this used to be the audio(4) driver

Re: patch for calendar.openbsd

2011-09-25 Thread Ingo Schwarze
Hi, Sergey Bronnikov wrote on Sat, Sep 24, 2011 at 09:39:55PM +0400: calendar.openbsd updated by latest hackaton events It looks like that file was started and maintained by mickey@ and hasn't been updated systematically for the last five years. But i see nothing wrong with making it a bit

Re: patch for calendar.openbsd

2011-09-25 Thread Ingo Schwarze
Christian Weisgerber wrote on Sun, Sep 25, 2011 at 01:07:34PM +: Ingo Schwarze schwa...@usta.de wrote: The c2k and c2k3 dates are inconstistent between hackathons.html and calendar.openbsd: c2k c2k3 hackathons.html Jul 15 May 08 calendar.openbsd Jun 15

Re: counting zero-length matches in sed(1)

2011-09-17 Thread Ingo Schwarze
Before the 5.0 lock, i rewrote the sed(1) s/// loop to fix multiple bugs with respect to zero-length matches. The patch that went in, http://marc.info/?l=openbsd-techm=131145325927701w=2 had to be backed out due to a regression: When the input did not end in a trailing newline character and

Re: counting zero-length matches in sed(1)

2011-07-26 Thread Ingo Schwarze
, s++, 1, APPEND); + if (*s != '\0') + cspace(SS, s++, 1, APPEND); lastempty = 1; } else lastempty = 0; - Forwarded message from Ingo Schwarze schwa...@cvs.openbsd.org - From

Re: counting zero-length matches in sed(1)

2011-07-21 Thread Ingo Schwarze
; - } /* * Swap the substitute space and the pattern space, and make sure - 8 - schnipp - 8 - 8 - schnapp - 8 - Otto Moerbeek wrote on Fri, Jun 24, 2011 at 09:41:00PM +0200: On Sat, Jun 18, 2011 at 08:16:05PM -0600, Ingo Schwarze wrote: When a regular expression has zero-length

Re: user(8) man page sync and userinfo -v clean up

2011-07-18 Thread Ingo Schwarze
Hi Glen, Glen Anderson wrote on Mon, Jul 18, 2011 at 10:10:23PM +0100: userinfo -v was removed from userinfo.8 and usage() a couple of years ago. Yes, we did that because the -v option has no effect for userinfo(8) and people might be confused what it does when it is documented. Remove it

source manuals: select the right one

2011-06-25 Thread Ingo Schwarze
14:36:04 - 1.41 +++ man.c 25 Jun 2011 13:46:30 - @@ -2,7 +2,7 @@ /* $NetBSD: man.c,v 1.7 1995/09/28 06:05:34 tls Exp $ */ /* - * Copyright (c) 2010 Ingo Schwarze schwa...@openbsd.org + * Copyright (c) 2010, 2011 Ingo Schwarze schwa...@openbsd.org * * Permission

counting zero-length matches in sed(1)

2011-06-18 Thread Ingo Schwarze
When a regular expression has zero-length matches in a string, both sed(1) global replacement (/g) and replacement of numbered instances (e.g. /2) are broken. This is not even limited to sed -E. Both Otto's patch and my own refactoring patch on misc@ only address global replacement and leave

wildcards in changelist(5)

2011-06-11 Thread Ingo Schwarze
Hi, some time ago, mk@ and matthew@ discovered a regression in security(8): Our rewrite killed changelist(5) wildcard support. I think we should put it back, the changelist(5) manual explicitly documents the feature. Comments? OKs? Yours, Ingo Index: security

Re: Small pgrep/pkill enhancement

2011-06-11 Thread Ingo Schwarze
Hi, Jonathan Perkin wrote on Tue, May 17, 2011 at 11:02:05PM +0100: Add -i to ignore case when matching process name It seems nobody picked this up, so i had a look at it. NetBSD has that since March 2005 (committed by sketch@). FreeBSD copied it from NetBSD a few days later.

Re: km(4) support for AMD Family 14h processors

2011-05-31 Thread Ingo Schwarze
Hi Brad, Brad wrote on Mon, May 30, 2011 at 07:44:37PM -0400: Index: share/man/man4/km.4 [...] +.\ .%O http://support.amd.com/us/Processor_TechDocs/31116.pdf Why are the URIs commented out? Both mandoc and modern groff handle %U properly. OK for the following diff? Ingo Index: km.4

Re: make \user syntax more explicit in forward(5)

2011-05-31 Thread Ingo Schwarze
Hi Thomas, i don't use .forward(5) and am not sure whether what you say is true or false, but the form is definitely not ok. Thomas de Grivel wrote on Tue, May 31, 2011 at 08:15:28PM +0200: --- forward.5.~1.9.~ Wed May 4 17:24:22 2011 +++ forward.5 Tue May 31 19:53:17 2011 @@ -35,8 +35,8

Re: Filesystem Hierarchy Standard (FHS) and OpenBSD

2011-05-10 Thread Ingo Schwarze
Hi Jeff, Jeff Licquia wrote on Mon, May 09, 2011 at 11:33:27PM -0400: (Sorry if this isn't the proper list for this discussion. If not, please point me in the right direction.) Since your enquiry is not backed up by a patch proposing specific changes to the OpenBSD operating system, this

Re: /etc/daily ROOTBACKUP duid patch

2011-05-09 Thread Ingo Schwarze
Hi, RD Thrush wrote on Sun, May 08, 2011 at 11:03:32AM -0400: I noticed that ROOTBACKUP stopped working when I converted the /altroot fstab entry to a disklabel UID. The attached patch seems to work for me but I'm sure there's a better way. Indeed. The existing code is already scary, and

Re: wprintf man page

2011-04-29 Thread Ingo Schwarze
Hi Stefan, Stefan Sperling wrote on Fri, Apr 29, 2011 at 01:16:37PM +0200: Does this look alright? Based on FreeBSD's page. Looks good from the mdoc(7) side, except that we haven't support yet for measuring the width of formatted strings, so for now, better provide width strings unformatted.

Re: uname(3) return values

2011-04-18 Thread Ingo Schwarze
Hi Matthew and Joachim, Matthew Dempsky wrote on Mon, Apr 18, 2011 at 11:34:50AM -0700: The issue here (if any) is that we over-specify the *successful* return value as precisely 0, rather than generally non-negative. I like the general idea, so i'd suggest the following. Note that the exact

<    4   5   6   7   8   9   10   >