From: Alex Kost <alezost@gmail.com>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] emacs: Add 'pretty-sha-path'.
Date: Wed, 05 Nov 2014 14:24:49 +0300 [thread overview]
Message-ID: <87d291uby6.fsf@gmail.com> (raw)
In-Reply-To: <87h9ye8x6d.fsf@gnu.org> ("Ludovic \=\?utf-8\?Q\?Court\=C3\=A8s\=22'\?\= \=\?utf-8\?Q\?s\?\= message of "Tue, 04 Nov 2014 22:37:14 +0100")
[-- Attachment #1: Type: text/plain, Size: 3800 bytes --]
Ludovic Courtès (2014-11-05 00:37 +0300) wrote:
> Alex Kost <alezost@gmail.com> skribis:
>
>> Also I forgot to mention “emacs/guix-messages.el” in “emacs.am” in
>> commit 62f261d, so I did it in this patch (I hope it's not too evil :-))
>
> Maybe “evil” is too strong a word ;-), but please keep the
> emacs/guix-messages.el addition in a separate commit.
>
> Commits are cheap and easy, so let’s favor clarity.
Yes, cheap, I know, but not very easy for me as I never sure what to
write in a commit message and I have to ask guix-devel even about such
trivial changes (the patch is attached :-)).
>> From e7ca6550d7f33d894e0023e3305938fce365fdba Mon Sep 17 00:00:00 2001
>> From: Alex Kost <alezost@gmail.com>
>> Date: Tue, 4 Nov 2014 19:38:27 +0300
>> Subject: [PATCH] emacs: Add 'pretty-sha-path'.
>>
>> * emacs/pretty-sha-path.el: New file.
>> * emacs.am (ELFILES): Add it.
>> * doc/emacs.texi (Emacs Pretty Path): New node.
>
> [...]
>
>> +@node Emacs Pretty Path
>> +@subsection Pretty SHA Path Mode
>
> What about adding, at the end of the first paragraph of the “Features”
> section, something like:
>
> ... where @code{xxx} is a base32 string (note that Guix comes with an
> Emacs extension to shorten those file names, @ref{Emacs Pretty Path}.)
OK, will do.
>> +Along with ``guix.el'', GNU@tie{}Guix comes with ``pretty-sha-path.el''.
>> +It provides a minor mode for abbreviating store paths by replacing
>> +SHA-sequences of symbols with ``@dots{}'':
>> +
>> +@example
>> +/gnu/store/onecansee32lettersandnumbershere-foo-0.1 @result{} /gnu/store/…-foo-0.1
>
> Perhaps insert a line break before @result{}, otherwise the PDF output
> may be truncated.
OK.
>> +@item M-x pretty-sha-path-global-mode
>> +Enable/disable prettifying globally.
>
> It seemed to me that the convention would be to call it
> global-pretty-sha-path-mode rather, no?
Ah, thanks! I didn't thought about that; I just always used a package
prefix for all symbols.
According to (info "(elisp) Coding Conventions"):
--8<---------------cut here---------------start------------->8---
• You should choose a short word to distinguish your program from
other Lisp programs. The names of all global symbols in your
program, that is the names of variables, constants, and functions,
should begin with that chosen prefix. Separate the prefix from the
rest of the name with a hyphen, ‘-’. This practice helps avoid
name conflicts, since all global variables in Emacs Lisp share the
same name space, and all functions share another name space(1).
--8<---------------cut here---------------end--------------->8---
And also:
--8<---------------cut here---------------start------------->8---
Occasionally, for a command name intended for users to use, it is
more convenient if some words come before the package’s name
prefix.
--8<---------------cut here---------------end--------------->8---
So I'm going to add ‘global-pretty-sha-path-mode’ and mention it in the
documentation, but I also leave ‘pretty-sha-path-global-mode’ just in
case (I'll make it an alias).
> Other than that, LGTM!
>
> Mark is right about refining the regexp, but OTOH false positives are
> very unlikely, and it might make things slightly slower, no? So either
> way is fine with me.
I think it shouldn't be visibly slower.
I've realized that "pretty-sha-path" is a bad name, because those 32
numbers and letters have nothing to do with SHA-sequences as I thought
initially. So maybe it would be better to rename it into
"pretty-hash-path" or "guix-pretty-path" (as it will be a part of Guix)
or something else. Or is it OK to leave it as it is?
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-build-Add-missing-emacs-guix-messages.el.patch --]
[-- Type: text/x-diff, Size: 611 bytes --]
From 4648af5bab6a9e98cc4c725b8fcca159874d4380 Mon Sep 17 00:00:00 2001
From: Alex Kost <alezost@gmail.com>
Date: Wed, 5 Nov 2014 14:00:05 +0300
Subject: [PATCH] build: Add missing "emacs/guix-messages.el".
* emacs.am (ELFILES): Add "emacs/guix-messages.el".
---
emacs.am | 1 +
1 file changed, 1 insertion(+)
diff --git a/emacs.am b/emacs.am
index 35470a3..9279078 100644
--- a/emacs.am
+++ b/emacs.am
@@ -24,6 +24,7 @@ ELFILES = \
emacs/guix-history.el \
emacs/guix-info.el \
emacs/guix-list.el \
+ emacs/guix-messages.el \
emacs/guix-utils.el \
emacs/guix.el
--
2.1.2
next prev parent reply other threads:[~2014-11-05 11:25 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-04 16:48 [PATCH] emacs: Add 'pretty-sha-path' Alex Kost
2014-11-04 19:39 ` Mark H Weaver
2014-11-05 11:24 ` Alex Kost
2014-11-04 21:37 ` Ludovic Courtès
2014-11-05 11:24 ` Alex Kost [this message]
2014-11-05 20:12 ` Ludovic Courtès
2014-11-05 21:38 ` Alex Kost
2014-11-06 9:37 ` Ludovic Courtès
2014-11-06 13:13 ` [PATCH] emacs: Add 'guix-prettify' Alex Kost
2014-11-06 16:05 ` 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=87d291uby6.fsf@gmail.com \
--to=alezost@gmail.com \
--cc=guix-devel@gnu.org \
--cc=ludo@gnu.org \
/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.