all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Andreas Enge <andreas@enge.fr>
To: Leo Famulari <leo@famulari.name>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] gnu: Add ircii.
Date: Tue, 2 Aug 2016 09:58:34 +0200	[thread overview]
Message-ID: <20160802075834.GB1957@solar> (raw)
In-Reply-To: <20160802005413.GB15847@jasmine>

On Mon, Aug 01, 2016 at 08:54:13PM -0400, Leo Famulari wrote:
> > +             (snippet
> > +              '(begin
> > +                 (substitute* "bsdinstall"
> > +                   (("/bin/strip") "strip")
> > +                   (("/bin/cp") "cp")
> > +                   (("/bin/chmod") "chmod")
> > +                   (("/etc/chown") "chown")
> > +                   (("/bin/chgrp") "chgrp")
> > +                   (("/bin/mkdir") "mkdir")
> > +                   (("/bin/rm") "rm")
> > +                   (("/bin/mv") "mv"))))))
> Does anyone else think we should put this origin snippet in a build phase
> instead? Or, should we leave it in the origin, where it will be applied
> to the source code provided by `guix build --source ircii`?

I think it would be more in line with our usual practices to have it in
a build phase, indeed. Snippets are rather used to remove non-free code
or bundled software.

Andreas

  reply	other threads:[~2016-08-02  7:58 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-01 19:34 [PATCH] gnu: Add ircii ng0
2016-08-02  0:54 ` Leo Famulari
2016-08-02  7:58   ` Andreas Enge [this message]
2016-08-02 11:30     ` ng0
2016-08-05  6:46       ` Ricardo Wurmus
2016-08-05 10:29         ` ng0
2016-08-05 11:17           ` Ricardo Wurmus
2016-08-05 12:55         ` ng0
2016-08-05 12:57           ` ng0
2016-08-05 13:01           ` ng0
2016-08-05 13:22             ` ng0
2016-08-05 15:48               ` ng0
2016-08-05 15:57                 ` Leo Famulari
2016-08-05 16:11                   ` ng0
2016-08-05 17:55                     ` Leo Famulari
2016-08-05 18:27                       ` ng0
2016-08-05 19:11                         ` Leo Famulari
2016-08-09 19:38             ` Ricardo Wurmus
2016-08-09 19:47               ` Leo Famulari
2016-08-13  7:06                 ` ng0
2016-08-02 12:42   ` 宋文武
2016-08-02 15:19     ` ng0
2016-08-04 19:03       ` ng0

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=20160802075834.GB1957@solar \
    --to=andreas@enge.fr \
    --cc=guix-devel@gnu.org \
    --cc=leo@famulari.name \
    /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.