vmd(4) close vmm parent socket

2018-04-13 Thread Martijn van Duren
Hello tech@, Playing with vmd I noticed that a vm process has vmm's socket to the parent process still open. Patch below works for me. OK? martijn@ Index: vmm.c === RCS file: /cvs/src/usr.sbin/vmd/vmm.c,v retrieving revision 1.80

Re: vmd(8) sends global config too late

2018-03-09 Thread Martijn van Duren
Hello Carlos, On 02/22/18 03:30, Carlos Cardenas wrote: > On Wed, Feb 21, 2018 at 11:27:05PM +0100, Martijn van Duren wrote: >> Hello tech@, >> >> When playing around with local prefix in vm.conf(5) I noticed that it >> wasn't picked up by my vm. The reason fo

vmd(8) sends global config too late

2018-02-21 Thread Martijn van Duren
Hello tech@, When playing around with local prefix in vm.conf(5) I noticed that it wasn't picked up by my vm. The reason for this was that config_setvm was called before config_setconfig. Diff below fixes this. OK? martijn@ Index: vmd.c ==

Re: Make systat(1) more human readable

2018-02-11 Thread Martijn van Duren
Some documentation nits found by jmc. On 02/11/18 12:31, Martijn van Duren wrote: > Hello tech@, > > Since my brain is incapable of quickly processing numbers larger than a > million (even with the thousand separators enabled), I would like to > introduce the -h flag to systat. &g

Make systat(1) more human readable

2018-02-11 Thread Martijn van Duren
Hello tech@, Since my brain is incapable of quickly processing numbers larger than a million (even with the thousand separators enabled), I would like to introduce the -h flag to systat. In order not to loose too much information I do the rollover at values from 10. up. I didn't use fmt_scal

Re: Make systat(1) list ordering

2018-02-07 Thread Martijn van Duren
Some manpage nits found by jmc. Index: engine.c === RCS file: /cvs/src/usr.bin/systat/engine.c,v retrieving revision 1.21 diff -u -p -r1.21 engine.c --- engine.c5 Apr 2017 15:57:11 - 1.21 +++ engine.c7 Feb 2018 19:49

Re: Make systat(1) list ordering

2018-02-07 Thread Martijn van Duren
Someone? On 12/30/17 17:37, Martijn van Duren wrote: > Anyone willing to comment on this? > > On 12/18/17 23:00, Martijn van Duren wrote: >> Hello tech@, >> >> I got a bit annoyed by the fact that it isn't clear what the current >> ordering is for states. I

Re: Fix for vi(1) manpage Visual command

2018-02-02 Thread Martijn van Duren
When running in ex-mode it sets itself in vi-mode. When running in vi-mode it's basically an alias to the ex and edit commands. This means that an uppercase first letter splits the screen in two. Personally I find this rather confusing, since we overload the function to mean two different things b

Re: Make systat(1) list ordering

2017-12-30 Thread Martijn van Duren
Anyone willing to comment on this? On 12/18/17 23:00, Martijn van Duren wrote: > Hello tech@, > > I got a bit annoyed by the fact that it isn't clear what the current > ordering is for states. I'm not very familiar with systat, so I might > have missed something obvious

Make systat(1) list ordering

2017-12-18 Thread Martijn van Duren
Hello tech@, I got a bit annoyed by the fact that it isn't clear what the current ordering is for states. I'm not very familiar with systat, so I might have missed something obvious. Since there was no obvious room available for always printing the active ordering and very few fields do support o

Re: w: don't print any header with -h

2017-12-15 Thread Martijn van Duren
On 12/15/17 09:15, Sebastian Benoit wrote: > Martijn van Duren(openbsd+t...@list.imperialat.at) on 2017.12.14 14:03:24 > +0100: >> On 12/14/17 13:56, Jasper Lievisse Adriaanse wrote: >>> On Thu, Dec 14, 2017 at 01:35:18PM +0100, Martijn van Duren wrote: >>>> Hello

Re: w: don't print any header with -h

2017-12-14 Thread Martijn van Duren
On 12/14/17 13:56, Jasper Lievisse Adriaanse wrote: > On Thu, Dec 14, 2017 at 01:35:18PM +0100, Martijn van Duren wrote: >> Hello Jasper, >> >> On 12/14/17 13:22, Jasper Lievisse Adriaanse wrote: >>> Hi, >>> >>> currently w(1) on OpenBSD differs fr

Re: w: don't print any header with -h

2017-12-14 Thread Martijn van Duren
Hello Jasper, On 12/14/17 13:22, Jasper Lievisse Adriaanse wrote: > Hi, > > currently w(1) on OpenBSD differs from other implementations > (GNU/Darwin/FreeBSD/SmartOS) in that 'w -h' does print the > 'USER TTY FROM ...' header whereas the others don't. > > Is there a specific reason for it or co

