all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#41228] [PATCH 0/2] gnu: radio: Simplify instalation instructions
@ 2020-05-13  8:50 Brice Waegeneire
  2020-05-13  9:00 ` [bug#41228] [PATCH 1/2] gnu: rtl-sdr: Simplify installation instructions Brice Waegeneire
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Brice Waegeneire @ 2020-05-13  8:50 UTC (permalink / raw)
  To: 41228

Brice Waegeneire (2):
  gnu: rtl-sdr: Simplify installation instructions.
  gnu: hackrf: Simplify installation instructions.

 gnu/packages/radio.scm | 31 ++++++-------------------------
 1 file changed, 6 insertions(+), 25 deletions(-)

-- 
2.26.2





^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#41228] [PATCH 1/2] gnu: rtl-sdr: Simplify installation instructions.
  2020-05-13  8:50 [bug#41228] [PATCH 0/2] gnu: radio: Simplify instalation instructions Brice Waegeneire
@ 2020-05-13  9:00 ` Brice Waegeneire
  2020-05-13  9:00 ` [bug#41228] [PATCH 2/2] gnu: hackrf: " Brice Waegeneire
  2020-05-14  9:02 ` [bug#41228] [PATCH 0/2] gnu: radio: Simplify instalation instructions Guillaume Le Vaillant
  2 siblings, 0 replies; 5+ messages in thread
From: Brice Waegeneire @ 2020-05-13  9:00 UTC (permalink / raw)
  To: 41228

* gnu/packages/radio.scm (rtl-sdr)[description]: Replace
'modify-services' snippet with 'udev-rules-service'.
---
 gnu/packages/radio.scm | 13 ++-----------
 1 file changed, 2 insertions(+), 11 deletions(-)

diff --git a/gnu/packages/radio.scm b/gnu/packages/radio.scm
index 41105fb8c5..1e50be1bcc 100644
--- a/gnu/packages/radio.scm
+++ b/gnu/packages/radio.scm
@@ -152,17 +152,8 @@ system configuration:
 (kernel-arguments '(\"modprobe.blacklist=dvb_usb_rtl28xxu\"))
 @end lisp
 
-To install the rtl-sdr udev rules, you must add this package in the
-configuration of the udev system service. E.g.:
-
-@lisp
-(services
- (modify-services %desktop-services
-  (udev-service-type config =>
-   (udev-configuration (inherit config)
-    (rules (cons rtl-sdr
-            (udev-configuration-rules config)))))))
-@end lisp")
+To install the rtl-sdr udev rules, you must extend 'udev-service-type' with
+this package.  E.g.: @code{(udev-rules-service 'rtl-sdr rtl-sdr)}")
     (license license:gpl2+)))
 
 (define-public chirp
-- 
2.26.2





^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [bug#41228] [PATCH 2/2] gnu: hackrf: Simplify installation instructions.
  2020-05-13  8:50 [bug#41228] [PATCH 0/2] gnu: radio: Simplify instalation instructions Brice Waegeneire
  2020-05-13  9:00 ` [bug#41228] [PATCH 1/2] gnu: rtl-sdr: Simplify installation instructions Brice Waegeneire
@ 2020-05-13  9:00 ` Brice Waegeneire
  2020-05-14  9:02 ` [bug#41228] [PATCH 0/2] gnu: radio: Simplify instalation instructions Guillaume Le Vaillant
  2 siblings, 0 replies; 5+ messages in thread
From: Brice Waegeneire @ 2020-05-13  9:00 UTC (permalink / raw)
  To: 41228

* gnu/packages/radio.scm (hackrf)[description]: Replace
'modify-services' snippet with 'udev-rules-service'.
---
 gnu/packages/radio.scm | 18 ++++--------------
 1 file changed, 4 insertions(+), 14 deletions(-)

diff --git a/gnu/packages/radio.scm b/gnu/packages/radio.scm
index 1e50be1bcc..6a919548f0 100644
--- a/gnu/packages/radio.scm
+++ b/gnu/packages/radio.scm
@@ -741,20 +741,10 @@ for correctness.")
      (synopsis "User-space library and utilities for HackRF SDR")
      (description
       "Command line utilities and a C library for controlling the HackRF
-Software Defined Radio (SDR) over USB.  Installing this package installs
-the userspace hackrf utilities and C library.  To install the hackrf
-udev rules, you must add this package as a system service via
-modify-services.  E.g.:
-
-@lisp
-(services
- (modify-services
-  %desktop-services
-  (udev-service-type config =>
-   (udev-configuration (inherit config)
-    (rules (cons hackrf
-            (udev-configuration-rules config)))))))
-@end lisp")
+Software Defined Radio (SDR) over USB.  Installing this package installs the
+userspace hackrf utilities and C library.  To install the hackrf udev rules,
+you must extend 'udev-service-type' with this package.  E.g.:
+@code{(udev-rules-service 'hackrf hackrf #:groups '(\"dialout\"))}.")
      (license license:gpl2))))
 
 (define-public hamlib
-- 
2.26.2





^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [bug#41228] [PATCH 0/2] gnu: radio: Simplify instalation instructions
  2020-05-13  8:50 [bug#41228] [PATCH 0/2] gnu: radio: Simplify instalation instructions Brice Waegeneire
  2020-05-13  9:00 ` [bug#41228] [PATCH 1/2] gnu: rtl-sdr: Simplify installation instructions Brice Waegeneire
  2020-05-13  9:00 ` [bug#41228] [PATCH 2/2] gnu: hackrf: " Brice Waegeneire
@ 2020-05-14  9:02 ` Guillaume Le Vaillant
  2020-05-14 13:56   ` bug#41228: " Brice Waegeneire
  2 siblings, 1 reply; 5+ messages in thread
From: Guillaume Le Vaillant @ 2020-05-14  9:02 UTC (permalink / raw)
  To: Brice Waegeneire; +Cc: 41228

[-- Attachment #1: Type: text/plain, Size: 318 bytes --]


Brice Waegeneire <brice@waegenei.re> skribis:

> Brice Waegeneire (2):
>   gnu: rtl-sdr: Simplify installation instructions.
>   gnu: hackrf: Simplify installation instructions.
>
>  gnu/packages/radio.scm | 31 ++++++-------------------------
>  1 file changed, 6 insertions(+), 25 deletions(-)

It looks good to me.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#41228: [PATCH 0/2] gnu: radio: Simplify instalation instructions
  2020-05-14  9:02 ` [bug#41228] [PATCH 0/2] gnu: radio: Simplify instalation instructions Guillaume Le Vaillant
@ 2020-05-14 13:56   ` Brice Waegeneire
  0 siblings, 0 replies; 5+ messages in thread
From: Brice Waegeneire @ 2020-05-14 13:56 UTC (permalink / raw)
  To: Guillaume Le Vaillant; +Cc: 41228-done

On 2020-05-14 09:02, Guillaume Le Vaillant wrote:
> Brice Waegeneire <brice@waegenei.re> skribis:
> 
>> Brice Waegeneire (2):
>>   gnu: rtl-sdr: Simplify installation instructions.
>>   gnu: hackrf: Simplify installation instructions.
>> 
>>  gnu/packages/radio.scm | 31 ++++++-------------------------
>>  1 file changed, 6 insertions(+), 25 deletions(-)
> 
> It looks good to me.

Thanks for the review Guillaume.

Pushed as 32290b2c35 and aee7d000f8.




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-05-14 14:02 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-05-13  8:50 [bug#41228] [PATCH 0/2] gnu: radio: Simplify instalation instructions Brice Waegeneire
2020-05-13  9:00 ` [bug#41228] [PATCH 1/2] gnu: rtl-sdr: Simplify installation instructions Brice Waegeneire
2020-05-13  9:00 ` [bug#41228] [PATCH 2/2] gnu: hackrf: " Brice Waegeneire
2020-05-14  9:02 ` [bug#41228] [PATCH 0/2] gnu: radio: Simplify instalation instructions Guillaume Le Vaillant
2020-05-14 13:56   ` bug#41228: " Brice Waegeneire

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.