Re: CVS commit: src/lib/libc/stdio

2010-05-04 Thread Robert Elz
Date:Fri, 30 Apr 2010 06:00:14 + From:Jukka Ruohonen jru...@netbsd.org Message-ID: 20100430060014.7739117...@cvs.netbsd.org | Module Name:src | Committed By: jruoho | Date: Fri Apr 30 06:00:14 UTC 2010 | | Modified Files: |

Re: CVS commit: src/lib/libc/stdio

2010-05-04 Thread Robert Elz
Date:Tue, 4 May 2010 16:39:16 +0300 From:Jukka Ruohonen jruoho...@iki.fi Message-ID: 20100504133916.ga9...@marx.bitnet | In my manual page: [...] | What was the problem again? Something odd - sorry - it looks as if my (older) nroff -mandoc on current sources

Re: CVS commit: src/lib/libc/stdio

2010-10-24 Thread Robert Elz
Date:Sun, 24 Oct 2010 16:00:46 +0100 From:Matthias Scheler t...@netbsd.org Message-ID: 20101024150046.ga...@colwyn.zhadum.org.uk | It doesn't have to be a macro anyway as it used only ones. Yes, I meant to ask about that, simply written inline would be no problem.

Re: CVS commit: src/usr.bin/units

2012-12-28 Thread Robert Elz
Date:Fri, 28 Dec 2012 17:57:46 + From:Alan Barrett a...@netbsd.org Message-ID: 20121228175746.e19f117...@cvs.netbsd.org | because unit names cannot contain hyphens. Should that not be mentioned in the man page? Perhaps together with a suggestion that if the

Re: CVS commit: src/usr.sbin/rtsold

2013-05-24 Thread Robert Elz
Date:Fri, 24 May 2013 16:47:08 + From:David Holland dholland-sourcechan...@netbsd.org Message-ID: 20130524164708.ga...@netbsd.org | : -The default interval for router advertisements, which is on the order of 10 | : +The default interval for router

Re: CVS commit: src/usr.sbin/rtsold

2013-05-26 Thread Robert Elz
Date:Sun, 26 May 2013 12:54:44 +0200 From:Thomas Klausner w...@netbsd.org Message-ID: 20130526105444.gu9...@danbala.tuwien.ac.at | Could one of the native speakers please fix it in a way that we won't | have this discussion again? Try the patch below, which I

Re: CVS commit: src/sys/fs/puffs

2014-10-06 Thread Robert Elz
Date:Mon, 6 Oct 2014 14:26:44 + From:Havard Eidnes h...@netbsd.org Message-ID: 20141006142644.7693...@cvs.netbsd.org | Make this build again without debugging enabled; DPRINTF() can end up | as empty, and in an if conditional, you then need braces if that's

Re: CVS commit: src/sys/arch/amd64/conf

2014-10-15 Thread Robert Elz
Date:Sun, 12 Oct 2014 00:17:22 +0900 From:Masao Uebayashi uebay...@gmail.com Message-ID: cadbf7ecem8n2vhq_9ats2ysjzx-zl-4cg65jtd7vuu0nigt...@mail.gmail.com For what it is worth, this change (below) just bit me - I did my first new builds for a couple of weeks, and my

Re: CVS commit: src/sys/fs/puffs

2014-10-15 Thread Robert Elz
Date:Wed, 15 Oct 2014 19:41:40 +0100 From:David Laight da...@l8s.co.uk Message-ID: 20141015184140.gl23...@snowdrop.l8s.co.uk | Consider what happens if you write: | When DPRINTF() expands 'if (xxx) yyy' it all goes horribly wrong. Of course, the do { } while(0)

Re: CVS commit: src/sys/dev/hdaudio

2017-08-03 Thread Robert Elz
Date:Fri, 4 Aug 2017 00:25:24 + From:"matthew green" Message-ID: <20170804002524.16ba1f...@cvs.netbsd.org> | put all the ELD debugging messages under #ifdef HDAFG_HDMI_DEBUG. I am not sure that doing that with the ones that are hda_error() is

Re: CVS commit: src/tests/lib/libm

2017-07-25 Thread Robert Elz
Date:Wed, 26 Jul 2017 00:17:49 + From:co...@sdf.org Message-ID: <20170726001748.gd3...@sdf.org> | And it adds an actual test for functionality, to be sure we weren't | wrong in adding the function for that arch. That's useful, but you can get that if you

Re: CVS commit: src/sys/dev/hdaudio

