bug#33325: xpra doesn't work out-of-the-box (implicit dependencies)

2018-11-26 Thread Rutger Helling
Hi Ludo,

apologies for the delayed reaction, I've been quite busy. Oddly enough
the commands seem to work fine on GuixSD. Can you maybe tell me what
distro you're using?

I don't have access to a foreign distro at the moment, but I'll try to
look into it next weekend.

Do you have the 'xauth' command? If not, could you install that and see
if it makes a difference?

On Sun, 25 Nov 2018 17:15:22 +0100
ludovic.cour...@inria.fr (Ludovic Courtès) wrote:

> Hello,
> 
> ludovic.cour...@inria.fr (Ludovic Courtès) skribis:
> 
> > $ guix environment --ad-hoc xpra --pure
> > [env]$ xpra start :100
> > Warning: Xorg binary not found, assuming the wrapper is needed!
> > ludo@ribbon ~/src/guix [env]$ Entering daemon mode; any further
> > errors will be reported to: /run/user/1000/xpra/:100.log
> >
> > [env]$ xpra list
> > Warning: Xorg binary not found, assuming the wrapper is needed!
> > No xpra sessions found  
> 
> FWIW I get the same problem with the recent xpra 2.4.2 update.
> 
> > I believe the problem should be reproducible in a ‘--pure’
> > environment. Perhaps the missing Python modules mentioned in the
> > log are the root cause?  
> 
> Thoughts?
> 
> Thanks in advance,
> Ludo’.



pgpmUSejA8poS.pgp
Description: OpenPGP digital signature


bug#33325: xpra doesn't work out-of-the-box (implicit dependencies)

2018-11-26 Thread Ludovic Courtès
Hi Rutger,

Rutger Helling  skribis:

> apologies for the delayed reaction, I've been quite busy. Oddly enough
> the commands seem to work fine on GuixSD. Can you maybe tell me what
> distro you're using?

I’m using GuixSD.  :-)

If we use ‘-C’ instead of ‘--pure’, we should both get the same result:

--8<---cut here---start->8---
$ guix environment --ad-hoc xpra -C 
[env]# xpra start :100
Warning: Xorg binary not found, assuming the wrapper is needed!

Warning: running as root
[env]# Entering daemon mode; any further errors will be reported to:
  /tmp/:100.log

[env]# xpra list
Warning: Xorg binary not found, assuming the wrapper is needed!

Warning: running as root
Found the following xpra sessions:
/home/ludo/.xpra:
UNKNOWN session at :100
Re-probing unknown sessions in: /home/ludo/.xpra

UNKNOWN session at :100 (cleaned up)
[env]# 
[env]# xpra list
Warning: Xorg binary not found, assuming the wrapper is needed!

Warning: running as root
No xpra sessions found
--8<---cut here---end--->8---

Same for you?

Thanks for your feedback,
Ludo’.





bug#33325: xpra doesn't work out-of-the-box (implicit dependencies)

2018-11-26 Thread Rutger Helling
So it should actually work in the container if you run the command xpra
upgrade, like so:

container$ xpra start :100 # wait a few seconds for it to start
container$ xpra upgrade :100
container$ xpra list

Just starting it in a container isn't terribly useful though since that
way you can't attach to it. If you run
guix environment --ad-hoc xpra -C --share=/tmp/.X11-unix
--share=/home/$USER/.xpra

and then start the server with the above commands you should be
able to attach to it outside of the container, like:
$ xpra attach :100
$ DISPLAY=:100 gui-program # on a different terminal

Does this work for you too?

I'm not sure why the 'xpra upgrade' command seems to be required in a
container. I have no idea how to fix that.

On Mon, 26 Nov 2018 10:00:37 +0100
ludovic.cour...@inria.fr (Ludovic Courtès) wrote:

