guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] gnu: Add glibc-hurd-headers


From: Ludovic Courtès
Subject: Re: [PATCH] gnu: Add glibc-hurd-headers
Date: Tue, 15 Apr 2014 21:39:47 +0200
User-agent: Gnus/5.130009 (Ma Gnus v0.9) Emacs/24.3 (gnu/linux)

Manolis Ragkousis <address@hidden> skribis:

> +(define-public glibc/hurd-headers
> +  (package (inherit glibc)
> +           (name "glibc-hurd-headers")

Indentation still wrong.

> +           (arguments
> +            `(#:strip-binaries? #f
> +              #:out-of-source? #t
> +              #:tests? #f
> +              #:configure-flags
> +              (list "--host=i686-pc-gnu"
> +                    "--enable-addons"
> +
> +                    ;; Installs NIS and RPC related headers that
> +                    ;; are not installed by default
> +                    "--enable-obsolete-rpc"
> +
> +                    (string-append "--with-headers="
> +                                   (assoc-ref %build-inputs 
> "gnumach-headers")
> +                                   "/include:"
> +                                   (assoc-ref %build-inputs "hurd-headers")
> +                                   "/include")
> +
> +                    ;; Use our Bash instead of /bin/sh.
> +                    (string-append "BASH_SHELL="
> +                                   (assoc-ref %build-inputs "bash")
> +                                   "/bin/bash"))
> +              #:phases (alist-replace
> +                        'install
> +                        (lambda _
> +                          (zero? (system* "make" "install-headers")))
> +                        (alist-delete
> +                         'build
> +                         (alist-cons-before
> +                          'configure 'pre-configure
> +                          (lambda _
> +                            (substitute* "configure"
> +                              (("/bin/pwd") "pwd")))
> +                          %standard-phases)))))))

Sorry, I didn’t get initially that it needed this much of the initial
arguments.  #:phases is the only thing that differs, right?

If that is the case, then indeed something like this is enough:

  (arguments
    `(#:tests? #f
      ,@(substitute-keyword-arguments (package-arguments glibc/hurd)
          ((#:phases _)
           '(alist-replace ...)))))

In general the guideline is to keep things as simple and concise as
possible.  In some cases, making things more concise will actually make
them more difficult to understand, which is not desirable.

HTH!

Ludo’.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]