2017-08-04 Thread Robert Elz
Date:Sat, 05 Aug 2017 04:29:33 +1000 From:matthew green Message-ID: <3108.1501871...@splode.eterna.com.au> | we are talking about 7 lines that appear every time my monitor | goes out of power save. That is strange, and suggests that the X

Re: CVS commit: src/sys/netinet

2017-07-28 Thread Robert Elz
Date:Fri, 28 Jul 2017 19:16:41 + From:"Maxime Villard" Message-ID: <20170728191641.cf10cf...@cvs.netbsd.org> | Remove TCP_COMPAT_42. This feature is a workaround for a bug in the TCP | stack of BSD4.2. Having such features just does not make

Re: CVS commit: src/tests/net/can

2017-05-28 Thread Robert Elz
Date:Sun, 28 May 2017 10:49:00 -0400 From:"Christos Zoulas" Message-ID: <20170528144900.b8390f...@cvs.netbsd.org> | Modified Files: | src/tests/net/can: t_can.c | | Log Message: | fix format. If socklen_t really needs the cast &

Re: CVS commit: src/tests/net/can

2017-05-28 Thread Robert Elz
OK, ignore me .. I see it was undone already... (should remember to read all mail before replying!) kre

Re: CVS commit: src/bin/sh

2017-06-08 Thread Robert Elz
Date:Thu, 8 Jun 2017 17:42:43 +0200 From:Joerg Sonnenberger Message-ID: <20170608154243.ga25...@britannica.bec.de> | /bin/sh is still broken. Try to build pkgtools/cwrappers. I have just committed a fix/workaround which allows cwrappers to build (for

Re: CVS commit: src/sys/modules/wsbell

2017-06-11 Thread Robert Elz
Date:Sun, 11 Jun 2017 22:21:34 + From:"Paul Goyette" Message-ID: <20170611222134.0ec01f...@cvs.netbsd.org> | Added Files: | src/sys/modules/wsbell: Makefile wsbell.ioconf | | Log Message: | Actually create the wsbell(4) module

Re: CVS commit: src/usr.bin/tic

2017-05-05 Thread Robert Elz
Date:Fri, 5 May 2017 11:49:28 +0100 From:Roy Marples Message-ID: | In this case, exit() isn't called, the main function returns. Same thing, return from main is defined as implicitly calling

Re: CVS commit: src/usr.bin/tic

2017-05-05 Thread Robert Elz
Date:Fri, 5 May 2017 09:07:14 +0100 From:Roy Marples Message-ID: | As we know, parts of NetBSD code are used outside of NetBSD. Of course, and if there are any of those for which calling exit()

Re: CVS commit: src/usr.bin/tic

2017-05-04 Thread Robert Elz
Date:Thu, 4 May 2017 14:07:33 + From:"Roy Marples" Message-ID: <20170504140733.b327df...@cvs.netbsd.org> | Module Name:src | Committed By: roy | Date: Thu May 4 14:07:33 UTC 2017 | | Modified Files: |

Re: CVS commit: src/sys/conf

2017-10-09 Thread Robert Elz
Date:Mon, 9 Oct 2017 14:37:00 +0200 From:Joerg Sonnenberger Message-ID: <20171009123700.gb7...@britannica.bec.de> | It would help if you wrote what problem you actually tried to fix :) Oh ... sorry, but it is in the b5 build log for evbarm-earmv7hf

Re: CVS commit: src/lib/libc/gen

2017-09-09 Thread Robert Elz
Date:Sat, 9 Sep 2017 18:00:40 + From:"Kamil Rytarowski" Message-ID: <20170909180040.222c2f...@cvs.netbsd.org> | Not that devname(3) does not set errno on failure and document it as a bug. Not everything in libc is expected to set errno on

Re: CVS commit: src/sys/kern

2017-08-25 Thread Robert Elz
Date:Fri, 25 Aug 2017 17:28:24 + (UTC) From:chris...@astron.com (Christos Zoulas) Message-ID: | Doesn't __nothing work? No idea, never heard of it ... feel free to change it to that if it works and is better. kre

Re: CVS commit: src/sys (and netbsd-8 stalled pullup request #390)

2017-11-27 Thread Robert Elz
Date:Mon, 27 Nov 2017 09:42:47 + From:Roy Marples Message-ID: | I don't understand the change to ip_output.c (ie why yours works and | mine doesn't) but as long as it works then I'm

Re: CVS commit: src/tests/lib/libc/locale

2017-11-27 Thread Robert Elz
Date:Mon, 27 Nov 2017 18:44:38 +0100 From:Joerg Sonnenberger Message-ID: <20171127174438.ga20...@britannica.bec.de> | Parsing a string constant is a well-defined | operation with precise result. A cross-compiler that doesn't do that | correctly is

