GNU bug report logs - #31954
[PATCH] gnu: Add emacs-org-brain.

Previous Next

Full log

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

From: Vasile Dumitrascu <va511e <at> yahoo.com>
To: Marius Bakke <mbakke <at> fastmail.com>, 31954 <at> debbugs.gnu.org,
 chewzerita <at> posteo.net
Subject: Re: [bug#31954] [PATCH] gnu: Add emacs-org-brain.
Date: Wed, 04 Jul 2018 16:30:00 +0000

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

  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 <at> 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!
> 

[0001-gnu-Add-emacs-org-brain.patch (text/x-patch, attachment)]

Previous Next

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