GNU bug report logs - #30798
[PATCH 2/2] gnu: Add emacs-deft

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 30798 in the body.
You can then email your comments to 30798 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Konrad Hinsen <konrad.hinsen <at> fastmail.net>
To: guix-patches <at> gnu.org
Subject: [PATCH 2/2] gnu: Add emacs-deft
Date: Tue, 13 Mar 2018 16:47:41 +0100

* 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"))
+       (sha256
+        (base32
+         "1vb9cjxskc7c0yyf9pvxy1fzypg1vrcgwnjz0m3hslinsgdyig58"))))
+    (build-system emacs-build-system)
+    (home-page "https://jblevins.org/projects/deft/")
+    (synopsis
+     "Quickly browse, filter, and edit plain text notes")
+    (description
+     "Deft is an Emacs mode for quickly browsing, filtering, and editing
+directories of plain text notes, inspired by Notational Velocity.")
+    (license license:gpl3+)))
-- 
2.16.2

Message #8 received at 30798 <at> debbugs.gnu.org (full text, mbox):

From: Oleg Pykhalov <go.wigust <at> gmail.com>
To: Konrad Hinsen <konrad.hinsen <at> fastmail.net>
Cc: 30798 <at> debbugs.gnu.org
Subject: Re: [bug#30798] [PATCH 2/2] gnu: Add emacs-deft
Date: Thu, 15 Mar 2018 11:38:12 +0300

[Message part 1 (text/plain, inline)]

Hello Konrad,

First of all thank you for your contribution!

Konrad Hinsen <konrad.hinsen <at> 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.

[signature.asc (application/pgp-signature, inline)]

Message #11 received at 30798 <at> debbugs.gnu.org (full text, mbox):

From: Konrad Hinsen <konrad.hinsen <at> fastmail.net>
To: Oleg Pykhalov <go.wigust <at> gmail.com>
Cc: 30798 <at> debbugs.gnu.org
Subject: Re: [bug#30798] [PATCH 2/2] gnu: Add emacs-deft
Date: Thu, 15 Mar 2018 10:45:19 +0100

Hi Oleg,

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

Are you sure this also applies to *stable* MELPA?

Konrad.

Message #14 received at 30798 <at> debbugs.gnu.org (full text, mbox):

From: Oleg Pykhalov <go.wigust <at> gmail.com>
To: Konrad Hinsen <konrad.hinsen <at> fastmail.net>
Cc: 30798 <at> debbugs.gnu.org
Subject: Re: [bug#30798] [PATCH 2/2] gnu: Add emacs-deft
Date: Thu, 15 Mar 2018 17:58:00 +0300

[Message part 1 (text/plain, inline)]

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

>> 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.
>
> Are you sure this also applies to *stable* MELPA?

I'm not sure about *stable* MELPA.  I see more package pulled from MELPA
stable in ‘emacs.scm’ and think we could stick to MELPA stable here.

Then, change URL to *https*, please.

Oleg.

[signature.asc (application/pgp-signature, inline)]

Message #17 received at 30798 <at> debbugs.gnu.org (full text, mbox):

From: Konrad Hinsen <konrad.hinsen <at> fastmail.net>
To: Oleg Pykhalov <go.wigust <at> gmail.com>
Cc: 30798 <at> debbugs.gnu.org
Subject: Re: [bug#30798] [PATCH 2/2] gnu: Add emacs-deft
Date: Fri, 16 Mar 2018 13:10:28 +0100

* 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 35acfdd20..f51f696fd 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -7386,3 +7386,26 @@ 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 "https://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")
+    (description
+     "Deft is an Emacs mode for quickly browsing, filtering, and editing
+directories of plain text notes, inspired by Notational Velocity.")
+    ;; A comment in the code says "may be freely distributed and modified
+    ;; under the BSD license". I take this to mean the original
+    ;; BSD license, called bsd-4 in Guix.
+    (license license:bsd-4)))
-- 
2.16.2

Message #20 received at 30798 <at> debbugs.gnu.org (full text, mbox):

From: Oleg Pykhalov <go.wigust <at> gmail.com>
To: Konrad Hinsen <konrad.hinsen <at> fastmail.net>
Cc: 30798 <at> debbugs.gnu.org, 30798-done <at> debbugs.gnu.org
Subject: Re: [bug#30798] [PATCH 2/2] gnu: Add emacs-deft
Date: Fri, 16 Mar 2018 17:13:33 +0300

[Message part 1 (text/plain, inline)]

Konrad Hinsen <konrad.hinsen <at> 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 35acfdd20..f51f696fd 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -7386,3 +7386,26 @@ 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+)))
> +

[…]

> +    ;; A comment in the code says "may be freely distributed and modified
> +    ;; under the BSD license". I take this to mean the original
> +    ;; BSD license, called bsd-4 in Guix.
> +    (license license:bsd-4)))

Thank you for investigation, but it's a BSD 3-Clause license.  It's easy
to check, because there are *3 clauses* in the license text:

    ;; Redistribution and use in source and binary forms, with or
    ;; without modification, are permitted provided that the following
    ;; conditions are met:
    ;; 1. Redistributions of source code must retain the above copyright
    ;;    notice, this list of conditions and the following disclaimer.
    ;; 2. Redistributions in binary form must reproduce the above
    ;;    copyright notice, this list of conditions and the following
    ;;    disclaimer in the documentation and/or other materials
    ;;    provided with the distribution.
    ;; 3. Neither the names of the copyright holders nor the names of
    ;;    any contributors may be used to endorse or promote products
    ;;    derived from this software without specific prior written
    ;;    permission.


As a tip,  we have a ‘licensecheck’ program in Guix package collection,
which could parse a source code file and tell a license.


Also don't forget a dot at the end in commit message.


Pushed with changes above as 5d818b3557cc3b546d5bd0639359c14c7c0ab685

I'll close the bug report.

Thanks,
Oleg.

[signature.asc (application/pgp-signature, inline)]

Message #28 received at 30798 <at> debbugs.gnu.org (full text, mbox):

From: Konrad Hinsen <konrad.hinsen <at> fastmail.net>
To: Oleg Pykhalov <go.wigust <at> gmail.com>
Cc: 30798 <at> debbugs.gnu.org, 30798-done <at> debbugs.gnu.org
Subject: Re: [bug#30798] [PATCH 2/2] gnu: Add emacs-deft
Date: Fri, 16 Mar 2018 15:29:34 +0100

Hi Oleg,

> Thank you for investigation, but it's a BSD 3-Clause license.  It's easy
> to check, because there are *3 clauses* in the license text:

Makes sense - but it isn't exactly obvious either to
non-lawyers. Therefore...

> As a tip,  we have a ‘licensecheck’ program in Guix package collection,
> which could parse a source code file and tell a license.

... this looks like a nice tool to have. Installed :-)

> Pushed with changes above as 5d818b3557cc3b546d5bd0639359c14c7c0ab685
>
> I'll close the bug report.

Ok, thanks!

Konrad.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.