[PATCH 0/8] updates!

[PATCH 0/8] updates!

[Efraim Flashner]

Its that time again when I look at my package list and decide its time to go out and get updates.  Actually when I read that mpv 0.11.0 was out and we were still on 0.9.0 I realized I missed it last time and it was time to stop putting off enlightenment updates.

Graphviz has two patches.  The first updates graphviz from 2.28.0 to 2.38.0, and comes with the downside that `guix refresh -l graphviz` says 56 dependencies and 105 secondary dependencies, which includes the guix binary itself.  The second patch moves the doc output to a separate output, and I haven't checked to see if that would also cause all the rebuilds, but since it should change the hash I assume it would also cause a bunch of rebuilds.

Efraim Flashner (8):
  gnu: enlightenment: Update to 0.19.11.
  gnu: terminology: Update to 0.9.1.
  gnu: youtube-dl: Update to 2015.09.28.
  gnu: mpv: Update to 0.11.0.
  gnu: mpg123: Update to 1.22.4.
  gnu: graphviz: Update to 2.38.0.
  gnu: graphviz: Add "doc" output.
  gnu: powertop: Update to 2.7.

 gnu/packages/enlightenment.scm |  8 ++++----
 gnu/packages/graphviz.scm      | 20 +++++++++++++++-----
 gnu/packages/linux.scm         |  7 +++----
 gnu/packages/mp3.scm           |  4 ++--
 gnu/packages/video.scm         |  8 ++++----
 5 files changed, 28 insertions(+), 19 deletions(-)

-- 
2.6.0

[PATCH 1/8] gnu: enlightenment: Update to 0.19.11.

[Efraim Flashner]

* gnu/packages/enlightenment.scm (enlightenment): Update to 0.19.11.
---
 gnu/packages/enlightenment.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/enlightenment.scm b/gnu/packages/enlightenment.scm
index 0a7eaf1..96ad676 100644
--- a/gnu/packages/enlightenment.scm
+++ b/gnu/packages/enlightenment.scm
@@ -270,14 +270,14 @@ Libraries with some extra bells and whistles.")
 (define-public enlightenment
   (package
     (name "enlightenment")
-    (version "0.19.9")
+    (version "0.19.11")
     (source (origin
               (method url-fetch)
               (uri
                (string-append "https://download.enlightenment.org/rel/apps/"
                               name "/" name "-" version ".tar.xz"))
               (sha256
-               (base32 "1ajpyw1bfbmxpniw041vbvrs9wqgzyw9kccx72nnfiy2wxspjcr5"))))
+               (base32 "0b8m6062i62q31gp9nd4hd7sckwxfvc6dgyck3f9gjdhrwvcn764"))))
     (build-system gnu-build-system)
     (native-inputs
      `(("pkg-config" ,pkg-config)))
-- 
2.6.0

[PATCH 2/8] gnu: terminology: Update to 0.9.1.

[Efraim Flashner]

* gnu/packages/enlightenment.scm (terminology): Update to 0.9.1.
---
 gnu/packages/enlightenment.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/enlightenment.scm b/gnu/packages/enlightenment.scm
index 96ad676..584cab6 100644
--- a/gnu/packages/enlightenment.scm
+++ b/gnu/packages/enlightenment.scm
@@ -218,7 +218,7 @@ The only supported now is VLC.")
 (define-public terminology
   (package
     (name "terminology")
-    (version "0.9.0")
+    (version "0.9.1")
     (source (origin
               (method url-fetch)
               (uri
@@ -226,7 +226,7 @@ The only supported now is VLC.")
                               "terminology/terminology-" version ".tar.xz"))
               (sha256
                (base32
-                "0iwid9cvd96kwl0hjhbby84kkz5sgb4p8454nnkf7wjvdz2f9b96"))))
+                "1kwv9vkhngdm5v38q93xpcykghnyawhjjcb5bgy0p89gpbk7mvpc"))))
     (build-system gnu-build-system)
     (native-inputs
      `(("pkg-config" ,pkg-config)))
-- 
2.6.0

[PATCH 3/8] gnu: youtube-dl: Update to 2015.09.28.

[Efraim Flashner]