> Hi Rutger,
> 
> Rutger Helling  skribis:
> 
> > apologies for the delayed reaction, I've been quite busy. Oddly
> > enough the commands seem to work fine on GuixSD. Can you maybe tell
> > me what distro you're using?  
> 
> I’m using GuixSD.  :-)
> 
> If we use ‘-C’ instead of ‘--pure’, we should both get the same
> result:
> 
> --8<---cut here---start->8---
> $ guix environment --ad-hoc xpra -C 
> [env]# xpra start :100
> Warning: Xorg binary not found, assuming the wrapper is needed!
> 
> Warning: running as root
> [env]# Entering daemon mode; any further errors will be reported to:
>   /tmp/:100.log
> 
> [env]# xpra list
> Warning: Xorg binary not found, assuming the wrapper is needed!
> 
> Warning: running as root
> Found the following xpra sessions:
> /home/ludo/.xpra:
> UNKNOWN session at :100
> Re-probing unknown sessions in: /home/ludo/.xpra
> 
> UNKNOWN session at :100 (cleaned up)
> [env]# 
> [env]# xpra list
> Warning: Xorg binary not found, assuming the wrapper is needed!
> 
> Warning: running as root
> No xpra sessions found
> --8<---cut here---end--->8---
> 
> Same for you?
> 
> Thanks for your feedback,
> Ludo’.



pgpkDZDK7fH8f.pgp
Description: OpenPGP digital signature


bug#33496: bug#33500: bug#33496: Guix pull failing to compute derivation

2018-11-26 Thread Ludovic Courtès
Jan Nieuwenhuizen  skribis:

> Ludovic Courtès writes:
>
>> My bad, I introduced the regression 16 hours ago.  This is fixed in
>> commit 0a9d1c5ab713bf525972f651c3cb63570e8c083d, thanks!
>
> Ah, great...there were at least three reports about this.
>
> Reading the commit it looks to me that the message and the content do
> not match.  The message says that install-nodist_pkglibexecSCRIPTS is
> removed, while the patch removes install-nodist_libexecSCRIPTS.

Oh sorry about that!

> Hoping the end result is okay!

Yes it is!  :-)

Thank you,
Ludo’.





bug#33507: nss 3.39 output is not deterministic

2018-11-26 Thread Ludovic Courtès
Hello,

The contents of .chk files differ:

--8<---cut here---start->8---
$ wget -q -O - 
https://berlin.guixsd.org/nar/gzip/fx35yrin8ssdmpqzska50pinzzafa2fr-nss-3.39 | 
gunzip -c | guix archive -x /tmp/t
$ diffoscope --exclude-directory-metadata /tmp/t 
/gnu/store/fx35yrin8ssdmpqzska50pinzzafa2fr-nss-3.39
--- /tmp/t
+++ /gnu/store/fx35yrin8ssdmpqzska50pinzzafa2fr-nss-3.39
├── lib
│ ├── nss
│ │ ├── libfreebl3.chk
│ │ │┄ xxd not available in path. Falling back to Python hexlify.
│ │ │ @@ -11,19 +11,19 @@
│ │ │  5c80e3430a9e943586d458a1ca22b973460bfb3e33f1d5d3b426bf50d7f20933
│ │ │  6ec0311b6d077086ca57f70b4a63f06fc88aed5060f311c744f3ce4e50422d85
│ │ │  335457038ddc664d6183171c7b0d65bc8f2c1986fce29f5d67fcd4a5f823a11a
│ │ │  a2e5843201ee88f15530e9743c1a2b54452e39b977e132af2d97e021ecf5
│ │ │  58e1c72ee0713d29a4d6e25f859c0504464189033cfab2cffad567ccec68fc83
│ │ │  d91f2e4e9a5e77a1ffe66f048bf96b47c649d2886e29a31baee04f728a28940c
│ │ │  1d8c99a26ff8ba9990c7e5b13c1034866a6a1f396358e15e9795454038456f02
│ │ │ -b5866eae2f327ea13a342c1cd3ff4e2c381caa2e66be323e3c065f010057
│ │ │ -6caa3fd8ec4cb53d699cda29b807f80ce730ba4e827c253e99cb9d320ea11a85
│ │ │ -1355d712105b1e6fcc2572e727d505d4426caa46848592d4cf81508721035236
│ │ │ -475158597860821870fc45854b8aaf4c31dd925983b94b966c6ab43935e50ddd
│ │ │ -0f9c5d8bb11a0d502ce7614de8967b405f04c2fd564da5e96dcd93858dbdd05a
│ │ │ -76d704513351b68718fe8f63a07a4f6f6efe9e91577de790d2821010530ff461
│ │ │ -ae06a4b14f52da5779a16c136984e8f28ef9301d18ff4c49cb67b0da12e17b7e
│ │ │ -65c73c3f65059370ca0018b564d08de1dbad8a156cce805db7f7ee650440f8f3
│ │ │ -642f28c939169e5783f858563f666ce8f1f87b22b83cda8038c28cb909a93300
│ │ │ -408c414c5ad6582289aad9f891aba2c4b481a2a6a866d2b85d85ac6aadcc
│ │ │ -7755ef5be789ec256097b9605eb1d14ea71f4d0f3e32b29666e0d14680e8fbe1
│ │ │ -f76a89
│ │ │ +b5866eae2f327ea13a342c1cd3ff4e2c381caa2e66be323e3c065f00ff2b
│ │ │ +3653b84f33ad6dd501466317b1172d6d261f37eb09f5f6862d594663fc0fdc5c
│ │ │ +51d2dc5b2925324338451cf0fab376d6ea24f33869be4693ce434827a059fc5d
│ │ │ +df792b14a2276514a23c57263706b60a04d808d9f86a008832eb35b3d31a104a
│ │ │ +ee2f7b1a241ff5b91a525f1e5dd879a0af580bd435bc78d4eba30be92a44ea3d
│ │ │ +3294e4d11e9c267f6a94709fa4a6b908e3a2c2a05c4079ffd9d86ffe5c2945d8
│ │ │ +40bf37df1da0f7b960d3042762e2bf3b1183ee01319b1f2ff5115042d15db8a0
│ │ │ +6bf010f2ea396495f80e6eee3254fe4272b33260c5656b005b2657f8b87451b5
│ │ │ +ade90b027de4be0a79bbdc962999669fe50c30d6f9e590a0f9c77f8b07ea
│ │ │ +00403393cf12977baa7cb130d3e810df1d667f975b85fddfd40bd60d789e
│ │ │ +e5ba187f6561417c1cbd11639a20024becb24121da351ff9151012848a943dc5
│ │ │ +f745
│ │ ├── libfreeblpriv3.chk
│ │ │┄ xxd not available in path. Falling back to Python hexlify.
│ │ │ @@ -11,19 +11,19 @@
│ │ │  5c80e3430a9e943586d458a1ca22b973460bfb3e33f1d5d3b426bf50d7f20933
│ │ │  6ec0311b6d077086ca57f70b4a63f06fc88aed5060f311c744f3ce4e50422d85
│ │ │  335457038ddc664d6183171c7b0d65bc8f2c1986fce29f5d67fcd4a5f823a11a
│ │ │  a2e5843201ee88f15530e9743c1a2b54452e39b977e132af2d97e021ecf5
│ │ │  58e1c72ee0713d29a4d6e25f859c0504464189033cfab2cffad567ccec68fc83
│ │ │  d91f2e4e9a5e77a1ffe66f048bf96b47c649d2886e29a31baee04f728a28940c
│ │ │  1d8c99a26ff8ba9990c7e5b13c1034866a6a1f396358e15e9795454038456f02
│ │ │ -b5866eae2f327ea13a342c1cd3ff4e2c381caa2e66be323e3c065f010098
│ │ │ -64bc9b90209d5094b1d589bef4fa8bc6ae4486bc77d9116495345a6f9e1429bd
│ │ │ -af1d2bc4ee46d765dd3a721c328a4d2b4eecb865bcc23e1ace20ea5b388a2aef
│ │ │ -e185e55b5a0df65b56701f38d2457bd1ae6a49363bc6724c926479222965c909
│ │ │ -49463cc1c730e7028b0e2237bdc9e5ea2c36050b541caaf58354cffc693539a2
│ │ │ -02ca25101e54a4e5764287b0a3adc829a1d1b275be37818b82499426ec9cd6cb
│ │ │ -8f696b5d24412ae3d70c23e143ab3bb19ff689b9a0d331589a8c7c0c782b1dd7
│ │ │ -ff6bb0a9e85433c33c4f1582bdd1fc740b504a8cef03772e432eb7ffe146b255
│ │ │ -1471ca0148f48da48e6a6f3cc10d12ac3078aac9ca55b15562fdd354db1bee00
│ │ │ -408aa20ddcd0e7b630d7128bb026bf79b8ba703203d1cb482634a56ae914
│ │ │ -365b38694ca9062d516b8c0ebb6135186ab142d72ab45083273572b06e1f6839
│ │ │ -d542bb
│ │ │ +b5866eae2f327ea13a342c1cd3ff4e2c381caa2e66be323e3c065f010069
│ │ │ +82a9c4efe012b14935e217617e7de432fc1ba95dc37faad3f4bfe4d217ea6a64
│ │ │ +7aec507b62f5d829e50baf048bd1306942dc0540340169e210b0719d871bfe6f
│ │ │ +efeb003aa3b58a7cbfed31479cfd0ecbc2416cea9602a0312ff2f4a71a9a7e2c
│ │ │ +614c008ed6021bfef3b4cb17a95c74834bc1f411adcb8b711584c314a902092d
│ │ │ +ba853cf39b254f780fe47c16abb4dd412c25930d30f76f48d773e3c0bff4fa9f
│ │ │ +01812e1b9bc0138765ceef665da1842712d442c294e7fda414d01e37d525e3ea
│ │ │ +1f8e542902d4671220e1c4c71bc55780865d56767b5e65d69d17581767ba1016
│ │ │ +6ff3e46d7c1967d066e804d9790e7cd4975df17fa678851059bad50e8a439100
│ │ │ +403757deb728e18996468318e0c59b8dd2bac37d998c1e5e0efca2443672
│ │ │ +52465a76e0607a8192fe2e07e3b0dc1007624885d3ffb653a0e5dbfd41dbdc84
│ │ │ +0324b6
│ │ ├── libnssdbm3.chk
│ │ │┄ xxd not available in path. Falling back to Python hexlify.
│ │ │ @@ -11,19 