Re: CVS commit: src/tests/lib/libc/locale

2017-11-28 Thread Robert Elz
Date:Tue, 28 Nov 2017 15:34:19 +0100 From:Joerg Sonnenberger Message-ID: <20171128143418.ga8...@britannica.bec.de> | Hidding things until then doesn't actually fix something. No, it doesn't, but when I made the change I wasn't hiding anything, just

Re: CVS commit: src/tests/lib/libc/locale

2017-11-28 Thread Robert Elz
Date:Wed, 29 Nov 2017 06:12:02 + From:Taylor R Campbell Message-ID: <20171129061642.e8dcb60...@jupiter.mumble.net> | That's pretty interesting! That is what I thought, it was certainly not what I expected. | Can

Re: CVS commit: src/tests/lib/libc/locale

2017-11-30 Thread Robert Elz
Date:Thu, 30 Nov 2017 23:09:07 +0100 From:Manuel Bouyer Message-ID: <20171130220907.ga2...@antioche.eu.org> | Shouldn't it be made Xfail on i386 in this case ? I don't think so, especially not now the problem is understood - it is trivial

Re: CVS commit: src/sys

2017-11-25 Thread Robert Elz
Date:Sat, 25 Nov 2017 18:11:22 +0900 (JST) From:Ryo ONODERA Message-ID: <20171125.181122.805047996391646417@tetera.org> | After the following commit, my IPv6 network does not work at all. | What information is required to investigate my

Re: CVS commit: src/sys

2017-11-25 Thread Robert Elz
Date:Sat, 25 Nov 2017 18:11:22 +0900 (JST) From:Ryo ONODERA Message-ID: <20171125.181122.805047996391646417@tetera.org> | After the following commit, my IPv6 network does not work at all. | What information is required to investigate my

Re: CVS commit: src/tests/lib/libc/locale

2017-11-29 Thread Robert Elz
Date:Wed, 29 Nov 2017 16:51:56 + From:Taylor R Campbell Message-ID: <20171129165637.6fa4960...@jupiter.mumble.net> | This is starting to smell like a compiler bug in fp correctness...but | I'm out of time to

Re: CVS commit: src/sys/compat

2017-12-01 Thread Robert Elz
Date:Sat, 2 Dec 2017 02:27:33 +0100 From:Joerg Sonnenberger Message-ID: <20171202012733.ga17...@britannica.bec.de> | > tell us what is "broken", exactly | | Every Linux binary failing by default is the very definition of broken. I agree - I was

Re: CVS commit: src/sys

2017-12-16 Thread Robert Elz
Date:Fri, 15 Dec 2017 21:24:20 +0700 From:Robert Elz <k...@munnari.oz.au> Message-ID: <11642.1513347...@andromeda.noi.kre.to> | Unfortunately, because of the way it gets called, this might not be an | easy change to make. It wouldn't be, but it al

Re: CVS commit: src/sys

2017-12-15 Thread Robert Elz
Date:Fri, 15 Dec 2017 05:01:17 + From:"Kengo NAKAHARA" Message-ID: <20171215050117.241baf...@cvs.netbsd.org> | Fix pullup'ed mbuf leaks. | The match function just requires enough mbuf length. This is still not correct. There are no

Re: CVS commit: src/bin/sh

2017-11-20 Thread Robert Elz
Date:Tue, 21 Nov 2017 01:58:24 + From:m...@netbsd.org Message-ID: <20171121015824.ga17...@homeworld.netbsd.org> | This might be the reason the atari floppies are overflowing, I didn't know they were, and yes, I suppose it might be, though the size increase from

Re: CVS commit: src/bin/sh

2017-11-20 Thread Robert Elz
Date:Tue, 21 Nov 2017 03:27:11 + (UTC) From:chris...@astron.com (Christos Zoulas) Message-ID: | We are working trying to fix it. Regardless, upon reflection, there is no need for -X in a SMALL shell, as used in install media,

Re: CVS commit: src/sys/net

2017-11-21 Thread Robert Elz
Date:Wed, 22 Nov 2017 04:56:52 + From:"SAITOH Masanobu" Message-ID: <20171122045652.2ee75f...@cvs.netbsd.org> | Return EINVAL in vlan_config() when a VLAN ID is 0 or 65535. | The spec states 0 and 65535 are reserved. Tags are 12 bigs,

Re: CVS commit: src/etc/rc.d

2017-12-05 Thread Robert Elz
Date:Tue, 5 Dec 2017 16:00:25 + From:David Holland Message-ID: <20171205160025.ga22...@netbsd.org> | Test -o isn't well specified? Or is the issue the precedence of ! vs. -o? -o is a "to be deprecated one day" option, but