* gnu/packages/video.scm (youtube-dl): Update to 2015.09.28.
---
 gnu/packages/video.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 28185fe..4a5ba41 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -791,7 +791,7 @@ projects while introducing many more.")
 (define-public youtube-dl
   (package
     (name "youtube-dl")
-    (version "2015.09.03")
+    (version "2015.09.28")
     (source (origin
               (method url-fetch)
               (uri (string-append "https://youtube-dl.org/downloads/"
@@ -799,7 +799,7 @@ projects while introducing many more.")
                                   version ".tar.gz"))
               (sha256
                (base32
-                "0wxjbqr07rm26iih12yhv0qwspfrg9safhgnrp8misqjjk4fz86z"))))
+                "0q3s2a91s1lr1db2ngacq0iapyr4jngx1dqp4z5dc6zma0qyx5k3"))))
     (build-system python-build-system)
     (inputs `(("setuptools" ,python-setuptools)))
     (home-page "http://youtube-dl.org")
-- 
2.6.0

[PATCH 4/8] gnu: mpv: Update to 0.11.0.

[Efraim Flashner]

* gnu/packages/video.scm (mpv): Update to 0.11.0.
---
 gnu/packages/video.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 4a5ba41..b46a986 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -671,7 +671,7 @@ SVCD, DVD, 3ivx, DivX 3/4/5, WMV and H.264 movies.")
 (define-public mpv
   (package
     (name "mpv")
-    (version "0.9.0")
+    (version "0.11.0")
     (source (origin
               (method url-fetch)
               (uri (string-append
@@ -679,7 +679,7 @@ SVCD, DVD, 3ivx, DivX 3/4/5, WMV and H.264 movies.")
                     ".tar.gz"))
               (sha256
                (base32
-                "08nx0g6ji2d90f5w62g327szhkb7id7jzwgf3x069rc5id1x3bx7"))
+                "1njvmqzj8akan5y485gx4blynwiy52adw7zbbnnnvd3dwis725d2"))
               (file-name (string-append name "-" version ".tar.gz"))))
     (build-system waf-build-system)
     (native-inputs
-- 
2.6.0

[PATCH 5/8] gnu: mpg123: Update to 1.22.4.

[Efraim Flashner]

* gnu/packages/mp3.scm (mpg123): Update to 1.22.4.
---
 gnu/packages/mp3.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/mp3.scm b/gnu/packages/mp3.scm
index d5ca155..4ce6917 100644
--- a/gnu/packages/mp3.scm
+++ b/gnu/packages/mp3.scm
@@ -283,14 +283,14 @@ This package contains the binary.")
 (define-public mpg123
   (package
     (name "mpg123")
-    (version "1.19.0")
+    (version "1.22.4")
     (source (origin
               (method url-fetch)
               (uri (string-append "mirror://sourceforge/mpg123/mpg123-"
                                   version ".tar.bz2"))
               (sha256
                (base32
-                "06xhd68mj9yp0r6l771aq0d7xgnl402a3wm2mvhxmd3w3ph29446"))))
+                "1lj0xv0b6sgqsbhx10dg60cnzgz98i76gxy51kqh11hka0pf0sah"))))
     (build-system gnu-build-system)
     (arguments '(#:configure-flags '("--with-default-audio=pulse")))
     (native-inputs `(("pkg-config" ,pkg-config)))
-- 
2.6.0

[PATCH 6/8] gnu: graphviz: Update to 2.38.0.

[Efraim Flashner]

* gnu/packages/graphviz.scm (graphviz): Update to 2.38.0.
---
 gnu/packages/graphviz.scm | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/graphviz.scm b/gnu/packages/graphviz.scm
