First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 182639
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: GNU Emacs Team <emacs@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Ulrich Müller <ulm@gentoo.org>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 182639 depends on: Show dependency tree
Show dependency graph
Bug 182639 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)







View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug

Description:   Opened: 2007-06-20 05:58 0000 
The following packages inherit elisp-common.eclass but do not use the right
condition for calling elisp-site-regen:

   app-i18n/anthy/anthy
   app-i18n/anthy-ss
   app-i18n/uim-svn
   kde-base/kdvi
   app-office/magicpoint
   dev-util/bzr
   dev-util/mercurial

The typical pattern (with some variations) is:

   pkg_postinst() {
       use emacs && elisp-site-regen
   }

   pkg_postrm() {
       has_version virtual/emacs && elisp-site-regen
   }

This is WRONG. The condition should be "use emacs" both in pkg_postinst() and
pkg_postrm(). Please fix it in your packages.

------- Comment #1 From Wulf Krueger (RETIRED) 2007-06-20 16:15:26 0000 ------- 
kde-base/kdvi is fixed. KDE is out of here. :-)

------- Comment #2 From Marien Zwart (RETIRED) 2007-06-22 00:02:51 0000 ------- 
Fixed dev-util/bzr, thanks for complaining :)

------- Comment #3 From MATSUU Takuto 2007-06-24 03:33:11 0000 ------- 
i think its wrong solution.

For example,
# USE=emacs emerge anthy
# USE=-emacs emerge anthy

50anthy-gentoo.el was removed and elisp-site-regen didn't run.

------- Comment #4 From Ulrich Müller 2007-06-24 06:33:45 0000 ------- 
(In reply to comment #3)
> i think its wrong solution.

But obviously, you didn't bother testing it. ;-)

> For example,
> # USE=emacs emerge anthy
> # USE=-emacs emerge anthy

The status of USE=emacs when unmerging is taken from the database, so unsetting
it between merge and unmerge has no effect.

------- Comment #5 From MATSUU Takuto 2007-06-24 15:56:24 0000 ------- 
I'm sorry. that's right. I didn't test it.

app-i18n/anthy
app-i18n/anthy-ss
app-i18n/uim-svn
fixed.

------- Comment #6 From Andrew Ross (RETIRED) 2007-06-30 07:25:53 0000 ------- 
Thanks for pointing this out. dev-util/mercurial fixed with no rev bump.

------- Comment #7 From Ulrich Müller 2007-06-30 08:33:58 0000 ------- 
Fixed app-office/magicpoint myself, since usata is away.
First Last Prev Next    No search results available      Search page      Enter new bug