Re: CVS commit: src/tests/lib/libc/locale

2017-12-01 Thread Robert Elz
Date:Sat, 2 Dec 2017 00:09:29 +0100 From:Joerg Sonnenberger Message-ID: <20171201230929.ga13...@britannica.bec.de> | This is *not* true. With C11 the standard is very explicit that return | must discard excessive precision. Even before, it was

Re: CVS commit: src/sys/compat

2017-12-08 Thread Robert Elz
Date:Fri, 8 Dec 2017 16:54:24 +0100 From:Maxime Villard Message-ID: <96c690fd-8577-0e4d-dcbb-84c51aed0...@m00nbsd.net> | No, it wasn't barbaric, it made perfect sense: It was, and made no sense. | when modloaded the linux execsw is not

Re: CVS commit: src/lib/libc/time

2017-10-29 Thread Robert Elz
Date:Sun, 29 Oct 2017 22:55:35 +0530 From:Abhinav Upadhyay Message-ID: | While mandoc -Tlint is not complaining about this, but it isn't | working as expected (I

Re: CVS commit: src/lib/libc/time

2017-10-29 Thread Robert Elz
Date:Sun, 29 Oct 2017 06:07:48 + From:"Abhinav Upadhyay" Message-ID: <20171029060748.45c43f...@cvs.netbsd.org> | Modified Files: | src/lib/libc/time: time2posix.3 | | Log Message: | Fix the escape used for em dash It was

Re: CVS commit: src/sbin/cgdconfig

2018-05-08 Thread Robert Elz
Date:Tue, 8 May 2018 19:15:28 +0100 From:Alexander Nasonov Message-ID: <20180508180815.GA5990@neva> | I think it broke the tool. If you run | cgdconfig -p | it will crash. I shall take a look. kre

Re: CVS commit: src/sbin/cgdconfig

2018-05-08 Thread Robert Elz
Date:Tue, 8 May 2018 19:15:28 +0100 From:Alexander Nasonov Message-ID: <20180508180815.GA5990@neva> | I think it broke the tool. If you run | | cgdconfig -p | | it will crash. Sorry, I cannot reproduce this, it looks to work OK to me. Can

Re: CVS commit: src/sbin/cgdconfig

2018-05-09 Thread Robert Elz
Date:Wed, 9 May 2018 08:59:55 +0100 From:Alexander Nasonov Message-ID: <20180509075955.GA7743@neva> | Adding (argc > 0) check before calling opendisk1 fixes the crash. Thanks - and I see what is wrong now, but (for whatever reason) that did not

Re: CVS commit: src/sys

2018-05-11 Thread Robert Elz
Date:Fri, 11 May 2018 12:49:11 +0100 From:Roy Marples Message-ID: <99536ae5-2bcf-c8d5-7c39-f0436c6d1...@marples.name> [Taking your reply out of order...] | The cvs commit increases the default size of the ICMP6 receive socket, | so it's not

Re: CVS commit: src/sys

2018-05-11 Thread Robert Elz
Date:Fri, 11 May 2018 09:43:59 + From:"Roy Marples" Message-ID: <20180511094359.e8785f...@cvs.netbsd.org> | This mitigates recent reports of socket overflow errors | and fixes PR bin/53247. As I read the PR and what followed, the fix for

Re: CVS commit: src/sys

2018-05-11 Thread Robert Elz
Date:Fri, 11 May 2018 12:03:19 +0100 From:Roy Marples Message-ID: | However, the normal amount of noise generated in his environment is no | longer generating any overflow messages so his

Re: CVS commit: src/usr.bin/gzip

