Extend description of protected-subnet in iked.conf(5)

2019-02-22 Thread Sevan Janiyan
Hi, Attached patch adds more details about what protected-subnet's role is in configuration file, it may be useful for someone unfamiliar with the terminology used in IKEv2. Sevan Index: sbin/iked/iked.conf.5 === RCS file:

Re: minor changes to isakmpd(8)

2019-02-18 Thread Sevan Janiyan
On 18/02/2019 14:35, Sevan Janiyan wrote: > I've dropped the changes with the exception of > s/CERTIP/CERTFQDN that is an actual bug and changing the file name to > indicate a copy. > > If you look at /etc/x509v3.cnf you'll see that for the x509v3_FQDN > extension, the su

Re: minor changes to isakmpd(8)

2019-02-18 Thread Sevan Janiyan
Hi Jason, On 18/02/2019 07:23, Jason McIntyre wrote: > if you agree with my comments below, could you mail us an updated diff > and i'll try to prod some folks for oks. Thanks for taking a look. I've dropped the changes with the exception of s/CERTIP/CERTFQDN that is an actual bug and changing

minor changes to isakmpd(8)

2019-02-15 Thread Sevan Janiyan
Use an It tag to label the additional steps referred to at the beginning of step 2: "This step, as well as the next one, needs to be done for every peer. Furthermore the last step will need to be done once for each ID you want the peer to have" Change the file name passed to openssl for extfile

DragonFly BSD fingerprint

2017-12-16 Thread Sevan Janiyan
Hello, Attached patch adds the fingerprints for DragonFly to pf.os. From http://gitweb.dragonflybsd.org/dragonfly.git/blob_plain/947927fb129de774317974bc923ca1ade3d4cc60:/etc/pf.os Sevan Index: etc/pf.os === RCS file:

Re: Add HISTORY section to clri(8)

2016-09-12 Thread Sevan Janiyan
On 13/09/2016 01:43, Theo de Raadt wrote: > I honestly don't see the value these annotations add, especially for > such tiny commands. Ok, fair enough. Sevan

Add HISTORY section to clri(8)

2016-09-12 Thread Sevan Janiyan
Hello, clri showed up in V6 UNIX http://minnie.tuhs.org/cgi-bin/utree.pl?file=V6/usr/man/man8/clri.8 Attached patch adds a history section to man page. Sevan Index: sbin/clri/clri.8 === RCS file: /cvs/src/sbin/clri/clri.8,v

Re: Drop main() prototype

2016-09-09 Thread Sevan Janiyan
On 05/09/2016 11:05, Marc Espie wrote: > On Mon, Sep 05, 2016 at 04:04:23AM -0400, Ted Unangst wrote: >> Sevan Janiyan wrote: >>> Hello, >>> Attached patches remove the main() prototype from >>> src/{sbin,usr.bin,usb.sbin} >> >> yes! > >

usr.bin/fgen/fgen.l - ANSIfy

2016-09-04 Thread Sevan Janiyan
Hello, Attached patch removes the K style function & main() prototype. Sevan Index: usr.bin/fgen/fgen.l === RCS file: /cvs/src/usr.bin/fgen/fgen.l,v retrieving revision 1.12 diff -u -p -r1.12 fgen.l --- usr.bin/fgen/fgen.l 9 Oct

Drop main() prototype

2016-09-04 Thread Sevan Janiyan
Hello, Attached patches remove the main() prototype from src/{sbin,usr.bin,usb.sbin} Next sweep will be to mark usage() as static __dead. Sevan Index: usr.bin/cap_mkdb/cap_mkdb.c === RCS file: /cvs/src/usr.bin/cap_mkdb/cap_mkdb.c,v

Re: Amend history & authors info in bin/expr/expr.1

2016-08-29 Thread Sevan Janiyan
Hi Ingo, On 27/08/2016 14:40, Ingo Schwarze wrote: >> -This version was written by >> +It was largely rewritten by >> .An John T. Conklin >> -in 1994. >> +in 1993. > > That's outright misleading. OpenBSD uses Conklin's NetBSD 1.1 > version (written in C in 1994). NetBSD uses jdolecek@'s

Adjust HISTORY for src/bin/stty/stty.1

2016-08-18 Thread Sevan Janiyan
Hello, The 2nd Edition UNIX source is incomplete over at TUHS[1] however there is a scan of a printed draft copy of the 2nd Edition manual[2], stty appears to be listed in the table of contents (page 7). Attached patch amends the HISTORY section of stty(1) to say v2 rather than v3 and bumps date.

Re: Add / Amend history info for some utilities in /bin

2016-08-18 Thread Sevan Janiyan
On 18/08/2016 06:07, Theo de Raadt wrote: > It appears the goal here is to be accurate regarding minor amusing > documentation details. That's good, the same detailing will reflects > in code as well. > > Well, then don't send a diff alone. Provide all the proof. History > is complicated and

Re: Add / Amend history info for some utilities in /bin

