Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 259695 - app-emacs/distel-4.03: bump version and fix paths
Summary: app-emacs/distel-4.03: bump version and fix paths
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Emacs project
URL: http://code.google.com/p/distel/issue...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-20 11:44 UTC by Sergey Sinkovskiy
Modified: 2009-02-21 15:32 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey Sinkovskiy 2009-02-20 11:44:21 UTC
Distel is a Emacs mode for Erlang development.
Package includes .erl which are compiled to .beam and .el files.

In Elisp source it's defined to search for .beam files in <site-lisp>/<distel>/../ebin/ directory, which means that 
placing .beam and .el files under different roots how it's implemented in
app-emacs/distel-3.2_p20071103 doesn't work.
Comment 1 Ulrich Müller gentoo-dev 2009-02-20 17:35:09 UTC
> In Elisp source it's defined to search for .beam files in
> <site-lisp>/<distel>/../ebin/ directory, which means that 
> placing .beam and .el files under different roots how it's implemented in
> app-emacs/distel-3.2_p20071103 doesn't work.

Thanks for the hint, this is broken behaviour. Clearly the elisp files belong in /usr/share/emacs/site-lisp/ and not in /usr/share/distel/elisp/.
Comment 2 Ulrich Müller gentoo-dev 2009-02-20 18:15:46 UTC
distel-4.03 committed to CVS. I've added a patch to make the path of the ebin directory configurable, so that we can set it in the site-init file.

Could you please test if it works for you?

The path issue already existed in the 3.2 snapshot, if I understand you correctly?
Comment 3 Sergey Sinkovskiy 2009-02-20 18:20:14 UTC
(In reply to comment #1)
> > In Elisp source it's defined to search for .beam files in
> > <site-lisp>/<distel>/../ebin/ directory, which means that 
> > placing .beam and .el files under different roots how it's implemented in
> > app-emacs/distel-3.2_p20071103 doesn't work.
> 
> Thanks for the hint, this is broken behaviour. Clearly the elisp files belong
> in /usr/share/emacs/site-lisp/ and not in /usr/share/distel/elisp/.
> 

So possible fixes can be:
1) patch distel source to search for .beam in different place
2) put elisp files to /usr/share/distel/elisp and create symlinks to them form emacs/site-lisp

Comment 4 Sergey Sinkovskiy 2009-02-20 18:21:21 UTC
(In reply to comment #2)
> distel-4.03 committed to CVS. I've added a patch to make the path of the ebin
> directory configurable, so that we can set it in the site-init file.
> 
> Could you please test if it works for you?
Can you provide me with instructions how to get CVS version? 

> The path issue already existed in the 3.2 snapshot, if I understand you
> correctly?
> 
Correct.
Comment 5 Ulrich Müller gentoo-dev 2009-02-20 18:28:07 UTC
> > distel-4.03 committed to CVS. I've added a patch to make the path of the
> > ebin directory configurable, so that we can set it in the site-init file.
> > 
> > Could you please test if it works for you?
> Can you provide me with instructions how to get CVS version? 

CVS = Portage tree ;-)
4.03 should appear on the normal rsync mirrors in a few hours.
Comment 6 Sergey Sinkovskiy 2009-02-20 21:46:41 UTC
(In reply to comment #5)
> > > distel-4.03 committed to CVS. I've added a patch to make the path of the
> > > ebin directory configurable, so that we can set it in the site-init file.
> > > 
> > > Could you please test if it works for you?
> > Can you provide me with instructions how to get CVS version? 
> 
> CVS = Portage tree ;-)
> 4.03 should appear on the normal rsync mirrors in a few hours.
> 

Works for me. Thanks.

P.S. I'm a newbie and not familiar with ticket flow here - so changed resolution as FIXED.
Comment 7 Ulrich Müller gentoo-dev 2009-02-20 22:37:58 UTC
> Works for me. Thanks.

Patch submitted upstream: <http://code.google.com/p/distel/issues/detail?id=14>

> P.S. I'm a newbie and not familiar with ticket flow here - so changed
> resolution as FIXED.

This is O.K., but you may as well leave it to the maintainer. The rule is that a bug can be marked as FIXED when a working solution has been committed to the Portage tree.

Thank you for reporting this issue.
Comment 8 mats cronqvist 2009-02-21 15:32:37 UTC
fixed upstream.
thanks for the patch.

 - distel Gentoo Task Force