[PATCH] gnu: flac: Install man pages in share/man

[PATCH] gnu: flac: Install man pages in share/man

[Mark H Weaver]

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

I'm not sure if this should go in master or core-updates.
What do you think?

    Mark



[-- Attachment #2: [PATCH] gnu: flac: Install man pages in share/man --]
[-- Type: text/x-patch, Size: 1553 bytes --]

From c90bdada66d79c04d42a7c6a6210fd73b866d328 Mon Sep 17 00:00:00 2001
From: Mark H Weaver <mhw@netris.org>
Date: Fri, 21 Mar 2014 03:14:54 -0400
Subject: [PATCH] gnu: flac: Install man pages in share/man.

* gnu/packages/xiph.scm (flac): Add --mandir configure flag.
---
 gnu/packages/xiph.scm | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/xiph.scm b/gnu/packages/xiph.scm
index 0e8cb5f..49db245 100644
--- a/gnu/packages/xiph.scm
+++ b/gnu/packages/xiph.scm
@@ -3,6 +3,7 @@
 ;;; Copyright © 2013 Nikita Karetnikov <nikita@karetnikov.org>
 ;;; Copyright © 2013 David Thompson <dthompson2@worcester.edu>
 ;;; Copyright © 2014 Sree Harsha Totakura <sreeharsha@totakura.in>
+;;; Copyright © 2014 Mark H Weaver <mhw@netris.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -201,7 +202,12 @@ OpenBSD's sndio.")
              (list (search-patch "flac-fix-memcmp-not-declared.patch")))))
    (build-system gnu-build-system)
    (arguments
-    `(#:parallel-tests? #f))
+    `(#:parallel-tests? #f
+      ;; By default, man pages are put in PREFIX/man,
+      ;; but we want them in PREFIX/share/man.
+      #:configure-flags (list (string-append "--mandir="
+                                             (assoc-ref %outputs "out")
+                                             "/share/man"))))
    ;; FIXME: configure also looks for xmms, input could be added once it exists
    (inputs `(("libogg" ,libogg)))
    (synopsis "flac free lossless audio codec")
-- 
1.8.4

Re: [PATCH] gnu: flac: Install man pages in share/man

[Ludovic Courtès]

Mark H Weaver <mhw@netris.org> skribis:

> I'm not sure if this should go in master or core-updates.
> What do you think?

It’s a leaf, so OK for master.

Ludo’.