bug#48389: Guile 3.0.7 test suite fails non-deterministically

2021-07-15 Thread Carl Dong
Hi all,

I’m also encountering this exact error when time-machining from 
0961807915d7955e70a081272d857b6cbf92c0c8 to 
83d9e2ee56cb0ca2a644074ad232503e25ac7116

I’m able to reproduce this (I’ve tried building the derivation 5 times now with 
different --cores, and this error happens every time)

Build logs here: https://nextcloud.carl.homeserver.net/s/yas2SwmST8Z3jRG
Keep-failed directory here: 
https://nextcloud.carl.homeserver.net/s/sSrjPZn5NqikeoJ

My system:
- AMD Ryzen Threadripper 2970WX 24-Core Processor
- Guix on Arch Linux
- tmpfs mounted on /tmp

Cheers,
Carl Dong
cont...@carldong.me
"I fight for the users"






bug#49583: [PATH] gnu: Add transmission-remote-gtk.

2021-07-15 Thread Leo Famulari
On Thu, Jul 15, 2021 at 02:31:08PM -0400, Justin Veilleux wrote:
> From 426aa9bd6b758628249df7c0925179ce39054f29 Mon Sep 17 00:00:00 2001
> From: terramorpha 
> Date: Thu, 15 Jul 2021 14:27:35 -0400
> Subject: [PATCH] gnu: Add transmission-remote-gtk.
> 
> * gnu/packages/bittorrent.scm (transmission-remote-gtk): New variable.

Thanks! Overall looks good.

I ran `guix lint` on it and got these warnings which should be addressed:

/home/leo/work/guix/gnu/packages/bittorrent.scm:165:5: 
transmission-remote-gtk@1.4.1: 'gettext' should probably be a native input
/home/leo/work/guix/gnu/packages/bittorrent.scm:161:0: 
transmission-remote-gtk@1.4.1: parentheses feel lonely, move to the previous or 
next line
/home/leo/work/guix/gnu/packages/bittorrent.scm:170:14: 
transmission-remote-gtk@1.4.1: synopsis should start with an upper-case letter 
or digit
/home/leo/work/guix/gnu/packages/bittorrent.scm:170:14: 
transmission-remote-gtk@1.4.1: no article allowed at the beginning of the 
synopsis

