GNU bug report logs - #36093
[PATCH 0/2] 'guix pack --entry-point' and Singularity service

Previous Next

Package: guix-patches;

Reported by: Ludovic Courtès <ludo <at> gnu.org>

Date: Tue, 4 Jun 2019 20:53:03 UTC

Severity: normal

Tags: patch

Done: Ludovic Courtès <ludo <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #14 received at 36093 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: Ludovic Courtès <ludovic.courtes <at> inria.fr>,
 36093 <at> debbugs.gnu.org
Subject: Re: [bug#36093] [PATCH 1/2] services: Add Singularity.
Date: Wed, 5 Jun 2019 17:02:17 +0200
[Message part 1 (text/plain, inline)]
Hi Ludo,

On Tue,  4 Jun 2019 23:01:14 +0200
Ludovic Courtès <ludo <at> gnu.org> wrote:

> +@defvr {Scheme Variable} singularity-service-type
> +This is the type of the service that runs
> +@url{https://www.sylabs.io/singularity/, Singularity}, 

Does it?
Doesn't it just "allow you to invoke"?

> +                  (substitute* (find-files "libexec/cli" "\\.exec$")
> +                    (("\\$SINGULARITY_libexecdir/singularity/bin/([a-z]+)-suid"
> +                      _ program)
> +                     (string-append "/run/setuid-programs/singularity-"
> +                                    program "-helper")))

Is absolute path OK?  There have been some efforts to get guix to relocate in
the past.  Does this apply here?

> +        ;; Create the directories that Singularity 2.6 expects to find.
> +        (for-each (lambda (directory)
> +                    (mkdir-p (string-append "/var/singularity/mnt/"
> +                                            directory)))
> +                  '("container" "final" "overlay" "session")))))

Are permissions OK?

LGTM!
[Message part 2 (application/pgp-signature, inline)]

This bug report was last modified 6 years and 41 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.