| Summary: | elisp-common.eclass and xemacs-elisp-common.eclass namespace collision (was: app-admin/puppet installs xemacs files instead of emacs files) | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Tom Prince <tom.prince> |
| Component: | Eclasses | Assignee: | XEmacs team <xemacs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | emacs, matsuu |
| Priority: | Normal | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Attachments: |
emerge --info
build.log |
||
Created attachment 303519 [details]
build.log
Fixed in cvs. Why has nobody CCed the emacs and xemacs teams here? The problem lies deeper, elisp-common.eclass and xemacs-elisp-common.eclass both assign the SITELISP variable. Probably one of them (at least) should rename it. AFAICS, xemacs-common.eclass is inherit by the following packages only: app-admin/puppet app-xemacs/gentoo-syntax sci-mathematics/octave Out of these, only puppet-2.7.12.ebuild uses the SITELISP variable directly. @graaff: Would it be possible for you to rename the variable in xemacs-elisp-common.eclass? (And maybe rename SITEPACKAGE too, for consistency?) Renaming it in elisp-common.eclass would require many times more effort, because it is inherited by some 500 ebuilds. (In reply to comment #3) > @graaff: Would it be possible for you to rename the variable in > xemacs-elisp-common.eclass? (And maybe rename SITEPACKAGE too, for > consistency?) Renaming it in elisp-common.eclass would require many times > more effort, because it is inherited by some 500 ebuilds. Sure, that sounds like the best solution. I'll see if I can find some time for that this weekend. Fixed in xemacs-elisp-common.eclass, as discussed in IRC:
SITELISP is now XEMACS_SITELISP,
SITEPACKAGES is now XEMACS_SITEPACKAGES.
I've also changed the app-admin/puppet-2.7.{12,13} ebuilds accordingly.
(In reply to comment #5) > SITEPACKAGES is now XEMACS_SITEPACKAGES. This should read "SITEPACKAGE" and "XEMACS_SITEPACKAGE". |
Created attachment 303517 [details] emerge --info /usr/share/xemacs is created and populated, while nothing is put in /usr/share/emacs