Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 404327 - x11-wm/dwm does not respect EPREFIX
Summary: x11-wm/dwm does not respect EPREFIX
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Prefix
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-18 02:21 UTC by Merlijn Wajer
Modified: 2012-02-19 19:07 UTC (History)
3 users (show)

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


Attachments
Patch to respect EPREFIX. (dwm-eprefix.patch,473 bytes, patch)
2012-02-18 02:23 UTC, Merlijn Wajer
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Merlijn Wajer 2012-02-18 02:21:17 UTC
x11-wm/dwm does not respect EPREFIX in src_install. It tries to install dwm outside the EPREFIX, which fails. This happens with both stable and keyworded versions of dwm.

Reproducible: Always
Comment 1 Merlijn Wajer 2012-02-18 02:23:17 UTC
Created attachment 302309 [details, diff]
Patch to respect EPREFIX.

I am not entirely sure this is an proper fix as I am relatively new to Gentoo/Prefix, but it fixed the bug in my case. Note that it should work for all versions of dwm currently in portage.
Comment 2 Fabian Groffen gentoo-dev 2012-02-18 07:54:39 UTC
Hi Merlijn,

Because the package has no Prefix keywords, it's likely not to work for Prefix.  Just so you know.  Prefix doesn't do stable or unstable keywords.

Your patch looks like the absolute minimum thing that needs to be done to the ebuild, but it requires a bump to EAPI=3 as well.  It may be that config.mk needs some changes to search in the Prefix, as the sed in src_unpack suggests, but I would have to check that.
Comment 3 Jeroen Roovers (RETIRED) gentoo-dev 2012-02-18 16:27:37 UTC
cvs/gentoo-x86/x11-wm/dwm $ ebuildvar EAPI
dwm-5.6.1.ebuild : EAPI="0"
dwm-5.7.2.ebuild : EAPI="0"
dwm-5.8.2.ebuild : EAPI="0"
  dwm-5.9.ebuild : EAPI="4"
  dwm-6.0.ebuild : EAPI="4"

It's EAPI 4 already.
Comment 4 Fabian Groffen gentoo-dev 2012-02-18 16:28:57 UTC
I should probably cvs update then :/
Comment 5 Fabian Groffen gentoo-dev 2012-02-19 19:07:31 UTC
as I expected, some additional prefixes were necessary, committed now, should fix this bug

Thanks Merlijn!