Summary: | dev-util/global gtags elisp install path mismatch | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | IGARASHI Masanao <igarashi> |
Component: | [OLD] Development | Assignee: | No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | emacs, jer |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | AMD64 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
IGARASHI Masanao
2009-09-19 17:01:17 UTC
Could you create the patch in unidiff format? (In reply to comment #1) > Could you create the patch in unidiff format? Er, so the fix was very simple and you could readily imagine the patch, yet you insisted on a unified diff? :-\ CC'ing emacs@ (since you maintain elisp-common.eclass) for confirmation. The file in question appears to be gentoo-x86/dev-util/global/files/50gtags-gentoo.el and I can't see what's wrong with it. I think this change might make sense: Index: global-5.9.2.ebuild =================================================================== RCS file: /var/cvsroot/gentoo-x86/dev-util/global/global-5.9.2.ebuild,v retrieving revision 1.1 diff -u -B -r1.1 global-5.9.2.ebuild --- global-5.9.2.ebuild 1 Sep 2010 13:40:39 -0000 1.1 +++ global-5.9.2.ebuild 1 Sep 2010 14:27:28 -0000 @@ -59,7 +59,7 @@ fi if use emacs; then - elisp-install gtags *.{el,elc} + elisp-install global *.{el,elc} elisp-site-file-install "${FILESDIR}/${SITEFILE}" fi } (In reply to comment #3) > I think this change might make sense: Right, either that, or just use ${PN}. And add a "||die" command. So the canonical solution would be: if use emacs; then elisp-install ${PN} *.el *.elc || die elisp-site-file-install "${FILESDIR}/${SITEFILE}" || die fi Thanks. That's fixed in 5.9.1 and 5.9.2. |