Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 419654

Summary: app-office/ledger-2.6.3-r1 fails with USE=emacs
Product: Gentoo/Alt Reporter: Palmer Dabbelt <palmer>
Component: Prefix SupportAssignee: Tim Harder <radhermit>
Status: RESOLVED FIXED    
Severity: normal CC: prefix
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: Fix for app-office/ledger-2.6.3-r1
Updated app-office/ledger-2.6.3-r1 ebuild

Description Palmer Dabbelt 2012-06-04 20:06:58 UTC
Created attachment 314222 [details]
Fix for app-office/ledger-2.6.3-r1

app-office/ledger-2.6.3-r1 fails to install on my ~amd64-linux prefix with USE=emacs because it installs files outside the prefix.  I get the normal error message for this kind of thing (something about a QA concern).

I believe the problem is that configure is being passed ${SITELISP}/${PN} instead of ${ROOT}${EPREFIX}${SITELISP}/${PN} (I got the longer string from the elisp-common eclass, so I think that's OK).  With the change it seems that ledger installs its elisp files in the correct place.

I've attached a patch.  It builds for me on ~amd64-linux.
Comment 1 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2012-06-06 02:01:01 UTC
Also, worth noting that ROOT/EPREFIX == EROOT
Comment 2 Palmer Dabbelt 2012-06-06 02:06:55 UTC
Created attachment 314367 [details]
Updated app-office/ledger-2.6.3-r1 ebuild

(In reply to comment #1)
> Also, worth noting that ROOT/EPREFIX == EROOT

OK, I've added an updated ebuild.
Comment 3 Tim Harder gentoo-dev 2012-06-06 04:14:38 UTC
(In reply to comment #2)
> OK, I've added an updated ebuild.

Attaching a patch like you did the first time is the preferred way of doing things. :)

@prefix: Feel free to commit whatever is necessary for ledger to build and install properly in a prefix environment.
Comment 4 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2012-06-06 17:35:14 UTC
Actually went with this:

-               $(use_with emacs lispdir "${SITELISP}/${PN}") \
+               $(use_with emacs lispdir "${EPREFIX}/${SITELISP}/${PN}") \