all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Efraim Flashner <efraim@flashner.co.il>
To: Alex Kost <alezost@gmail.com>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 1/3] gnu: iproute: Use 'modify-phases'.
Date: Thu, 11 Feb 2016 13:11:55 +0200	[thread overview]
Message-ID: <20160211131155.5561dd68@debian-netbook> (raw)
In-Reply-To: <1455188599-5715-2-git-send-email-alezost@gmail.com>

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

On Thu, 11 Feb 2016 14:03:17 +0300
Alex Kost <alezost@gmail.com> wrote:

> * gnu/packages/linux.scm (iproute)[arguments]: Use 'modify-phases'.
> ---
>  gnu/packages/linux.scm | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
> index e7f7bcd..a6e3ec9 100644
> --- a/gnu/packages/linux.scm
> +++ b/gnu/packages/linux.scm
> @@ -7,6 +7,7 @@
>  ;;; Copyright © 2015 Taylan Ulrich Bayırlı/Kammer <taylanbayirli@gmail.com>
>  ;;; Copyright © 2015, 2016 Efraim Flashner <efraim@flashner.co.il>
>  ;;; Copyright © 2016 Christopher Allan Webber <cwebber@dustycloud.org>
> +;;; Copyright © 2016 Alex Kost <alezost@gmail.com>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -887,13 +888,12 @@ packet filter.")
>                              (string-append "DOCDIR=" out "/share/doc/"
>                                             ,name "-" ,version)
>                              (string-append "MANDIR=" out "/share/man")))
> -       #:phases (alist-cons-before
> -                 'install 'pre-install
> -                 (lambda _
> -                   ;; Don't attempt to create /var/lib/arpd.
> -                   (substitute* "Makefile"
> -                     (("^.*ARPDDIR.*$") "")))
> -                 %standard-phases)))
> +       #:phases (modify-phases %standard-phases
> +                  (add-before 'install 'pre-install
> +                    (lambda _
> +                      ;; Don't attempt to create /var/lib/arpd.
> +                      (substitute* "Makefile"
> +                        (("^.*ARPDDIR.*$") "")))))))
>      (inputs
>       `(("iptables" ,iptables)
>         ("db4" ,bdb)))

LGTM!

-- 
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 --]

  reply	other threads:[~2016-02-11 11:16 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-11 11:03 [PATCH 0/3]: Update iproute and add it to %base-packages Alex Kost
2016-02-11 11:03 ` [PATCH 1/3] gnu: iproute: Use 'modify-phases' Alex Kost
2016-02-11 11:11   ` Efraim Flashner [this message]
2016-02-11 11:03 ` [PATCH 2/3] gnu: iproute: Update to 4.4.0 Alex Kost
2016-02-11 11:12   ` Efraim Flashner
2016-02-13 18:50     ` Alex Kost
2016-02-11 17:29   ` Andreas Enge
2016-02-11 11:03 ` [PATCH 3/3] system: Add iproute to '%base-packages' Alex Kost
2016-02-11 11:15   ` Efraim Flashner
2016-02-12 10:59     ` Alex Kost
2016-02-18  9:24   ` Alex Kost

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160211131155.5561dd68@debian-netbook \
    --to=efraim@flashner.co.il \
    --cc=alezost@gmail.com \
    --cc=guix-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.