Re: remove one bug, add three

2017-12-12 Thread Martijn van Duren
On 12/12/17 15:02, kshe wrote: > On Tue, 12 Dec 2017 12:44:03 +, Todd C. Miller wrote: >> On Tue, 12 Dec 2017 11:57:58 +, kshe wrote: >> >>> Perhaps the worst part of all this, though, is how the change of >>> behaviour, which made sed fail hard where it previously handled input in >>> a pe

Re: vi: remove awk scripts

2017-12-11 Thread Martijn van Duren
On 12/11/17 14:14, Marc Espie wrote: >> @@ -37,6 +37,16 @@ SRCS= cl_funcs.c cl_main.c cl_read.c cl_ >> >> LINKS= ${BINDIR}/vi ${BINDIR}/ex >> LINKS+= ${BINDIR}/vi ${BINDIR}/view >> + >> +options_def.h: common/options.awk common/options.c >> +awk -f ${.CURDIR}/../common/options.

Re: vi: remove awk scripts

2017-12-11 Thread Martijn van Duren
Both options have their risks and aren't particularly pretty. Maybe the auto-generation option is a bit safer, because a comment is harder to overlook than a separate file. Two minor nits on your patch (modified patch below): 1) You forgot to change ex/ex.awk to enum 2) I don't like the trailing c

Re: vi: remove awk scripts

2017-12-11 Thread Martijn van Duren
ping On 11/11/17 10:59, Martijn van Duren wrote: > Hello tech@, > > I've gotten confused by the awk scripts a few times now. > They seem to unused and I had to really look where they were originally > intended. For the curious: > awk -f common/options.awk common/options.c

Re: sed: Fix up y command

2017-12-11 Thread Martijn van Duren
Here's a new diff with the issues addressed. But instead of a check I fixed the starting offset, which also caused issues with y///. OK? Index: compile.c === RCS file: /cvs/src/usr.bin/sed/compile.c,v retrieving revision 1.43 diff -u

Re: sed: Fix read/write file not optional

2017-12-07 Thread Martijn van Duren
On 12/07/17 21:25, Todd C. Miller wrote: > Previously, 'p' would be traversed until eol. With your diff this > is not longer the case. Since the return value of compile_flags() > is used to assign a new value for 'p', isn't that a problem? > > - todd > Since the w flag works to eol everything

sed: Fix up y command

2017-12-07 Thread Martijn van Duren
Hello tech@, Our sed command goes a bit bit bonkers on the y command in a couple of different scenarios and was found thanks to the sed.1 diff from kshe. 1) is a posix[0] violation, which states: If a followed by an 'n' appear in string1 or string2, the two characters shall be handled as a sin

sed: Fix read/write file not optional

2017-12-07 Thread Martijn van Duren
Hello tech@, Based on the diff from kshe I found a small inconsistency in our implementation. Right now we allow r to have no argument, where posix states[0]: The r and w command verbs, and the w flag to the s command, take an rfile (or wfile) parameter, separated from the command verb letter or f

Re: Fix segfault on vi(1)

2017-11-23 Thread Martijn van Duren
OK martijn@ On 11/23/17 14:14, Ricardo Mestre wrote: > Index: common/delete.c > === > RCS file: /cvs/src/usr.bin/vi/common/delete.c,v > retrieving revision 1.11 > diff -u -p -u -r1.11 delete.c > --- common/delete.c 6 Jan 2016 22:28:

vi: remove awk scripts

2017-11-11 Thread Martijn van Duren
Hello tech@, I've gotten confused by the awk scripts a few times now. They seem to unused and I had to really look where they were originally intended. For the curious: awk -f common/options.awk common/options.c > include/options_def.h awk -f ex/ex.awk ex/ex_cmd.c > include/ex_def.h The options_d

vi: flush trace to disk early

2017-11-10 Thread Martijn van Duren
Hello tech@, When debugging some crashes in vi it can come in handy if TRACE calls are actually send to disk before the crash. This shouldn't be a problem, since we're running debugging code anyway. OK? martijn@ Index: common/main.c =

[patch] setsid.2 inconsistency

2017-09-13 Thread Martijn van Duren
Hello tech@, When reading up on setsid I found ERRORS section confusing. It indicates that EPERM may occur if the process group ID of a process other than the calling process matches the process ID of the calling process. To me this appears to be in contradiction with the fork manpage, which stat

Re: [patch] Add -z and -Z to apmd for automatic suspend/hibernate