2016-08-14 Thread Sevan Janiyan
On 12/08/2016 05:21, Sevan Janiyan wrote: > Hello, > Attached diff adds history sections to chio(1), dd(1), echo(1), pax(1), > sleep(1) and test(1). > > Corrects the version info in sync(8). > > Ammends kill(1) to note originally the command appeared in section eight. &g

Add / Amend history info for some utilities in /bin

2016-08-11 Thread Sevan Janiyan
Hello, Attached diff adds history sections to chio(1), dd(1), echo(1), pax(1), sleep(1) and test(1). Corrects the version info in sync(8). Ammends kill(1) to note originally the command appeared in section eight. Info obtained from the CSRG CD set and hosted copies of man pages at TUHS. By the

Re: Amend history info for some utilities in /bin

2016-08-11 Thread Sevan Janiyan
Thanks Ingo, will man.openbsd.org update automatically at some point or is it a manual process? Sevan

Amend history info for some utilities in /bin

2016-08-10 Thread Sevan Janiyan
Hello, Attached diff corrects the version info utilities first appears in the man pages of df, ln, ls, ps. Version information was obtained from TUHS and cat-v.org which host copies of the man pages. Sevan Index: bin/df/df.1 ===

patch to bin/cat/cat.c to align output between -be & -ne

2016-06-26 Thread Sevan Janiyan
. [1] http://marc.info/?l=openbsd-tech=146665028627169=2 Regards Sevan Janiyan Index: cat.c === RCS file: /cvs/src/bin/cat/cat.c,v retrieving revision 1.24 diff -u -p -u -r1.24 cat.c --- bin/cat/cat.c 4 Nov 2015 21:28:01

unit test for fix included in r1.13 of bin/cat/cat.c

2016-06-22 Thread Sevan Janiyan
to the tree over 14 years ago, I understand if it's not of interest, I benefited from putting together the test. Sevan Janiyan Index: regress/bin/Makefile === RCS file: /cvs/src/regress/bin/Makefile,v retrieving revision 1.9 diff -u

eradicate "fat diminisher system" in spamd.conf(5)

2016-05-31 Thread Sevan Janiyan
Hello, spews.org which is referenced in spamd.conf(5) is dead & now redirects to a dieting system. Attached diff swaps the spews reference in the man page to nixspam list actually found in spamd.conf Second diff removes the pointer to spamlinks.net which is also dead. Sevan Index:

Re: fan control on macppc G5 smu

2016-03-19 Thread Sevan Janiyan
On 18/03/2016 12:02, Martin Pieuchot wrote: Could you test the diff below and tell me if it also helps? It is adapted from a submission from Dominic Marks: https://marc.info/?l=openbsd-ppc=142836014007157=2 It definitely helps here. As the kernel works its way through probing various

Re: Grammatical changes to httpd.conf(5)

2016-03-08 Thread Sevan Janiyan
On 08/03/2016 07:04, Jason McIntyre wrote: > i think it's better to submit changes where there is a clear > improvement, or where we can definitely say what's there now is > incorrect. Ok, no worries. Sevan

Re: Grammatical changes to httpd.conf(5)

2016-03-07 Thread Sevan Janiyan
On 07/03/2016 19:14, Jason McIntyre wrote: > all of your changes appear to be for the sake of preference, rather than > that what's there is incorrect. it would be better then to state why you > think your change is neccessary. i don;t feel any of these changes are > improvements. > > language

Grammatical changes to httpd.conf(5)

2016-03-07 Thread Sevan Janiyan
Hi, Attached diff makes some grammatical adjustments to httpd.conf(5) Sevan Index: src/usr.sbin/httpd/httpd.conf.5 === RCS file: /cvs/src/usr.sbin/httpd/httpd.conf.5,v retrieving revision 1.68 diff -u -p -u -r1.68 httpd.conf.5 ---

Typo in smtpd.conf(5)

2016-03-06 Thread Sevan Janiyan
Hi, Attached diff corrects a typo to correspond with the commit message. s/ca/certificate http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin/smtpd/smtpd.conf.5?rev=1.142=text/x-cvsweb-markup Sevan Index: usr.sbin/smtpd/smtpd.conf.5

Re: /32 routes don't stick

2016-02-26 Thread Sevan Janiyan
On 26/02/2016 16:53, Sevan Janiyan wrote: > in my hostname.if I have > !/sbin/route add -net 192.168.0.1 -netmask 255.255.255.255 -link > -cloning -iface vio0 > !/sbin/route add default -ifa 10.0.0.1 192.168.0.1 > > Which works. To clarify 192.168.0.1 is the address assign

Re: /32 routes don't stick

2016-02-26 Thread Sevan Janiyan
On 26/02/2016 16:38, Matthieu Herrb wrote: > hostname.em0: > > inet 92.224.149.DDD/32 > ! route add 91.224.148.0/32 -link -iface em0 in my hostname.if I have !/sbin/route add -net 192.168.0.1 -netmask 255.255.255.255 -link -cloning -iface vio0 !/sbin/route add default -ifa 10.0.0.1

Re: Detect more keyboard cases when starting X

