Re: Improve error message in rcctl(8) again

2023-07-13 Thread Antoine Jacoutot
On Mon, Jul 10, 2023 at 04:46:28PM -0400, Anthony Coulter wrote: > Seven years ago I tried to restart a configuration file and it didn't > work out: https://marc.info/?l=openbsd-tech&m=147318006722787&w=2 > > This morning I tried to disable a different configuration file and got > similar results.

Re: [PATCH] [src] share/man/man8/rc.subr.8 - normalise markup

2022-10-22 Thread Antoine Jacoutot
On Wed, Oct 19, 2022 at 10:42:18PM +0100, Raf Czlonka wrote: > On Fri, May 27, 2022 at 01:28:07PM BST, Antoine Jacoutot wrote: > > > > rc.subr.8 committed, thanks. > > Hi Antoine, > > I only just noticed that the diff hasn't been applied/committed in > its ent

Re: /etc/rc.d/iked regression after r1.9

2022-09-03 Thread Antoine Jacoutot
On Sat, Sep 03, 2022 at 11:51:41PM +, Lucas wrote: > Hello tech@, > > commit r1.9 removed the rc_exec call in iked's rc_pre. Because of that, > the exit code of rc_pre is that of the && list. In the case of > sasyncd_flags=NO, this means that rc_pre fails and triggers the break > in the while

Re: [PATCH] [src] share/man/man8/rc.{d,subr}.8 - normalise markup

2022-05-27 Thread Antoine Jacoutot
On Fri, May 27, 2022 at 12:34:16PM +0100, Raf Czlonka wrote: > On Fri, May 27, 2022 at 12:06:53PM BST, Stuart Henderson wrote: > > On 2022/05/27 11:43, Raf Czlonka wrote: > > > Hello, > > > > > > daemon_logger was the odd one out in rc.subr(8). > > > > I think that is correct. > > > > > While th

Re: rc.subr: implement rc_execdir

2022-05-22 Thread Antoine Jacoutot
Here's new diff that implements daemon_execdir. Index: etc/rc.d/rc.subr === RCS file: /cvs/src/etc/rc.d/rc.subr,v retrieving revision 1.153 diff -u -p -r1.153 rc.subr --- etc/rc.d/rc.subr21 May 2022 10:50:09 - 1.153 +++ e

Re: rc.subr: implement rc_execdir

2022-05-21 Thread Antoine Jacoutot
>> On 5/21/2022 12:24 PM, Antoine Jacoutot wrote: >>> On Sat, May 21, 2022 at 12:04:03PM -0400, A Tammy wrote: >>>> On 5/21/2022 10:53 AM, Antoine Jacoutot wrote: >>>>> Hi. >>>>> >>>>> This diff allows to configure an direct

Re: rc.subr: implement rc_execdir

2022-05-21 Thread Antoine Jacoutot
On Sat, May 21, 2022 at 12:04:03PM -0400, A Tammy wrote: > > On 5/21/2022 10:53 AM, Antoine Jacoutot wrote: > > Hi. > > > > This diff allows to configure an directory from which we run rc.d commands. > > This can be useful for daemons that must cd into a specifi

rc.subr: implement rc_execdir

2022-05-21 Thread Antoine Jacoutot
Hi. This diff allows to configure an directory from which we run rc.d commands. This can be useful for daemons that must cd into a specific directory prio launching. Here's an example for the automounter. Some rc.d scripts in ports can benefit from this as well (so we can remove handcrafted rc_st

Re: turn rc_exec variable into an rcexec function

