Re: new option for rcctl ls

2021-10-23 Thread Antoine Jacoutot
On Fri, Oct 22, 2021 at 07:27:28PM +0100, Stuart Henderson wrote: > On 2021/10/22 12:56, Stuart Henderson wrote: > > On 2021/10/22 12:20, Antoine Jacoutot wrote: > > > On Thu, Oct 21, 2021 at 04:45:47PM +0100, Stuart Henderson wrote: > > > > Sometimes I find it useful

Re: new option for rcctl ls

2021-10-22 Thread Antoine Jacoutot
On Thu, Oct 21, 2021 at 04:45:47PM +0100, Stuart Henderson wrote: > Sometimes I find it useful to list daemons which are set to 'disabled' > but are actually running. Either those where I have started them by hand > forgotten to enable in rc.conf.local, or to check for services which > shouldn't

Re: rc(8): start unwind earlier

2021-01-29 Thread Antoine Jacoutot
On Fri, Jan 29, 2021 at 04:53:34PM +0100, Florian Obser wrote: > Start unwind earlier. > > OK? > > diff --git rc rc > index 94465add54f..7b5f835f0af 100644 > --- rc > +++ rc > @@ -442,6 +442,7 @@ fill_baddynamic tcp > sysctl_conf > > start_daemon slaacd >/dev/null 2>&1 > +start_daemon unwind

Re: Improving the syspatch(8) and sysupgrade(8) test for the invoking user's privileges

2021-01-29 Thread Antoine Jacoutot
On Fri, Jan 29, 2021 at 03:31:10AM -0500, Horia Racoviceanu wrote: > Hi, > > I noticed the syspatch(8) and sysupgrade(8) ksh scripts test if the > invoking user has the right privileges i.e. (($(id -u) != 0)) > > The test for the effective user ID number can be bypassed in > syspatch(8) and

Re: syspatch exit state

2020-12-07 Thread Antoine Jacoutot
On Mon, Dec 07, 2020 at 01:30:55PM +0100, Ingo Schwarze wrote: > Hello Antoine, > > Antoine Jacoutot wrote on Mon, Dec 07, 2020 at 01:01:27PM +0100: > > > I just tested this change and it seems to work: > > I did not repeat my testing, but here is some quick fee

Re: syspatch exit state

2020-12-07 Thread Antoine Jacoutot
On Mon, Dec 07, 2020 at 11:54:04AM +0100, Ingo Schwarze wrote: > > Index: syspatch.sh > > === > > RCS file: /cvs/src/usr.sbin/syspatch/syspatch.sh,v > > retrieving revision 1.166 > > diff -u -p -r1.166 syspatch.sh > > --- syspatch.sh

Re: syspatch exit state

2020-12-07 Thread Antoine Jacoutot
On Sun, Dec 06, 2020 at 10:52:37PM +0100, Alexander Hall wrote: > > > On December 6, 2020 8:13:26 PM GMT+01:00, Antoine Jacoutot > wrote: > >On Sun, Dec 06, 2020 at 05:20:31PM +, Stuart Henderson wrote: > >> On 2020/12/06 16:39, Otto Moerbeek wrote: > >

Re: syspatch exit state

2020-12-06 Thread Antoine Jacoutot
On Sun, Dec 06, 2020 at 05:20:31PM +, Stuart Henderson wrote: > On 2020/12/06 16:39, Otto Moerbeek wrote: > > On Sun, Dec 06, 2020 at 03:31:19PM +, SW wrote: > > > > > On 06/12/2020 14:32, Otto Moerbeek wrote: > > > > On Sun, Dec 06, 2020 at 02:19:05PM +, SW wrote: > > > > > > > >>

Re: add a section exit status in syspatch.8

2020-07-21 Thread Antoine Jacoutot
On Tue, Jul 21, 2020 at 03:21:24PM +0200, Solene Rapenne wrote: > this diff adds an EXIT STATUS section to syspatch man page. > > syspatch exit with 1 on errors and 2 if syspatch updates itself. > > If syspatch does nothing (because no syspatch) or install syspatches > successfully, the exit

Re: empty rc.firsttime when installing

2020-07-14 Thread Antoine Jacoutot
Hmm. rc.firsttime could be populated in advance by configuration tools and what now and in this case you will loose the changes, no? — Antoine > On 14 Jul 2020, at 15:04, Denis Fondras wrote: > > I was upgrading an EdgeRouter and it restarted multiple times instead of > booting > /bsd > >

Re: Add note about example dhclient.conf

2020-02-08 Thread Antoine Jacoutot
On Sat, Feb 08, 2020 at 05:33:34PM +0100, Ingo Schwarze wrote: > Hi, > > i think i said it before: i hate /etc/examples/ and think that the > directory ought to be mostly empty. With the exception of rare > cases like bgpd(8), where you have to provide a lot of information > before you can

Re: sshd proctitle [Re: CVS: cvs.openbsd.org: src]