2017-08-13 Thread Martijn van Duren
Hello Jesper, On 08/12/17 23:39, Jesper Wallin wrote: > Hello tech@ > > I've been stuck with fsck for way to many times now because of me > forgetting to connect the AC power. This patch will add -z and -Z > followed by the percentage. apmd will try to suspend or hibernate the > system if the AC

Re: less(1) - segmentation fault with '-g'

2017-08-02 Thread Martijn van Duren
You're right. Maybe I should read up on my less. :-) OK martijn@ for the full diff. On 08/02/17 19:55, Anton Lindqvist wrote: > On Wed, Aug 02, 2017 at 07:19:50PM +0200, Martijn van Duren wrote: >> OK martijn@ for the pattern.c part. >> >> The search.c part seems unnee

Re: less(1) - segmentation fault with '-g'

2017-08-02 Thread Martijn van Duren
OK martijn@ for the pattern.c part. The search.c part seems unneeded. - If we don't have a match we don't enter hilite_line at all. - If we have a match and don't negate get sp/ep for every match. - If we have a match and negate, with your process.c patch we get a sp/ep with value NULL and return

Re: [patch] Remove binc from vi(1)

2017-07-13 Thread Martijn van Duren
No one? On 07/02/17 19:58, Martijn van Duren wrote: > Any takers? > > On 06/22/17 21:32, Martijn van Duren wrote: >> Hello tech@, >> >> Attached a patch to remove the binc function from vi and replace it with >> recallocarray. The functions effectively do the s

Re: CVS: cvs.openbsd.org: src

2017-07-13 Thread Martijn van Duren
On 07/03/17 08:30, Martijn van Duren wrote: > On 07/01/17 18:14, Mark Kettenis wrote: >> CVSROOT: /cvs >> Module name: src >> Changes by: kette...@cvs.openbsd.org2017/07/01 10:14:10 >> >> Modified files: >> sys/dev/pci

Re: cwm: remove ssh auto-completion

2017-07-09 Thread Martijn van Duren
As a sysadmin with access to over a couple of 100 machines I find this feature extremely useful. It would require me quite a bit of extra brainpower to remember the correct/full hostname of each and every one of them, especially since I can start typing halfway through the hostname. So yeah, I thi

Re: CVS: cvs.openbsd.org: src

2017-07-03 Thread Martijn van Duren
On 07/01/17 18:14, Mark Kettenis wrote: > CVSROOT: /cvs > Module name: src > Changes by: kette...@cvs.openbsd.org2017/07/01 10:14:10 > > Modified files: > sys/dev/pci/drm: drm_irq.c drm_linux.c drm_linux.h >drm_linux_list.h drm_mm.c drm_mm.h drm_mode.

Re: systemd compat for doas

2017-07-02 Thread Martijn van Duren
Where would this be needed and make sense? I wasn't aware of this systemd behaviour until now, but a quick google search shows me that it's seen by many as a bug and I tend to agree. A type-O shouldn't imply root access. martijn@ On 07/02/17 20:59, Ted Unangst wrote: > If the username starts

Re: [patch] Remove binc from vi(1)

2017-07-02 Thread Martijn van Duren
Any takers? On 06/22/17 21:32, Martijn van Duren wrote: > Hello tech@, > > Attached a patch to remove the binc function from vi and replace it with > recallocarray. The functions effectively do the same thing since > BINC_{GOTO,RET} already do the nlen > llen comparison. I

Re: [patch] increase command buffer ksh

2017-06-28 Thread Martijn van Duren
On 06/28/17 19:00, Theo de Raadt wrote: > Sounds good. > > It might be nice to determine if these two variables should be one, > ie. tied together better. I was thinking the same thing. Both are used for input line buffering, so I reckon that's sane. Identical checksum compared to previous diff.

[patch] increase command buffer ksh

2017-06-28 Thread Martijn van Duren
Hello tech@, On monday I was playing with the build environment of PHP and all of a sudden I couldn't finish the command because the command buffer was full, so I was forced to use bash to finish what I set out to do. Of course I'd rather stay on ksh, so here's a patch (guided by anton@) which in

Wrong check vi(1)

2017-06-24 Thread Martijn van Duren
Hello tech@, I noticed that ADD_SPACE_{GOTO,RET} have their checks twisted around. I reckon we got lucky that L__gp is never NULL when called. I'm reckon it's worth while to remove the tmp space all together, but that diff would require quite a bit of extra work, so lets just fix the check for no

[patch] Remove binc from vi(1)