bug#33509: core-updates: arc-theme fails with "Invalid GNOME version: 3.24"

2018-11-26 Thread Björn Höfling
On core-updates the package arc-theme fails:

starting phase `configure'
source directory: "/tmp/guix-build-arc-theme-20170302.drv-0/arc-theme-20170302" 
build directory: "/tmp/guix-build-arc-theme-20170302.drv-0/arc-theme-20170302"
configure flags: ("CONFIG_SHELL=/gnu/store/q19l04vd2za80mk1845pz7r8cz29qk43-bash
configure: WARNING: unrecognized options: --enable-fast-install
checking for a BSD-compatible install... /gnu/store/5s2nib1lrd2101bbrivcl17kjx1m
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /gnu/store/5s2nib1lrd2101bbrivcl17kjx1msp
checking for gawk... gawk
checking whether make sets $(MAKE)... no
checking whether make supports nested variables... yes
checking for pkg-config... /gnu/store/xyiyf12w4i7zf5g5b4ah0bi1mmd6i8xa-pkg-confi
checking pkg-config is at least version 0.9.0... yes
configure: error: Invalid GNOME version: 3.24

It seams that only GNOME versions until 3.22 are valid.

Nix has a workaround that sets the version down:

https://github.com/NixOS/nixpkgs/commit/165fa89762d7156bb3f2774cd3ff0d40cd0a9606

If I set the version hard to 3.22 the package builds again
successfully:

- '(#:phases
+ '(#:configure-flags
+   '("--with-gnome=3.22")
+   #:phases

Is it legitimate to patch it like that or do we need more tests?

Björn




pgpYOX6krky8q.pgp
Description: OpenPGP digital signature


bug#33359: On emacs-clang-format

2018-11-26 Thread Pierre Neidhardt
I went ahead and implemented the aforementioned approach:

> - Keep emacs-clang-format but use the Clang source (as per Tim suggestion), 
> and
>   add emacs-clang-rename from the same source.

-- 
Pierre Neidhardt
https://ambrevar.xyz/


signature.asc
Description: PGP signature


bug#33517: Problem booting when using btrfs subvolume for /gnu/store

2018-11-26 Thread Christopher Baines
I'm loosing track of this issue a bit, as I've been dealing with it for
a while. I have a machine that I've setup where /gnu/store is a btrfs
subvolume. I do this so that I can make flexible use of the space on
that btrfs filesystem.

Unfortunately, the grub configuration generated for this doesn't seem to
account for this, and so it requires some tweaking to get it to boot.

A long while back, I discovered I could make the following change, then
the generated grub configuration would be fine.


---
 gnu/bootloader/grub.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/bootloader/grub.scm b/gnu/bootloader/grub.scm
index 06856dd58c..c3ddc3e128 100644
--- a/gnu/bootloader/grub.scm
+++ b/gnu/bootloader/grub.scm
@@ -320,8 +320,8 @@ entries corresponding to old generations of the system."
   ;; Use the right file names for KERNEL and INITRD in case
   ;; DEVICE-MOUNT-POINT is not "/", meaning that the store is on a
   ;; separate partition.
-  (let ((kernel  (strip-mount-point device-mount-point kernel))
-(initrd  (strip-mount-point device-mount-point initrd)))
+  (let ((kernel kernel)
+(initrd initrd))
 #~(format port "menuentry ~s {
   ~a
   linux ~a ~a
--
2.19.2


Unfortunately, it's not a proper solution, as it obviously breaks when
you actually want to strip the mount point off so that grub can find the
right files.

I'm creating a bug for this, as I think it would be good to track the
issue. I've also written a system test that I believe reproduced the
issue.

From 7eee5685f95d0b6baeb97f5fdd947fe5223a61c9 Mon Sep 17 00:00:00 2001
From: Christopher Baines 
Date: Fri, 26 Oct 2018 18:48:32 +0100
Subject: [PATCH] WIP Btrfs store subvolume test

---
 gnu/tests/install.scm | 91 ++-
 1 file changed, 90 insertions(+), 1 deletion(-)

diff --git a/gnu/tests/install.scm b/gnu/tests/install.scm
index 4764de..cfa071187c 100644
--- a/gnu/tests/install.scm
+++ b/gnu/tests/install.scm
@@ -43,7 +43,8 @@
 %test-separate-home-os
 %test-raid-root-os
 %test-encrypted-os
-%test-btrfs-root-os))
+%test-btrfs-root-os
+%test-btrfs-root-with-store-subvolume-os))
 
 ;;; Commentary:
 ;;;
@@ -826,4 +827,92 @@ build (current-guix) and then store a couple of full system images.")
  (command (qemu-command/writable-image image)))
   (run-basic-test %btrfs-root-os command "btrfs-root-os")
 
+
+;;;
+;;; Btrfs root file system with store subvolume.
+;;;
+
+(define-os-with-source (%btrfs-root-with-store-subvolume-os
+%btrfs-root-with-store-subvolume-os-source)
+  ;; The OS we want to install.
+  (use-modules (gnu) (gnu tests) (srfi srfi-1))
+
+  (operating-system
+(host-name "liberigilo")
+(timezone "Europe/Paris")
+(locale "en_US.UTF-8")
+
+(bootloader (bootloader-configuration
+ (bootloader grub-bootloader)
+ (target "/dev/vdb")))
+(kernel-arguments '("console=ttyS0"))
+(file-systems (cons* (file-system
+   (device (file-system-label "my-root"))
+   (mount-point "/")
+   (type "btrfs"))
+ (file-system
+   (device (file-system-label "my-root"))
+   (mount-point "/gnu/store")
+   (type "btrfs")
+   (options "subvol=/gnu/store"))
+ %base-file-systems))
+(users (cons (user-account
+  (name "charlie")
+  (group "users")
+  (home-directory "/home/charlie")
+  (supplementary-groups '("wheel" "audio" "video")))
+ %base-user-accounts))
+(services (cons (service marionette-service-type
+ (marionette-configuration
+  (imported-modules '((gnu services herd)
+  (guix combinators)
+%base-services
+
+(define %btrfs-root-with-store-subvolume-installation-script
+  ;; Shell script of a simple installation.
+  "\
+. /etc/profile
+set -e -x
+guix --version
+
+export GUIX_BUILD_OPTIONS=--no-grafts
+ls -l /run/current-system/gc-roots
+parted --script /dev/vdb mklabel gpt \\
+  mkpart primary ext2 1M 3M \\
+  mkpart primary ext2 3M 2G \\
+  set 1 boot on \\
+  set 1 bios_grub on
+mkfs.btrfs -L my-root /dev/vdb2
+mount /dev/vdb2 /mnt
+btrfs subvolume create /mnt/home
+mkdir /mnt/gnu
+btrfs subvolume create /mnt/gnu/store
+herd start cow-store /mnt
+mkdir /mnt/etc
+cp /etc/target-config.scm /mnt/etc/config.scm
+guix system build /mnt/etc/config.scm
+guix system init /mnt/etc/config.scm /mnt --no-substitutes
+sync
+reboot\n")
+
+(define %test-btrfs-root-with-store-subvolume-os
+  (system-test
+   (name "btrfs-root-with-store-subvolum

bug#33509: core-updates: arc-theme fails with "Invalid GNOME version: 3.24"

2018-11-26 Thread Efraim Flashner
On Mon, Nov 26, 2018 at 12:55:15PM +0100, Björn Höfling wrote:
> On core-updates the package arc-theme fails:
> 
> starting phase `configure'
> source directory: 
> "/tmp/guix-build-arc-theme-20170302.drv-0/arc-theme-20170302" 
> build directory: "/tmp/guix-build-arc-theme-20170302.drv-0/arc-theme-20170302"
> configure flags: 
> ("CONFIG_SHELL=/gnu/store/q19l04vd2za80mk1845pz7r8cz29qk43-bash
> configure: WARNING: unrecognized options: --enable-fast-install
> checking for a BSD-compatible install... 
> /gnu/store/5s2nib1lrd2101bbrivcl17kjx1m
> checking whether build environment is sane... yes
> checking for a thread-safe mkdir -p... 
> /gnu/store/5s2nib1lrd2101bbrivcl17kjx1msp
> checking for gawk... gawk
> checking whether make sets $(MAKE)... no
> checking whether make supports nested variables... yes
> checking for pkg-config... 
> /gnu/store/xyiyf12w4i7zf5g5b4ah0bi1mmd6i8xa-pkg-confi
> checking pkg-config is at least version 0.9.0... yes
> configure: error: Invalid GNOME version: 3.24
> 
> It seams that only GNOME versions until 3.22 are valid.
> 
> Nix has a workaround that sets the version down:
> 
> https://github.com/NixOS/nixpkgs/commit/165fa89762d7156bb3f2774cd3ff0d40cd0a9606
> 
> If I set the version hard to 3.22 the package builds again
> successfully:
> 
> - '(#:phases
> + '(#:configure-flags
> +   '("--with-gnome=3.22")
> +   #:phases
> 
> Is it legitimate to patch it like that or do we need more tests?
> 
> Björn
> 

I'd look into if you can use (version-major+minor (package-version gnome))
so it doesn't have to be updated.


-- 
Efraim Flashner  אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted


signature.asc
Description: PGP signature