2020-01-24 Thread Antoine Jacoutot
Great :-) Ok aja — Antoine > On 24 Jan 2020, at 13:02, Stuart Henderson wrote: > > On 2020/01/23 21:20, Damien Miller wrote: >>> On Thu, 23 Jan 2020, Damien Miller wrote: >>> On Thu, 23 Jan 2020, Damien Miller wrote: >>> What information would you like there? We could put the

Re: sshd proctitle [Re: CVS: cvs.openbsd.org: src]

2020-01-22 Thread Antoine Jacoutot
On Thu, Jan 23, 2020 at 02:36:43PM +1100, Damien Miller wrote: > On Wed, 22 Jan 2020, Stuart Henderson wrote: > > > On 2020/01/21 15:39, Damien Miller wrote: > > > CVSROOT: /cvs > > > Module name: src > > > Changes by: d...@cvs.openbsd.org2020/01/21 15:39:57 > > > > > > Modified

Re: syspatch -c as non-root

2019-12-14 Thread Antoine Jacoutot
On Sat, Dec 14, 2019 at 10:12:36AM +0300, Vadim Zhukov wrote: > Hello all (long time no see!) > > TL;DR: Allow syspatch -c run under non-priviledged user. > > Reasoning: instead of putting syspatch -c in crontab, I've implemented > a Zabbix trigger. Since the Zabbix agent runs as unpriviledged

Re: /etc/rc: remove useless cat