2018-06-12 Thread Robert Elz
Date:Tue, 12 Jun 2018 19:52:43 +0200 From:Kamil Rytarowski Message-ID: <90519caf-289c-b4d3-7ebc-d14d4efc9...@gmx.com> | I will try to meet the expectations in commit messages for *San fixes. That would be good (and not just those, unless the board required it,

Re: CVS commit: src/sys/compat/sys

2018-06-15 Thread Robert Elz
Date:Fri, 15 Jun 2018 12:41:56 +0300 From:Valery Ushakov Message-ID: <20180615094156.gd3...@pony.stderr.spb.ru> | Re memset - I now wonder if | compiler is even allowed to be smart here b/c strict aliasing Another way would be static const struct

Re: CVS commit: src/external/cddl/osnet/dev/dtrace/i386

2018-06-04 Thread Robert Elz
Date:Mon, 4 Jun 2018 21:13:23 -0400 From:chris...@zoulas.com (Christos Zoulas) Message-ID: <20180605011323.e275017f...@rebar.astron.com> | It is unfortunate that the files have diverged unnecessarly. Perhaps we | should fix that and push the changes upstream. I

Re: CVS commit: src/external/cddl/osnet/dev/dtrace/i386

2018-06-04 Thread Robert Elz
Date:Mon, 4 Jun 2018 21:31:34 + (UTC) From:chris...@astron.com (Christos Zoulas) Message-ID: | That should come after ... In general "infrastructure" | headers before "feature" headers... I will fix it (it happens to work, | but)... Yes, that's what I

Re: CVS commit: src/sys/kern

2018-06-30 Thread Robert Elz
Date:Sat, 30 Jun 2018 22:47:51 + From:"Taylor R Campbell" Message-ID: <20180630224751.777cdf...@cvs.netbsd.org> | Module Name:src | Committed By: riastradh | Date: Sat Jun 30 22:47:51 UTC 2018 | | Modified Files: |

Re: CVS commit: src/sys/kern

2018-06-30 Thread Robert Elz
Date:Sun, 1 Jul 2018 01:54:35 + From:Taylor R Campbell Message-ID: <20180701015435.93fa760...@jupiter.mumble.net> | Sorry about the breakage with xen_machdep.c that made finding this | tricky. No problem, for me it turned out to actually help ... I hadn't

Re: CVS commit: src/sys/kern

2017-12-22 Thread Robert Elz
Date:Fri, 22 Dec 2017 15:02:57 + From:"Kamil Rytarowski" Message-ID: <20171222150257.8e519f...@cvs.netbsd.org> | ptrace: Partially undo PT_{READ,WRITE}_{I,D} and unbreak these commands | | The refactored code did not work and was

Re: CVS commit: src/sys/kern

2017-12-22 Thread Robert Elz
Date:Fri, 22 Dec 2017 23:22:37 +0700 From:Robert Elz <k...@munnari.oz.au> Message-ID: <1080.1513959...@andromeda.noi.kre.to> | where vm was declared as | | struct vmspace *vm; | | but is not otherwise initialised Well, not quite, I miss

Re: CVS commit: src/sys/kern

2017-12-22 Thread Robert Elz
Date:Fri, 22 Dec 2017 23:35:12 +0700 From:Robert Elz <k...@munnari.oz.au> Message-ID: <19521.1513960...@andromeda.noi.kre.to> | The EFAULT that is returned there is probably the one Kamil mentioned. And of course it is, as in the test ... if ((

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

2018-08-03 Thread Robert Elz
Date:Fri, 3 Aug 2018 02:17:33 + From:"Kamil Rytarowski" Message-ID: <20180803021733.b2002f...@cvs.netbsd.org> | GCC with -fsanitize=undefiend detects a potential overflow in the code. | Cast the return value of ntohs(3) to size_t. I don't understand the

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

2018-08-03 Thread Robert Elz
Date:Fri, 3 Aug 2018 15:02:28 +0200 From:Martin Husemann Message-ID: <20180803130227.ga23...@mail.duskware.de> | What exactly makes the code safe now? If ntohs(p->ip.ip_len) < | (sizeof(p->ip) + sizeof(p->udp)) then we are now in even more serious | trouble.

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

2018-08-03 Thread Robert Elz
Date:Fri, 3 Aug 2018 15:54:24 +0200 From:Martin Husemann Message-ID: <20180803135424.gc23...@mail.duskware.de> | Ah, stupid C integer promotion rules. uint16_t is promoted to int | here, not unsigned int or size_t. Even with that, there should be no problem, in

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

2018-08-03 Thread Robert Elz
Date:Sat, 4 Aug 2018 02:15:15 +0200 From:Kamil Rytarowski Message-ID: | In general there shall not be a relation between -O level and | sanitizers. Sanitizers do not need -O0 or -g for operation. That's fine. But are you doing compiles that way? (necessary

Re: CVS commit: src/sys/arch/evbarm/bcm53xx

2018-08-04 Thread Robert Elz
Date:Sat, 4 Aug 2018 14:33:13 +0100 From:Nick Hudson Message-ID: | I'll take a look Great, thanks. http://releng.netbsd.org/builds/HEAD/201808040510Z/ provides links to the error logs of the failing builds (ignore the one that fails building "file" -

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

2018-08-03 Thread Robert Elz
Date:Fri, 3 Aug 2018 19:46:01 +0100 From:Roy Marples Message-ID: | Considering both methods work and the result of htons is a uint16_t (but | is this always guaranteed?) ntohs() (not that it matters) and "always" is a big word, but that is how it is defined by

Re: CVS commit: src/external/bsd/dhcpcd/dist/src

2018-08-03 Thread Robert Elz
Date:Sat, 4 Aug 2018 04:24:01 +0200 From:Kamil Rytarowski Message-ID: <568544f4-36d5-853e-cdf8-248f84fad...@gmx.com> | I haven't changed any optimization or similar flags for the builds. Then why did the ssh example start giving "perhaps used uninit" warnings when

Re: CVS commit: src/sys

2018-08-22 Thread Robert Elz
Martin Husemann writes: | On Wed, Aug 22, 2018 at 02:35:15PM +, Christos Zoulas wrote: |> But it was already fixed, why did you break it again? It was more or less put back to yesterday - what was in asan.h for the !KASAN case is more or less irrelevant, as all the calls are #ifdef'd and

Re: CVS commit: src/sys/sys

2018-08-22 Thread Robert Elz
Maxime Villard writes: | Shouldn't we make kern_malloc take a size_t? In the end it calls kmem_alloc | which does take a size_t. Someone else's decision, I am not sure of all of the ramifications. | The reason we should not use __nothing is because it leaves unused | variables, and

Re: CVS commit: src/sys

2018-08-22 Thread Robert Elz
Christos Zoulas writes: | I think that my commits are last now. So if it is broken, it is my fault! I think your recent commit is more or less irrelevant right now. It defines macros in the ! KSAN case that are (currently) only used #if KSAN So if something is broken, it is probably

Re: CVS commit: src/sys

2018-08-22 Thread Robert Elz
Christos Zoulas writes: | Yes, I don't know if they caught the changes, but nevertheless I am running | an evbarm build right now anyway.. The current buildbot build started 14:09, your changes were 14:11 and 14:12 (all UTC), so no ... (and the source time is rounded back to an even 10 mins,

Re: CVS commit: src/lib/csu/common

2018-07-13 Thread Robert Elz
Date:Fri, 13 Jul 2018 09:38:02 + From:m...@netbsd.org Message-ID: <20180713093802.ga20...@homeworld.netbsd.org> | ack. we should probably leave the build broken, anyone who tries out the | resulting netbsd might have to spend some time figuring out how to boot

Re: CVS commit: src/sys

2018-07-09 Thread Robert Elz
Date:Mon, 9 Jul 2018 12:28:04 + (UTC) From:chris...@astron.com (Christos Zoulas) Message-ID: | >Avoid redefining SET/CLR/ISSET | Why not remove them?!? Too lazy to validate that would work ... a couple of them caused no build breakage (that I'm aware of)

Re: CVS commit: src/bin/sh

2018-03-16 Thread Robert Elz
Date:Thu, 15 Mar 2018 01:20:43 + From:"Valeriy E. Ushakov" Message-ID: <20180315012043.c3d92f...@cvs.netbsd.org> | The manual is still rather inconsistent e.g. when referring to | parameters where it randomly uses both $0 and 0 or $@ and @ -

Re: CVS commit: src/sys/kern

2018-04-13 Thread Robert Elz
Date:Sat, 14 Apr 2018 05:24:40 +1000 From:matthew green Message-ID: <8304.1523647...@splode.eterna.com.au> | i'd recommend going for at least 100ns intervals, if not 10ns. I have implemented it in a way that the precision can be controlled by

Re: CVS commit: src/sys/kern

2018-04-13 Thread Robert Elz
Date:Fri, 13 Apr 2018 05:21:16 -0400 From:"Christos Zoulas" Message-ID: <20180413092116.b3b08f...@cvs.netbsd.org> | src/sys/kern: subr_prf.c | make this narrower by popular request. Would it perhaps be reasonable to just print

Re: CVS commit: src/sys/netinet

2018-03-31 Thread Robert Elz
Date:Sat, 31 Mar 2018 15:33:30 +0100 From:Alexander Nasonov Message-ID: <20180331143330.GB22190@neva> | After looking at email archives I see that many active developers | are "guilty" of committing 'hi dev@' messages. It used to be common.

Re: CVS commit: src

2018-04-01 Thread Robert Elz
Sorry, replied to the wrong message ... Ignore that last one! kre

Re: CVS commit: src

2018-04-01 Thread Robert Elz
And actually when I read the man page more closely, I do see that "args" is listed as an alias for "command" so that is not undocumented either. kre

Re: CVS commit: src/sys/dev/scsipi

2018-03-23 Thread Robert Elz
Date:Sat, 24 Mar 2018 02:50:05 +0100 From:Kamil Rytarowski Message-ID: <8b2f90cf-0c7f-1154-b3fb-c4c600d07...@gmx.com> | > To generate a diff of this commit: | > cvs rdiff -u -r1.231 -r1.232 src/sys/dev/scsipi/st.c | | I had to revert this in

Re: CVS commit: src/sys/netinet

2018-03-30 Thread Robert Elz
Date:Sat, 31 Mar 2018 03:07:29 + From:Taylor R Campbell Message-ID: <20180331030950.2325d60...@jupiter.mumble.net> | but a passive observer can't tell the difference between | one colleague joshing another and

Re: CVS commit: src/bin/sh

2018-03-16 Thread Robert Elz
Date:Fri, 16 Mar 2018 17:24:55 +0300 From:Valery Ushakov Message-ID: <20180316142455.gf3...@pony.stderr.spb.ru> | mdoc.samples(7) is a very handy reference. For me that is the same as mdoc(7) (a copy of the same man page) - is that what is

Re: CVS commit: src/bin/sh

2018-03-16 Thread Robert Elz
Date:Fri, 16 Mar 2018 14:15:47 +0300 From:Valery Ushakov Message-ID: <20180316111547.gc3...@pony.stderr.spb.ru> | Or: | | When a shell function is executed, all of the shell positional | parameters (except $0, which remains

Re: CVS commit: src/bin/sh

2018-03-17 Thread Robert Elz
Date:Sun, 18 Mar 2018 03:29:54 +0300 From:Valery Ushakov Message-ID: <20180318002954.gl3...@pony.stderr.spb.ru> | At this point I'm inclined to just chalk it up to English culture | being nominalistic par excelence and give up any further

Re: CVS commit: src/bin/sh

2018-03-17 Thread Robert Elz
Date:Sat, 17 Mar 2018 22:52:51 +0300 From:Valery Ushakov Message-ID: <20180317195251.gj3...@pony.stderr.spb.ru> | Environment variable is a name/value pair stored in an environment If you stopped there I'd agree with you. | that a C program

Re: CVS commit: src/bin/sh

2018-03-17 Thread Robert Elz
Date:Sat, 17 Mar 2018 23:10:36 +0300 From:Valery Ushakov Message-ID: <20180317201036.gk3...@pony.stderr.spb.ru> | [this probably belong to tech-userlevel] Perhaps - we don't really have a good place to discuss the doc. | ksh(1) has

Re: CVS commit: src/bin/sh

2018-03-17 Thread Robert Elz
Date:Sat, 17 Mar 2018 16:18:58 +0300 From:Valery Ushakov Message-ID: <20180317131858.gi3...@pony.stderr.spb.ru> | Exactly. And when all of them are, it kinda defeats the purpose of | them being in the ENVIRONMENT section. Why?What

Re: CVS commit: src/bin/sh

2018-03-16 Thread Robert Elz
Date:Sat, 17 Mar 2018 03:46:44 +0300 From:Valery Ushakov Message-ID: <20180317004644.gh3...@pony.stderr.spb.ru> | That reminds me... We currently abuse ENVIRONMENT section to document | variables like PS1 or HISTSIZE which are quite obviously

Re: CVS commit: src/bin/sh

2018-03-16 Thread Robert Elz
Date:Fri, 16 Mar 2018 14:41:22 +0300 From:Valery Ushakov Message-ID: <20180316114122.gd3...@pony.stderr.spb.ru> | The yak is much more entangled now, at least its source form is! | | -.Dv 0 \" $0 | +.Dv 0 | | Why did you delete

Re: CVS commit: src/libexec/rpc.rstatd

2018-10-31 Thread Robert Elz
Date:Wed, 31 Oct 2018 20:27:53 +1100 From:matthew green Message-ID: <16475.1540978...@splode.eterna.com.au> | hmmm... feels like KERN_BOOTTIMESPEC should be added instead, | and the compat restored for this old API. | | or should have been 9 years ago :-(

Re: CVS commit: src/libexec/rpc.rstatd

2018-10-31 Thread Robert Elz
Date:Wed, 31 Oct 2018 20:27:53 +1100 From:matthew green Message-ID: <16475.1540978...@splode.eterna.com.au> | hmmm... feels like KERN_BOOTTIMESPEC should be added instead, | and the compat restored for this old API. | | or should have been 9 years ago :-( A

Re: CVS commit: src/etc

2018-09-30 Thread Robert Elz
Date:Sun, 30 Sep 2018 21:40:18 +0200 From:Joerg Sonnenberger Message-ID: <20180930194018.ga11...@britannica.bec.de> | Just for the future, a better way would be to use :Uno or similar. Yes, I thought there was a method like that, but make "programmming" is not my

Re: CVS commit: src/sys/external/bsd/drm2/dist/drm/i915

2018-09-22 Thread Robert Elz
Date:Sat, 22 Sep 2018 22:03:56 +0200 From:Kamil Rytarowski Message-ID: <164fe80f-089f-ea38-1751-e442e6125...@gmx.com> | I know, but in this case the symbol name (preprocessor symbol) was | available in other functions in the same file. I've decided that it will

CVS commit: src/etc/rc.d

2019-05-12 Thread Robert Elz
Module Name:src Committed By: kre Date: Sun May 12 11:55:47 UTC 2019 Modified Files: src/etc/rc.d: smtoff Log Message: I should know better! Don't rely upon the way the shell implements pipes. Skip the "error" printf from GetSmtId() as there is no easy portable way to

CVS commit: src/etc/rc.d

2019-05-12 Thread Robert Elz
Module Name:src Committed By: kre Date: Sun May 12 11:55:47 UTC 2019 Modified Files: src/etc/rc.d: smtoff Log Message: I should know better! Don't rely upon the way the shell implements pipes. Skip the "error" printf from GetSmtId() as there is no easy portable way to

CVS commit: src/sbin/dmesg

2019-06-04 Thread Robert Elz
Module Name:src Committed By: kre Date: Tue Jun 4 11:37:39 UTC 2019 Modified Files: src/sbin/dmesg: dmesg.c Log Message: PR toolchain/54269 Make a SMALL dmesg even smaller. When compiled -DSMALL, dmesg does nothing at all with the kernel timestamps (it processes no

CVS commit: src/sbin/dmesg

2019-06-04 Thread Robert Elz
Module Name:src Committed By: kre Date: Tue Jun 4 11:37:39 UTC 2019 Modified Files: src/sbin/dmesg: dmesg.c Log Message: PR toolchain/54269 Make a SMALL dmesg even smaller. When compiled -DSMALL, dmesg does nothing at all with the kernel timestamps (it processes no

CVS commit: src/sbin/dmesg

2019-06-04 Thread Robert Elz
Module Name:src Committed By: kre Date: Tue Jun 4 11:59:05 UTC 2019 Modified Files: src/sbin/dmesg: dmesg.c Log Message: Delete a redundant (useless) assignment. NFCI. To generate a diff of this commit: cvs rdiff -u -r1.42 -r1.43 src/sbin/dmesg/dmesg.c Please note

CVS commit: src/sbin/dmesg

2019-06-04 Thread Robert Elz
Module Name:src Committed By: kre Date: Tue Jun 4 11:59:05 UTC 2019 Modified Files: src/sbin/dmesg: dmesg.c Log Message: Delete a redundant (useless) assignment. NFCI. To generate a diff of this commit: cvs rdiff -u -r1.42 -r1.43 src/sbin/dmesg/dmesg.c Please note

CVS commit: src/sys/rump/librump/rumpkern

2019-06-02 Thread Robert Elz
Module Name:src Committed By: kre Date: Sun Jun 2 19:41:51 UTC 2019 Modified Files: src/sys/rump/librump/rumpkern: emul.c Log Message: Apply a patch from hannken@ which adds a weak alias for rump_getcwd_common() allowing -lrump to be used without -lrumpvfs. This is an

CVS commit: src

2019-06-01 Thread Robert Elz
Module Name:src Committed By: kre Date: Sat Jun 1 06:59:18 UTC 2019 Modified Files: src/tests/dev/usb/t_hid: Makefile src/tests/lib/semaphore/pthread: Makefile src/tests/net/bpfjit: Makefile src/tests/net/icmp: Makefile src/tests/net/if:

CVS commit: src

2019-06-01 Thread Robert Elz
Module Name:src Committed By: kre Date: Sat Jun 1 06:59:18 UTC 2019 Modified Files: src/tests/dev/usb/t_hid: Makefile src/tests/lib/semaphore/pthread: Makefile src/tests/net/bpfjit: Makefile src/tests/net/icmp: Makefile src/tests/net/if:

Re: CVS commit: src

2019-06-01 Thread Robert Elz
Date:Sat, 1 Jun 2019 10:34:41 +0200 From:"J. Hannken-Illjes" Message-ID: | This looks wrong -- you should have added an weak alias to | sys/rump/librump/rumpkern/emul.c to handle the case where | librumpvfs is not present. I have no idea how that could have

  1   2   3   4   5   6   7   8   9   10   >