Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 160607 - app-emacs/tramp-cvs ebuild
Summary: app-emacs/tramp-cvs ebuild
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Emacs project
URL:
Whiteboard:
Keywords: EBUILD
Depends on:
Blocks:
 
Reported: 2007-01-07 01:23 UTC by Hans Halvorson
Modified: 2007-02-04 10:53 UTC (History)
0 users

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


Attachments
user submitted ebuild for tramp-cvs (tramp-cvs-2.8_pre.ebuild,1.13 KB, text/plain)
2007-01-07 01:25 UTC, Hans Halvorson
Details
modified tramp-cvs ebuild (tramp-cvs-2.1.8_pre.ebuild,1.44 KB, text/plain)
2007-01-09 02:25 UTC, Hans Halvorson
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Hans Halvorson 2007-01-07 01:23:34 UTC
There is a known incompatibility (with autocompletion) between tramp versions 2.7 and earlier, and emacs versions 22.0.92 and higher.  (Version 2.0.55 of tramp is distributed with app-editors/emacs-cvs-22.0.92 as of January 6.  Version 2.7 of tramp is the latest release and is available in portage as app-emacs/tramp-2.7.)  This bug is fixed in tramp CVS as of Dec 20, 2006, but tramp CVS is not in portage. 

Reproducible: Always

Steps to Reproduce:
Try to autocomplete with TAB on "/su:localhost:/etc/"



Actual Results:  
Error: "invalid number of arguments"

Expected Results:  
displays list of possible completions
Comment 1 Hans Halvorson 2007-01-07 01:25:55 UTC
Created attachment 105743 [details]
user submitted ebuild for tramp-cvs

Comments and criticisms welcome
Comment 2 Hans Halvorson 2007-01-09 02:25:49 UTC
Created attachment 106139 [details]
modified tramp-cvs ebuild

Tramp configure.ac changes the permissions on /dev/null.  Added a little sed to fix the problem.
Comment 3 Christian Faulhammer (RETIRED) gentoo-dev 2007-01-29 07:50:42 UTC
Looks ok:

1) Put quotes around variable as ${S} or ${D} (as they can contain spaces)
2) You don't need to DEPEND on virtual/emacs, elisp.eclass does that already
3) nostrip is needed when binaries are stripped by the Makefile of the package, else Portage does that.

Else, I am not that interested in putting it into Portage as cvs ebuilds are a bit ungrateful to maintain and tramp team is fast with releases.  Maybe you want to put into Sunrise Overlay (http://www.gentoo-sunrise.org/), which is a good way to start.
Comment 4 Hans Halvorson 2007-02-04 10:53:59 UTC
Thank you very much for the feedback.

Tramp has since been bumped to v. 2.1.8, with an ebuild in portage.  So, I agree that portage does not yet need a tramp-cvs ebuild.

(In reply to comment #3)
> Looks ok:
> 
> 1) Put quotes around variable as ${S} or ${D} (as they can contain spaces)
> 2) You don't need to DEPEND on virtual/emacs, elisp.eclass does that already
> 3) nostrip is needed when binaries are stripped by the Makefile of the package,
> else Portage does that.
> 
> Else, I am not that interested in putting it into Portage as cvs ebuilds are a
> bit ungrateful to maintain and tramp team is fast with releases.  Maybe you
> want to put into Sunrise Overlay (http://www.gentoo-sunrise.org/), which is a
> good way to start.
>