2017-06-22 Thread Martijn van Duren
Hello tech@, Attached a patch to remove the binc function from vi and replace it with recallocarray. The functions effectively do the same thing since BINC_{GOTO,RET} already do the nlen > llen comparison. I've run this without any issues, but since recallocarray does extra checks and binc ALWAYS

Re: sync root.mail

2017-03-30 Thread Martijn van Duren
On 03/30/17 08:55, Otto Moerbeek wrote: > On Thu, Mar 30, 2017 at 06:44:37AM +, Miod Vallat wrote: > > Ambiguous: choose package for emacs--no_x11 > a 0: > 1: emacs-21.4p37-no_x11 > 2: emacs-25.1p3-no_x11 > Your choice: Time to choose anothe

recallocarray in tail(1)

2017-03-26 Thread Martijn van Duren
Hello, There's one instance in tail(1) where recallocarray is appropriate. In fact, it would have prevented the regression I introduced originally. OK? martijn@ Index: read.c === RCS file: /cvs/src/usr.bin/tail/read.c,v retrieving

Fix off by 1 in sed

2016-10-11 Thread Martijn van Duren
Hello tech@, As reported and fixed by pfg@freebsd. Fixes the following case: % echo z | gsed -n -e 's/^a*/b/2p' # empty -- correct % echo z | sed -n -e 's/^a*/b/2p' z # incorrect Fix in FreeBSD r302973 OK? martijn@ Index: process.c =

Re: mount(8): strlen + malloc + snprintf == asprintf

2016-09-05 Thread Martijn van Duren
On 09/05/16 10:06, Ali H. Fardan wrote: > On 2016-09-05 11:04, Otto Moerbeek wrote: >> On Mon, Sep 05, 2016 at 10:47:06AM +0300, Ali H. Fardan wrote: >> >>> On 2016-09-05 10:44, David Gwynne wrote: > On 5 Sep 2016, at 17:39, Ali H. Fardan wrote: > > and why is he telling me this? I jus

Re: UTF-8 support for column(1)

2016-09-03 Thread Martijn van Duren
I forgot to add the iswspace(3) support. Updated diff below. On 09/03/16 18:46, Martijn van Duren wrote: > Here's the diff for UTF-8 support in column(1). > > OK? > Index: column.c === RCS file: /cvs/src/usr.bin/

UTF-8 support for column(1)

2016-09-03 Thread Martijn van Duren
Here's the diff for UTF-8 support in column(1). OK? Index: column.c === RCS file: /cvs/src/usr.bin/column/column.c,v retrieving revision 1.24 diff -u -p -r1.24 column.c --- column.c31 Aug 2016 20:43:57 - 1.24 +++ column.

rewrite column(1)

2016-08-31 Thread Martijn van Duren
Hello tech@, Here's a complete restructuring of the column(1) utility. It's aim is to ease the implementation of UTF-8 support, which will come in a separate patch, but also fixes a few bugs while here: - Fix the width-calculation of a tab-character - Correct treatment of files without trailing n

gethostbyname(3) RES_OPTIONS

2016-08-05 Thread Martijn van Duren
Hello tech@, In gethostbyname(3) the RES_OPTIONS refers to resolver(3), which in turn merely refers to resolv.conf(5). So why not make the referral directly? martijn@ Index: gethostbyname.3 === RCS file: /cvs/src/lib/libc/net/gethos

Re: [PATCH] shutdown: dot not write non-printable characters to wall(1)

2016-08-01 Thread Martijn van Duren
On 08/01/16 20:58, Consus wrote: > The wall(1) utility now replaces non-printable ASCII characters with a > quotation mark. This results in a funny message: > > ?*** System shutdown message from consus@localhost ***? > > This patch fixes the issue by removing '\007' from the shutdown > noti

Re: Remove infinite loop check from ed

2016-07-10 Thread Martijn van Duren
anyone? On 06/28/16 15:00, Martijn van Duren wrote: > ping > > On 06/18/16 11:13, Martijn van Duren wrote: >> Hello tech@, >> >> This patch is build on top of my REG_STARTEND patch. >> >> It removes the infinite loop check and just makes sure that infinite

Re: Fix begin of word bug in ed

2016-07-10 Thread Martijn van Duren
anyone? On 06/28/16 15:00, Martijn van Duren wrote: > ping > > On 06/18/16 11:13, Martijn van Duren wrote: >> Hello tech@, >> >> Here's a second jab at implementing REG_STARTEND in ed. This patch just >> focusses on getting REG_STARTEND in there and the gen

Re: Remove infinite loop check from ed

2016-06-28 Thread Martijn van Duren
ping On 06/18/16 11:13, Martijn van Duren wrote: > Hello tech@, > > This patch is build on top of my REG_STARTEND patch. > > It removes the infinite loop check and just makes sure that infinite > loops won't happen. This brings the behaviour closer to sed and vi. >