index 52522af..e1eb654 100644
--- a/gnu/packages/graphviz.scm
+++ b/gnu/packages/graphviz.scm
@@ -36,16 +36,15 @@
 (define-public graphviz
   (package
     (name "graphviz")
-    (version "2.28.0")
+    (version "2.38.0")
     (source (origin
              (method url-fetch)
              (uri (string-append
                    "http://www.graphviz.org/pub/graphviz/ARCHIVE/graphviz-"
-                   version
-                   ".tar.gz"))
+                   version ".tar.gz"))
              (sha256
               (base32
-               "0xpwg99cd8sp0c6r8klsmc66h1pday64kmnr4v6f9jkqqmrpkank"))))
+               "17l5czpvv5ilmg17frg0w4qwf89jzh2aglm9fgx0l0aakn6j7al1"))))
     (build-system gnu-build-system)
     (arguments
      ;; FIXME: rtest/rtest.sh is a ksh script (!).  Add ksh as an input.
-- 
2.6.0

[PATCH 7/8] gnu: graphviz: Add "doc" output.

[Efraim Flashner]

* gnu/packages/graphviz.scm (graphviz)[outputs, arguments]: New fields.
---
 gnu/packages/graphviz.scm | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/graphviz.scm b/gnu/packages/graphviz.scm
index e1eb654..09f475b 100644
--- a/gnu/packages/graphviz.scm
+++ b/gnu/packages/graphviz.scm
@@ -1,5 +1,6 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2013, 2015 Ludovic Courtès <ludo@gnu.org>
+;;; Copyright © 2015 Efraim Flashner <efraim@flashner.co.il>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -59,7 +60,16 @@
                    (substitute* "lib/gvc/Makefile"
                      (("am__append_5 *=.*")
                       "am_append_5 =\n")))
-                 %standard-phases)))
+                 (alist-cons-after
+                  'install 'move-docs
+                  (lambda* (#:key outputs #:allow-other-keys)
+                           (let ((out (assoc-ref outputs "out"))
+                                 (doc (assoc-ref outputs "doc")))
+                             (mkdir-p (string-append doc "/share/graphviz"))
+                             (rename-file (string-append out "/share/graphviz/doc")
+                                          (string-append doc "/share/graphviz/doc"))
+                             #t))
+                 %standard-phases))))
     (inputs
      `(("libXrender" ,libxrender)
        ("libX11" ,libx11)
@@ -76,6 +86,7 @@
     (native-inputs
      `(("bison" ,bison)
        ("pkg-config" ,pkg-config)))
