Re: [PATCH 1/2] gnu: Add LIRC.

[ludo@gnu.org (Ludovic Courtès)]

Alex Kost <alezost@gmail.com> skribis:

> Ludovic Courtès (2015-03-24 10:12 +0100) wrote:
>
>> Alex Kost <alezost@gmail.com> skribis:
>>
>>> From 75c91629aeddacd054c78d8fc769fb17b6bac10a Mon Sep 17 00:00:00 2001
>>> From: Alex Kost <alezost@gmail.com>
>>> Date: Sun, 22 Mar 2015 16:23:47 +0300
>>> Subject: [PATCH 1/2] gnu: Add LIRC.
>>>
>>> * gnu/packages/lirc.scm: New file.
>>> * gnu/packages/patches/lirc-localstatedir.patch: New file.
>>> * gnu-system.am: Add them.
>>
>> [...]
>>
>>> +    (synopsis "Linux Infrared Remote Control")
>>> +    (description
>>> +     "LIRC allows to send and receive IR signals of many commonly used
>>> +remote controls.
>>> +
>>> +The most important part of LIRC is the 'lircd' daemon that will decode IR
>>> +signals received by the device drivers and provide the information on a
>>> +socket.  It will also accept commands for IR signals to be sent if the
>>> +hardware supports this.  The second daemon program called 'lircmd' will
>>> +connect to lircd and translate the decoded IR signals to mouse movements.
>>> +You can e.g. configure X to use your remote control as an input device.
>>> +
>>> +The user space applications will allow you to control your computer with
>>> +your remote control.  You can send X events to applications, start programs
>>> +and much more on just one button press.  The possible applications are
>>> +obvious: Infra-red mouse, remote control for your TV tuner card or CD-ROM,
>>> +shutdown by remote, program your VCR and/or satellite tuner with your
>>> +computer, etc.")
>>> +    (license license:gpl2+)))
>>
>> It would be ideal if you could make the description more concise
>> (roughly stripping a paragraph.)
>>
>> The rest LGTM!
>
> OK, thanks; what about this one?:
>
>     (description
>      "LIRC allows to send and receive IR signals of many commonly used

s/allows/allows computers/

> remote controls.  The most important part of LIRC is the 'lircd' daemon
> that decodes IR signals received by the device drivers.  The second daemon
> program 'lircmd' allows to translate IR signals to mouse movements.  The
> user space applications allow to control your computer with your remote

s/allow/allow you/

> control: you can send X events to applications, start programs and much
> more on just one button press.")

OK to push with these changes.

Thank you,
Ludo’.