bug#49550: Enable to boot from emmc on rockpro64

2021-07-16 Thread Vagrant Cascadian
On 2021-07-13, Pierre Langlois wrote:
> I'm afraid since commit 3a851d45576e046d696fcf35b34d57b2cd28ea49 [0]
> I've not been able to boot from eMMC on the rockpro64 board, it freezes
> before loading the kernel. Re-introducing the work-around fixed the
> issue for me (see patch attached).
>
> Did we mean to remove the work-around? I'm wondering if there's
> something I'm missing, or if I should instead boot from a USB device.

I definitely tested this on pinebook-pro and probably just assumed it
would also fix the issue on rockpro64...


> I've also updated u-boot to 2021.07 but I'm seeing the same boot issue
> so it doesn't appear to be fixed upstream :-/
>
> Anybody know of any other work around? If not, are you happy with the
> patch attached?

I'll try to test on rockpro64 and see if I can confirm the issue and
your fix...


live well,
  vagrant

> From f3c647bb447706f465a3fb4b8d6e09cd089dbed9 Mon Sep 17 00:00:00 2001
> From: Pierre Langlois 
> Date: Sat, 8 May 2021 16:19:23 +0100
> Subject: [PATCH 1/3] gnu: u-boot-rockpro64-rk3399: Disable USB boot.
>
> * gnu/packages/bootloaders.scm (u-boot-rockpro64-rk3399)[arguments]: Add
> 'patch-rockpro64-config phase.
> ---
>  gnu/packages/bootloaders.scm | 11 ---
>  1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/bootloaders.scm b/gnu/packages/bootloaders.scm
> index 742992a119..75705a27c1 100644
> --- a/gnu/packages/bootloaders.scm
> +++ b/gnu/packages/bootloaders.scm
> @@ -12,7 +12,7 @@
>  ;;; Copyright © 2019 Mathieu Othacehe 
>  ;;; Copyright © 2020 Björn Höfling 
>  ;;; Copyright © 2018, 2019, 2020 Vagrant Cascadian 
> -;;; Copyright © 2020 Pierre Langlois 
> +;;; Copyright © 2020, 2021 Pierre Langlois 
>  ;;; Copyright © 2021 Vincent Legoll 
>  ;;; Copyright © 2021 Brice Waegeneire 
>  ;;;
> @@ -931,11 +931,16 @@ to Novena upstream, does not load u-boot.img from the 
> first partition.")
>  (substitute-keyword-arguments (package-arguments base)
>((#:phases phases)
> `(modify-phases ,phases
> +  (add-after 'unpack 'patch-rockpro64-config
> +;; Fix regression in 2020.10 causing freezes on boot with 
> USB boot enabled.
> +;; See 
> https://gitlab.manjaro.org/manjaro-arm/packages/core/uboot-rockpro64/-/issues/4
> +(lambda _
> +  (substitute* "configs/rockpro64-rk3399_defconfig"
> +(("CONFIG_USE_PREBOOT=y") "CONFIG_USE_PREBOOT=n"
>(add-after 'patch-rockpro64-config 'set-environment
>  (lambda* (#:key inputs #:allow-other-keys)
>(setenv "BL31" (string-append (assoc-ref inputs "firmware")
> -"/bl31.elf"))
> -  #t))
> +"/bl31.elf"
>;; Phases do not succeed on the bl31 ELF.
>(delete 'strip)
>(delete 'validate-runpath)
> -- 
> 2.32.0
>
>
>
> [0]: https://lists.gnu.org/archive/html/guix-commits/2021-05/msg00085.html


signature.asc
Description: PGP signature


bug#49583: [PATCH v2]

2021-07-16 Thread Justin Veilleux
I addressed the problems the previous patch had.

About the tests, the reason they fail is because (IIUC) an age rating tag
in some manifest.xml file is missing.
From e007667ecd2ba45c3b4862694f9609a7437be3c0 Mon Sep 17 00:00:00 2001
From: terramorpha 
Date: Fri, 16 Jul 2021 11:57:29 -0400
Subject: [PATCH] gnu: Add transmission-remote-gtk.

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

diff --git a/gnu/packages/bittorrent.scm b/gnu/packages/bittorrent.scm
index 0ca60d607c..52f1e39f66 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,40 @@ 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 "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
+ ;; the tests fail because of an age rating tag that is not present in the
+ ;; metadata
+ `(#:tests? #t))
+(native-inputs
+ `(("gettext" ,gnu-gettext)
+   ("pkg-config" ,pkg-config)))
+(inputs
+ `(("appstream-glib" ,appstream-glib)
+   ("curl" ,curl)
+   ("gtk+" ,gtk+)
+   ("json-glib" ,json-glib)
+
 (define-public libtorrent
   (package
 (name "libtorrent")
-- 
2.32.0



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

2021-07-16 Thread Hartmut Goebel

Hi Mathieu,
thanks for the review. I updated the doc-string, fixed the other parts 
and pushed as 3ee0f170c8bd883728d8abb2c2e00f445c13f17d.



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


False is used to omit elements from the query-string.

Booleans and other types are not handled, since this low-level function 
doesn't know how to convert them into a string to be put into the query. 
#t could be "1", "t", "true", depending on the API used.


--
Regards
Hartmut Goebel

| Hartmut Goebel  | h.goe...@crazy-compilers.com   |
| www.crazy-compilers.com | compilers which you thought are impossible |






bug#49600: [core-updates] gcc-11 fails to cross-compile from x86_64-linux

2021-07-16 Thread Thiago Jung Bauermann via Bug reports for GNU Guix
Hello,

On core-updates (at least as of commit 8456581375cf), the gcc package fails 
to cross-compile. This can be seen in the following CI jobs:

• https://ci.guix.gnu.org/build/667365/details
  gcc-11.1.0, arm-linux-gnueabihf.gcc.x86_64-linux
• https://ci.guix.gnu.org/build/667394/details
  gcc-11.1.0, aarch64-linux-gnu.gcc.x86_64-linux
• https://ci.guix.gnu.org/build/667452/details
  gcc-11.1.0, powerpc64le-linux-gnu.gcc.x86_64-linux
• https://ci.guix.gnu.org/build/667481/details
  gcc-11.1.0, riscv64-linux-gnu.gcc.x86_64-linux
• https://ci.guix.gnu.org/build/667336/details
  gcc-11.1.0, mips64el-linux-gnu.gcc.x86_64-linux
• https://ci.guix.gnu.org/build/667423/details
  gcc-11.1.0, powerpc-linux-gnu.gcc.x86_64-linux

They all fail with a similar error:

--8<---cut here---start->8---
⋮
In file included from 
/tmp/guix-build-gcc-11.1.0.drv-0/build/arm-linux-gnueabihf/libstdc++-v3/include/bits/move.h:57,
 from 
/tmp/guix-build-gcc-11.1.0.drv-0/build/arm-linux-gnueabihf/libstdc++-v3/include/bits/nested_exception.h:40,
 from 
../../../../gcc-11.1.0/libstdc++-v3/libsupc++/exception:148,
 from 
../../../../gcc-11.1.0/libstdc++-v3/libsupc++/eh_exception.cc:26:
/tmp/guix-build-gcc-11.1.0.drv-0/build/arm-linux-gnueabihf/libstdc++-v3/include/type_traits:971:25:
 error: there are no arguments to ?__is_nothrow_constructible? that depend on a 
template parameter, so a declaration of ?__is_nothrow_constructible? must be 
available [-fpermissive]
  971 |   = __bool_constant<__is_nothrow_constructible(_Tp, _Args...)>;
  | ^~
/tmp/guix-build-gcc-11.1.0.drv-0/build/arm-linux-gnueabihf/libstdc++-v3/include/type_traits:971:25:
 note: (if you use ?-fpermissive?, G++ will accept your code, but allowing the 
use of an undeclared name is deprecated)
/tmp/guix-build-gcc-11.1.0.drv-0/build/arm-linux-gnueabihf/libstdc++-v3/include/type_traits:971:66:
 error: template argument 1 is invalid
  971 |   = __bool_constant<__is_nothrow_constructible(_Tp, _Args...)>;
  |  ^
/tmp/guix-build-gcc-11.1.0.drv-0/build/arm-linux-gnueabihf/libstdc++-v3/include/type_traits:976:45:
 error: expected template-name before ?::type
  | ^
/tmp/guix-build-gcc-11.1.0.drv-0/build/arm-linux-gnueabihf/libstdc++-v3/include/type_traits:976:45:
 error: expected ?{? before ?8---

In addition to that, arm-linux-gnueabihf.gcc.x86_64-linux has the following
error which doesn’t appear on the other arches:

no include path in which to search for stdc-predef.h

As a side note, gcc-11.1.0 also fails to build on
i586-pc-gnu.gcc.x86_64-linux¹, but with a very different error message,
so it appears to be an unrelated problem.

gcc-8.5.0 also fails to cross-compile on several arches, but with a
different error message as well, so it also appears to be a separate
problem.

-- 
Thanks,
Thiago

¹ https://ci.guix.gnu.org/build/667510/details







bug#49600: [core-updates] gcc-11 fails to cross-compile from x86_64-linux

2021-07-16 Thread Thiago Jung Bauermann via Bug reports for GNU Guix
Hi,

There were several commits added to GCC’s ‘releases/gcc-11’ branch since 
the 11.1.0 release, so I wanted to check whether this problem was fixed by 
any of them.

I applied the patch containing those commits to the gcc-11 package, 
obtained by running the following on a git checkout of the GCC repo:

```
$ git diff releases/gcc-11.1.0..origin/releases/gcc-11 \
> gcc-11-branch-update.patch
```

At the time, origin/releases/gcc-11 was:

419201f566bb libstdc++: Use function object for __decay_copy helper

Unfortunately it didn’t make a difference.

-- 
Thanks,
Thiago







bug#49583: [PATCH v2]

2021-07-16 Thread Leo Famulari
On Fri, Jul 16, 2021 at 12:01:41PM -0400, Justin Veilleux wrote:
> I addressed the problems the previous patch had.

Thanks!

> About the tests, the reason they fail is because (IIUC) an age rating tag
> in some manifest.xml file is missing.

Okay. I found the upstream bug report:

https://github.com/transmission-remote-gtk/transmission-remote-gtk/issues/95

I cherry-picked the bug fix, added it to your patch, and pushed as
9cb35c02164d929fcb8929e7f454df215df8cf25 after making some cosmetic
changes to the package definition. Also, I changed the license to "GPL 2
or later" since the primary source files include the "or later" clause
in their license headers.

Thanks again!