Bug 183482 - app-emacs/distel (new ebuild)
Bug#: 183482 Product:  Gentoo Linux Version: unspecified Platform: All
OS/Version: Linux Status: RESOLVED Severity: enhancement Priority: P2
Resolution: FIXED Assigned To: emacs@gentoo.org Reported By: justin-gentoo@openaether.org
Component: Ebuilds
URL: 
Summary: app-emacs/distel (new ebuild)
Keywords:  EBUILD
Status Whiteboard: 
Opened: 2007-06-28 06:17 0000
Description:   Opened: 2007-06-28 06:17 0000 
distel is a very useful pkg for erlang dev in emacs



Reproducible: Always

Steps to Reproduce:

------- Comment #1 From Justin 2007-06-28 06:18:17 0000 ------- 
Created an attachment (id=123275) [details]
distel ebuild 

This is the ebuild that pulls from google code's svn

------- Comment #2 From Justin 2007-06-28 06:19:19 0000 ------- 
Created an attachment (id=123276) [details]
gentoo site elisp

I am not sure if this is the proper place to put the init code. Or if it should
just go in an einfo.

------- Comment #3 From Christian Faulhammer 2007-06-28 06:43:54 0000 ------- 
You don't need virtual/emacs in DEPEND, elisp.eclass already provides that. 
Please put a comment after emake's die command.  Put quotes around the argument
of elisp-site-file-install().
Is the require in the site file really needed (is there no autoload)?  What
does the setup call do in the init?

Just a few comments, I am reluctant to put an live ebuild into Portage.  Can we
wait for an release?  Maybe I will put it into the overlay, or you add it to
Sunrise.

------- Comment #4 From Ulrich Müller 2007-06-28 06:53:52 0000 ------- 
(In reply to comment #3)
> Can we wait for an release?

There is a distel-3.2.tar.gz on Sourceforge.

------- Comment #5 From Christian Faulhammer 2007-06-28 06:59:10 0000 ------- 
(In reply to comment #4)
> (In reply to comment #3)
> > Can we wait for an release?
> There is a distel-3.2.tar.gz on Sourceforge.

 Which is obsoleted by upstream.

------- Comment #6 From Justin 2007-06-28 16:09:52 0000 ------- 
Created an attachment (id=123330) [details]
distel ebuild, updated

- removed virtual/emacs depend
- added comment to emake die
- put quotes around the site install


The distel-setup:
(defun distel-setup ()
  (add-hook 'erlang-mode-hook 'distel-erlang-mode-hook))

A few packages do this in site-gentoo.el   However, it does seem like something
that would be better placed in ~/.emacs   Its more a question convention. 

Putting this in the emacs overlay makes a lot of sense. (I don't know anything
about sunrise)

------- Comment #7 From Christian Faulhammer 2007-06-28 16:22:40 0000 ------- 
(In reply to comment #6)
> distel ebuild, updated

 Great.

> A few packages do this in site-gentoo.el   However, it does seem like something
> that would be better placed in ~/.emacs   Its more a question convention. 

 I think that is ok for a site init.

> Putting this in the emacs overlay makes a lot of sense. (I don't know anything
> about sunrise)

 http://www.gentoo-sunrise.org/ (a user overlay), the Emacs overlay is
available via "layman -a emacs" (sorry, look up the documentation yourself) and
we have a wiki on http://overlays.gentoo.org/proj/emacs/
 Let's wait for a release tarball someday.

------- Comment #8 From Ulrich Müller 2007-10-04 15:44:14 0000 ------- 
app-emacs/distel-3.2_p20070928, i.e. an SVN snapshot, has been committed to the
Emacs overlay, see <http://overlays.gentoo.org/proj/emacs/>.

------- Comment #9 From Christian Faulhammer 2008-01-23 08:24:29 0000 ------- 
Taken from overlay and prepared a newer snapshot.  Thanks for the submission,
closing