From: Nils Gillmann <niasterisk@grrlz.net>
To: Alex Kost <alezost@gmail.com>
Cc: Guix-devel <guix-devel@gnu.org>, Federico Beffa <beffa@ieee.org>
Subject: Re: emacs packaging (emacs-popup)
Date: Mon, 15 Feb 2016 19:00:42 +0100 [thread overview]
Message-ID: <874md9j1kl.fsf@grrlz.net> (raw)
In-Reply-To: <87a8n1rkex.fsf@gmail.com> (Alex Kost's message of "Mon, 15 Feb 2016 19:46:30 +0300")
Alex Kost <alezost@gmail.com> writes:
> Federico Beffa (2016-02-15 15:12 +0300) wrote:
>
>> Nils Gillmann <niasterisk@grrlz.net> writes:
>>
>>> I'm packaging emacs-popup.
>>>
>>> Currently the output builds with
>>> niasterisk@khazad-dum ~/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
Oh!
Thanks for the input and further reading. Then I'll look into
last changes and get a patch ready.
thanks,
--
ng
next prev parent reply other threads:[~2016-02-15 18:00 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-15 12:12 emacs packaging (emacs-popup) Federico Beffa
2016-02-15 16:46 ` Alex Kost
2016-02-15 18:00 ` Nils Gillmann [this message]
2016-02-16 7:58 ` Federico Beffa
2016-02-17 9:52 ` Alex Kost
2016-02-17 20:11 ` Federico Beffa
-- strict thread matches above, loose matches on Subject: below --
2016-02-15 2:26 Nils Gillmann
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=874md9j1kl.fsf@grrlz.net \
--to=niasterisk@grrlz.net \
--cc=alezost@gmail.com \
--cc=beffa@ieee.org \
--cc=guix-devel@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.