Summary: | gnome-extra/hamster-applet: 2.32.1 fails to build due to eclass changes | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Rémi Cardona (RETIRED) <remi> |
Component: | Eclasses | Assignee: | Gentoo Linux Gnome Desktop Team <gnome> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | scarabeus |
Priority: | High | Keywords: | InVCS |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 339225 | ||
Attachments: | environment |
Description
Rémi Cardona (RETIRED)
2011-01-17 07:24:49 UTC
Created attachment 260039 [details]
environment
IIRC, there was a forum post about it - eclass was broken, but it's fixed now. Well, arguably, you could say hamster-applet should use the call in pkg_setup. If you see it this way, reopen. problem is that waf-utils eclass should not define this variable in pkg_setup, it should be an eclass-scope variable like we usually do for this kind of stuff. CCing scarabeus for the previous comment eclass API changes are bad bad bad. (In reply to comment #4) > problem is that waf-utils eclass should not define this variable in pkg_setup, > it should be an eclass-scope variable like we usually do for this kind of > stuff. > Then you have problem that you have to define such variable PRIOR inherit, and given it spins of the S you need to define S prior inherit too. It is safer to expect ebuilds to use all provided package scopes. To bit expand my point: normaly we do and would not work: EAPI=4 MY_P=eeeee inherit waf-utils ... S=${WORKDIR}/${MY_P} ... required if global scope: EAPI=4 MY_P=eeeee S=${WORKDIR}/${MY_P} inherit waf-utils ... I bet most people would never spot this issue, thus i set it in function scope. Did I already say I hate waf ? Why is it that not all packages would use waf script and rename it to something silly ? Didn't (waf) upstream set a policy for this ? Or do upstream devs think it's funny to mess around with the build system's name. I wonder if we could package waf as-is and make all waf packages use that script we will have on system. But i bet they are version dependant and thats the reason why it is shipped with each package huh? waf upstream explicitly say in their doc that we should not do that. They could say their build system is broken by design, would be the same imho. But to the real question, is there a package using waf but named something else ? Could we not just symlink WAF_BINARY to waf in src_prepare ? (In reply to comment #11) > But to the real question, is there a package using waf but named something else > ? See talloc ebuild :D god help us... Anyway, could you (did you ?) send a mail to gentoo-dev saying waf-utils eclass should use pkg_setup always ? (In reply to comment #13) > god help us... > > Anyway, could you (did you ?) send a mail to gentoo-dev saying waf-utils eclass > should use pkg_setup always ? > Nope I did not, i probably should done it src_configure and drop pkg_setup as whole, cause it would make more sense, and always work. What do you think? fine by me. If possible the new feature should be documented (can't remember if it is right now). Eclass updated in main tree based on consensus with Gilles. Closing as fixed. |