Re: Fix begin of word bug in ed

2016-06-28 Thread Martijn van Duren
ping On 06/18/16 11:13, Martijn van Duren wrote: > Hello tech@, > > Here's a second jab at implementing REG_STARTEND in ed. This patch just > focusses on getting REG_STARTEND in there and the general restructuring > that comes with it. > This fixes the begin of word bug

Remove infinite loop check from ed

2016-06-18 Thread Martijn van Duren
Hello tech@, This patch is build on top of my REG_STARTEND patch. It removes the infinite loop check and just makes sure that infinite loops won't happen. This brings the behaviour closer to sed and vi. Although gnu eds has the same infinite loop quirk, I don't see any reference on doing so in

Fix begin of word bug in ed

2016-06-18 Thread Martijn van Duren
Hello tech@, Here's a second jab at implementing REG_STARTEND in ed. This patch just focusses on getting REG_STARTEND in there and the general restructuring that comes with it. This fixes the begin of word bug similar to sed. (Also fix a wrong comment while here) No other changes intended. OK?

(u)ral remove ralink url

2016-06-14 Thread Martijn van Duren
Hello tech@, It appears that http://www.ralinktech.com isn't available anymore. OK to remove? martijn@ Index: ral.4 === RCS file: /cvs/src/share/man/man4/ral.4,v retrieving revision 1.108 diff -u -p -r1.108 ral.4 --- ral.4 7 M

Re: Fix begin of word bug in ed

2016-05-29 Thread Martijn van Duren
eck: XbXbX This command isn't possible in the current ed, since it borks out on an infinite loop detection, so the previous diff would still be an improvement, but it's not what sed/vi does. On 05/29/16 15:18, Martijn van Duren wrote: > Hello tech@, > > Here's a first attem

Fix begin of word bug in ed

2016-05-29 Thread Martijn van Duren
Hello tech@, Here's a first attempt at fixing the begin of word bug as also found in sed. It does a little more then I initially sed^ht out to do, but I also found some other unexpected behaviour. 1) It removes the isbinary test and the accompanying NUL_TO_NEWLINE/NEWLINE_TO_NUL conversions. If a

Re: sed/regcomp bug?

