Alexis Ballier wrote: > Indeed; different names could be given to different implementations of > the same thing, but that might completely kill the point of abstracting > it. > Maybe eclasses should die on unknown eapi; the fact is I really hate the > current way it's done when switching an ebuild to EAPI-2 which uses > an eclass that exports src_compile; most eclasses don't special case > eapi-2 yet and we end up running econf twice at best. I fear that'll be > the same with eapi-3, eapi-4, etc. (supposing that they'll support > src_configure too) > >> > An EXPORT_FUNCTIONS ignoring any function its doesn't know for its >> > eapi would help too. >> > Ciaran McCreesh <[EMAIL PROTECTED]> wrote: >> An EXPORT_FUNCTIONS ignoring incorrect usage makes one less place >> checking for eclass screwups... > > yes; that's just a matter of choice though, but for eclasses it's > probably not luxury. > Well it's simple enough to check (and give a QA warning) for unknown functions; adding a check for a specific string prefix (or to exclude a certain subset) in EXPORT_FUNCTIONS (based on current EAPI) is simple enough too. Is that what you mean?
The behaviour to trigger could change eg for debug mode, or a repoman check. I don't quite see how that deals with an eclass calling econf in its exported src_compile? Seems like EAPI versioning for eclasses (with implicit 0 only) is more what you're after for that issue (so the PM could suppress src_configure if src_compile is going to resolve to an EAPI-0 eclass function, although the inheritance stack might prove problematic.) Having to die for an unsupported EAPI seems like the wrong approach; if it's not going to work the PM shouldn't source it. If it can be made to work by filtering certain functions, that's doable. In the worst case, an ebuild switching to EAPI will require eclass maintenance; this is where the separation of elibs (useful code) and eclasses (template ebuilds) would be useful, although that needs versioning too.
