Re: Improve error message in rcctl(8)

2016-09-06 Thread ludovic coues
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: ${cached_svc_is_special_smtpd.c

Re: Make rc scripts use [[ instead of [

2016-09-05 Thread ludovic coues
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

2016-07-19 Thread ludovic coues
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 unpack > the ports tree ta

Re: Small change suggestion to the front page design

2016-06-28 Thread ludovic coues
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] http://leaverou.github.io/contrast-ratio/#rgb%28

usbdump, a tool to print usb descriptor

2015-10-25 Thread Ludovic Coues
Usbdump is the result of my work during this summer of code. It is a tool inspired by pcidump. It reads usb device descriptor and print them with a bit of parsing. It aim to be a more friendly alternative to lsusb. There is two main difference with pcidump, boiling down to the nature of the usb pr

Re: merging usb_video_header_desc and usb_video_header_desc_all

2015-08-15 Thread Ludovic Coues
On 13/08/15(Thu) 19:41, ludovic coues wrote: > 2015-08-13 15:11 GMT+02:00 Martin Pieuchot : > > > > With this diff I get the following error on my x220: > > > > uvideo_vs_negotiation: uvideo0: no frame descriptors found! > > > > I have this camera: > >

Re: merging usb_video_header_desc and usb_video_header_desc_all

2015-08-13 Thread ludovic coues
2015-08-13 15:11 GMT+02:00 Martin Pieuchot : > 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 .

Re: Less strict requirement for USB_REQUEST ioctl

2015-08-10 Thread ludovic coues
2015-08-10 22:56 GMT+02:00 Martin Pieuchot : > 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 writ

Less strict requirement for USB_REQUEST ioctl

2015-07-30 Thread Ludovic Coues
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: /cvs/src/sys

Re: merging usb_video_header_desc and usb_video_header_desc_all

2015-07-23 Thread Ludovic Coues
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 tes

Re: Missing descriptor in uvideo.h

2015-07-21 Thread Ludovic Coues
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: doas failsafe

2015-07-21 Thread ludovic coues
2015-07-21 10:56 GMT+02:00 Theo de Raadt : >> 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 what you are > talki

Re: doas failsafe

2015-07-21 Thread ludovic coues
2015-07-21 8:58 GMT+02:00 Manuel Giraud : > "Ted Unangst" 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? Should it be done elsew

Re: Missing descriptor in uvideo.h

2015-07-20 Thread ludovic coues
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

2015-07-17 Thread Ludovic Coues
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 an

Re: Rational behind split struct in uvideo.h

2015-07-16 Thread ludovic coues
2015-07-16 21:45 GMT+02:00 Martin Pieuchot : > 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

Rational behind split struct in uvideo.h

2015-07-16 Thread ludovic coues
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 baInterfaceNR

Re: Missing descriptor in uvideo.h

2015-07-15 Thread ludovic coues
2015-07-15 17:04 GMT+02:00 Martin Pieuchot : > 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 n

Missing descriptor in uvideo.h

2015-07-15 Thread Ludovic Coues
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 r

Adding USB descriptor to uvideo.h

2015-07-15 Thread Ludovic Coues
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 rev

Re: tcpdump -A: really printable characters

2015-07-13 Thread ludovic coues
2015-07-13 13:14 GMT+02:00 : > 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 -- Cordialement, Coues

Re: Missing usb descriptor definition

2015-06-29 Thread ludovic coues
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 str

Missing usb descriptor definition

2015-06-26 Thread ludovic coues
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: sys

Re: pkg_info: print used repos

2015-06-24 Thread ludovic coues
2015-06-24 11:40 GMT+02:00 Marc Espie : > 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 coinci