all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Vasile Dumitrascu <va511e@yahoo.com>
To: Alex Kost <alezost@gmail.com>
Cc: 26268@debbugs.gnu.org
Subject: bug#26268: [PATCH] gnu: Add emacs-idle-highlight.
Date: Tue, 28 Mar 2017 18:59:00 +0000	[thread overview]
Message-ID: <68dcc0d8-ee90-ae6d-b47d-f17c4bfa3e3f@yahoo.com> (raw)
In-Reply-To: <87bmsnhzfx.fsf@gmail.com>

    Hi Alex

  I agree with your observations. To explain my previous choice: I was
using melpa as source for the packages as I have found the option to
import from melpa in
https://www.gnu.org/software/guix/manual/guix.html#Invoking-guix-import
so I thought that this is the recommended approach. The document should
be updated to mention this as not recommended for contributions, for the
reasons you have mentioned.

  Maybe rather we should have a predefined import task that easily
imports from github or git based repositories (I could not find one in
the docs).

  I will make the changes for this and the previous submitted packages
and refile patches. Thank you,

   Vasile

Alex Kost:
> Vasile Dumitrascu (2017-03-27 01:39 +0200) wrote:
> 
> Hello Vasile!
> 
>>     * gnu/packages/emacs.scm (emacs-idle-highlight): New variable.
>> ---
>>  gnu/packages/emacs.scm | 21 +++++++++++++++++++++
>>  1 file changed, 21 insertions(+)
>>
>> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
>> index 4fe2ffa4d..02f92bc26 100644
>> --- a/gnu/packages/emacs.scm
>> +++ b/gnu/packages/emacs.scm
>> @@ -3960,3 +3960,24 @@ abbreviation and automatically expand it into function templates.")
>>     (description "@code{emacs-memoize} is an Emacs library for
>>  memoizing functions.")
>>     (license license:unlicense)))
>> +
>> +(define-public emacs-idle-highlight
>> +  (package
>> +    (name "emacs-idle-highlight")
>> +    (version "20120920.948")
>> +    (source
>> +     (origin
>> +       (method url-fetch)
>> +       (uri (string-append
>> +             "http://melpa.org/packages/idle-highlight-mode-"
>> +             version ".el"))
> 
> We don't (at least we shouldn't IMO) use files from MELPA.  The problem
> is: once the original source will be updated on github, MELPA will build
> it and the previous MELPA version will not be available anymore.
> 
> So I think it's better to use the original source which is:
> 
>   https://github.com/nonsequitur/idle-highlight-mode/archive/1.1.3.tar.gz
> 
> Could you send an updated patch with this tarball?
> 
> Sorry, I missed your previous emacs packages (symon, monroe and
> keyfreq), which are committed already, but eventually they will become
> "broken": for example, when there will be a new commit in "monroe"
> repository, MELPA will build the new version and the current
> "monroe-20170220.540.el" file will be removed from MELPA, so the current
> 'emacs-monroe' package could never be built after that.
> 
> The situation was much worse in the past, when MELPA updated even
> untouched(!) packages every several hours, so their hashes were changing
> all the time, so the files from MELPA couldn't be used at all.
> Nowadays, MELPA updates packages only when there are some changes in the
> source.
> 
>> +       (sha256
>> +        (base32
>> +         "083ihf97a20l9pbqgn9jh6vdx97z40mp2fhi17qs90a16hpfhx5j"))))
>> +    (build-system emacs-build-system)
>> +    (home-page "http://www.emacswiki.org/cgi-bin/wiki/IdleHighlight")
> 
> For me it is redirected to
> <https://www.emacswiki.org/emacs/IdleHighlight> so I think it's better
> to use this page.
> 
>> +    (synopsis "Highlights all occurences of the word the point is on")
>> +    (description
>> +     "@code{idle-highlight-mode} sets an idle timer that highlights all
>> +occurences in the buffer of the word under the point.")
> 
> Usually we mention "Emacs" in synopsis or description, like:
> 
> "This Emacs package provides @code{idle-highlight-mode} that sets an
> idle timer to highlight all occurences in the buffer of the word under
> the point."
> 
>> +    (license license:gpl3+)))
> 

  reply	other threads:[~2017-03-28 19:00 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20170326233912.23146-1-va511e@yahoo.com>
2017-03-27  8:54 ` bug#26268: [PATCH] gnu: Add emacs-idle-highlight Alex Kost
2017-03-28 18:59   ` Vasile Dumitrascu [this message]
2017-03-30  8:49     ` Ludovic Courtès
2017-03-28 20:12 ` Vasile Dumitrascu
2017-03-29 11:40   ` Catonano
2017-03-29 11:59     ` Vasile Dumitrascu
2017-03-30  8:53   ` 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

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

  git send-email \
    --in-reply-to=68dcc0d8-ee90-ae6d-b47d-f17c4bfa3e3f@yahoo.com \
    --to=va511e@yahoo.com \
    --cc=26268@debbugs.gnu.org \
    --cc=alezost@gmail.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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.