2016-05-28 Thread Martijn van Duren
Hello tech@, Here's part two of the sed fix. Here's part two of the sed fix. It applies the just added REG_NOTBOL|REG_STARTEND change to sed, so that begin of word matches directly after a previous match, ending in not a word, can match. It passes regress and an earlier version of this patch (bas

Re: sed/regcomp bug?

2016-05-24 Thread Martijn van Duren
Hello tech@, Here's another shot at fixing the sed/regex bug. This time I've split it up in a libc/regex part and a sed part. The sed patch is still the same, but I'll resend it after the regex changes are in the tree.[1] I've had a lot of help from schwarze@ at testing the consequences of this p

Re: sed/regcomp bug?

2016-05-10 Thread Martijn van Duren
On 05/10/16 20:58, Ingo Schwarze wrote: > Hi Martijn, > > Martijn van Duren wrote on Tue, May 10, 2016 at 08:08:34PM +0200: >> On 05/10/16 19:29, Ingo Schwarze wrote: >>> Martijn van Duren wrote on Tue, May 10, 2016 at 02:43:54PM +0200: > >>&g

Re: sed/regcomp bug?

2016-05-10 Thread Martijn van Duren
On 05/10/16 19:29, Ingo Schwarze wrote: > Hi Martijn, > > Martijn van Duren wrote on Tue, May 10, 2016 at 02:43:54PM +0200: > >> Index: ./lib/libc/regex/engine.c >> === >> RCS file: /cvs/src/lib/libc/

Re: sed/regcomp bug?

2016-05-10 Thread Martijn van Duren
ping any OKs? On 05/05/16 00:17, Martijn van Duren wrote: > Hello tech@, > > On 04/23/16 07:21, Jonathan Gray wrote on bugs@: >> $ cat foo.sh >> #!/bin/sh >> >> $1 -r ' >> s/[[:space:]]//g >> s/\> ' >> $ cat foo.csv >>

[patch] remove custom getline from vi

2016-05-08 Thread Martijn van Duren
Hello tech@, I'm currently working on removing the binc family tree (including {GET,ADD}_SPACE_{RET,GOTO}) from vi. During this quest I found this gem which I reckon can be done better with the libc native function. This has the benefit of extra checks from getline and it echos all system erro

[patch] memory leak in vi

2016-05-07 Thread Martijn van Duren
Hello tech@, There's a memory leak in vi in the REALLOC{,ARRAY} macro. This patch should fix the leak. A similar diff is already present in FreeBSD's vi. OK? martijn@ Index: common/mem.h === RCS file: /cvs/src/usr.bin/vi/common/mem

Re: sed/regcomp bug?

2016-05-04 Thread Martijn van Duren
Hello tech@, On 04/23/16 07:21, Jonathan Gray wrote on bugs@: > $ cat foo.sh > #!/bin/sh > > $1 -r ' > s/[[:space:]]//g > s/\ ' > $ cat foo.csv > R32G32B32A32_FLOAT , 128, 1, 1, 1, sf32, sf32, sf32, sf32, , >,, linear, > $ ./foo.sh sed < foo.csv > R32G32B32A32_FLOAT,128

Re: changelist: rm unbound/db/root.key

2016-04-20 Thread Martijn van Duren
I have this in my config for quite some time, so OK martijn@ On 04/20/16 10:21, Stuart Henderson wrote: > This file changes twice a day if you're validating dnssec and > it's pretty pointless to warn about in security(8). > > OK? > > Index: changelist > ==

Re: libressl cert chain problem.

2016-04-19 Thread Martijn van Duren
On 12/22/15 14:56, Stuart Henderson wrote: > Could some libressl people look at this please? We have a problem where > the chain coming from a server roots from a certificate that isn't in > the root store, but an intermediary certificate *is* in the root store. > Thanks. I also just hit this bug

[patch] NULL check in libedit

2016-03-22 Thread Martijn van Duren
Hello tech@, I found this one by exiting bc -l via ^D. It should be fine since ct_encode_string also returns NULL when tmp is NULL. OK? Index: eln.c === RCS file: /cvs/src/lib/libedit/eln.c,v retrieving revision 1.12 diff -u -p -r

Re: [PATCH] - '^\' misaligned in vi's 'viusage'

2016-03-13 Thread Martijn van Duren
Fixed, thanks. On 03/13/16 10:59, Raf Czlonka wrote: > Hi all, > > A superfluous character causes '^\' to be misaligned in vi's > 'viusage' command. > > Regards, > > Raf > > Index: usr.bin/vi/vi/v_cmd.c > === > RCS file: /cvs/src

[patch] add UTF-8 support to column(1)

2016-02-27 Thread Martijn van Duren
Hello tech@, Here's my attempt to implement UTF-8 support in column(1). Besides the general UTF-8 conversions it does several other things to make it behave properly. Full changelist is as follow: - Make separator and input full UTF-8 aware. - Do proper character width count. This also fixes some

Re: [patch] vi enable -pedantic

2016-02-08 Thread Martijn van Duren
ue, Feb 09, 2016 at 12:57:37AM +0100, Martijn van Duren wrote: >> And of course you find out way to late that format=flowed was still on... >> Here's a new diff. >> >> Any OKs for this? >> >> On 02/01/16 23:34, Martijn van Duren wrote: >>> Hello t

Re: [patch] vi enable -pedantic

2016-02-08 Thread Martijn van Duren
And of course you find out way to late that format=flowed was still on... Here's a new diff. Any OKs for this? On 02/01/16 23:34, Martijn van Duren wrote: > Hello tech@, > > This patch enables -pedantic and does the appropriate cleanup that comes > with it. It's

doas do CR before printing challenge

2016-02-08 Thread Martijn van Duren
Hello tech@, The following patch adds a carriage return before printing the regular challenge. If you're like me and you sometimes forget to wait before the prompt comes up before typing your password it might be a little less stressful if the message returns to the start of the line instead of a

[patch] vi enable -pedantic

2016-02-01 Thread Martijn van Duren
Hello tech@, This patch enables -pedantic and does the appropriate cleanup that comes with it. It's mostly a CHAR_T->char conversion, which should be quite harmless, but edge-cases can be missed. I'd like to have multiple OKs for this one as well as testing on multiple architectures, just to

[Patch] (lack of) UTF-8 support in write(1)

2016-02-01 Thread Martijn van Duren
Hello tech@, I've been working on this patch with schwarze@. It introduces UTF-8 support in write(1), or more precise lack thereof. The specifications say that write(1) should write characters to the console in conformance with the senders locale. Since the receiving tty might not be UTF-8 c

Re: [patch] cleanup vi

2016-01-23 Thread Martijn van Duren
Here's a small update: On 01/22/16 21:18, Martijn van Duren wrote: 3) 3_vi_remove_progname.diff: Don't keep a copy of the progname in memory, use getprogname instead. Attached without the setprogname. The reason I included setprogname was because getprogname in the manpage doesn'

Re: Writing Documentation

2016-01-23 Thread Martijn van Duren
On 01/23/16 09:35, Timo wrote: Hi who is responsible for writing documentation for OpenBSD? I'd like to get involved in writing documentation for OpenBSD as I really like OpenBSD and I feel technical writing is one of my strong points. Regards. Timo The best way would be to just start writing

[patch] cleanup vi

2016-01-22 Thread Martijn van Duren
Hello tech@, Attached are 6 patches to do some initial cleanup in vi (these complement the perr patch that got comitted earlier). They are intended to be applied in sequential order, because there may be slight overlap, but they (almost) all do more or less the same thing: Remove custom code

[patch] ls + utf-8 support

2016-01-17 Thread Martijn van Duren
h -t and the content is redirected client-side, but you can't win them all. Sincerely, Martijn van Duren Index: ls.c === RCS file: /cvs/src/bin/ls/ls.c,v retrieving revision 1.44 diff -u -p -r1.44 ls.c --- ls.c1 Dec

Re: start UTF-8 support for ksh(1) vi editing mode

2016-01-17 Thread Martijn van Duren
d through mbtowc and based on it's return printed or presented as an ? per byte, or at the very least treated as a single byte. Sincerely, Martijn van Duren

pledge position in vi

2016-01-09 Thread Martijn van Duren
Sincerely, Martijn van Duren Index: common/main.c === RCS file: /cvs/src/usr.bin/./vi/common/main.c,v retrieving revision 1.32 diff -u -p -r1.32 main.c --- common/main.c 6 Jan 2016 22:27:39 - 1.32 +++ common/main.c 9 Jan 2016 09:3

Re: remove language catalogs from vi

2016-01-05 Thread Martijn van Duren
On 01/05/16 04:46, Theo Buehler wrote: On Mon, Jan 04, 2016 at 05:16:57PM +0100, Martijn van Duren wrote: Slight change in that lenght wasn't always preserved in msg.c. This revision works well for me and millert@ The diff looks good to me and the only remark I have is that catalog/R

Re: remove language catalogs from vi

2016-01-04 Thread Martijn van Duren
Slight change in that lenght wasn't always preserved in msg.c. This revision works well for me and millert@ On 12/16/15 17:43, Martijn van Duren wrote: ping On 12/02/15 20:36, Martijn van Duren wrote: Hello tech@, I've had a discussion with bentley@ about some patches for vi. Som

Re: [PATCH] vi remove custom perr from cl_main

2015-12-28 Thread Martijn van Duren
n Sun, 27 Dec 2015 10:12:23 +0100, Martijn van Duren wrote: This patch has been accepted by the nvi2 project.[1] There are more patches, but I'm waiting till these have been reviewed by nvi2. Committed, thanks. Thanks for taking care of this. There's a lot more where that came from

[PATCH] vi remove custom perr from cl_main

2015-12-27 Thread Martijn van Duren
Hello tech@, This patch has been accepted by the nvi2 project.[1] There are more patches, but I'm waiting till these have been reviewed by nvi2. Sincerely, Martijn van Duren [1] https://github.com/lichray/nvi2/pull/43 Index: cl/cl_m

Re: remove language catalogs from vi

2015-12-16 Thread Martijn van Duren
ping On 12/02/15 20:36, Martijn van Duren wrote: Hello tech@, I've had a discussion with bentley@ about some patches for vi. Some of which I've send to Zhihao from the nvi2 project to keep the projects somewhat in sync. I'm still awaiting his response on those before sending t

remove language catalogs from vi

2015-12-02 Thread Martijn van Duren
easier reviewing, to incrementally remove support. After the last diff the catalog directory can be removed entirely. Martijn van Duren diff --git common/gs.h common/gs.h index 8d64493..8859ba7 100644 --- common/gs.h +++ common/gs.h @@ -74,7 +74,6 @@ struct _gs { #define GO_TERM 3 /* Global op

Re: [patch] cwm: Preserve stacking order during cycling

2015-11-21 Thread Martijn van Duren
I use cwm on all my desktops and I really love its simplicity, but this has been one of the minor things that annoyed me. I tried the patch and it worked like a charm. So as an active user, feature-wise: Yes please. :) On 11/21/15 02:10, Vadim Vygonets wrote: Hi, After cycling through many wi

Re: [patch] tail(1) follow multiple files

2015-11-16 Thread Martijn van Duren
Isn't anyone willing to take a stab at this patch? On 11/09/15 12:56, Martijn van Duren wrote: ping On 11/04/15 23:29, Martijn van Duren wrote: Hello tech@, I got somewhat annoyed by the fact that OpenBSD's tail can't follow multiple files and since the last attempt at it

Re: [patch] tail(1) follow multiple files

2015-11-09 Thread Martijn van Duren
ping On 11/04/15 23:29, Martijn van Duren wrote: Hello tech@, I got somewhat annoyed by the fact that OpenBSD's tail can't follow multiple files and since the last attempt at it was from 2008 I thought I'd give it a shot. I already sent an earlier version of this diff to Landr

[patch] tail(1) follow multiple files

2015-11-04 Thread Martijn van Duren
not always be available for kqueue monitoring. Sincerely, Martijn van Duren Index: extern.h === --- extern.h (revision 1) +++ extern.h (revision 34) @@ -36,17 +36,23 @@ if (write(STDOUT_FILENO, p, size) != size) \ oerr(); +struct tai

pledge(2) and exec

2015-10-09 Thread Martijn van Duren
written out of ignorance and merely intended to get myself educated on the subject and to poke the bear to see it's (future) capabilities. Not to criticise, or to get mauled by said bear. Sincerely, Martijn van Duren [1] a new version against the latest doas is up at http://imperialat.at/vias.diff for the potential few actually using it.

Re: CVS: cvs.openbsd.org: src

2015-09-30 Thread Martijn van Duren
On 09/30/15 14:15, Joerg Jung wrote: CVSROOT:/cvs Module name:src Changes by: j...@cvs.openbsd.org2015/09/30 06:15:12 Modified files: share/man/man4 : isa.4 sys/arch/amd64/conf: GENERIC sys/arch/i386/conf: GENERIC sys/dev/isa: files.isa Add

Re: rcctl action multiple daemons

2015-09-30 Thread Martijn van Duren
ping On 09/25/15 14:44, Martijn van Duren wrote: On 09/25/15 14:03, Stuart Henderson wrote: On 2015/09/25 13:17, Martijn van Duren wrote: I don't always want all daemons running all the time on my desktop, but when I want to start multiple daemons (for instance when doing some webdev) I

Re: rcctl action multiple daemons

2015-09-25 Thread Martijn van Duren
On 09/25/15 14:03, Stuart Henderson wrote: On 2015/09/25 13:17, Martijn van Duren wrote: I don't always want all daemons running all the time on my desktop, but when I want to start multiple daemons (for instance when doing some webdev) I have to call rcctl multiple times, which resul

rcctl action multiple daemons

2015-09-25 Thread Martijn van Duren
e seem to be no regressions. I gave disable and enable the same treatment as action, because it was slightly easier that way. Any interest in (or comments on) this patch? Sincerely, Martijn van Duren Index: rcctl.sh === RCS file: /cv

Re: [PATCH] doas: add path resolving

2015-09-19 Thread Martijn van Duren
On 09/19/15 23:58, Ted Unangst wrote: Martijn van Duren wrote: Hello tech@, The following patch adds $PATH resolving to doas. This has two advantages: 1) People are forced to configure doas with a full path, which makes sure that applications can't be spoofed by users changing their $PATH