2019-11-10 Thread Antoine Jacoutot
On Sun, Nov 10, 2019 at 02:51:32PM +0100, Christian Weisgerber wrote: > Since /etc/rc uses ksh constructs anyway, we should use $( of $(cat file). > > ok? OK aja > > Index: rc > === > RCS file: /cvs/src/etc/rc,v > retrieving

Re: sysupgrade(8) and http_proxy

2019-11-03 Thread Antoine Jacoutot
On Sun, Nov 03, 2019 at 12:16:56PM +0100, Florian Obser wrote: > I like it, if someone who is fluent in ksh line noise could please > verify and commit, that would be awesome, thanks. Why not let the installer handle this? It already has code for it. sysupgrade ony needs to create the proper

Re: Questions about syspatch(8) mtree(8) behaviour

2019-09-03 Thread Antoine Jacoutot
On Mon, Sep 02, 2019 at 08:58:01PM +0200, Hiltjo Posthuma wrote: > On Mon, Sep 02, 2019 at 12:07:59PM -0600, Theo de Raadt wrote: > > Hiltjo Posthuma wrote: > > > > > Hi, > > > > > > I have three questions regarding a behaviour of syspatch(8) with mtree(8). > > > > > > 1. I noticed when

Re: ld.so speedup (part 2)

2019-04-27 Thread Antoine Jacoutot
On Sat, Apr 27, 2019 at 09:55:33PM +0800, Nathanael Rensen wrote: > The diff below speeds up ld.so library intialisation where the dependency > tree is broad and deep, such as samba's smbd which links over 100 libraries. > > See for example https://marc.info/?l=openbsd-misc=155007285712913=2 > >

Re: vmctl: usage on extra arguments

2019-03-01 Thread Antoine Jacoutot
On Fri, Mar 01, 2019 at 01:28:58PM +0100, Klemens Nanni wrote: > I blatantly missed the argc/argv adjustments after getopt(3), resulting > in valid commands like `vmctl create a -s 1G' to fail. > > Noticed by ajacoutot the hard way. > > OK? Works for me (tm). ok :-) > > Index:

Re: add getrun for rcctl(8)

2019-02-18 Thread Antoine Jacoutot
On Mon, Feb 18, 2019 at 11:21:38AM +0900, YASUOKA Masahiko wrote: > Hi, > > On Sun, 17 Feb 2019 10:55:11 +0100 > Antoine Jacoutot wrote: > > On Fri, Feb 15, 2019 at 02:50:22PM +0900, YASUOKA Masahiko wrote: > >> On Fri, 15 Feb 2019 14:45:14 +0900 (JST) &

Re: add getrun for rcctl(8)

2019-02-17 Thread Antoine Jacoutot
On Fri, Feb 15, 2019 at 02:50:22PM +0900, YASUOKA Masahiko wrote: > On Fri, 15 Feb 2019 14:45:14 +0900 (JST) > YASUOKA Masahiko wrote: > > The diff adds "getrun" command for rcctl(8) which shows the daemon > > variables from the running daemon. > > > > ok? comment? > > Sorry, previous diff is

Re: allow weak passwd

2018-12-12 Thread Antoine Jacoutot
On Tue, Dec 11, 2018 at 10:46:18PM +0100, Mark Kettenis wrote: > > Date: Tue, 11 Dec 2018 22:39:37 +0100 > > From: Antoine Jacoutot > > > > On Tue, Dec 11, 2018 at 04:27:19PM -0500, Ted Unangst wrote: > > > Mark Kettenis wrote: > > > > > From: &q

Re: allow weak passwd

2018-12-11 Thread Antoine Jacoutot
On Tue, Dec 11, 2018 at 04:27:19PM -0500, Ted Unangst wrote: > Mark Kettenis wrote: > > > From: "Ted Unangst" > > > Date: Mon, 10 Dec 2018 14:14:08 -0500 > > > Content-Type: text/plain; charset=utf-8 > > > > > > So I was actually looking at the passwd check rules because I wanted > > > to add a

Re: rcctl, tiny typo in comment

2018-10-21 Thread Antoine Jacoutot
On Sun, Oct 21, 2018 at 01:37:51PM +0200, Hiltjo Posthuma wrote: > Hi, > > Below a patch for a very small typo in a comment in /usr/sbin/rcctl > > > diff --git a/usr.sbin/rcctl/rcctl.sh b/usr.sbin/rcctl/rcctl.sh > index 61ad76a4987..3b2f0cabe49 100644 > --- a/usr.sbin/rcctl/rcctl.sh > +++

Re: Add $daemon_nice to rc.subr

2018-09-03 Thread Antoine Jacoutot
sept. 2018 à 23:10, Antoine Jacoutot a > écrit : > > > > On Mon, Sep 03, 2018 at 10:34:51PM +0200, Thomas de Grivel wrote: > > > Hello, > > > > Hi. > > > > > Following patch allows sysadmins to configure nice values for RC daemons. >

Re: Add $daemon_nice to rc.subr

2018-09-03 Thread Antoine Jacoutot
On Mon, Sep 03, 2018 at 10:34:51PM +0200, Thomas de Grivel wrote: > Hello, Hi. > Following patch allows sysadmins to configure nice values for RC daemons. > Default nice value is set to 10 as I wish to prioritize interactive > applications over system daemons and I think most probably do too. >

Re: tracking down sources of spin cpu%

2018-07-25 Thread Antoine Jacoutot
> I see the exact same issue as you do. > I'll try with s/modesetting/intel and see if it improves things. OK I can already confirm this "fixes" the issue for me. -- Antoine

Re: tracking down sources of spin cpu%

2018-07-25 Thread Antoine Jacoutot
On Wed, Jul 25, 2018 at 10:22:15AM +0100, Stuart Henderson wrote: > On 2018/07/24 23:07, Stuart Henderson wrote: > > My workstation is freezing up a lot again (usually in 30-60ish second > > bursts with no or very slow response to mouse or keyboard or screen > > updates). > > > > I have a status

Re: syspatch(8) exit code

2018-04-13 Thread Antoine Jacoutot
On Fri, Apr 13, 2018 at 07:37:24AM -0600, Theo de Raadt wrote: > No. There are no patches yet, and it can't handle an empty directory. > > I am waiting for aja and robert to decide what to do about it. Otherwise > we'll have thus fuss at the start of every new release. > > Come on guys, make a

Re: Allow syspatch to use ftp

2018-03-10 Thread Antoine Jacoutot
On Sun, Mar 04, 2018 at 05:49:52PM +, Andrew Marks wrote: > The below allows "ftp://; url in /etc/installurl. > > Now the discussion part, there are at least three utilities I know of > which use /etc/installurl to find OpenBSD "sources" > 1. syspatch > 2. pkg_add > 3. installation (I don't

Re: syspatch / reorder_kernel

2017-12-12 Thread Antoine Jacoutot
On Tue, Dec 12, 2017 at 11:56:20AM +, Stuart Henderson wrote: > Quick thought (I don't have time to look further now but I just noticed > this and wanted to get it down before I forget). > > I just upgraded an Alix to 6.2 release and then wanted to run syspatch > afterwards. I logged in just

Re: clang: Avoid EBX/RBX

2017-11-17 Thread Antoine Jacoutot
On Fri, Nov 17, 2017 at 08:46:28AM +, Stuart Henderson wrote: > On 2017/11/17 09:36, Antoine Jacoutot wrote: > > > here are. Personally I'd like to see devel/libexecinfo to be removed > > > from ports if possible. > > > > +1 > > I even volunteer to d

Re: clang: Avoid EBX/RBX

2017-11-17 Thread Antoine Jacoutot
> here are. Personally I'd like to see devel/libexecinfo to be removed > from ports if possible. +1 I even volunteer to do that part. AFAIK it's not a *hard* requirement for anything. -- Antoine

Re: Mention start, stop, restart, reload, and check in rcctl.8

2017-09-15 Thread Antoine Jacoutot
On Fri, Sep 15, 2017 at 10:16:22AM +0100, George Brown wrote: > Arguably these options are fairly obvious but I was prompted by the fact > that "check" is not mentioned in the man page and this differs from the > nomenclature of "status" often used in other systems. > > This is the first time

Re: [PATCH] Syspatch not clearing needed free space counter

2017-08-17 Thread Antoine Jacoutot
On Thu, Aug 03, 2017 at 06:26:47PM -0400, trondd wrote: > Stuart Henderson wrote: > > > Ah, perhaps the change to disk behaviour wasn't reflected in calculations > > then.. > > I got it figured out. > > In the checkfs function, the 'eval $(stat...)' command stores a list

Re: [PATCH] Do not mention newaliases(8) in /etc/mail/aliases

2017-05-31 Thread Antoine Jacoutot
On May 31, 2017 11:35:28 AM GMT+02:00, Consus wrote: >OpenBSD defaults to file table now so there is no need in running >newaliases(8). >--- > etc/mail/aliases | 8 ++-- > 1 file changed, 2 insertions(+), 6 deletions(-) > >diff --git a/etc/mail/aliases b/etc/mail/aliases >index

Re: printcap(5) error

2017-05-14 Thread Antoine Jacoutot
On Sat, May 13, 2017 at 12:52:04PM -0400, mar...@martinbrandenburg.com wrote: > printcap(5) says: > > NameTypeDefault Description > sd str /var/spool/outputspool directory > > However as far as I can tell, the actual default is > /var/spool/output/lpd.

Re: OpenBSD Errata: May 2nd, 2017

2017-05-03 Thread Antoine Jacoutot
On Tue, May 02, 2017 at 10:26:02PM -0400, T.J. Townsend wrote: > Errata patches for dhcpd, vmm, LibreSSL and softraid have been released > for OpenBSD 6.1 today. > > Details can be found on this page: https://www.openbsd.org/errata61.html > > Binary updates for the amd64 and i386 platforms are

Re: PATCH: change tiling behaviour in cwm(1)

2017-04-24 Thread Antoine Jacoutot
On April 24, 2017 2:19:31 PM GMT+02:00, Okan Demirmen <o...@demirmen.com> wrote: >On Wed 2017.04.12 at 10:38 +0200, Antoine Jacoutot wrote: >> On Tue, Apr 11, 2017 at 06:00:21PM +0200, Gerrit Meyerheim wrote: >> > Hi @tech, >> > >> > The way cwm(1) c

Re: PATCH: change tiling behaviour in cwm(1)

2017-04-12 Thread Antoine Jacoutot
On Tue, Apr 11, 2017 at 06:00:21PM +0200, Gerrit Meyerheim wrote: > Hi @tech, > > The way cwm(1) currently implements tiling (off by default) is > corresponding to this for vtile, > > - > | | | > | | 1 | > |

Re: sync root.mail

2017-03-29 Thread Antoine Jacoutot
On Wed, Mar 29, 2017 at 09:40:32PM +0200, Christian Weisgerber wrote: > Antoine Jacoutot: > > > Why not just: > > > > # pkg_add -v rsync chromium emacs--no_x11 > > > > So we don't have to change it each release? > > Because people won't let

Re: sync root.mail

2017-03-29 Thread Antoine Jacoutot
On Wed, Mar 29, 2017 at 09:10:49PM +0200, Christian Weisgerber wrote: > Sync the version of the example package. ok? Why not just: # pkg_add -v rsync chromium emacs--no_x11 So we don't have to change it each release? > Index: root.mail >

Re: wpa key vs wep key

2017-03-21 Thread Antoine Jacoutot
On Tue, Mar 21, 2017 at 02:10:48PM +0900, Stefan Sperling wrote: > I see no reason to leave WEP enabled if a WPA key is set, and leaving > WPA enabled when a WEP key is set. > > Several cases of "my wifi suddenly stopped working" turned out to be due > to stale WEP keys interfering with WPA. I

Re: rcctl(8) does not set flags

2017-02-15 Thread Antoine Jacoutot
On Wed, Feb 15, 2017 at 02:22:31PM -0200, Daniel Bolgheroni wrote: > Hi tech@, > > Setting, for example > > # rcctl enable nfsd > # rcctl set nfsd flags -tun 4 > > has no effect on /etc/rc.conf.local. This is also true for other cases > where the default flags for the daemon are equal to the

Re: add empty /root/.ssh/authorized_keys to mtree/sets ?

2017-02-10 Thread Antoine Jacoutot
On Fri, Feb 10, 2017 at 10:00:51AM +0100, Landry Breuil wrote: > On Fri, Feb 10, 2017 at 09:36:16AM +0100, Antoine Jacoutot wrote: > > On Thu, Feb 09, 2017 at 06:19:54PM +0100, Landry Breuil wrote: > > > On Sun, Feb 05, 2017 at 08:37:31PM +, Stuart Henderson wrote: > >

Re: add empty /root/.ssh/authorized_keys to mtree/sets ?

2017-02-10 Thread Antoine Jacoutot
On Thu, Feb 09, 2017 at 06:19:54PM +0100, Landry Breuil wrote: > On Sun, Feb 05, 2017 at 08:37:31PM +, Stuart Henderson wrote: > > On 2017/02/05 09:53, Robert Peichaer wrote: > > > On Sun, Feb 05, 2017 at 10:46:41AM +0100, Landry Breuil wrote: > > > > Hi, > > > > > > > > when installing

syspatch early testing

2016-12-01 Thread Antoine Jacoutot
Hi. syspatch(8), a "binary" patch system for -release is now ready for early testing. This does not use binary diffing to update the system, but regular signed tarballs containing the updated files (ala installer). I would appreciate feedback on the tool. But please send it directly to *me*,

Re: reloading pf through ansible easy hook

2016-11-23 Thread Antoine Jacoutot
On Wed, Nov 23, 2016 at 09:40:48AM -0500, sven falempin wrote: > On Mon, Nov 21, 2016 at 5:48 PM, Antoine Jacoutot <ajacou...@bsdfrog.org> > wrote: > > > > On Mon, Nov 21, 2016 at 05:34:35PM -0500, sven falempin wrote: > > > Ansible is already managing

Re: reloading pf through ansible easy hook

2016-11-21 Thread Antoine Jacoutot
On Mon, Nov 21, 2016 at 05:34:35PM -0500, sven falempin wrote: > Ansible is already managing pkg and service of openBSD , cool > > If one want to manage pf with it, and push or modify a few files, > on must run - command: /sbin/pfctl -f {{ dank.config }} > > Yet - service could be use, if this

Re: syspatch(8): How to install patches?

2016-11-09 Thread Antoine Jacoutot
On Wed, Nov 09, 2016 at 07:12:38PM +1100, bytevolc...@safe-mail.net wrote: > Hello, > > In the manual page for syspatch(8), it provides a list of options, yet > does not specify how to invoke it for fetching and installing the > latest patches. > > To avoid this confusion, it is worth adding a

Re: rcctl ls - services and daemons

2016-10-10 Thread Antoine Jacoutot
On Mon, Oct 10, 2016 at 06:43:05PM +0200, Jan Stary wrote: > The rcctl(8) manpage makes the distinction between 'daemons' and 'services', > and the description of some items in the output of 'rcctl ls' > mentions only 'daemons'. But they are both 'services and deamons' > in all cases (tested with

Re: Improve error message in rcctl(8)

2016-09-06 Thread Antoine Jacoutot
On Tue, Sep 06, 2016 at 04:09:49PM -0400, Anthony Coulter wrote: > Regarding Jiri's suggestion: Here is a diff that makes > `rcctl ls all' only list executable files with valid service > names. > > This diff also fixes two problems with my original submission: > 1. The use of `[' instead of `[['

Re: Improve error message in rcctl(8)

2016-09-06 Thread Antoine Jacoutot
On Tue, Sep 06, 2016 at 09:01:08PM +0200, ludovic coues wrote: > 2016-09-06 20:53 GMT+02:00 Antoine Jacoutot <ajacou...@bsdfrog.org>: > > On Tue, Sep 06, 2016 at 12:29:58PM -0400, Anthony Coulter wrote: > >> Sometimes when I restart a service after changing its c

Re: Improve error message in rcctl(8)

2016-09-06 Thread Antoine Jacoutot
On Tue, Sep 06, 2016 at 12:29:58PM -0400, Anthony Coulter wrote: > Sometimes when I restart a service after changing its configuration file > I accidentally type: > > # rcctl restart smtpd.conf > /usr/sbin/rcctl: ${cached_svc_is_special_smtpd.conf}: bad substitution > /usr/sbin/rcctl[556]:

Re: sysmerge.8: Mention PAGER behavior when undefined/empty

2016-09-06 Thread Antoine Jacoutot
On Mon, Sep 05, 2016 at 05:56:03PM -0400, Michael Reed wrote: > As is done in other man pages. Committed, thank you. > === > RCS file: /cvs/src/usr.sbin/sysmerge/sysmerge.8,v > retrieving revision 1.78 > diff -u -p -r1.78 sysmerge.8

Re: libtool -bindir support

2016-08-01 Thread Antoine Jacoutot
> > > Index: Link.pm > > > === > > > RCS file: /cvs/src/usr.bin/libtool/LT/Mode/Link.pm,v > > > retrieving revision 1.31 > > > diff -u -p -p -u -r1.31 Link.pm > > > --- Link.pm 27 Apr 2016 09:50:57 - 1.31 > > > +++

Re: libtool -bindir support

2016-07-31 Thread Antoine Jacoutot
On July 31, 2016 7:14:21 PM GMT+02:00, j...@wxcvbn.org wrote: > >Making read(2) return EISDIR for directories breaks two ports, both >because they use libtool -bindir. cc(1) gets executed with an unknown >option, -bindir, and a path such as /usr/local/bin, which then gets >passed to ld(1). ld(1)

Re: rcctl ls faulty -> failed

2016-03-29 Thread Antoine Jacoutot
On Tue, Mar 29, 2016 at 10:48:17AM -0400, Ian Darwin wrote: > On Tue, Mar 29, 2016 at 03:29:27PM +0200, Antoine Jacoutot wrote: > > Hi. > > > > We'd like to rename the 'faulty' listing to 'failed'. > > i.e. rcctl ls faile

rcctl ls faulty -> failed

2016-03-29 Thread Antoine Jacoutot
Hi. We'd like to rename the 'faulty' listing to 'failed'. i.e. rcctl ls failed 'faulty' does sound a bit weird and is not obvious to remember. Now the question is should we keep supporting the 'faulty' keyword or not? I'm not in favor of adding a knob especially when it's just an alias; that'd

Re: Xorg stipple

2016-03-09 Thread Antoine Jacoutot
On Wed, Mar 09, 2016 at 05:09:13PM -0600, joshua stein wrote: > Is anyone seriously finding video/Xorg bugs through the default X > stipple pattern anymore? Xorg changed the default to draw a black > background a while ago (with stipple enabled using the -retro flag), > but we have this local

Re: diff to /etc/examples/printcap and sysmerge

2016-02-26 Thread Antoine Jacoutot
On Fri, Feb 26, 2016 at 09:17:16AM -0600, Chris Bennett wrote: > On Thu, Feb 25, 2016 at 07:56:54PM -0700, Theo de Raadt wrote: > > Chris, you continue to amaze me. > > > > Upon running sysmerge, that will break everyone's setup. > > > > Like, can you try stuff before you send it out? > > > >

Re: libgtop2: Use getifaddrs(3) instead of KVM

2015-12-05 Thread Antoine Jacoutot
On Sat, Dec 05, 2015 at 05:05:07PM +0100, Martin Pieuchot wrote: > Here's a rewrite of glibtop_get_netload_p(). I tested it with custom > code because I could not trigger this code path with our ports. > > This unbreaks devel/libgtop2 after the recent commit. > > I believe this should go

Re: rcctl action multiple daemons

2015-10-03 Thread Antoine Jacoutot
On Wed, Sep 30, 2015 at 01:40:44PM +0200, Antoine Jacoutot wrote: > On Wed, Sep 30, 2015 at 12:30:52PM +0200, Martijn van Duren wrote: > > ping > > I am still pondering the pros and cons... I'll probably commit your diff tomorrow, thanks. -- Antoine

Re: rc script for ftp-proxy with v6

2015-09-12 Thread Antoine Jacoutot
On Sat, Sep 12, 2015 at 01:20:39PM +0100, Stuart Henderson wrote: > ftp-proxy can only open one listening socket, so to use it with > both IPv4 and IPv6, two instances are required. Now that rc.subr > does exact matches of the process name, we could add a separate > rc script for this. Do we want

Re: rcctl ls: give an argument to needs_root()

2015-08-01 Thread Antoine Jacoutot
On Sat, Aug 01, 2015 at 11:23:56AM +0200, Theo Buehler wrote: If the `rcctl ls' command is given an argument that requires root, the error message has a awkward space in front of the colon since no argument is passed to the needs_root() function: $ rcctl ls started rcctl : need root

Re: sed -i

2015-07-17 Thread Antoine Jacoutot
On Fri, Jul 17, 2015 at 12:57:12PM -0400, Brian Callahan wrote: On 07/17/15 12:10, Jasper Lievisse Adriaanse wrote: Hi, Here's a diff to add the '-i' flag to sed to do inplace edits. It's mostly from FreeBSD with some adjustments to prevent a race with unlink() and fopen()

Re: [patch] Path typo in FAQ

2015-07-13 Thread Antoine Jacoutot
Fixed, thank you. On Mon, Jul 13, 2015 at 01:01:32AM -0400, Michael McConville wrote: Index: faq/faq15.html === RCS file: /cvs/www/faq/faq15.html,v retrieving revision 1.109 diff -u -p -r1.109 faq15.html --- faq/faq15.html2

Re: nsd.conf in /etc/changelist

2015-07-07 Thread Antoine Jacoutot
On Sun, May 17, 2015 at 11:42:24AM +0200, Tim van der Molen wrote: nsd.conf may contain keys, so store its checksum only. Also update the path to unbound's root.key. Thanks Tim, committed. I did change /var/unbound/db/root.key to be stored as a checksum as well. Index: changelist

Re: ftp://ftp.fr

2015-06-27 Thread Antoine Jacoutot
On Fri, Jun 26, 2015 at 05:40:19PM +0200, Antoine Jacoutot wrote: Everything else will remain the same (cvs, rsync, ...); it's *only* the FTP service that is going away. Thank you. The hosting place of ftp.fr was going under major electrical maintenance this morning. Everything should

ftp://ftp.fr

2015-06-26 Thread Antoine Jacoutot
Hi. As of tomorrow morning (CET), ftp.fr will stop serving files over FTP. It is time people move to HTTP. Everything else will remain the same (cvs, rsync, ...); it's *only* the FTP service that is going away. Thank you. -- Antoine

Re: don't error for duplicate identical type definitions

2015-06-10 Thread Antoine Jacoutot
Any objections? Considering I am the one who requested it, obviously no objection :-) I've successfully tested this with ports that used to need ports/lang/gcc/4.9 . So as far as I am concerned, OK for me. Index: c-decl.c ===