2015-12-26 Thread Sevan Janiyan
Hi Anthony, On 26/12/2015 00:55, Anthony J. Bentley wrote: As X starts, it will attempt to detect features from the kbd(8) setting--for example, us.dvorak will enable dvorak in X, and fr.dvorak will enable French dvorak in X. However, it detects these features with equality checks, which will

Re: Detect more keyboard cases when starting X

2015-12-26 Thread Sevan Janiyan
On 26/12/2015 17:37, Sevan Janiyan wrote: > Hi Anthony, > > On 26/12/2015 00:55, Anthony J. Bentley wrote: >> As X starts, it will attempt to detect features from the kbd(8) >> setting--for example, us.dvorak will enable dvorak in X, and >> fr.dvorak will enable

Permitting the override of MACHINE_ARCH in amd64/param.h

2015-10-12 Thread Sevan Janiyan
l in scenarios where you group arch name variants under a common alias, at the moment using such a mechanism fails under OpenBSD/amd64 as whatever is passed in during build time is overwritten by the inclusion of . Is this viable? Regards, Sevan Janiyan

Re: Permitting the override of MACHINE_ARCH in amd64/param.h

2015-10-12 Thread Sevan Janiyan
On 12/10/2015 16:30, Theo de Raadt wrote: > You have failed to explain the usage case; I don't believe there is > any scenario where people benefit from more architecture divergence. As an example, in pkgsrc, CPUs architectures are grouped under a common alias so they can be addressed once

Re: Permitting the override of MACHINE_ARCH in amd64/param.h

2015-10-12 Thread Sevan Janiyan
On 12/10/2015 17:11, Theo de Raadt wrote: > You want 10 layers up of high level software, to redefine fields in > a system .h file -- for an ecosystem that isn't part of OpenBSD at > all. OpenBSD is the exception to the rule within the framework, there is no specific changes needed to the said

Re: Permitting the override of MACHINE_ARCH in amd64/param.h

2015-10-12 Thread Sevan Janiyan
On 12/10/2015 18:32, Ted Unangst wrote: > just don't include or look at param.h. Looking into doing that now. Sevan

Re: fixes for coverity warnings to cat(1)

2015-07-29 Thread Sevan Janiyan
On 21/07/2015 00:16, Sevan Janiyan wrote: Hi, Attached is a diff for a couple of issues raised by coverity, obtained from NetBSD src/bin/cat/cat.c r1.53 r1.54 From the commit message in NetBSD CVS: bin/cat/cat.c 976654 Argument cannot be negative (missing check

Re: fixes for coverity warnings to cat(1)

2015-07-29 Thread Sevan Janiyan
On 29/07/2015 23:24, Todd C. Miller wrote: I don't really see any need for this diff. ugh, sorry if I wasted your time, thanks for the explanation. Sevan

Re: Patch to add -l flag to cat(1)

2015-07-22 Thread Sevan Janiyan
Hi Philip, On 23/07/2015 00:54, Philip Guenther wrote: This is the second time you've sent a patching adding a feature without saying *why* the feature should be added. That's not very helpful. Apologies. Your first patch for cat added a feature (-f option) to solve problem in NetBSD

Patch to add -l flag to cat(1)

2015-07-22 Thread Sevan Janiyan
Hi, Attached patch adds the -l flag to cat This option causes cat(1) to use fcntl(2) to set an exclusive advisory lock on stdout. which was used to guarantee orderly writing to file. Obtained from NetBSD cat.c r1.26 Sevan Janiyan From NetBSD cat.c r1.32 cat.1 r1.26 Index: bin/cat/cat.1

fixes for coverity warnings to cat(1)

2015-07-20 Thread Sevan Janiyan
Improper use of negative value (missing check for fileno result, stdin) Diff also adds a skip label used by the changes which appeared in NetBSD at a different revision in the past. Sevan Janiyan From NetBSD cat.c r1.53 r1.54 Index: bin/cat/cat.c

Re: Patch to add -f flag to cat(1)

2015-07-19 Thread Sevan Janiyan
On 18/07/2015 07:40, Philip Guenther wrote: You have in mind a place where this would be used? Where are there bugs that this would resolve? Hi Philip, I originally thought it was meant to be a performance thing in busy environments but that's because I'd misinterpreted things due to

Re: Patch to add -f flag to cat(1)

2015-07-19 Thread Sevan Janiyan
On 19/07/2015 16:13, Ted Unangst wrote: I could maybe be convinced. However, fopen is the C standard stdio function. One reason you may be using stdio is because you want portability, so adding nonportable extensions to it seems counter productive. Understood, I'll leave it as it's not

Patch to add -f flag to cat(1)

2015-07-17 Thread Sevan Janiyan
Hi, Attached is a patch to add the -f flag to cat(1). -f ensures that cat is opening a regular file in non blocking mode aborts otherwise. Obtained from NetBSD src/bin/cat/cat.c r1.22 r1.34 Sevan Janiyan From NetBSD cat.c r1.22, r1.34 cat.1 r1.18, r1.25 Index: bin/cat/cat.c