unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Carlo Zancanaro <carlo@zancanaro.id.au>
To: Marius Bakke <mbakke@fastmail.com>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: [PATCH] gnu: Add kakoune
Date: Fri, 27 Jan 2017 10:57:42 +1100	[thread overview]
Message-ID: <87poj9z85l.fsf@zancanaro.id.au> (raw)
In-Reply-To: <87vat1324b.fsf@kirby.i-did-not-set--mail-host-address--so-tickle-me>


[-- Attachment #1.1: Type: text/plain, Size: 512 bytes --]


On Thu, Jan 26 2017, Marius Bakke wrote
> Would you like to fix it? Good catch, sorry for making the mistake on
> your behalf!

No worries!

I've attached a patch to move it to a snippet. I wasn't sure how to word
the commit message (this is the first time I've updated a package), so I
hope it's okay. If not, feel free to change it.

Thanks for the comments regarding patches/snippets/phases. I think I'll
just need to hang around a bit longer and see how things are done to get
a feel for it myself.

Carlo


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0001-gnu-kakoune-Move-substitution-phase-to-snippet.patch --]
[-- Type: text/x-patch, Size: 1912 bytes --]

From b66c5ce74f15994f7976d405171771c185a646ff Mon Sep 17 00:00:00 2001
From: Carlo Zancanaro <carlo@zancanaro.id.au>
Date: Fri, 27 Jan 2017 10:48:12 +1100
Subject: [PATCH] gnu: kakoune: Move substitution phase to snippet.

* gnu/packages/text-editors.scm (kakoune): Move substitution phase to snippet.
---
 gnu/packages/text-editors.scm | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/gnu/packages/text-editors.scm b/gnu/packages/text-editors.scm
index cb4855860..465c088a8 100644
--- a/gnu/packages/text-editors.scm
+++ b/gnu/packages/text-editors.scm
@@ -98,7 +98,13 @@ based command language.")
                (commit commit)))
          (sha256
           (base32
-           "19qs99l8r9p1vi5pxxx9an22fvi7xx40qw3jh2cnh2mbacawvdyb"))))
+           "19qs99l8r9p1vi5pxxx9an22fvi7xx40qw3jh2cnh2mbacawvdyb"))
+         (modules '((guix build utils)))
+         (snippet
+          ;; Kakoune uses 'gzip' to compress its manpages. Make sure
+          ;; timestamps are not preserved for reproducibility.
+          '(substitute* "src/Makefile"
+            (("gzip -f") "gzip -f --no-name")))))
       (build-system gnu-build-system)
       (arguments
        `(#:make-flags (list (string-append "PREFIX=" (assoc-ref %outputs "out"))
@@ -117,10 +123,6 @@ based command language.")
                  (("if \\(m_shell.empty\\(\\)\\)" line)
                   (string-append "m_shell = \"" (which "sh")
                                  "\";\n        " line)))
-               ;; Kakoune uses 'gzip' to compress its manpages. Make sure
-               ;; timestamps are not preserved for reproducibility.
-               (substitute* "src/Makefile"
-                 (("gzip -f") "gzip -f --no-name"))
                #t))
            (delete 'configure)
            ;; kakoune requires us to be in the src/ directory to build
-- 
2.11.0


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

  reply	other threads:[~2017-01-26 23:57 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-30  3:43 [PATCH] gnu: Add kakoune Carlo Zancanaro
2016-12-30  4:44 ` Carlo Zancanaro
2016-12-30 19:25   ` Leo Famulari
2016-12-30 23:19     ` Carlo Zancanaro
2017-01-10 22:26       ` Leo Famulari
2017-01-21  1:49         ` Carlo Zancanaro
2017-01-26 10:20           ` Ludovic Courtès
2017-01-26 12:26             ` Carlo Zancanaro
2017-01-26 20:19               ` Marius Bakke
2017-01-26 21:43                 ` Carlo Zancanaro
2017-01-26 22:09                   ` Marius Bakke
2017-01-26 23:57                     ` Carlo Zancanaro [this message]
2017-01-27  0:47                       ` Marius Bakke
2017-01-27 23:00                         ` gzip embedded timestamps Ludovic Courtès

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

  List information: https://guix.gnu.org/

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

  git send-email \
    --in-reply-to=87poj9z85l.fsf@zancanaro.id.au \
    --to=carlo@zancanaro.id.au \
    --cc=guix-devel@gnu.org \
    --cc=mbakke@fastmail.com \
    /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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).