Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 419654 - app-office/ledger-2.6.3-r1 fails with USE=emacs
Summary: app-office/ledger-2.6.3-r1 fails with USE=emacs
Status: RESOLVED FIXED
Alias: None
Product: Gentoo/Alt
Classification: Unclassified
Component: Prefix Support (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Tim Harder
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-06-04 20:06 UTC by Palmer Dabbelt
Modified: 2012-06-06 17:35 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Fix for app-office/ledger-2.6.3-r1 (app-office_ledger-2.6.3-r1.patch,1.94 KB, text/plain)
2012-06-04 20:06 UTC, Palmer Dabbelt
Details
Updated app-office/ledger-2.6.3-r1 ebuild (ledger-2.6.3-r1.ebuild,1.98 KB, text/plain)
2012-06-06 02:06 UTC, Palmer Dabbelt
Details

Note You need to log in before you can comment on or make changes to this bug.
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}") \