guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Remove Emacs interface.


From: Ludovic Courtès
Subject: Re: [PATCH] Remove Emacs interface.
Date: Thu, 29 Dec 2016 18:40:57 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

Hi!

Alex Kost <address@hidden> skribis:

> The attached patch removes Emacs UI and its documentation (12000 lines
> removed, hooray!).  I added the only mention about 'emacs-guix' package
> in "Package Management" chapter.

OK!

I was thinking that, even if the Emacs-Guix manual isn’t available
online currently, this change would lead users following the link in the
HTML manual to the Emacs-Guix home page (with a wrong anchor), which is
better than 404:

diff --git a/doc/htmlxref.cnf b/doc/htmlxref.cnf
index bd2eb5f..999edfe 100644
--- a/doc/htmlxref.cnf
+++ b/doc/htmlxref.cnf
@@ -219,6 +219,8 @@ emacs               node    ${EMACS}/html_node/emacs/
 easejs         mono    ${GS}/easejs/manual/easejs.html
 easejs         node    ${GS}/easejs/manual/
 
+emacs-guix     mono    https://notabug.org/...
+
 emacs-muse     node    ${GS}/emacs-muse/manual/muse.html
 emacs-muse     node    ${GS}/emacs-muse/manual/html_node/
 
Feel free to add it to the patch!

> Now 'emacs' directory contains a single git-controlled file -
> "guix-emacs.el".  It is used to patch our 'emacs' package (to find emacs
> packages in "share/emacs/site-lisp/guix.d").  But it is misplaced.

I agree.  May you can commit this patch first while we look for a
solution?

> What about making "gnu/packages/files" (or another dirname) directory,
> and putting "guix-emacs.el" and kernel configs ("linux-libre-*.conf")
> there?
>
> I mean "gnu/packages/patches" is not always enough, sometimes we want to
> use real files in the package recipes (configs for 'linux-libre' or
> "guix-emacs.el" for 'emacs'), so having "gnu/packages/files" seems
> reasonable to me, WDYT?

Sounds good.

In general, we can create other subdirectories alongside patches/ and
bootstrap/.  So we could have linux-conf/ and emacs/ there.  We can also
add regular files like guix-emacs.el directly under gnu/packages (like
ld-wrapper.in, linux-conf.*, etc.)

All this is fine as long as the files get installed in the right place
wrt. %load-path.

I’m not sure a catch-all files/ (or aux-files/?) directory is helpful
though.  There’s a case for having a linux-conf/ subdirectory now; for
guix-emacs.el, I am slightly inclined to put it directly in
gnu/packages, but no strong opinion.

Thoughts?

Thank you!

Ludo’.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]