Re: luarexlib patch

2015-05-22 Thread Antoine Jacoutot
On Sun, May 10, 2015 at 10:20:10PM +0200, Mark Kettenis wrote: [ Sending this to tech@ since I don't just want the ports people to know about this ] Invoking the linker directly to build executables or shared libraries is strongly discouraged. It really doesn't work properly unless you

Re: chroot: add -c to use login class with -u

2015-05-18 Thread Antoine Jacoutot
On Mon, May 18, 2015 at 04:30:33PM -0600, Todd C. Miller wrote: Currently, chroot -u doesn't use the settings in /etc/login.conf. This adds a -c option to apply the via setusercontext(). We can't use LOGIN_SETALL since the uid change has to happen after chroot(2) and the groups may be

Re: [DIFF] /etc/rc: gracefully shut down base daemons too

2015-02-17 Thread Antoine Jacoutot
I see how a clean shutdown might matter for, say, postgres. But what is the point in shutting down cron, sshd, ntpd, or unbound right before the system is going down anyway? Shutting down stuff like pflogd and syslogd before the system is actually going down might even be harmful. If it

Re: [DIFF] /etc/rc: gracefully shut down base daemons too

2015-02-17 Thread Antoine Jacoutot
On Tue, Feb 17, 2015 at 01:20:03PM +, Craig Skinner wrote: Produces (on 5.6 release) - with start up order reversed: root@box:~ 0# halt -p stopping package daemons: greyscanner postfix sshguard. stopping base daemons: cron spamlogd spamd sshd ntpd unbound nsd pflogd syslogd. syncing

Re: unnbound vs file descriptors

2014-12-16 Thread Antoine Jacoutot
may just a naive question.. but did you sudo vipw and put unbound class for unbound user? That's not neccesary anymore these days, I believe. The rc.d subsystem takes case of setting the proper class, if available. At least it That's correct. does not document setting the login

Re: Binary code patching and paravirtualization

2014-12-09 Thread Antoine Jacoutot
in summer, I posted some paravirt patches for amd64. In response to the comments I received then, I have created some infrastructure to binary patch kernel code during boot. In order to get some feedback, I am posting the whole paravirt code patching diff here. Also, KVM users may be

fuck you pkg.conf

2014-11-29 Thread Antoine Jacoutot
Hi. So. Each time pkg_add runs, it tries to connect to either PKG_PATH or to installpath from pkg.conf to fetch quirks (I assume). When you have no network anymore (moving around with your laptop or whatever), and you want to install a local package, you're basically fucked: $ time sudo

Re: fuck you pkg.conf

2014-11-29 Thread Antoine Jacoutot
But that said, why does your pkg.conf keep returning? I don't have one on my laptop at all, I probably removed it once after installing, but it It returns each time I upgrade using bsd.rd. -- Antoine

OpenBSD 5.6 Released

2014-11-01 Thread Antoine Jacoutot
Fresh, Anil Madhavapeddy, Anthony J. Bentley, Antoine Jacoutot, Austin Hook, Benoit Lecocq, Bob Beck, Brad Smith, Brandon Mercer, Brent Cook, Bret Lambert, Brett Mahar, Brian Callahan, Camiel Dobbelaar, Charles Longeau, Chris Cappuccio, Christian Weisgerber, Christopher Zimmermann

Re: uhub(4) USB device (dis)connect

2014-10-31 Thread Antoine Jacoutot
On Fri, Oct 31, 2014 at 11:19:22PM +0100, Martin Pieuchot wrote: Device below changes uhub(4) to no longer check *all* the ports status of *all* the hubs on bus when one of those hubs requests an explore. This has the nice effect of reducing the delay/bus traffic when a port status change is

Re: rcctl: find(1) service files in /etc/rc.d

2014-10-15 Thread Antoine Jacoutot
On Wed, Oct 15, 2014 at 10:11:18AM +0100, Craig R. Skinner wrote: On 2014-10-14 Tue 00:24 AM |, Antoine Jacoutot wrote: Makes sense yes. Not sure I'd want a function just for that one liner though. I'll commit something tomorrow. Nice one, using shell internals. This restricts

Re: rcctl: find(1) service files in /etc/rc.d

2014-10-13 Thread Antoine Jacoutot
On Mon, Oct 13, 2014 at 09:31:05PM +0100, Craig R. Skinner wrote: Move 2 duplicate searches into a function. The diff also ignores (RCS) subdirectories. $ find /etc/rc.d ! -type f /etc/rc.d /etc/rc.d/RCS Makes sense yes. Not sure I'd want a function just for that one liner though. I'll

Re: rcctl(8): Fetch default service flags

2014-10-10 Thread Antoine Jacoutot
The problem occurs when I try to manage a service like nfsd which has a default set of flags. Consider the following commands, which basically performs what ansible does when the user is not supplying any extra arguments: --- # rcctl status nfsd NO # rcctl enable nfsd

Re: rcctl(8): Fetch default service flags

2014-10-10 Thread Antoine Jacoutot
On Fri, Oct 10, 2014 at 02:50:40PM +0200, Patrik Lundin wrote: On Fri, Oct 10, 2014 at 01:50:52PM +0200, Antoine Jacoutot wrote: If I now run the tool again, still without any supplied arguments, it will compare the empty set of user supplied flags to -tun 4 which always differs. So

Re: rcctl(8): Fetch default service flags

2014-10-10 Thread Antoine Jacoutot
4. Enable the service *without* any arguments (even if it has default arguments). It's been discussed at lenght in Slovenia -- that's a known issue; the move of rc.conf to a parsed config file has had similar side effects on other stuffs. I see no way to do that with rcctl(8).

Re: rcctl(8): Fetch default service flags

2014-10-10 Thread Antoine Jacoutot
Maby the expected results of our tools are different. My goal for the ansible module is that you either keep a specific set of flags in the playbook which will then get set if necessary, or you remove the flags and will instead get the system default. Not passing any arguments does not

Re: rcctl(8): Fetch default service flags

2014-10-10 Thread Antoine Jacoutot
On Fri, Oct 10, 2014 at 08:02:17PM +0100, Stuart Henderson wrote: On 2014/10/10 15:19, Patrik Lundin wrote: Not passing any arguments does not mean do not modify any flags, it means make sure we have the default flags. Because of this I will need to check if the current set of flags are the

Re: rcctl(8): special services and flags

2014-10-09 Thread Antoine Jacoutot
On Thu, Oct 09, 2014 at 09:02:14PM +0200, Patrik Lundin wrote: Hello, While working on rcctl(8) support for ansible I have run into a situation I am not sure how to deal with. Basically, if the user has supplied arguments we append flags whatever the user requested and this works good.

Re: rcctl(8): special services and flags

2014-10-09 Thread Antoine Jacoutot
An example would be if the user started out with supplying -s to ntpd and then changes his mind and removes the flag. The idea is to then call rcctl with an empty flags argument to get the default set of flags instead. It was my understanding from the man page that this was the approriate

Re: rcctl(8): Fetch default service flags

2014-10-09 Thread Antoine Jacoutot
The module wants to figure out if it should reset the flags in rc.conf.local to the default set if no arguments are supplied by the user. Can you provide an example. Because if not arguments are supplied, according to your previous mail, it should run like: rcctl enable foobar flags which

Re: rcctl(8): Fetch default service flags

2014-10-09 Thread Antoine Jacoutot
Sorry, I should probably have been more clear on the reason I need it :). The main issue from the ansible perspective is to know if we are _going_ to change something or not. The user may for example run ansible in --check mode where it should only report if it is about to change something