> +(arguments
> + `(#:tests? #f ; the tests fail
> +))

We need more of an explanation for why the tests should be disabled. Did
you look into the failure and try to determine if it represents some
real problems?





bug#49583: [PATH] gnu: Add transmission-remote-gtk.

2021-07-15 Thread Justin Veilleux

From 426aa9bd6b758628249df7c0925179ce39054f29 Mon Sep 17 00:00:00 2001
From: terramorpha 
Date: Thu, 15 Jul 2021 14:27:35 -0400
Subject: [PATCH] gnu: Add transmission-remote-gtk.

* gnu/packages/bittorrent.scm (transmission-remote-gtk): New variable.
---
 gnu/packages/bittorrent.scm | 36 
 1 file changed, 36 insertions(+)

diff --git a/gnu/packages/bittorrent.scm b/gnu/packages/bittorrent.scm
index 0ca60d607c..90dabb40ed 100644
--- a/gnu/packages/bittorrent.scm
+++ b/gnu/packages/bittorrent.scm
@@ -11,6 +11,7 @@
 ;;; Copyright © 2018 Ricardo Wurmus 
 ;;; Copyright © 2019, 2020 Brett Gilio 
 ;;; Copyright © 2020 Hartmut Goebel 
+;;; Copyright © 2021 Justin Veilleux 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -38,6 +39,7 @@
   #:use-module (guix utils)
   #:use-module (gnu packages)
   #:use-module (gnu packages adns)
+  #:use-module (gnu packages autotools)
   #:use-module (gnu packages boost)
   #:use-module (gnu packages check)
   #:use-module (gnu packages compression)
@@ -45,6 +47,7 @@
   #:use-module (gnu packages curl)
   #:use-module (gnu packages cyrus-sasl)
   #:use-module (gnu packages freedesktop)
+  #:use-module (gnu packages gettext)
   #:use-module (gnu packages glib)
   #:use-module (gnu packages gnome)
   #:use-module (gnu packages gnupg)
@@ -138,6 +141,39 @@ DHT, µTP, PEX and Magnet Links.")
 ;; A few files files carry an MIT/X11 license header.
 (license (list l:gpl2 l:gpl3
 
+(define-public transmission-remote-gtk
+  (package
+(name "transmission-remote-gtk")
+(version "1.4.1")
+(source
+ (origin
+   (uri
+(string-append
+ "https://github.com/transmission-remote-gtk/;
+ "transmission-remote-gtk/releases/download/"
+ version "/transmission-remote-gtk-" version ".tar.xz"))
+   (method url-fetch)
+   (sha256
+(base32 "1aqjl5rgamgcgqvcldd1gzyfh2xci0m7070924d6vz2qln0q75sr"
+(build-system gnu-build-system)
+(synopsis "a gtk frontend to the transmission daemon")
+(description "transmission-remote-gtk is a GTK client for remote management
+of the Transmission BitTorrent client, using its HTTP RPC protocol.")
+(license license:gpl2)
+(home-page
+ "https://github.com/transmission-remote-gtk/transmission-remote-gtk;)
+(arguments
+ `(#:tests? #f ; the tests fail
+))
+(native-inputs
+ `(("pkg-config" ,pkg-config)))
+(inputs
+ `(("gettext" ,gnu-gettext)
+   ("json-glib" ,json-glib)
+   ("curl" ,curl)
+   ("gtk+" ,gtk+)
+   ("appstream-glib" ,appstream-glib)
+
 (define-public libtorrent
   (package
 (name "libtorrent")
-- 
2.32.0



bug#49582: [PATCH] gnu: Add tosdr-chromium.

2021-07-15 Thread Justin Veilleux
this patch adds the tos;dr initiative's extension
From c6881b9b065cf2a85b604c8a1e5ddbae77822034 Mon Sep 17 00:00:00 2001
From: terramorpha 
Date: Thu, 15 Jul 2021 14:03:19 -0400
Subject: [PATCH] gnu: Add tosdr-chromium.

* gnu/packages/browser-extensions.scm (tosdr, tosdr-chromium): New variables.
---
 gnu/packages/browser-extensions.scm | 50 -
 1 file changed, 49 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/browser-extensions.scm b/gnu/packages/browser-extensions.scm
index a6120baf96..7a1a5bda69 100644
--- a/gnu/packages/browser-extensions.scm
+++ b/gnu/packages/browser-extensions.scm
@@ -1,5 +1,6 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2020, 2021 Marius Bakke 
+;;; Copyright © 2021 Justin Veilleux 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -21,10 +22,14 @@
   #:use-module (guix git-download)
   #:use-module (guix build-system copy)
   #:use-module (guix build-system gnu)
+  #:use-module (guix build-system trivial)
   #:use-module ((guix licenses) #:prefix license:)
   #:use-module (gnu build chromium-extension)
   #:use-module (gnu packages compression)
-  #:use-module (gnu packages python))
+  #:use-module (gnu packages python)
+  #:use-module (gnu packages base)
+  #:use-module (gnu packages bash)
+  #:use-module (gnu packages node))
 
 (define play-to-kodi
   (package
@@ -121,3 +126,46 @@ ungoogled-chromium.")
 
 (define-public ublock-origin/chromium
   (make-chromium-extension ublock-origin "chromium"))
+
+(define tosdr
+  (package
+(name "tosdr")
+(version "4.1.1")
+(source
+ (origin
+   (uri
+(git-reference
+ (url "https://github.com/tosdr/browser-extensions.git;)
+ (commit version)))
+   (sha256
+(base32 "0wz31f9rz087yw1a7cdhdgqvgnhbk569jywv846n122m4bpk3yw0"))
+   (method git-fetch)))
+(inputs
+ `(("bash" ,bash)
+   ("coreutils" ,coreutils)
+   ("node" ,node)
+   ("zip" ,zip)
+   ("unzip" ,unzip)))
+(arguments
+ `(#:builder
+   (begin
+ (use-modules (guix build utils))
+ (set-path-environment-variable "PATH"
+	'("bin")
+	(map cdr %build-inputs))
+ (copy-recursively (assoc-ref %build-inputs "source") ".")
+ (invoke "sh" "./build.sh")
+ (invoke "unzip" "dist/chrome.zip" "-d" %output)
+ #t)
+   #:modules ((guix build utils
+(build-system trivial-build-system)
+(synopsis "extension to inform of the important aspects of the terms and
+conditions of a web service")
+(description "This extension informs you instantly of your rights online by
+showing an unintrusive icon in the toolbar. You can click on this icon to get
+summaries from the Terms of Service; Didn't Read initiative.")
+(license license:agpl3)
+(home-page "https://tosdr.org/;)))
+
+(define-public tosdr-chromium
+  (make-chromium-extension tosdr))
-- 
2.32.0



bug#49510: [PATCH]

2021-07-15 Thread Leo Prikler
Hi,

I've cleaned up the patch and pushed it.

Thanks,
Leo






bug#49510: [PATCH]

2021-07-15 Thread Justin Veilleux
from another address
From aae47588a9efc24d68a1df34b05c0a85a00c4d1e Mon Sep 17 00:00:00 2001
From: terramorpha 
Date: Mon, 12 Jul 2021 12:38:57 -0400
Subject: [PATCH] gnu: Add putty.

* gnu/packages/networking.scm (putty): New variable.
---
 gnu/packages/networking.scm | 35 +++
 1 file changed, 35 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 19b58501e9..ce00a016e6 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -4100,3 +4100,38 @@ on hub/switched networks.  It is based on @acronym{ARP} packets, it will send
 @acronym{ARP} requests and sniff for replies.")
(home-page "https://github.com/netdiscover-scanner/netdiscover;)
(license license:gpl3+)))
+
+(define-public putty
+  (package
+(name "putty")
+(version "0.75")
+(source
+ (origin
+   (method url-fetch)
+   (uri (string-append "http://www.putty.be/; version
+   "/putty-" version ".tar.gz"))
+   (sha256
+(base32 "1xgrr1fbirw79zafspg2b6crzfmlfw910y79md4r7gnxgq1kn5yk"
+(arguments
+ `(#:phases
+   (modify-phases %standard-phases
+ (add-before 'configure 'go-into-unix/
+   (lambda _ (begin
+  (chdir "unix")
+  #t))
+(build-system gnu-build-system)
+(native-inputs
+ `(("pkg-config" ,pkg-config)))
+(inputs
+ `(("gtk+" ,gtk+)
+   ("perl" ,perl)
+   ("python" ,python)
+   ("python2" ,python-2.7)))
+(synopsis "Graphical @acronym{SSH} and telnet client")
+(description
+ "Putty is a powerful terminal client.  It supports @acronym{SSH}, telnet,
+and raw socket connections with good terminal emulation.  It supports public key
+authentication and Kerberos single-sign-on.  It also includes command-line
+@acronym{SFTP} and @acronym{SCP} implementations.")
+(home-page "https://www.chiark.greenend.org.uk/~sgtatham/putty/;)
+(license license:expat)))
-- 
2.32.0



bug#48794: Add verification instructions to guix .iso

2021-07-15 Thread bo0od
yeah put that link in download page above "Installation instructions" 
and name it "Verification Instructions" which will be easy one place for 
the user to see how to install and how to verify, do you think its a bad 
suggestion?


Ludovic Courtès:

Hi,

bo0od  skribis:


Its better for the end user to know how to verify his guix .iso when
he download it to make sure there is no issue with it.

https://guix.gnu.org/en/download/


Verification instructions are in the manual section linked from that
page:

   https://guix.gnu.org/manual/en/html_node/USB-Stick-and-DVD-Installation.html

Ludo’.







bug#48796: Guix on Debian 11 - Cant run or find applications from Guix

2021-07-15 Thread bo0od



> I'll try do propose some patch for the Guix manual but... don't hold
> your breath, I need some testing.

Sure tyt, but its a disaster way for guix for not doing this 
automatically. Imagine new user he should do stuff manually after 
installing guix through apt.. yeah he (mostly) wont stay around after 
that to be a guix user.


Giovanni Biscuolo:

I'll try do propose some patch for the Guix manual but... don't hold
your breath, I need some testing.






bug#48796: Guix on Debian 11 - Cant run or find applications from Guix

2021-07-15 Thread bo0od

> Does this message [1] fix your issue?

"If you type "source ~/.guix-profile/etc/profile" from a Bash shell, it 
loads the needed environment variable"


yes it worked, but thats not really what im asking as this is workaround 
for the issue but im asking for a solution to the users as they can type 
the app name and it should run and icon should be shown somewhere on 
application menu or desktop or so.


flatpak , snap which work almost similarly to guix can do that then guix 
should do that as well.


otherwise guix should mention that there wont be icons nor ability to 
run the applications from terminal unless you do 1 2 3 after guix app 
installaion which is sadly a downside for new comers.





zimoun:

Does this message [1] fix your issue?






bug#47630: Improve guix manual installation docs

2021-07-15 Thread bo0od

> Could you propose a concrete wording solving
> the issues you see

Not really because english is not my mother tongue so there going to be 
errors in grammars, choosing of words..etc so better to be written by 
someone who has better english than me.


But for the commands to go on step by step i can give that with little 
text and any webadmin can pick it up and arrange it accordingly.


If guix documentation built on wikimedia it would be much easier because 
i can register and upload my changes and can be reviewed and updated, 
But using email for this is not the best way to achieve it but what to do..




zimoun:

Hi,

On Wed, 07 Apr 2021 at 04:55, bo0od  wrote:


Checking here:

https://guix.gnu.org/manual/en/html_node/Manual-Installation.html

This section actually misses alot of commands and its not nice to give only
hint commands and leave step by step explanation e.g:

https://guix.gnu.org/manual/en/html_node/Keyboard-Layout-and-Networking-and-Partitioning.html

"parted /dev/sda set 1 esp on"

This command wont work without labeling the partition "parted /dev/sda --
mklabel gpt" and so on..

This mean user cant copy and paste he need to figure out the errors while hes
follow the documentation because it doesn't give to the user the full picture.

I suggest to take a look at NixOS docs which goes step by step without jumping
any necessary one:

https://nixos.org/manual/nixos/stable/index.html#sec-installation-partitioning

(Unclear documentation causes unsolvable or ambiguous questions from users and
its troublesome to the support team to re-add the missing parts of the docs in
their support.(Assuming they figured out where is the issue user is falling
in.))


Thanks for your report.  Could you propose a concrete wording solving
the issues you see?

Thanks,
simon







bug#44906: [bug#49482] [PATCH 3/3] ci: Properly construct URLs.

2021-07-15 Thread Mathieu Othacehe


Hello Hartmut,

Thanks for this patchset!

> +(define* (api-url base-url path #:rest query)
> +  "Build a proper API url, taking into account BASE_URL's trailing slashes."

s/BASE_URL/BASE-URL/

You could also indicate what is the expect format for query: '("name"
"value") lists.

> +(((_ #f) . rest) (lp rest acc))
> +(((name val) . rest)
> + (lp rest (cons*
> +   name "="
> +   (if (string? val) (uri-encode val) (number->string val))

What about booleans? False is filtered above but true will throw an
exception.

> +(resolve-uri-reference ref base)))
> +
> +

There's an extra new line here.

> +(define* (json-api-fetch base-url path #:rest query)
> +  (json-fetch (apply api-url base-url path query)))
> +
> +

Here also.

Otherwise, it looks nice :)

Thanks,

Mathieu