unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Oleg Pykhalov <go.wigust@gmail.com>
To: Konrad Hinsen <konrad.hinsen@fastmail.net>
Cc: 30798@debbugs.gnu.org
Subject: [bug#30798] [PATCH 2/2] gnu: Add emacs-deft
Date: Thu, 15 Mar 2018 11:38:12 +0300	[thread overview]
Message-ID: <87sh91u423.fsf@gmail.com> (raw)
In-Reply-To: <m1woyg9drn.fsf@fastmail.net> (Konrad Hinsen's message of "Tue, 13 Mar 2018 16:47:41 +0100")

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

Hello Konrad,

First of all thank you for your contribution!

Konrad Hinsen <konrad.hinsen@fastmail.net> writes:

> * gnu/packages/emacs.scm (emacs-deft): New variable.
> ---
>  gnu/packages/emacs.scm | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
>
> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> index 9c9c399fd..07ac1f0fe 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -7389,3 +7389,26 @@ to inherit a login shell's environment variables.  It also allows other
>  environment variables to be retrieved from the shell, so that Emacs will
>  see the same values you get in a terminal.")
>      (license license:gpl3+)))
> +
> +(define-public emacs-deft
> +  (package
> +    (name "emacs-deft")
> +    (version "0.8")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append
> +             "http://stable.melpa.org/packages/deft-"
> +             version
> +             ".el"))

Unfortunately, we can't use sources from MELPA.  The problem is: when a
repository get a new commit, the old tarball is *REPLACED* with the new
one on MELPA, so the package could never be built anymore, since the
source disappeared.  That's why we use tarballs directly from source.

You could use Deft source tarball https://github.com/jrblevin/deft/tags


BTW, to make a recipe more polished you could remove unnecessary new
lines, for example:

       (uri (string-append "http://stable.melpa.org/packages/deft-"
             version ".el"))

> +       (sha256
> +        (base32
> +         "1vb9cjxskc7c0yyf9pvxy1fzypg1vrcgwnjz0m3hslinsgdyig58"))))
> +    (build-system emacs-build-system)
> +    (home-page "https://jblevins.org/projects/deft/")
> +    (synopsis
> +     "Quickly browse, filter, and edit plain text notes")

Likewise, unnecessary new line.

> +    (description
> +     "Deft is an Emacs mode for quickly browsing, filtering, and editing
> +directories of plain text notes, inspired by Notational Velocity.")
> +    (license license:gpl3+)))

The home page says Deft is licensed under BSD license.  Could you check
it, please?

Oleg.

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

  reply	other threads:[~2018-03-15  8:39 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-13 15:47 [bug#30798] [PATCH 2/2] gnu: Add emacs-deft Konrad Hinsen
2018-03-15  8:38 ` Oleg Pykhalov [this message]
2018-03-15  9:45   ` Konrad Hinsen
2018-03-15 14:58     ` Oleg Pykhalov
2018-03-16 12:10   ` Konrad Hinsen
2018-03-16 14:13     ` bug#30798: " Oleg Pykhalov
2018-03-16 14:29       ` [bug#30798] " Konrad Hinsen

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=87sh91u423.fsf@gmail.com \
    --to=go.wigust@gmail.com \
    --cc=30798@debbugs.gnu.org \
    --cc=konrad.hinsen@fastmail.net \
    /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).