Re: groupdel(8): preserve `+' line

2014-10-04 Thread Antoine Jacoutot
Do people run this crap as root? I wonder. I certainly don't. Yes. Just like useradd and such. These historical tools are harcoded in so many places and are expected to be present. That said, having someone rewrites them in a sane way would be awesome. -- Antoine

Re: sysmerge auto update

2014-07-20 Thread Antoine Jacoutot
On Sun, Jul 20, 2014 at 03:52:14PM +0800, Nathanael Rensen wrote: Since the switch to SHA256, sysmerge(8) has stopped auto-upgrading files without local changes. Oh right, I forgot that part. Thanks, I'll have a look. $ diff -u a b | grep -E '^\+' | sed '1d' +SHA256 (./etc/rc) =

Re: DNS control port additions to /etc/services

2014-07-15 Thread Antoine Jacoutot
On Tue, Jul 15, 2014 at 12:22:37PM +0100, Craig R. Skinner wrote: Suggestion of add NSD, Unbound BIND control ports to /etc/services: Makes sense to me. Anyone want to OK this? Index: etc/services === RCS file:

Re: DNS control port additions to /etc/services

2014-07-15 Thread Antoine Jacoutot
On Tue, Jul 15, 2014 at 10:06:10AM -0400, Simon Perreault wrote: Le 2014-07-15 09:51, Antoine Jacoutot a écrit : +unbound-cntl 8953/tcp# Unbound validating, recursive, and caching DNS server control The IANA name for this port is ub-dns-control. http

Re: DNS control port additions to /etc/services

2014-07-15 Thread Antoine Jacoutot
I'll discuss tweaks to the diff below but I'm in two minds about whether we want it. We don't enable the control socket in unbound by default at present (there is a diff somewhere to move this to unix domain sockets which we'd much prefer over network sockets..) Be aware, there is a downside

Re: DNS control port additions to /etc/services

2014-07-15 Thread Antoine Jacoutot
Well it depends what policy we want. Looking at the file most entries have both even if only one protocol is effectively in use. Looking at the file though, most of those are older entries - I think Yes. The reason is this: # Note that it is presently the policy of IANA to assign a

Re: DNS control port additions to /etc/services

2014-07-15 Thread Antoine Jacoutot
On Tue, Jul 15, 2014 at 04:35:58PM +0100, Stuart Henderson wrote: On 2014/07/15 17:17, Antoine Jacoutot wrote: Well it depends what policy we want. Looking at the file most entries have both even if only one protocol is effectively in use. Looking at the file though, most of those

Re: /etc/services records for squid cvsyncd

2014-07-15 Thread Antoine Jacoutot
On Tue, Jul 15, 2014 at 08:37:31PM +0100, Craig R. Skinner wrote: The IANA names don't match these popular OpenBSD package's port numbers: http://www.iana.org/assignments/service-names-port-numbers/service-names-port-numbers.xhtml?search=3128 = ndl-aas, not web cache/squid

  1   2   >