Re: 03/03: gnu: nss, nss-certs: Update to 3.27.2.

[Leo Famulari]

On Tue, Dec 20, 2016 at 01:56:03PM -0500, Mark H Weaver wrote:
> l...@famulari.name (Leo Famulari) writes:
> 
> > lfam pushed a commit to branch master
> > in repository guix.
> >
> > commit 7ab3ea426640e4e7ae798a8f72b3c90b383cb824
> > Author: Leo Famulari 
> > Date:   Tue Dec 13 18:59:50 2016 -0500
> >
> > gnu: nss, nss-certs: Update to 3.27.2.
> > 
> > * gnu/packages/gnuzilla.scm (nss): Update to 3.27.2.
> > * gnu/packages/certs.scm (nss-certs): Update to 3.27.2.
> 
> Thanks for this, but unfortunately this version of 'nss' seems to
> consistently fail its test suite on armhf, or at least it has failed 3
> times in a row.
> 
>   https://hydra.gnu.org/build/1712083

At least some of these failures are caused by an expired test
certificate:

https://bugzilla.mozilla.org/show_bug.cgi?id=1323978

This breaks compilation of NSS 3.27.1 and 3.27.2 across all of our
architectures.

Re: [PATCH 01/11] gnu: Add python-flask-wtf.

[Danny Milosavljevic]

Since flask-WTF directly imports flask, this package should also propagate 
flask.

time out in shepherd service?

[ng0]

Is there a time out in shepherd service start/checks, or is the
"service could not be started" simply a result of when a service
encounters a fatal error and quits?
-- 
♥Ⓐ  ng0  | PGP keys and more: https://n0is.noblogs.org/
 |http://ng0.chaosnet.org

Re: Remove duplicates in AUTHORS file before the release

[David Hashe]

Hi Alex,

david.ha...@dhashe.com is correct.

Thanks,
David

On Tue, Dec 20, 2016 at 7:57 AM, Alex Kost  wrote:

> Hello, if it's not too late for the release, I think it would be good to
> adjust ".mailmap" to avoid duplicates in the generated AUTHORS file (if
> it's too late, it would be good anyway).
>
> As you can check with "git --no-pager shortlog -nse", there are some
> duplicates in the output (double names or emails for the same person).
> This output is used to generate AUTHORS file in release tarballs.  So to
> disambiguate this, I would like to ask the people what are their
> preference:
>
> - Alex:  or ?
>
> - Ben: "Ben Woodcroft" or "Ben J Woodcroft"?
>
> - Daniel:  or ?
>
> - David: , right? (you have one commit with
>   )
>
> - Ivan: "Ivan Vilata-i-Balaguer" or "Ivan Vilata i Balaguer"?
>
> - Jeff:  or ?
>
> - John: "John J. Foerch" or "John J Foerch"?
>
> - Marek:  or ?
>
> - Marius:  or ?
>
> - Pjotr:  or ?
>
> - Thomas:  or ?
>
> - Vincent:  or ?
>
>
> Please do not Cc other people in your answer, just reply to
> , thanks!
>
> --
> Alex
>

Re: gnunet-service, first draft in need of review.

[Hartmut Goebel]

Am 20.12.2016 um 18:11 schrieb ng0:
> I found the files created (for example $directory/hostlists/new
> (or whatever the name was) is owned by root. this should not
> happen, it should be gnunet:gnunet.

Then maybe the process is run as root. postgresql-shepherd-service has
an example for how to switch the user.

-- 
Regards
Hartmut Goebel

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

Re: 03/03: gnu: nss, nss-certs: Update to 3.27.2.

[Leo Famulari]

On Tue, Dec 20, 2016 at 01:56:03PM -0500, Mark H Weaver wrote:
> > gnu: nss, nss-certs: Update to 3.27.2.
> > 
> > * gnu/packages/gnuzilla.scm (nss): Update to 3.27.2.
> > * gnu/packages/certs.scm (nss-certs): Update to 3.27.2.
> 
> Thanks for this, but unfortunately this version of 'nss' seems to
> consistently fail its test suite on armhf, or at least it has failed 3
> times in a row.
> 
>   https://hydra.gnu.org/build/1712083

Thanks for pointing this out.

> Given the importance of the proper functioning of this package, I'm not
> comfortable disabling the tests.

I agree.

> Do we have reason to believe that this update fixes security flaws?  Is
> there a compelling reason not to revert this update until a version is
> released that passes the test suite on our supported systems?

Not as far as I know, although I assume there are some sort of trust
"problems" fixed in each release of nss-certs.

I'll revert it and investigate. I'd rather not wait for an upstream fix
if we can help it.

I notice know that this release appears to require a newer version of
nspr than we package [0]:

"The HG tag is NSS_3_27_2_RTM. NSS 3.27.2 requires NSPR 4.13 or newer."

What do you recommend I do? How about I make an nss-updates branch with
updates to nspr, nss, nss-certs, and possibly other updates in (gnu
packages gnuzilla), and build it on Hydra when resources are available?

[0]
https://developer.mozilla.org/en-US/docs/Mozilla/Projects/NSS/NSS_3.27.2_Release_Notes


signature.asc
Description: PGP signature

Re: 03/03: gnu: nss, nss-certs: Update to 3.27.2.

[Mark H Weaver]

l...@famulari.name (Leo Famulari) writes:

> lfam pushed a commit to branch master
> in repository guix.
>
> commit 7ab3ea426640e4e7ae798a8f72b3c90b383cb824
> Author: Leo Famulari 
> Date:   Tue Dec 13 18:59:50 2016 -0500
>
> gnu: nss, nss-certs: Update to 3.27.2.
> 
> * gnu/packages/gnuzilla.scm (nss): Update to 3.27.2.
> * gnu/packages/certs.scm (nss-certs): Update to 3.27.2.

Thanks for this, but unfortunately this version of 'nss' seems to
consistently fail its test suite on armhf, or at least it has failed 3
times in a row.

  https://hydra.gnu.org/build/1712083

Given the importance of the proper functioning of this package, I'm not
comfortable disabling the tests.

Do we have reason to believe that this update fixes security flaws?  Is
there a compelling reason not to revert this update until a version is
released that passes the test suite on our supported systems?

 Thanks,
   Mark

Re: [PATCH 6/6] gnu: Add grub-efi.

[Marius Bakke]

Ludovic Courtès  writes:

> Marius Bakke  skribis:
>
>> Ludovic Courtès  writes:
>>
>>> Hello!
>>>
>>> Marius Bakke  skribis:
>>>
>> OK. I'll try to find out why tests don't work with the UEFI variant
>> first in order to at least write a meaningful comment. Maybe qemu needs
>> UEFI support or something like that.
>
> It might be that we no longer need QEMU 1.3.1 to run the tests (see the
> top of gnu/packages/grub.scm)?

 The problem is missing UEFI firmware for the qemu calls. But we indeed
 no longer need qemu@1.3.1 for the tests, at least on x86_64. I replaced
 it with 'qemu-minimal'. Pushed!
>>>
>>> Great!
>>>
>>> I’m failing at installing GuixSD on a new laptop I have here.
>>> ‘efibootmgr’ exits with code 2 and this message:
>>>
>>>   EFI variables are not supported on this system.
>>>
>>> (which ‘grub-install’ happily ignores.)
>>>
>>> This is because /sys/firmware/efi is missing, which apparently is
>>> because I booted off the GuixSD USB image (“legacy”) and not in EFI
>>> mode.
>>>
>>> What would you suggest?  :-)
>>
>> What I did was a normal BIOS install, backup the grub.cfg, switch laptop
>> to UEFI (only) and boot a Debian live CD. From there "apt-get install
>> grub-efi; grub-install /dev/sda" and afterwards copy grub.cfg in place.
>>
>> You may want to add "insmod efi_gop" and "insmod efi_uga" to grub.cfg,
>> otherwise you won't get a framebuffer until the proper video driver is
>> loaded (which may require unlocking root partition etc).
>
> Wait, all I need is /sys/firmware/efi in the install image.  Is it
> impossible?

Perhaps you can trick Linux into creating it without booting UEFI mode.
Not sure if grub/efibootmgr actually need to read or write there.

>> Not the most user friendly installation instructions! I'm researching
>> methods to make the base install image hybrid BIOS/UEFI.
>
> What would it take?

The scripts I've looked at so far seems to use ISOLINUX as the initial
bootloader and then chainload to grub. Didn't experiment much, haven't
been able to get syslinux packaged yet.

It's *probably* possible do it with grub only, by partitioning the
installation image and create both a "bios_grub" GPT partition and an
EFI system partition and install to both with a different --target.


signature.asc
Description: PGP signature

Re: gnunet-service, first draft in need of review.

[ng0]

Hartmut Goebel  writes:

> Am 19.12.2016 um 20:30 schrieb ng0:
>> I'd like someone to review the shepherd service so I can be sure that
>> the errors I'm getting aren't gnunet related. You have to configure a
>
> Maybe this is due to wrong file-system permissions: In gnunet-activation
> you write:
>
> +   (chmod "/var/lib/gnunet/" #o600)
> +   (mkdir-p "/var/lib/gnunet/.local/share/gnunet")
>
>
> So the service first removes the write permissions and then tries to
> create a sub-directory.

I found the files created (for example $directory/hostlists/new
(or whatever the name was) is owned by root. this should not
happen, it should be gnunet:gnunet.
The permissions have to be very restrictive, at least from what I
remember writing the Gentoo package and service.
I'll try to adjust the chmod level then, maybe this fixed the not
starting problem.

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

-- 
♥Ⓐ  ng0  | PGP keys and more: https://n0is.noblogs.org/
 |http://ng0.chaosnet.org

Re: [PATCH 6/6] gnu: Add grub-efi.

[Ludovic Courtès]

Marius Bakke  skribis:

> Ludovic Courtès  writes:
>
>> Hello!
>>
>> Marius Bakke  skribis:
>>
> OK. I'll try to find out why tests don't work with the UEFI variant
> first in order to at least write a meaningful comment. Maybe qemu needs
> UEFI support or something like that.

 It might be that we no longer need QEMU 1.3.1 to run the tests (see the
 top of gnu/packages/grub.scm)?
>>>
>>> The problem is missing UEFI firmware for the qemu calls. But we indeed
>>> no longer need qemu@1.3.1 for the tests, at least on x86_64. I replaced
>>> it with 'qemu-minimal'. Pushed!
>>
>> Great!
>>
>> I’m failing at installing GuixSD on a new laptop I have here.
>> ‘efibootmgr’ exits with code 2 and this message:
>>
>>   EFI variables are not supported on this system.
>>
>> (which ‘grub-install’ happily ignores.)
>>
>> This is because /sys/firmware/efi is missing, which apparently is
>> because I booted off the GuixSD USB image (“legacy”) and not in EFI
>> mode.
>>
>> What would you suggest?  :-)
>
> What I did was a normal BIOS install, backup the grub.cfg, switch laptop
> to UEFI (only) and boot a Debian live CD. From there "apt-get install
> grub-efi; grub-install /dev/sda" and afterwards copy grub.cfg in place.
>
> You may want to add "insmod efi_gop" and "insmod efi_uga" to grub.cfg,
> otherwise you won't get a framebuffer until the proper video driver is
> loaded (which may require unlocking root partition etc).

Wait, all I need is /sys/firmware/efi in the install image.  Is it
impossible?

> Not the most user friendly installation instructions! I'm researching
> methods to make the base install image hybrid BIOS/UEFI.

What would it take?

Woow, I didn’t expect UEFI to be that much of a pain.  :-/

Ludo’.

Re: gnunet-service, first draft in need of review.

[Hartmut Goebel]

Am 19.12.2016 um 20:30 schrieb ng0:
> I'd like someone to review the shepherd service so I can be sure that
> the errors I'm getting aren't gnunet related. You have to configure a

Maybe this is due to wrong file-system permissions: In gnunet-activation
you write:

+   (chmod "/var/lib/gnunet/" #o600)
+   (mkdir-p "/var/lib/gnunet/.local/share/gnunet")


So the service first removes the write permissions and then tries to
create a sub-directory.

-- 
Regards
Hartmut Goebel

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

Re: Remove duplicates in AUTHORS file before the release

[Ivan Vilata i Balaguer]

Alex Kost (2016-12-20 16:57:50 +0300) wrote:

> Hello, if it's not too late for the release, I think it would be good
> to adjust ".mailmap" to avoid duplicates in the generated AUTHORS file
> (if it's too late, it would be good anyway). […]
> 
> - Ivan: "Ivan Vilata-i-Balaguer" or "Ivan Vilata i Balaguer"?

Hi Alex, thanks for taking care of this.  In my case the second version
is the native Catalan one, but since it causes lots of issues with
programs dividing it into first name and surname, I'd rather use the
first one (with the dashes), which clusters name and surnames
separately.

Cheers,

-- 
Ivan Vilata i Balaguer -- https://elvil.net/

Re: Remove duplicates in AUTHORS file before the release

[Marek Benc]

On Tue, 20 Dec 2016 16:57:50 +0300
Alex Kost  wrote:

> Hello, if it's not too late for the release, I think it would be good
> to adjust ".mailmap" to avoid duplicates in the generated AUTHORS
> file (if it's too late, it would be good anyway).
> 
> As you can check with "git --no-pager shortlog -nse", there are some
> duplicates in the output (double names or emails for the same person).
> This output is used to generate AUTHORS file in release tarballs.  So
> to disambiguate this, I would like to ask the people what are their
> preference:
> 
> - Alex:  or ?
> 
> - Ben: "Ben Woodcroft" or "Ben J Woodcroft"?
> 
> - Daniel:  or ?
> 
> - David: , right? (you have one commit with
>   )
> 
> - Ivan: "Ivan Vilata-i-Balaguer" or "Ivan Vilata i Balaguer"?
> 
> - Jeff:  or ?
> 
> - John: "John J. Foerch" or "John J Foerch"?
> 
> - Marek:  or ?
> 
> - Marius:  or ?
> 
> - Pjotr:  or ?
> 
> - Thomas:  or ?
> 
> - Vincent:  or
> ?
> 
> 
> Please do not Cc other people in your answer, just reply to
> , thanks!
> 

Marek Benc here, my preferred email address is .

-- 
Marek

Re: alsa-plugins, new version of patch

[Kei Kebreau]

ng0  writes:

> Thanks Kei for catching this mistake. I tried to solve it with a
> directory-excursion, but ended up using chdir and removed the
> dedicated phase to delete the empty dir, as deleting the directory to
> me is a closure/finishing touch to the split phase.
> Any objections, further comments or can this be added now?

LGTM. Pushed as 7d2511bc6b467c056e7e0bcb0760f7b9652ba083.


signature.asc
Description: PGP signature

Re: [PATCH][WIP] gnu: Add debbugs.

[ng0]

ng0  writes:

> Ludovic Courtès  writes:
>
>> ng0  skribis:
>>
>>> So, here's debbugs. Disclaimer: I never ran or did setup or got in any
>>> way in touch with debbugs on a server.
>>>
>>> Input welcome to adjust. All I know is that we'll need a service to make
>>> this work. I have no reasonable interest in this, so I won't go all the
>>> way. I'd like to fix this package up as far as usage goes.
>>> I can not test it running, but from the build/install perspective it is
>>> done. There are structures in code which can be changed etc, but this
>>> was done in about ~30 minutes to get some input as I thought, hey we use
>>> debbugs.. why do we not have debbugs packaged already?
>>
>> Because nobody did it I guess.
>>
>> Could you send an non-WIP patch?  Since you mark it as WIP, people most
>> likely don’t look at it, because there’s already so many patches to
>> review.  Please take extra time to test it and resend when you think
>> it’s ready to apply.
>>
>> Thank you!
>>
>> Ludo’.
>>
>
> Technically this is finished and can be applied on master. Why I can't
> test it I have described in the text above the email - I have no idea
> how debbugs behaves, what is  expected etc.. that's why I think someone
> with an interest to replace debbugs on their (mostly) debian based
> server with debbugs from Guix should test it.

I try to get to the point where I have an local or external
server running GuixSD some time between now and end of spring, so
I will be able to easily test this.
I'll work on this and come up with an patch which is no longer
"work in progress", nevertheless some feedback on the original
patch from someone (search the archives for "gnu: Add debbugs", I
don't provide links I don't need) would be highly
appreciated. I haven't reviewed myself after the time which
passed, I hope it's not too horrible.

-- 
♥Ⓐ  ng0  | PGP keys and more: https://n0is.noblogs.org/
 |http://ng0.chaosnet.org

Re: Remove duplicates in AUTHORS file before the release

[Alex Sassmannshausen]

Alex Kost writes:

> Hello, if it's not too late for the release, I think it would be good to
> adjust ".mailmap" to avoid duplicates in the generated AUTHORS file (if
> it's too late, it would be good anyway).
>
> As you can check with "git --no-pager shortlog -nse", there are some
> duplicates in the output (double names or emails for the same person).
> This output is used to generate AUTHORS file in release tarballs.  So to
> disambiguate this, I would like to ask the people what are their
> preference:
>
> - Alex:  or ?

I'm using a...@pompo.co .

Thank you for doing this!

Alex

Re: [PATCH 3/3] gnu: Add paprefs

[ng0]

ng0  writes:

> I found neither paprefs nor gconfmm in our package database as I was
> about to move this to my 'done' tag, out of sight.
> Can someone (Christopher? Leo?) finish this?

As I got no reply after my initial question, I will fix both of
these packages up next month.

> Thanks.
> Leo Famulari  writes:
>
>> On Wed, Feb 17, 2016 at 03:02:41PM +, Christopher Baines wrote:
>>> * gnu/packages/pulseaudio.scm (paprefs): New variable
>>
>> Cool, thanks for the patches! This review is rather superficial since I
>> don't have a GuixSD desktop system, so I can't really test the
>> functionality of the software provided by this patch series. I assume it
>> works for you. Did you test it on GuixSD or on a foreign distro?
>>
>>> ---
>>>  gnu/packages/pulseaudio.scm | 34 ++
>>>  1 file changed, 34 insertions(+)
>>
>> [...]
>>
>>> +(define-public paprefs
>>> +  (package
>>> +(name "paprefs")
>>> +(version "0.9.10")
>>> +(source (origin
>>> + (method url-fetch)
>>> + (uri (string-append
>>> +   
>>> "https://freedesktop.org/software/pulseaudio/paprefs/paprefs-;
>>> +   version
>>> +   ".tar.xz"))
>>
>> I'd prefer if the URL started directly after string-append, and then
>> used the minimum number of lines, while staying within the 80 character
>> limit. I can correct this sort of thing when applying patches, but I
>> must say it out loud once in a while ;)
>>
>>> + (sha256
>>> +  (base32
>>> +   "1c5b3sb881szavly220q31g7rvpn94wr7ywlk00hqb9zaikml716"
>>> +(build-system gnu-build-system)
>>> +(arguments
>>> +  '(#:configure-flags '("CXXFLAGS=-std=c++11"))) ; required by gtkmm
>>> +(inputs
>>> + `(("pulseaudio" ,pulseaudio)
>>> +   ("gconfmm" ,gconfmm)
>>> +   ("lynx" ,lynx) ; used for documentation generation
>>
>> If it's only used to generate the docs, can we make it a native-input,
>> so it's only available while building?
>>
>>> +   ("libglademm" ,libglademm)
>>> +   ("dbus-glib" ,dbus-glib)))
>>> +(native-inputs
>>> + `(("intltool" ,intltool)
>>> +   ("pkg-config" ,pkg-config)))
>>> +(home-page "https://freedesktop.org/software/pulseaudio/paprefs/;)
>>> +(synopsis "simple configuration dialog for PulseAudio")
>>
>> The linter whines that the synopsis does not start with a capital
>> letter. What, you don't like the linter? ;)
>>
>>> +(description
>>> + "PulseAudio Preferences (paprefs) is a simple GTK+ based configuration
>>> +dialog for the PulseAudio sound server.")
>>> +(license l:gpl2+)))
>>> -- 
>>> 2.7.0
>>> 
>>> 
>>
>
> -- 
>   ng0
>
>

-- 
♥Ⓐ  ng0  | PGP keys and more: https://n0is.noblogs.org/
 |http://ng0.chaosnet.org

New package: reducelcs (v2)

[ng0]

Added a comment about the commit used, tried to make the synopsis shorter.
It can be made even shorter in my opinion, but I don't see how right now.
Feedback welcome.

[PATCH] gnu: Add reducelcs.

[ng0]

From: ng0 

* gnu/packages/lcs.scm: New file.
* gnu/local.mk (GNU_SYSTEM_MODULES): Add lcs.scm.
---
 gnu/local.mk |  1 +
 gnu/packages/lcs.scm | 67 
 2 files changed, 68 insertions(+)
 create mode 100644 gnu/packages/lcs.scm

diff --git a/gnu/local.mk b/gnu/local.mk
index 3a56c840b..ce09ed238 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -201,6 +201,7 @@ GNU_SYSTEM_MODULES =\
   %D%/packages/key-mon.scm \
   %D%/packages/kodi.scm\
   %D%/packages/language.scm\
+  %D%/packages/lcs.scm  \
   %D%/packages/ldc.scm \
   %D%/packages/lego.scm\
   %D%/packages/less.scm\
diff --git a/gnu/packages/lcs.scm b/gnu/packages/lcs.scm
new file mode 100644
index 0..6974b25e9
--- /dev/null
+++ b/gnu/packages/lcs.scm
@@ -0,0 +1,67 @@
+;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2016 ng0 
+;;;
+;;; This file is part of GNU Guix.
+;;;
+;;; GNU Guix is free software; you can redistribute it and/or modify it
+;;; under the terms of the GNU General Public License as published by
+;;; the Free Software Foundation; either version 3 of the License, or (at
+;;; your option) any later version.
+;;;
+;;; GNU Guix is distributed in the hope that it will be useful, but
+;;; WITHOUT ANY WARRANTY; without even the implied warranty of
+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;;; GNU General Public License for more details.
+;;;
+;;; You should have received a copy of the GNU General Public License
+;;; along with GNU Guix.  If not, see .
+
+(define-module (gnu packages lcs)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix packages)
+  #:use-module (gnu packages)
+  #:use-module (gnu packages maths)
+  #:use-module (guix git-download)
+  #:use-module (guix build-system gnu))
+
+(define-public reducelcs
+  ;; This is the last commit which is available upstream, no
+  ;; tarball release happened since 2010.
+  (let ((commit "474f88deb968061abe8cf11c959e02319b8ae5c0")
+(revision "1"))
+(package
+  (name "reducelcs")
+  (version (string-append "0.0.0-" revision "." (string-take commit 7)))
+  (source (origin
+(method git-fetch)
+(uri (git-reference
+  (url "https://github.com/gdv/Reduce-Expand-for-LCS;)
+  (commit commit)))
+(file-name (string-append name "-" version "-checkout"))
+(sha256
+ (base32
+  "1rllzcfwc042c336mhq262a8ha90x6afq30kvk60r7i4761j4yjm"
+  (build-system gnu-build-system)
+  (inputs
+   `(("openlibm" ,openlibm)))
+  (arguments
+   `(#:tests? #f ; no tests
+ #:phases
+ (modify-phases %standard-phases
+   (delete 'configure) ; no configure script
+   (replace 'install ; no install phase either
+ (lambda* (#:key outputs #:allow-other-keys)
+   (let* ((out (assoc-ref outputs "out"))
+  (bin (string-append out "/bin")))
+ (install-file "Approximation" bin)
+ (install-file "CollectResults" bin)
+ (install-file "GenerateInstances" bin)
+ #t))
+  (synopsis "Compute the approximate Longest Commons Subsequence
+of a set of strings")
+  (description
+   "@code{reduceLCS} is an implementation of the Reduce-Expand
+algorithm for LCS.  It is a fast program to compute the approximate
+Longest Commons Subsequence of a set of strings.")
+  (home-page "http://gianluca.dellavedova.org/software/;)
+  (license license:gpl2
-- 
2.11.0

Re: Remove duplicates in AUTHORS file before the release

[Vincent Legoll]

Hello,

> - Vincent:  or ?

I'll keep: "Vincent Legoll "

Thanks

-- 
Vincent Legoll

Re: [PATCH 6/6] gnu: Add grub-efi.

[Marius Bakke]

Ludovic Courtès  writes:

> Hello!
>
> Marius Bakke  skribis:
>
 OK. I'll try to find out why tests don't work with the UEFI variant
 first in order to at least write a meaningful comment. Maybe qemu needs
 UEFI support or something like that.
>>>
>>> It might be that we no longer need QEMU 1.3.1 to run the tests (see the
>>> top of gnu/packages/grub.scm)?
>>
>> The problem is missing UEFI firmware for the qemu calls. But we indeed
>> no longer need qemu@1.3.1 for the tests, at least on x86_64. I replaced
>> it with 'qemu-minimal'. Pushed!
>
> Great!
>
> I’m failing at installing GuixSD on a new laptop I have here.
> ‘efibootmgr’ exits with code 2 and this message:
>
>   EFI variables are not supported on this system.
>
> (which ‘grub-install’ happily ignores.)
>
> This is because /sys/firmware/efi is missing, which apparently is
> because I booted off the GuixSD USB image (“legacy”) and not in EFI
> mode.
>
> What would you suggest?  :-)

What I did was a normal BIOS install, backup the grub.cfg, switch laptop
to UEFI (only) and boot a Debian live CD. From there "apt-get install
grub-efi; grub-install /dev/sda" and afterwards copy grub.cfg in place.

You may want to add "insmod efi_gop" and "insmod efi_uga" to grub.cfg,
otherwise you won't get a framebuffer until the proper video driver is
loaded (which may require unlocking root partition etc).

Not the most user friendly installation instructions! I'm researching
methods to make the base install image hybrid BIOS/UEFI.

Once booted, you should apply the following two patches when
reconfiguring the system. I think they are safe for BIOS systems too,
but haven't done extensive testing.



signature.asc
Description: PGP signature
>From fa12cd92a2e4eead22f640d6231813e50b8191bf Mon Sep 17 00:00:00 2001
From: Marius Bakke 
Date: Sun, 6 Nov 2016 17:26:06 +
Subject: [PATCH 1/2] build: Make grub-install command UEFI aware.

* gnu/build/install.scm (install-grub): Extend grub-install command with
'--bootloader-id' and '--efi-directory'.
---
 gnu/build/install.scm | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/gnu/build/install.scm b/gnu/build/install.scm
index 5c2b35632..ddd95bbf6 100644
--- a/gnu/build/install.scm
+++ b/gnu/build/install.scm
@@ -54,6 +54,9 @@ GC'd."
   (unless (zero? (system* "grub-install" "--no-floppy"
   "--boot-directory"
   (string-append mount-point "/boot")
+			  "--bootloader-id=GNU"
+			  "--efi-directory"
+  (string-append mount-point "/boot")
   device))
 (error "failed to install GRUB")))
 
-- 
2.11.0

>From dd71d9b334ceccc09cd42484c6deac2079c44c70 Mon Sep 17 00:00:00 2001
From: Marius Bakke 
Date: Mon, 7 Nov 2016 12:24:01 +
Subject: [PATCH 2/2] system: Load efi modules in grub.cfg.

* gnu/system/grub.scm (eye-candy): Load 'efi_gop' and 'efi_uga' grub modules.
---
 gnu/system/grub.scm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gnu/system/grub.scm b/gnu/system/grub.scm
index 4657b06b5..9477b2494 100644
--- a/gnu/system/grub.scm
+++ b/gnu/system/grub.scm
@@ -183,6 +183,8 @@ system string---e.g., \"x86_64-linux\"."
 (if (string-match "^(x86_64|i[3-6]86)-" system)
 "
   # Leave 'gfxmode' to 'auto'.
+  insmod efi_gop
+  insmod efi_uga
   insmod vbe
   insmod vga
   insmod video_bochs
-- 
2.11.0

[PATCH] gnu: Add alsa-plugins.

[ng0]

From: ng0 

* gnu/packages/linux.scm (alsa-plugins): New variable.
---
 gnu/packages/linux.scm | 65 ++
 1 file changed, 65 insertions(+)

diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
index 827ee8b37..78fab4e18 100644
--- a/gnu/packages/linux.scm
+++ b/gnu/packages/linux.scm
@@ -17,6 +17,7 @@
 ;;; Copyright © 2016 John Darrington 
 ;;; Copyright © 2016 Marius Bakke 
 ;;; Copyright © 2016 Rene Saavedra 
+;;; Copyright © 2016 ng0 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -72,6 +73,8 @@
   #:use-module (gnu packages slang)
   #:use-module (gnu packages texinfo)
   #:use-module (gnu packages tls)
+  #:use-module (gnu packages video)
+  #:use-module (gnu packages xiph)
   #:use-module (gnu packages xml)
   #:use-module (gnu packages xdisorg)
   #:use-module (gnu packages xorg)
@@ -894,6 +897,68 @@ MIDI functionality to the Linux-based operating system.")
 ;; GPLv2-only.
 (license license:gpl2)))
 
+(define-public alsa-plugins
+  (package
+(name "alsa-plugins")
+(version "1.1.1")
+(source (origin
+ (method url-fetch)
+ (uri (string-append "ftp://ftp.alsa-project.org/pub/plugins/;
+ name "-" version ".tar.bz2"))
+ (sha256
+  (base32
+   "1w81z5jlwqhd1l2m7qrq69lc4k9dnrg1wn52jsl2hrf3hbhd394f"
+(build-system gnu-build-system)
+;; TODO: Split libavcodec and speex if possible. It looks like they can not
+;; be split, there are references to both in files.
+;; TODO: Remove OSS related plugins, they add support to run native
+;; ALSA applications on OSS however we do not offer OSS and OSS is
+;; obsolete.
+(outputs '("out" "pulseaudio"))
+(arguments
+ `(#:phases
+   (modify-phases %standard-phases
+ (add-after 'install 'split
+   (lambda* (#:key inputs outputs #:allow-other-keys)
+ ;; Distribute the binaries to the various outputs.
+ (let* ((out (assoc-ref outputs "out"))
+(pua (assoc-ref outputs "pulseaudio"))
+(pualib (string-append pua "/lib/alsa-lib"))
+(puaconf (string-append pua "/share/alsa/alsa.conf.d")))
+   (mkdir-p puaconf)
+   (mkdir-p pualib)
+   (chdir (string-append out "/share"))
+   (for-each (lambda (file)
+   (rename-file file (string-append puaconf "/" 
(basename file
+ (find-files out "\\.(conf|example)"))
+   (for-each (lambda (file)
+   (rename-file file (string-append pualib "/" 
(basename file
+ (find-files out ".*pulse\\.(la|so)"))
+   (chdir "..")
+   ;; We have moved the files to output pulsaudio, the
+   ;; directory is now empty.
+   (delete-file-recursively (string-append out "/share"))
+   #t))
+(inputs
+ `(("alsa-lib" ,alsa-lib)
+   ("speex" ,speex) ; libspeexdsp resampling plugin
+   ("libsamplerate" ,libsamplerate) ; libsamplerate resampling plugin
+   ("ffmpeg" ,ffmpeg) ; libavcodec resampling plugin, a52 plugin
+   ("pulseaudio" ,pulseaudio))) ; PulseAudio plugin
+(native-inputs
+ `(("pkg-config" ,pkg-config)))
+(home-page "http://www.alsa-project.org/;)
+(synopsis "Plugins for the Advanced Linux Sound Architecture (ALSA)")
+(description
+ "The Advanced Linux Sound Architecture (ALSA) provides audio and
+MIDI functionality to the Linux-based operating system.  This package enhances 
ALSA
+by providing additional plugins which include: upmixing, downmixing, jackd and
+pulseaudio support for native alsa applications, format conversion (s16 to 
a52), and
+external rate conversion.")
+(license (list license:gpl2+
+   ;; `rate/rate_samplerate.c': LGPL v2.1 or later.
+   license:lgpl2.1+
+
 (define-public iptables
   (package
 (name "iptables")
-- 
2.11.0

alsa-plugins, new version of patch

[ng0]

Thanks Kei for catching this mistake. I tried to solve it with a 
directory-excursion, but ended up using chdir and removed the dedicated phase 
to delete the empty dir, as deleting the directory to me is a closure/finishing 
touch to the split phase.
Any objections, further comments or can this be added now?

Remove duplicates in AUTHORS file before the release

[Alex Kost]

Hello, if it's not too late for the release, I think it would be good to
adjust ".mailmap" to avoid duplicates in the generated AUTHORS file (if
it's too late, it would be good anyway).

As you can check with "git --no-pager shortlog -nse", there are some
duplicates in the output (double names or emails for the same person).
This output is used to generate AUTHORS file in release tarballs.  So to
disambiguate this, I would like to ask the people what are their
preference:

- Alex:  or ?

- Ben: "Ben Woodcroft" or "Ben J Woodcroft"?

- Daniel:  or ?

- David: , right? (you have one commit with
  )

- Ivan: "Ivan Vilata-i-Balaguer" or "Ivan Vilata i Balaguer"?

- Jeff:  or ?

- John: "John J. Foerch" or "John J Foerch"?

- Marek:  or ?

- Marius:  or ?

- Pjotr:  or ?

- Thomas:  or ?

- Vincent:  or ?


Please do not Cc other people in your answer, just reply to
, thanks!

-- 
Alex

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

[John Darrington]

On Tue, Dec 20, 2016 at 12:03:31PM +0100, John Darrington wrote:
 On Tue, Dec 20, 2016 at 02:17:42AM -0500, Mark H Weaver wrote:

 The scheme code contains a number of procedures similar to 
 (load-extension "libguile-ncurses" "func"). We need the first
 string to contain the absolute path where the shared object is 
 installed.   The previous version had done this by patching the 

  ^^^ 
For clarification, when I say "installed" here,  I mean it's location 
in /gnu/store.

 .scm file post-installation.  However that meant that the compiled
 .go file was compiled with the old, unmodified version.  
 This patch ensures that the .scm files are patched first, and that
 the .go files are built from the patched .scm files.
 



-- 
Avoid eavesdropping.  Send strong encrypted email.
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://sks-keyservers.net or any PGP keyserver for public key.



signature.asc
Description: Digital signature

Re: GuixSD not build

[Ludovic Courtès]

Hi Daniel,

"Daniel Pimentel (d4n1)"  skribis:

> I tried install guixsd recently and all time there are errors about build 
> (printscreen attached). I tried with:
> -guix system init ...
> -guix system init --fallback ...
>
> Note: my network works. I tried with desktop.scm and lightweight-desktop.scm 
> too.

Looks like you sent too big an attachment.  Could you shrink it so that
it takes at most 500KB?

TIA!

Ludo’.

Re: Funny reference in bootstrap bash-static

[Ludovic Courtès]

Hello!

Pjotr Prins  skribis:

> The static version of bash contains a number of non-existant references. They 
> look harmless enough and are probably used
> for bootstrapping. 
>
> Contains /gnu/store reference in 
> gnu/store/7jhakv1r1nbs2sr2f7ammq256w7niarh-bash-static-4.3.39/bin/bash
> 1127848 
> /gnu/store/-bash-static-4.3.39/share/bashdb/bashdb-main.inc
> 1179560 
> /gnu/store/-glibc-intermediate-2.22/lib/gconv/gconv-modules.cache
> 1183296 
> /gnu/store/-glibc-intermediate-2.22/share/locale
> 1183432 
> /gnu/store/-glibc-intermediate-2.22/share/locale
> 1222784 
> /gnu/store/-glibc-intermediate-2.22/share/zoneinfo
> 1226344 
> /gnu/store/-glibc-intermediate-2.22/libexec/getconf
> 1235648 
> /gnu/store/-glibc-intermediate-2.22/lib/

This is on purpose: this is done by the ‘remove-store-references’ call
in the ‘bash-static’ recipe so that ‘bash-static’ has no dependencies
other than itself.

> The thing is that the hello package ultimately references these through
>
> Contains /gnu/store reference in 
> gnu/store/qv7bk62c22ms9i11dhfl71hnivyc82k2-glibc-2.22/bin/xtrace
>   0 
> #!/gnu/store/7jhakv1r1nbs2sr2f7ammq256w7niarh-bash-static-4.3.39/bin/bash
> 903 
> pcprofileso='/gnu/store/qv7bk62c22ms9i11dhfl71hnivyc82k2-glibc-2.22/\$LIB/libpcprofile.so'
> 994 
> pcprofiledump='/gnu/store/qv7bk62c22ms9i11dhfl71hnivyc82k2-glibc-2.22/bin/pcprofiledump'
>
> which means that most packages do. It would probably make sense to have 
> xtrace point to a full bash. No?

‘xtrace’ just needs a minimal shell interpreter, so that’s fine.  See
‘glibc-final-with-bootstrap-bash’ and below in (gnu packages
commencement) for the whole story.

HTH!

Ludo’.

Re: Emacs-Guix released outside from Guix

[Ludovic Courtès]

Howdy,

Thanks for taking the time to explain.

Make sure to let us know what new things happen in Emacs-Guix, you know
it has its fan club here!  :-)

Ludo’.

Re: [PATCH 6/6] gnu: Add grub-efi.

[Ludovic Courtès]

Hello!

Marius Bakke  skribis:

>>> OK. I'll try to find out why tests don't work with the UEFI variant
>>> first in order to at least write a meaningful comment. Maybe qemu needs
>>> UEFI support or something like that.
>>
>> It might be that we no longer need QEMU 1.3.1 to run the tests (see the
>> top of gnu/packages/grub.scm)?
>
> The problem is missing UEFI firmware for the qemu calls. But we indeed
> no longer need qemu@1.3.1 for the tests, at least on x86_64. I replaced
> it with 'qemu-minimal'. Pushed!

Great!

I’m failing at installing GuixSD on a new laptop I have here.
‘efibootmgr’ exits with code 2 and this message:

  EFI variables are not supported on this system.

(which ‘grub-install’ happily ignores.)

This is because /sys/firmware/efi is missing, which apparently is
because I booted off the GuixSD USB image (“legacy”) and not in EFI
mode.

What would you suggest?  :-)

TIA,
Ludo’.

GuixSD not build

[Daniel Pimentel (d4n1)]

GNUs,

I tried install guixsd recently and all time there are errors about build 
(printscreen attached). I tried with:
-guix system init ...
-guix system init --fallback ...

Note: my network works. I tried with desktop.scm and lightweight-desktop.scm 
too.

Thanks,

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

[John Darrington]

On Tue, Dec 20, 2016 at 02:17:42AM -0500, Mark H Weaver wrote:
 Hi John,
 
 John Darrington  writes:
 > * gnu/packages/guile.scm (guile-ncurses) [arguments]: Install shared 
object before
 > attempting to build the package.  Patch load-extension path before 
building instead
 > of after.
 
 The first sentence above is mistaken or misleading:
 
   "Install shared object before attempting to build the package."

How about I change it to "... attempting to build the rest of the package" ??

 > + (add-before 'build 'fix-libguile-ncurses-file-name
 > (lambda* (#:key outputs #:allow-other-keys)
 > - [...]
 > + (and (zero? (system* "make" "install"
 > +  "-C" "src/ncurses"
 > +  "-j" (number->string
 > +(parallel-job-count
 > +  (let* ((out   (assoc-ref outputs "out"))
 > + (dir   "src/ncurses")
 > + (files (find-files dir ".scm")))
 > +(substitute* files
 > +  (("\"libguile-ncurses\"")
 > +   (format #f 
"\"~a/lib/guile/2.0/libguile-ncurses\""
 > +   out)))
 > +#t)))
 
 By running "make install -C src/ncurses [...]", you are in fact causing
 most if not all of the package to be built and installed right here in
 the 'fix-libguile-ncurses-file-name' phase.

I didn't check to see what percentage of the package is contained in the 
src/ncurses directory, but it certainly isn't "all".

 This is no good.

Why do you think it is "no good"?  It may not be perfect, but it is better
than what we had before, and certainly fixes the problem.

 
 Can you help me understand what you're trying to do here, and why?
 
Sure (I would like to see a convention where such explanations are
put in the commit messaage, but I have previously been outvoted on
that issue):

The scheme code contains a number of procedures similar to 
(load-extension "libguile-ncurses" "func"). We need the first
string to contain the absolute path where the shared object is 
installed.   The previous version had done this by patching the 
.scm file post-installation.  However that meant that the compiled
.go file was compiled with the old, unmodified version.  
This patch ensures that the .scm files are patched first, and that
the .go files are built from the patched .scm files.

I hope this makes things clearer for you.

J'


-- 
Avoid eavesdropping.  Send strong encrypted email.
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://sks-keyservers.net or any PGP keyserver for public key.



signature.asc
Description: Digital signature