Re: emacs packaging (emacs-popup)

[Alex Kost]

Federico Beffa (2016-02-15 15:12 +0300) wrote:

> Nils Gillmann <address@hidden> writes:
>
>> I'm packaging emacs-popup.
>>
>> Currently the output builds with
>> address@hidden ~/projects/guix_project/guix$ ls -al
>> /gnu/store/knpjk9vflja0i6yhnz4ai57k29jpgm4p-emacs-popup-0.5.3/share/emacs/site-lisp/guix.d/popup-0.5.3/
>> total 180
>> dr-xr-xr-x 4 root guixbuild  4096 Jan  1  1970 ./
>> dr-xr-xr-x 3 root guixbuild  4096 Jan  1  1970 ../
>> -r--r--r-- 2 root guixbuild    90 Jan  1  1970 Cask
>> dr-xr-xr-x 3 root guixbuild  4096 Jan  1  1970 etc/
>> -r--r--r-- 2 root guixbuild    14 Jan  1  1970 .gitignore
>> -r--r--r-- 2 root guixbuild   626 Jan  1  1970 Makefile
>> -r--r--r-- 2 root guixbuild   334 Jan  1  1970 popup-autoloads.el
>> -r--r--r-- 2 root guixbuild 54848 Jan  1  1970 popup.el
>> -r--r--r-- 2 root guixbuild 75474 Jan  1  1970 popup.elc
>> -r--r--r-- 2 root guixbuild 10187 Jan  1  1970 README.md
>> dr-xr-xr-x 2 root guixbuild  4096 Jan  1  1970 tests/
>> -r--r--r-- 2 root guixbuild   569 Jan  1  1970 .travis.yml
>>
>> Can this be improved, ie remove some of the files to clean it up,
>> or is it all good?
>
> 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

-- 
Alex