2022-05-20 Thread Antoine Jacoutot
On Tue, May 17, 2022 at 09:12:08AM +0200, Antoine Jacoutot wrote: > Hi folks. > > I would like to move away from the rc_exec variable to an rcexec() function > for > rc.d scripts. > That will allow daemon_logger to work out of the box even for manually > crafted rc_start(

turn rc_exec variable into an rcexec function

2022-05-17 Thread Antoine Jacoutot
Hi folks. I would like to move away from the rc_exec variable to an rcexec() function for rc.d scripts. That will allow daemon_logger to work out of the box even for manually crafted rc_start() functions. I will also simplify the addition of new features like daemon_startdir. I have tested this w

Re: rc.subr - allow setting daemon start directory

2022-05-13 Thread Antoine Jacoutot
On Fri, May 13, 2022 at 07:06:23PM -0400, aisha wrote: > > On 5/13/2022 7:03 PM, Antoine Jacoutot wrote: > > On Fri, May 13, 2022 at 07:01:30PM -0400, aisha wrote: > > > On 22/05/06 08:28AM, Antoine Jacoutot wrote: > > > > On Thu, May 05, 2022 at 04:

Re: rc.subr - allow setting daemon start directory

2022-05-05 Thread Antoine Jacoutot
On Thu, May 05, 2022 at 04:49:10PM -0600, Aaron Bieber wrote: > > > On Thu, May 5, 2022, at 4:46 PM, Antoine Jacoutot wrote: > > Can you elaborate? > > Do they need to start from a specific directory or from a directory > > they have write access to? > > Because

Re: rc.subr - allow setting daemon start directory

2022-05-05 Thread Antoine Jacoutot
Can you elaborate? Do they need to start from a specific directory or from a directory they have write access to? Because we could cd /tmp unconditionally — Antoine > On 6 May 2022, at 00:43, Aaron Bieber wrote: > >  > >> On Thu, May 5, 2022, at 3:24 PM, Antoine Jacout

Re: rc.subr - allow setting daemon start directory

2022-05-05 Thread Antoine Jacoutot
Not a fan adding a knob for this. I don’t remember why amd needs to start from /etc/amd. — Antoine > On 5 May 2022, at 23:12, aisha wrote: > > Hi, > I've attached a patch which allows setting the daemon start directory using > the variable `daemon_startdir`. > I think this is useful in the

Re: rcctl.8: Add missing variable

2022-04-01 Thread Antoine Jacoutot
On Thu, Mar 31, 2022 at 09:46:07PM -0500, Matthew Martin wrote: > diff --git rcctl.8 rcctl.8 > index 6c3048b834c..93a76a937c1 100644 > --- rcctl.8 > +++ rcctl.8 > @@ -64,6 +64,7 @@ can be one of > .Cm class , > .Cm flags , > .Cm logger , > +.Cm rtable , > .Cm status , > .Cm timeout > or Inde

Re: patch: nm(1): add support for symbols created with -ffunction-sections

2021-11-06 Thread Antoine Jacoutot
On Sat, Nov 06, 2021 at 03:20:03PM +0100, Sebastien Marie wrote: > Hi, > > aja@ shows me some problems with x11/gnome/librsvg update (the port is > Rust based), and I finally tracked the problem inside nm(1). > > I will not speak of Rust anymore, and will use only C for the example. > > When an

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 be

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 sysup

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

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 stat

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 start

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 firs

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 f

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 us

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 revisi

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 auto_

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

2019-09-02 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 applyin

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&m=155007285712913&w=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: usr.sbin/vmctl/

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 br

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 f

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 > +++ b/usr.

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 b

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 k

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 a

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

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 I've

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 of disk > devices and

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 c1ac04b5a81..045

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 al

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 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) currently implements tiling (o

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&

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 thi

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 fl

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 'throw

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*, ther

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 > wrote: > > > > On Mon, Nov 21, 2016 at 05:34:35PM -0500, sven falempin wrote: > > > Ansible is already managing pkg and service of openBSD , cool > &

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 gl

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 li

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 e

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 `[[' c

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 : > > On Tue, Sep 06, 2016 at 12:29:58PM -0400, Anthony Coulter wrote: > >> Sometimes when I restart a service after changing its configuration file > &

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

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 > > > +++ Link.

Re: libtool -bindir support

2016-08-01 Thread Antoine Jacoutot
On Sun, Jul 31, 2016 at 07:21:39PM +0200, Antoine Jacoutot wrote: > 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 unkno

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'. >

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 a

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 chan

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? > > > > I'

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 upstre

Re: rc.d/rebound: check for rebound.conf

2015-12-04 Thread Antoine Jacoutot
On Fri, Dec 04, 2015 at 01:41:55PM +0100, Christian Weisgerber wrote: > Require /etc/rebound.conf to exist before starting rebound. > > Otherwise, if the configuration file doesn't exist, rebound will > error out, but only after daemonizing, so the rc system won't notice > the failure. > > ok? I

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: rcctl action multiple daemons

2015-09-30 Thread Antoine Jacoutot
On Wed, Sep 30, 2015 at 12:30:52PM +0200, Martijn van Duren wrote: > ping I am still pondering the pros and cons... > 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 ru

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 t

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 pr

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

Re: [patch] Path typo in FAQ

2015-07-12 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.html

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. Everyt

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 yo

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 sp

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

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: > > # halt -p > stopping package daemons: greyscanner postfix sshguard. > stopping base daemons: cron spamlogd spamd sshd ntpd unbound nsd pflogd > syslogd. > syncing disks..

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 l

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

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

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 pkg_add

OpenBSD 5.6 Released

2014-11-01 Thread Antoine Jacoutot
ially have still helped us with our goal of improving the quality of the software. Our developers are: Aaron Bieber, Alexander Bluhm, Alexander Hall, Alexandr Shadchin, Alexandre Ratchov, Andrew Fresh, Anil Madhavapeddy, Anthony J. Bentley, Antoine Jacoutot, Austin Hook, Benoit Lecocq,

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 chang

  1   2   3   >