Re: Improve error message in rcctl(8)

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 >> I accidentally type: >> >> # rcctl restart smtpd.conf >> /usr/sbin/rcctl:

Re: Make rc scripts use [[ instead of [

2016-09-05 20:11 GMT+02:00 : > The fact that one particular system still boots is not "good enough here". > > Kind regards, > Anton > That's why he share his work. So other people can try on other system. -- Cordialement, Coues Ludovic +336 148 743 42

Re: clarity recommendation for faq15.html

faq4.html [1] already deal with the problem. [1] http://www.openbsd.org/faq/faq4.html#AddThoughts 2016-07-19 3:29 GMT+02:00 Josh Grosse : > I had a conversation with a new OpenBSD user who thought that he > may have either misunderstood or been misled by the guidance to

Re: Small change suggestion to the front page design

Any reason for not putting the diff inline ? Why the mail don't mention the drastic change to the font like using sans-serif instead of serif or removing the underline to the link ? Any reason you reduced the contrast from 4.4 [1] to 3.9 [2] ? [1]

Re: merging usb_video_header_desc and usb_video_header_desc_all

On 13/08/15(Thu) 19:41, ludovic coues wrote: 2015-08-13 15:11 GMT+02:00 Martin Pieuchot m...@openbsd.org: With this diff I get the following error on my x220: uvideo_vs_negotiation: uvideo0: no frame descriptors found! I have this camera: uvideo0 at uhub3 port 6 configuration 1

Re: merging usb_video_header_desc and usb_video_header_desc_all

2015-08-13 15:11 GMT+02:00 Martin Pieuchot m...@openbsd.org: On 23/07/15(Thu) 17:52, Ludovic Coues wrote: On 17/07/15(Fri) 17:53, Ludovic Coues wrote: Following yesterday feedback, I wrote a patch merging usb_video_header_desc and usb_video_header_desc_all in uvideo.c . Current kernel

Re: Less strict requirement for USB_REQUEST ioctl

2015-08-10 22:56 GMT+02:00 Martin Pieuchot m...@openbsd.org: On 30/07/15(Thu) 12:18, Ludovic Coues wrote: Right now, an USB_REQUEST ioctl will fail with an EBADF error if done on a device opened as read-only. With this patch, the ioctl will fail only if the USB_REQUEST is a write request. Do

Less strict requirement for USB_REQUEST ioctl

Right now, an USB_REQUEST ioctl will fail with an EBADF error if done on a device opened as read-only. With this patch, the ioctl will fail only if the USB_REQUEST is a write request. Index: sys/dev/usb/usb.c === RCS file:

Re: merging usb_video_header_desc and usb_video_header_desc_all

On 17/07/15(Fri) 17:53, Ludovic Coues wrote: Following yesterday feedback, I wrote a patch merging usb_video_header_desc and usb_video_header_desc_all in uvideo.c . Current kernel compile fine with it on amd64 and video display image. At the moment, I can't test the patch on other platform

Re: doas failsafe

2015-07-21 10:56 GMT+02:00 Theo de Raadt dera...@cvs.openbsd.org: Ability to define alias in the doas config file might be nice. Just like ssh with the ssh_config file. I have always wanted a .lsrc file, which would allow me to override the special options for ls, as well. That's kind of

Re: doas failsafe

2015-07-21 8:58 GMT+02:00 Manuel Giraud man...@ledu-giraud.fr: Ted Unangst t...@tedunangst.com writes: Manuel Giraud wrote: Hi, I've just shot myself in the foot after /etc/doas.conf tweaking. This patch adds a failsafe permit :wheel rule in case of syntax error. Is this safe enough?

Re: Missing descriptor in uvideo.h

On 20/07/15(Mon) 16:44, Martin Pieuchot wrote: Is it because by ``wWord'' you mean uDword? Did you consider using an union with #define? I don't know what I could do with an union but I followed the advice. I rewrote the diff, dropped the data field and used a couple of macro to access the

Re: Missing descriptor in uvideo.h

There is a typo with struct usb_video_output_header_desc, corresponding to table 3-15. Last field should be bmaControls. -- Cordialement, Coues Ludovic +336 148 743 42

merging usb_video_header_desc and usb_video_header_desc_all

Following yesterday feedback, I wrote a patch merging usb_video_header_desc and usb_video_header_desc_all in uvideo.c . Current kernel compile fine with it on amd64 and video display image. At the moment, I can't test the patch on other platform. So I'm looking for feedback, on the pattern used

Rational behind split struct in uvideo.h

I am looking for some insight about some choice in the code. In the file /sys/dev/usb/uvideo.h, the structure usb_video_header_desc store a part of an USB descriptor. The entire descriptor is stored in usb_video_header_desc_all which consist of fix, a pointer to the former struct and

Re: Rational behind split struct in uvideo.h

2015-07-16 21:45 GMT+02:00 Martin Pieuchot m...@openbsd.org: On 16/07/15(Thu) 16:24, ludovic coues wrote: But I don't know why this code is done this way. As it haven't been touched since version 1.4 in 2008, I assume it's an old way of doing things or a short term solution. If that the case

Adding USB descriptor to uvideo.h

Following diff add missing descriptor from Video USB specification 1.5 to uvideo.h and update a couple of table reference to match that version. Index: sys/dev/usb/uvideo.h === RCS file: /cvs/src/sys/dev/usb/uvideo.h,v retrieving

Re: Missing descriptor in uvideo.h

2015-07-15 17:04 GMT+02:00 Martin Pieuchot m...@openbsd.org: On 15/07/15(Wed) 16:45, Ludovic Coues wrote: Following is a diff adding missing USB descriptor to uvideo.h according to USB Video spec 1.5 . It also update a couple of table reference from spec 1.1 to 1.5 Do not hesitate to explain

Missing descriptor in uvideo.h

Following is a diff adding missing USB descriptor to uvideo.h according to USB Video spec 1.5 . It also update a couple of table reference from spec 1.1 to 1.5 Index: sys/dev/usb/uvideo.h === RCS file: /cvs/src/sys/dev/usb/uvideo.h,v

Re: tcpdump -A: really printable characters

2015-07-13 13:14 GMT+02:00 li...@wrant.com: Ignore this if it wastes time, what is 'vis' and is it platform / architecture specific? Rather means to get what it was planned to achieve probably.. $ man vis [...] NAME vis ─ display non-printable characters in a visual format --

Re: Missing usb descriptor definition

mpi@ pointed out in private that both FreeBSD and DragonFly where already splitting struct declaration and typedef and that they use another naming convention. Namely, they don't shorten descriptor not interface. Following patch apply that to each of our usb struct and update the ioctl related

Missing usb descriptor definition

This diff add the missing definition for the interface association descriptor. The format isn't the same as with other descriptor, the struct definition is separated from the typedef. Another diff should follow doing that for every descriptor with long plan being to remove the typedefs. Index:

Re: pkg_info: print used repos

2015-06-24 11:40 GMT+02:00 Marc Espie es...@nerim.net: On Tue, Jun 23, 2015 at 08:53:15PM -0600, Theo de Raadt wrote: - $state-handle_options('cCdfF:hIKLmPQ:qr:RsSUe:E:Ml:aAt', + $state-handle_options('cCdfF:hIKLmpPQ:qr:RsSUe:E:Ml:aAt', Starting to look a lot like ls. What a