+    (outputs '("out" "doc"))                      ; 5 MiB of html + pdfs
     (home-page "http://www.graphviz.org/")
     (synopsis "Graph visualization software")
     (description
-- 
2.6.0

[PATCH 8/8] gnu: powertop: Update to 2.7.

[Efraim Flashner]

* gnu/packages/linux.scm (powertop): Update to 2.7.
  [arguments]: Add "pciutils" to inputs.
---
 gnu/packages/linux.scm | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
index d261933..046a105 100644
--- a/gnu/packages/linux.scm
+++ b/gnu/packages/linux.scm
@@ -1159,7 +1159,7 @@ devices.  It replaces 'iwconfig', which is deprecated.")
 (define-public powertop
   (package
     (name "powertop")
-    (version "2.5")
+    (version "2.7")
     (source
      (origin
        (method url-fetch)
@@ -1168,12 +1168,11 @@ devices.  It replaces 'iwconfig', which is deprecated.")
              version ".tar.gz"))
        (sha256
         (base32
-         "02rwqbpasdayl201v0549gbp2f82rd0hqiv3i111r7npanjhhb4b"))))
+         "1jkqqr3l1x98m7rgin1dgfzxqwj4vciw9lyyq1kl9bdswa818jwd"))))
     (build-system gnu-build-system)
     (inputs
-     ;; TODO: Add pciutils.
      `(("zlib" ,zlib)
-       ;; ("pciutils" ,pciutils)
+       ("pciutils" ,pciutils)
        ("ncurses" ,ncurses)
        ("libnl" ,libnl)))
     (native-inputs
-- 
2.6.0

Re: [PATCH 0/8] updates!

[Ludovic Courtès]

Efraim Flashner <efraim@flashner.co.il> skribis:

> Its that time again when I look at my package list and decide its time to go out and get updates.  Actually when I read that mpv 0.11.0 was out and we were still on 0.9.0 I realized I missed it last time and it was time to stop putting off enlightenment updates.

Heh.

> Graphviz has two patches.  The first updates graphviz from 2.28.0 to 2.38.0, and comes with the downside that `guix refresh -l graphviz` says 56 dependencies and 105 secondary dependencies, which includes the guix binary itself.  The second patch moves the doc output to a separate output, and I haven't checked to see if that would also cause all the rebuilds, but since it should change the hash I assume it would also cause a bunch of rebuilds.

It all LGTM, including the patch that adds a “doc” output for Graphviz.

Could you give me your Savannah account name so we can add you there for
commit access?

Thanks!

Ludo’.

Re: [PATCH 0/8] updates!

[Efraim Flashner]

[-- Attachment #1: Type: text/plain, Size: 523 bytes --]

On Thu, 01 Oct 2015 11:07:20 +0200
ludo@gnu.org (Ludovic Courtès) wrote:

> Could you give me your Savannah account name so we can add you there for
> commit access?

It turns out my Savannah account name is also efraim :)
 
> Thanks!
> 
> Ludo’.

No, thank you! This is exciting :)

-Efraim

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

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCH 8/8] gnu: powertop: Update to 2.7.

[Ricardo Wurmus]

Hi Efraim,

> * gnu/packages/linux.scm (powertop): Update to 2.7.
>   [arguments]: Add "pciutils" to inputs.

This should be

  * gnu/packages/linux.scm (powertop): Update to 2.7.
    [inputs]: Add "pciutils".

Is “pciutils” necessary for the update?  If it isn’t maybe it would be
better to split this into two commits, as these are unrelated changes.

~~ Ricardo

Re: [PATCH 5/8] gnu: mpg123: Update to 1.22.4.

[Ricardo Wurmus]

Efraim Flashner <efraim@flashner.co.il> writes:

> * gnu/packages/mp3.scm (mpg123): Update to 1.22.4.
> ---
>  gnu/packages/mp3.scm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/mp3.scm b/gnu/packages/mp3.scm
> index d5ca155..4ce6917 100644
> --- a/gnu/packages/mp3.scm
> +++ b/gnu/packages/mp3.scm
> @@ -283,14 +283,14 @@ This package contains the binary.")
>  (define-public mpg123
>    (package
>      (name "mpg123")
> -    (version "1.19.0")
> +    (version "1.22.4")
>      (source (origin
>                (method url-fetch)
>                (uri (string-append "mirror://sourceforge/mpg123/mpg123-"
>                                    version ".tar.bz2"))
>                (sha256
>                 (base32
> -                "06xhd68mj9yp0r6l771aq0d7xgnl402a3wm2mvhxmd3w3ph29446"))))
> +                "1lj0xv0b6sgqsbhx10dg60cnzgz98i76gxy51kqh11hka0pf0sah"))))
>      (build-system gnu-build-system)
>      (arguments '(#:configure-flags '("--with-default-audio=pulse")))
>      (native-inputs `(("pkg-config" ,pkg-config)))

Looks good to me!

~~ Ricardo

Re: [PATCH 4/8] gnu: mpv: Update to 0.11.0.

[Ricardo Wurmus]

Efraim Flashner <efraim@flashner.co.il> writes:

> * gnu/packages/video.scm (mpv): Update to 0.11.0.
> ---
>  gnu/packages/video.scm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> index 4a5ba41..b46a986 100644
> --- a/gnu/packages/video.scm
> +++ b/gnu/packages/video.scm
> @@ -671,7 +671,7 @@ SVCD, DVD, 3ivx, DivX 3/4/5, WMV and H.264 movies.")
>  (define-public mpv
>    (package
>      (name "mpv")
> -    (version "0.9.0")
> +    (version "0.11.0")
>      (source (origin
>                (method url-fetch)
>                (uri (string-append
> @@ -679,7 +679,7 @@ SVCD, DVD, 3ivx, DivX 3/4/5, WMV and H.264 movies.")
>                      ".tar.gz"))
>                (sha256
>                 (base32
> -                "08nx0g6ji2d90f5w62g327szhkb7id7jzwgf3x069rc5id1x3bx7"))
> +                "1njvmqzj8akan5y485gx4blynwiy52adw7zbbnnnvd3dwis725d2"))
>                (file-name (string-append name "-" version ".tar.gz"))))
>      (build-system waf-build-system)
>      (native-inputs

Looks good to me!

~~ Ricardo

Re: [PATCH 3/8] gnu: youtube-dl: Update to 2015.09.28.

[Ricardo Wurmus]

Efraim Flashner <efraim@flashner.co.il> writes:

> * gnu/packages/video.scm (youtube-dl): Update to 2015.09.28.
> ---
>  gnu/packages/video.scm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> index 28185fe..4a5ba41 100644
> --- a/gnu/packages/video.scm
> +++ b/gnu/packages/video.scm
> @@ -791,7 +791,7 @@ projects while introducing many more.")
>  (define-public youtube-dl
>    (package
>      (name "youtube-dl")
> -    (version "2015.09.03")
> +    (version "2015.09.28")
>      (source (origin
>                (method url-fetch)
>                (uri (string-append "https://youtube-dl.org/downloads/"
> @@ -799,7 +799,7 @@ projects while introducing many more.")
>                                    version ".tar.gz"))
>                (sha256
>                 (base32
> -                "0wxjbqr07rm26iih12yhv0qwspfrg9safhgnrp8misqjjk4fz86z"))))
> +                "0q3s2a91s1lr1db2ngacq0iapyr4jngx1dqp4z5dc6zma0qyx5k3"))))
>      (build-system python-build-system)
>      (inputs `(("setuptools" ,python-setuptools)))
>      (home-page "http://youtube-dl.org")

Also looks good!

~~ Ricardo

Re: [PATCH 2/8] gnu: terminology: Update to 0.9.1.

[Ricardo Wurmus]

Efraim Flashner <efraim@flashner.co.il> writes:

> * gnu/packages/enlightenment.scm (terminology): Update to 0.9.1.
> ---
>  gnu/packages/enlightenment.scm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/enlightenment.scm b/gnu/packages/enlightenment.scm
> index 96ad676..584cab6 100644
> --- a/gnu/packages/enlightenment.scm
> +++ b/gnu/packages/enlightenment.scm
> @@ -218,7 +218,7 @@ The only supported now is VLC.")
>  (define-public terminology
>    (package
>      (name "terminology")
> -    (version "0.9.0")
> +    (version "0.9.1")
>      (source (origin
>                (method url-fetch)
>                (uri
> @@ -226,7 +226,7 @@ The only supported now is VLC.")
>                                "terminology/terminology-" version ".tar.xz"))
>                (sha256
>                 (base32
> -                "0iwid9cvd96kwl0hjhbby84kkz5sgb4p8454nnkf7wjvdz2f9b96"))))
> +                "1kwv9vkhngdm5v38q93xpcykghnyawhjjcb5bgy0p89gpbk7mvpc"))))
>      (build-system gnu-build-system)
>      (native-inputs
>       `(("pkg-config" ,pkg-config)))

This, too, looks fine.

~~ Ricardo

Re: [PATCH 1/8] gnu: enlightenment: Update to 0.19.11.

[Ricardo Wurmus]

Efraim Flashner <efraim@flashner.co.il> writes:

