unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#31954] [PATCH] gnu: Add emacs-org-brain.
@ 2018-06-24  9:29 Vasile Dumitrascu
  2018-06-24 12:30 ` Charlie Ritter
  2018-06-25 19:36 ` Marius Bakke
  0 siblings, 2 replies; 5+ messages in thread
From: Vasile Dumitrascu @ 2018-06-24  9:29 UTC (permalink / raw)
  To: 31954; +Cc: Vasile Dumitrascu

* gnu/packages/emacs.scm (emacs-org-brain): New variable.
---
 gnu/packages/emacs.scm | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 3b2c41c5d..616175d54 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -11277,3 +11277,28 @@ file.")
        "@code{wgrep-helm} allows you to edit a @code{helm-grep-mode} buffer and
 apply those changes to the file buffer.")
       (license license:gpl3+))))
+
+(define-public emacs-org-brain
+  (let ((version "20180522")  ; no proper tag, use date of commit
+        (commit "3faf9303af3f2356e3444e69c22dc6c5774047d1")
+        (revision "1"))
+    (package
+      (name "emacs-org-brain")
+      (version (git-version version revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/Kungsgeten/org-brain.git")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32
+           "1ad681zk6kckw2zbk0r4iaj4bw8cfqrbd1s3gdwgdjlzq81q9mmj"))))
+      (build-system emacs-build-system)
+      (home-page "https://github.com/Kungsgeten/org-brain")
+      (synopsis "Org-mode wiki and concept-mapping for Emacs")
+      (description "@code{emacs-org-brain} implements a variant of concept
+mapping in Emacs, using @code{org-mode}. It is heavily inspired by a piece of
+software called The Brain.")
+      (license license:expat))))
-- 
2.18.0

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [bug#31954] [PATCH] gnu: Add emacs-org-brain.
  2018-06-24  9:29 [bug#31954] [PATCH] gnu: Add emacs-org-brain Vasile Dumitrascu
@ 2018-06-24 12:30 ` Charlie Ritter
  2018-06-25 19:36 ` Marius Bakke
  1 sibling, 0 replies; 5+ messages in thread
From: Charlie Ritter @ 2018-06-24 12:30 UTC (permalink / raw)
  To: Vasile Dumitrascu; +Cc: 31954


Thank you kind stranger!!!

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#31954] [PATCH] gnu: Add emacs-org-brain.
  2018-06-24  9:29 [bug#31954] [PATCH] gnu: Add emacs-org-brain Vasile Dumitrascu
  2018-06-24 12:30 ` Charlie Ritter
@ 2018-06-25 19:36 ` Marius Bakke
  2018-07-04 16:30   ` Vasile Dumitrascu
  1 sibling, 1 reply; 5+ messages in thread
From: Marius Bakke @ 2018-06-25 19:36 UTC (permalink / raw)
  To: Vasile Dumitrascu, 31954

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

Vasile Dumitrascu <va511e@yahoo.com> writes:

> * gnu/packages/emacs.scm (emacs-org-brain): New variable.

Thanks for this patch!  Overall it LGTM, but it does not apply to my git
checkout, so here are some nitpicks that I'd normally fix myself:

[...]

> +(define-public emacs-org-brain
> +  (let ((version "20180522")  ; no proper tag, use date of commit

'org-brain.el' has "Version: 0.5" in the source header.  Perhaps we
should use that instead?

[...]

> +      (description "@code{emacs-org-brain} implements a variant of concept
> +mapping in Emacs, using @code{org-mode}. It is heavily inspired by a piece of
> +software called The Brain.")

Make sure to use double spaces in descriptions (guix lint should complain
about this).  We also should not refer to nonfree software, can you try
to expand on this description without mentioning the inspiration?

The commentary inside 'org-brain.el' contains a pretty good description.

Can you send an updated patch please?  Thanks in advance!

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [bug#31954] [PATCH] gnu: Add emacs-org-brain.
  2018-06-25 19:36 ` Marius Bakke
@ 2018-07-04 16:30   ` Vasile Dumitrascu
  2018-07-07 15:51     ` bug#31954: " Ludovic Courtès
  0 siblings, 1 reply; 5+ messages in thread
From: Vasile Dumitrascu @ 2018-07-04 16:30 UTC (permalink / raw)
  To: Marius Bakke, 31954, chewzerita

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


  Hi

@Marius: Please find attached an updated patch with the requested
changes. Do let me know if anything else is to be adjusted.

@Charlie: I am glad you find it useful. You are welcome!

  Thank you,

  Vasile

Marius Bakke:
> Vasile Dumitrascu <va511e@yahoo.com> writes:
> 
>> * gnu/packages/emacs.scm (emacs-org-brain): New variable.
> 
> Thanks for this patch!  Overall it LGTM, but it does not apply to my git
> checkout, so here are some nitpicks that I'd normally fix myself:
> 
> [...]
> 
>> +(define-public emacs-org-brain
>> +  (let ((version "20180522")  ; no proper tag, use date of commit
> 
> 'org-brain.el' has "Version: 0.5" in the source header.  Perhaps we
> should use that instead?
> 
> [...]
> 
>> +      (description "@code{emacs-org-brain} implements a variant of concept
>> +mapping in Emacs, using @code{org-mode}. It is heavily inspired by a piece of
>> +software called The Brain.")
> 
> Make sure to use double spaces in descriptions (guix lint should complain
> about this).  We also should not refer to nonfree software, can you try
> to expand on this description without mentioning the inspiration?
> 
> The commentary inside 'org-brain.el' contains a pretty good description.
> 
> Can you send an updated patch please?  Thanks in advance!
> 

[-- Attachment #2: 0001-gnu-Add-emacs-org-brain.patch --]
[-- Type: text/x-patch, Size: 1886 bytes --]

From 097ea36392360028cec20d2e8908755c926dcbcb Mon Sep 17 00:00:00 2001
From: Vasile Dumitrascu <va511e@yahoo.com>
Date: Wed, 4 Jul 2018 18:24:50 +0200
Subject: [PATCH] gnu: Add emacs-org-brain.

* gnu/packges/emacs.scm (emacs-org-brain): New variable.
---
 gnu/packages/emacs.scm | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 467791e21..712dd1363 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -11290,3 +11290,29 @@ To use, add @code{allow-emacs-pinentry} to @code{~/.gnupg/gpg-agent.conf},
 reload the configuration with @code{gpgconf --reload gpg-agent}, and start the
 server with @code{M-x pinentry-start}.")
       (license license:gpl3+))))
+
+(define-public emacs-org-brain
+  (package
+    (name "emacs-org-brain")
+    (version "0.5")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/Kungsgeten/org-brain.git")
+             (commit "3faf9303af3f2356e3444e69c22dc6c5774047d1")))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32
+         "1ad681zk6kckw2zbk0r4iaj4bw8cfqrbd1s3gdwgdjlzq81q9mmj"))))
+    (build-system emacs-build-system)
+    (home-page "https://github.com/Kungsgeten/org-brain")
+    (synopsis "Org-mode wiki and concept-mapping for Emacs")
+    (description "@code{emacs-org-brain} implements a variant of concept
+mapping in Emacs, using @code{org-mode}.  An org-brain is a network of
+org-mode entries, where each entry is a file or a headline, and you can get a
+visual overview of the relationships between the entries: parents, children,
+siblings and friends.  This visual overview can also be used to browse your
+entries.  You can think of entries as nodes in a mind map, or pages in a
+wiki.")
+    (license license:expat)))
-- 
2.18.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* bug#31954: [PATCH] gnu: Add emacs-org-brain.
  2018-07-04 16:30   ` Vasile Dumitrascu
@ 2018-07-07 15:51     ` Ludovic Courtès
  0 siblings, 0 replies; 5+ messages in thread
From: Ludovic Courtès @ 2018-07-07 15:51 UTC (permalink / raw)
  To: Vasile Dumitrascu; +Cc: 31954-done, chewzerita

Hello Vasile,

Vasile Dumitrascu <va511e@yahoo.com> skribis:

> @Marius: Please find attached an updated patch with the requested
> changes. Do let me know if anything else is to be adjusted.

This updated patch appears to address Marius’ concerns so I’ve applied
it.

Thank you!

Ludo’.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-07-07 15:53 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-06-24  9:29 [bug#31954] [PATCH] gnu: Add emacs-org-brain Vasile Dumitrascu
2018-06-24 12:30 ` Charlie Ritter
2018-06-25 19:36 ` Marius Bakke
2018-07-04 16:30   ` Vasile Dumitrascu
2018-07-07 15:51     ` bug#31954: " Ludovic Courtès

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