GNU bug report logs - #59410
[PATCH] gnu: Add distrobox.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 59410 in the body.
You can then email your comments to 59410 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: Adam Faiz <adam.faiz <at> disroot.org>
To: guix-patches <at> gnu.org
Subject: [PATCH] gnu: Add distrobox.
Date: Sun, 20 Nov 2022 22:30:56 +0800

From 1fc7bf3f3c35490f95e223fef9c83ef7d2501dff Mon Sep 17 00:00:00 2001
From: AwesomeAdam54321 <adam.faiz <at> disroot.org>
Date: Fri, 18 Nov 2022 10:51:30 +0800
Subject: [PATCH] gnu: Add distrobox.

* gnu/packages/containers.scm (distrobox): New variable.
---
 gnu/packages/containers.scm | 39 ++++++++++++++++++++++++++++++++++++-
 1 file changed, 38 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/containers.scm b/gnu/packages/containers.scm
index 59eea423a1..384a263e2e 100644
--- a/gnu/packages/containers.scm
+++ b/gnu/packages/containers.scm
@@ -26,6 +26,7 @@ (define-module (gnu packages containers)
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix git-download)
+  #:use-module (guix build-system copy)
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system go)
   #:use-module (guix build-system meson)
@@ -46,7 +47,8 @@ (define-module (gnu packages containers)
   #:use-module (gnu packages selinux)
   #:use-module (gnu packages version-control)
   #:use-module (gnu packages virtualization)
-  #:use-module (gnu packages web))
+  #:use-module (gnu packages web)
+  #:use-module (gnu packages wget))

 (define-public crun
   (let ((commit "c381048530aa750495cf502ddb7181f2ded5b400"))
@@ -354,3 +356,38 @@ (define-public podman
 volumes mounted into those containers, and pods made from groups of
 containers.")
     (license license:asl2.0)))
+
+(define-public distrobox
+  (package
+    (name "distrobox")
+    (version "1.4.1")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/89luca89/distrobox")
+             (commit version)))
+       (sha256
+        (base32 "1qc66ghlq6b0nh4gk8ls64njwv96pkhz0p31k40fnicxwkfnb2jq"))
+       (file-name (git-file-name name version))))
+    (build-system copy-build-system)
+    (inputs
+     (list podman wget))
+    (arguments
+     (list #:phases
+           #~(modify-phases %standard-phases
+               (add-before 'install 'refer-to-inputs
+                 (lambda _
+                   (substitute* (find-files "." "^distrobox.*")
+                     (("podman") (string-append #$(this-package-input 
"podman") "/bin/podman"))
+                     (("wget") (string-append #$(this-package-input 
"wget") "/bin/wget"))
+                     (("command -v") "test -x"))))
+               (replace 'install
+                 (lambda _
+                   (invoke "./install" "--prefix" #$output))))))
+    (home-page "https://distrobox.privatedns.org/")
+    (synopsis "Fancy wrapper to create and start containers highly 
integrated with the hosts")
+    (description
+     "Distrobox is a fancy wrapper around Podman or Docker
+to create and start containers highly integrated with the hosts.")
+    (license license:gpl3+)))
-- 
2.38.0

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

From: Ricardo Wurmus <rekado <at> elephly.net>
To: 59410 <at> debbugs.gnu.org
Subject: [PATCH] gnu: Add distrobox.
Date: Mon, 19 Dec 2022 17:30:23 +0100

Hi,

thanks for the patch.

I’m always a bit worried when I see the copy-build-system, because the
lack of a configuration step often means that the tool makes possibly
invalid assumptions about the runtime environment.

I see that you’re patching the locations of “podman”, “wget”, and
“command -v”, but the tools contain even more references, e.g. to
“curl”, “grep”, “sed”, coreutils, “host-spawn”, “sudo”, “pinentry”,
“less”, …

It may be a better idea to wrap all the executables in a reasonable
default environment.

What do you think?

-- 
Ricardo

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

From: omlet <at> tuta.io
To: 59410 <at> debbugs.gnu.org
Subject: [PATCH] gnu: Add distrobox.
Date: Mon, 19 Dec 2022 15:48:14 +0100 (CET)

[Message part 1 (text/plain, inline)]

Its possible approved the package?
-- 
 Enviado de forma segura com Tutanota. Obtenha a sua caixa de correio criptografada, sem publicidade.

[Message part 2 (text/html, inline)]

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

From: Adam Faiz <adam.faiz <at> disroot.org>
To: 59410 <at> debbugs.gnu.org
Cc: rekado <at> elephly.net
Subject: [bug#59410] [PATCH] gnu: Add distrobox.
Date: Tue, 20 Dec 2022 01:25:18 +0000

> Hi,
> 
> thanks for the patch.
> 
> I’m always a bit worried when I see the copy-build-system, because the
> lack of a configuration step often means that the tool makes possibly
> invalid assumptions about the runtime environment.
> 
> I see that you’re patching the locations of “podman”, “wget”, and
> “command -v”, but the tools contain even more references, e.g. to
> “curl”, “grep”, “sed”, coreutils, “host-spawn”, “sudo”, “pinentry”,
> “less”, …
> 
> It may be a better idea to wrap all the executables in a reasonable
> default environment.
> 
> What do you think?
It sounds like a good idea.

> -- 
> Ricardo

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

From: omlet <at> tuta.io
To: 59410 <at> debbugs.gnu.org
Subject: [PATCH] gnu: Add distrobox.
Date: Sun, 15 Jan 2023 18:14:57 +0100 (CET)

[Message part 1 (text/plain, inline)]

I don't know how much longer the official packaging for guix will take, but with the distrobox it will be possible to download many free software applications that are taking to be brought to the guix package manager

[Message part 2 (text/html, inline)]

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

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: Adam Faiz via Guix-patches via <guix-patches <at> gnu.org>
Cc: Adam Faiz <adam.faiz <at> disroot.org>, 59410-done <at> debbugs.gnu.org
Subject: Re: [bug#59410] [PATCH] gnu: Add distrobox.
Date: Sat, 04 Feb 2023 00:16:04 +0100

Hello,

Adam Faiz via Guix-patches via <guix-patches <at> gnu.org> writes:

> Subject: [PATCH] gnu: Add distrobox.

Applied with the changes below. Thank you.

> +(define-public distrobox
> +  (package
> +    (name "distrobox")
> +    (version "1.4.1")

I updated it to latest version.

> +    (source
> +     (origin
> +       (method git-fetch)
> +       (uri (git-reference
> +             (url "https://github.com/89luca89/distrobox")
> +             (commit version)))
> +       (sha256
> +        (base32 "1qc66ghlq6b0nh4gk8ls64njwv96pkhz0p31k40fnicxwkfnb2jq"))
> +       (file-name (git-file-name name version))))
> +    (build-system copy-build-system)
> +    (inputs
> +     (list podman wget))
> +    (arguments
> +     (list #:phases
> +           #~(modify-phases %standard-phases
> +               (add-before 'install 'refer-to-inputs
> +                 (lambda _
> +                   (substitute* (find-files "." "^distrobox.*")
> +                     (("podman") (string-append #$(this-package-input 
> "podman") "/bin/podman"))
> +                     (("wget") (string-append #$(this-package-input 
> "wget") "/bin/wget"))

I used `search-input-file' instead of `this-package-input'.

> +    (license license:gpl3+)))

License seems to be GPL3 only, so I changed it.

Regards,
-- 
Nicolas Goaziou

Previous Next

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