[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: emacs packaging (emacs-popup)
From: |
Alex Kost |
Subject: |
Re: emacs packaging (emacs-popup) |
Date: |
Wed, 17 Feb 2016 12:52:24 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Federico Beffa (2016-02-16 10:58 +0300) wrote:
> On Mon, Feb 15, 2016 at 5:46 PM, Alex Kost <address@hidden> wrote:
>> Federico Beffa (2016-02-15 15:12 +0300) wrote:
[...]
>>> I would suggest to move the README.md (and in general documentation
>>> related files) to $datadir/doc/package-name-version/. Info files to
>>> $datadir/info/package-name-version/. Take a look at 'emacs-slime'.
>>>
>>> Remove unused files like .gitignore, ...
>>
>> I don't agree. This is not done for any other emacs package. I think
>> ".gitignore" and other files should not be removed in a package recipe.
>> This is a job for emacs-build-system, and I thought it was agreed¹. So
>> I think this "emacs-popup" package can be left as it is now, and once
>> our emacs-build-system will be improved (to remove useless file, to move
>> README into share/doc, etc.), it will be fixed automatically.
>>
>> ¹ http://lists.gnu.org/archive/html/guix-devel/2015-07/msg00416.html
>
> Doing so will break some packages. One example is 'emacs-slime' which
> includes .lisp and .asd files, other ones are 'emacs-auctex' and
> 'emacs-ob-ipython', ...
For such packages, a potential "cleanup" phase may be removed in a
package recipe. Also I don't think that .lisp, .asd and other files
should be placed in ".../share/emacs/site-lisp/" directory. My
understanding is that this directory (and it's subdirs) should be used
only for elisp files.
> While we could add a phase to remove some pretty common and useless
> files such as .gitignore, in general things are not standardized and
> more complex. Unless upstream fixes this situation it will be
> impossible to make a perfect 'emacs-build-system'.
I think that most packages are simple and very similar, and "cleanup"
phase can just remove everything except *.el[c]. For the complex
packages, this phases can be removed, besides it's likely that the
manual intervention in a package recipe will be required anyway.
Anyway, I'm just talking. Someday (I hope) I will propose patches for
improving emacs-build-system (as I see it).
--
Alex