Subject: [bug#33265] [WIP RFC] services: Add file system monitor service.

From: Clément Lassieur <[email-masked]>
Date: Mon, 05 Nov 2018 10:36:00 +0100

Hi Danny,

I don't have time to do a proper review, but just one comment:

Danny Milosavljevic <[email-masked]> writes:

> +          (test-assert "file system monitor running"
> +            (marionette-eval
> +             '(begin
> +                (use-modules (gnu services herd))
> +                (match (start-service 'monitor-file-system)
> +                  (#f #f)
> +                  (('service response-parts ...)
> +                   (match (assq-ref response-parts 'running)
> +                     ((pid) (number? pid))))))
> +             marionette))

How is that better than:

    (test-assert "file system monitor running"
      (marionette-eval
       '(begin
         (use-modules (gnu services herd))
         (start-service 'monitor-file-system))
       marionette))

'start-service' should return #f if the service fails to start.

Cheers,
Clément