> * gnu/packages/enlightenment.scm (enlightenment): Update to 0.19.11.
> ---
>  gnu/packages/enlightenment.scm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/enlightenment.scm b/gnu/packages/enlightenment.scm
> index 0a7eaf1..96ad676 100644
> --- a/gnu/packages/enlightenment.scm
> +++ b/gnu/packages/enlightenment.scm
> @@ -270,14 +270,14 @@ Libraries with some extra bells and whistles.")
>  (define-public enlightenment
>    (package
>      (name "enlightenment")
> -    (version "0.19.9")
> +    (version "0.19.11")
>      (source (origin
>                (method url-fetch)
>                (uri
>                 (string-append "https://download.enlightenment.org/rel/apps/"
>                                name "/" name "-" version ".tar.xz"))
>                (sha256
> -               (base32 "1ajpyw1bfbmxpniw041vbvrs9wqgzyw9kccx72nnfiy2wxspjcr5"))))
> +               (base32 "0b8m6062i62q31gp9nd4hd7sckwxfvc6dgyck3f9gjdhrwvcn764"))))
>      (build-system gnu-build-system)
>      (native-inputs
>       `(("pkg-config" ,pkg-config)))

As the ‘(base32 ...)’ line slightly exceeds the maximum line length, I’d
suggest to put the hash string on a line of its own.

Okay with this change.

~~ Ricardo

Re: [PATCH 8/8] gnu: powertop: Update to 2.7.

[Efraim Flashner]

[-- Attachment #1: Type: text/plain, Size: 961 bytes --]

On Thu, 1 Oct 2015 12:14:55 +0200
Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> wrote:

> Hi Efraim,
> 
> > * gnu/packages/linux.scm (powertop): Update to 2.7.
> >   [arguments]: Add "pciutils" to inputs.
> 
> This should be
> 
>   * gnu/packages/linux.scm (powertop): Update to 2.7.
>     [inputs]: Add "pciutils".
> 
> Is “pciutils” necessary for the update?  If it isn’t maybe it would be
> better to split this into two commits, as these are unrelated changes.
> 
> ~~ Ricardo

pciutils isn't necessary for the update. I saw that there was the to-do item
for adding it, and I didn't run into any problems when building and testing
with it added. It seemed like a small change so I threw it in with the
version bump.

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

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCH 0/8] updates!

[Ludovic Courtès]

Efraim Flashner <efraim@flashner.co.il> skribis:

> On Thu, 01 Oct 2015 11:07:20 +0200
> ludo@gnu.org (Ludovic Courtès) wrote:
>
>> Could you give me your Savannah account name so we can add you there for
>> commit access?
>
> It turns out my Savannah account name is also efraim :)

Here you are!  Please do read the “Commit Access” section in ‘HACKING’.
If you’re unsure with Git, rebasing, etc., do not hesitate to ask.

As a first test, you’re welcome to push those 8 patches.  :-)

Thanks,
Ludo’.

Re: [PATCH 8/8] gnu: powertop: Update to 2.7.

[Andreas Enge]

Hello,

there is a problem with downloading the tarball:
   http://hydra.gnu.org/build/703762/nixlog/1/tail-reload

Could you have a look, please?

Andreas

Re: [PATCH 8/8] gnu: powertop: Update to 2.7.

[Efraim Flashner]

[-- Attachment #1: Type: text/plain, Size: 731 bytes --]

On Thu, 1 Oct 2015 21:36:46 +0200
Andreas Enge <andreas@enge.fr> wrote:

> Hello,
> 
> there is a problem with downloading the tarball:
>    http://hydra.gnu.org/build/703762/nixlog/1/tail-reload
> 
> Could you have a look, please?
> 
> Andreas
> 

I pushed a patch to fix that. somehow the url got changed from
https://01.org/sites/default/files/downloads/powertop/powertop-2.7.tar.gz to
https://01.org/powertop/sites/default/files/downloads/powertop-2.7.tar.gz and
that's why it wasn't downloading.

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

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCH 8/8] gnu: powertop: Update to 2.7.

[Andreas Enge]

On Thu, Oct 01, 2015 at 11:00:56PM +0300, Efraim Flashner wrote:
> I pushed a patch to fix that.

Great, thanks for your quick reaction!

Andreas

Re: [PATCH 8/8] gnu: powertop: Update to 2.7.

[Ludovic Courtès]

Efraim Flashner <efraim@flashner.co.il> skribis:

> On Thu, 1 Oct 2015 12:14:55 +0200
> Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> wrote:
>
>> Hi Efraim,
>> 
>> > * gnu/packages/linux.scm (powertop): Update to 2.7.
>> >   [arguments]: Add "pciutils" to inputs.
>> 
>> This should be
>> 
>>   * gnu/packages/linux.scm (powertop): Update to 2.7.
>>     [inputs]: Add "pciutils".
>> 
>> Is “pciutils” necessary for the update?  If it isn’t maybe it would be
>> better to split this into two commits, as these are unrelated changes.
>> 
>> ~~ Ricardo
>
> pciutils isn't necessary for the update.

In this case this should rather be two separate commits, to make it
clear that the two things are unrelated.  I had overlooked that.

Thanks,
Ludo’.