[PATCH] doas: add path resolving

2015-09-19 Thread Martijn van Duren
ntial dangerous) binaries to said directories 2) People who were already using full paths in doas.conf(5) are now allowed to run `shutdown` as a simple `shutdown` instead of typing `/sbin/shutdown` every time. THIS PATCH CAN AND MIGHT BREAK YOUR doas.conf(5) CONFIGURATION FILE! Sincerely, Martij

Re: ksh INT32 type

2015-09-10 Thread Martijn van Duren
On 09/10/15 15:56, Michael McConville wrote: Martijn van Duren wrote: I already sent this diff on September 1st.[1] Pointed out in private to and ok by nicm@ [1]http://marc.info/?l=openbsd-tech&m=144112883814618&w=2 Ah, awkward. I hadn't seen this. Can you clarify why you use

Re: [PATCH] ksh getint cleanup

2015-09-10 Thread Martijn van Duren
ping On 09/04/15 08:50, Martijn van Duren wrote: Here's a slightly updated patch which makes sure that 0x0x isn't possible On 09/03/15 21:59, Martijn van Duren wrote: Hello tech@, I gave a look to getint in var.c, which turned out to roll out it's own version of string to

Re: ksh INT32 type

2015-09-10 Thread Martijn van Duren
I already sent this diff on September 1st.[1] Pointed out in private to and ok by nicm@ [1]http://marc.info/?l=openbsd-tech&m=144112883814618&w=2 On 09/10/15 11:05, Nicholas Marriott wrote: I think all of these except perhaps Coproc_id would be better as plain int not int32_t. The typedefs cou

<    1   2   3   4   5   6   7   >