guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] Remove Emacs interface.


From: Alex Kost
Subject: Re: [PATCH] Remove Emacs interface.
Date: Thu, 29 Dec 2016 23:43:44 +0300
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

Ludovic Courtès (2016-12-29 18:40 +0100) wrote:

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

Added, thanks!

>> 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?

Committed.

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

But we keep all patches in a single directory, why should there be
multiple dirs for auxiliary files?

> 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?

I don't like keeping such files in "gnu/packages" the most!  My opinion
is that only .scm files should be placed there.

I vote either for a single "aux-files" (I like this name) directory, or
for its sub-directories ("aux-files/emacs", "aux-files/linux").

-- 
Alex



reply via email to

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