Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug
Bug#: 158073
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Gunnar Wrobel <wrobel@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Stephan Weller <stephan.weller@web.de>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
ledger-2.5-r1.ebuild.diff app-office/ledger-2.5-r1.ebuild.diff patch Jakub Moc (RETIRED) 2006-12-13 13:51 0000 601 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 158073 depends on: Show dependency tree
Bug 158073 blocks: 184166
Votes: 0    Show votes for this bug    Vote for this bug

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


Not eligible to see or edit group visibility for this bug.






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

Description:   Opened: 2006-12-13 13:31 0000 
I tried to emerge ledger today, but the ebuild always wants to install emacs as
a dependency. The ebuild has
emacs? ( virtual/emacs )
in its DEPEND, which is of course ok, but the ebuild also inherits elisp, which
in turn depends on virtual/emacs.
So at the moment there is no possibility to install ledger without installing
emacs.

I am not that fluent in ebuild writing, is there something like a conditional
"inherit"-statement? If not, this should probably be a portage bug...

------- Comment #1 From Jakub Moc (RETIRED) 2006-12-13 13:38:50 0000 ------- 
(In reply to comment #0)
> I am not that fluent in ebuild writing, is there something like a conditional
> "inherit"-statement? 

No, it's strictly forbidden to inherit eclasses only if a use flag is set...

------- Comment #2 From Jakub Moc (RETIRED) 2006-12-13 13:47:53 0000 ------- 
A quick fix, ruby-style...

<snip>
--- elisp.eclass        2006-02-28 04:05:42.000000000 +0100
+++ elisp.eclass        2006-12-13 22:45:15.000000000 +0100
@@ -23,7 +23,11 @@
 #      S="${WORKDIR}/${P}"
 fi

-DEPEND="virtual/emacs"
+# If you specify EMACS_OPTIONAL you also need to take care of 
+# emacs useflag and dependency.
+if [[ ${EMACS_OPTIONAL} != "yes" ]]; then
+        DEPEND="virtual/emacs"
+fi
 IUSE=""

 elisp_src_unpack() {
</snip>

------- Comment #3 From Jakub Moc (RETIRED) 2006-12-13 13:51:32 0000 ------- 
Created an attachment (id=103982) [details]
app-office/ledger-2.5-r1.ebuild.diff

- make emacs dependency really optional

------- Comment #4 From Stephan Weller 2006-12-13 23:21:46 0000 ------- 
Yep, that solves it for me. Thanks a lot, guys!

And, btw, not bad: 20 minutes to solve this one. If only all bugs could be
fixed that fast ;-)

------- Comment #5 From Jakub Moc (RETIRED) 2007-01-03 09:50:40 0000 ------- 
*** Bug 159861 has been marked as a duplicate of this bug. ***

------- Comment #6 From Christian Faulhammer 2007-01-05 03:32:06 0000 ------- 
Use elisp-common.eclass instead of elisp.

------- Comment #7 From Christian Faulhammer 2007-02-14 12:46:08 0000 ------- 
As there was no reaction from maintainer, I did the eclass migration.

------- Comment #8 From Ulrich Müller 2007-07-06 06:17:06 0000 ------- 
The ebuild should have the following in pkg_postinst and pkg_postrm:
    use emacs && elisp-site-regen

This was automatically called with elisp.eclass, but isn't with
elisp-common.eclass anymore.

Reopening.

------- Comment #9 From Christian Faulhammer 2007-09-16 10:48:24 0000 ------- 
No reaction from maintainer for more than two monts, so closing

------- Comment #10 From Christian Faulhammer 2007-09-16 10:48:46 0000 ------- 
And of course I fixed it. :)
Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug