On 10/17/2015 02:38 PM, Rich Freeman wrote: > On Sat, Oct 17, 2015 at 8:25 AM, hasufell <[email protected]> wrote: >> On 10/17/2015 02:19 PM, Jason A. Donenfeld wrote: >>> >>> The other question is more critical -- could you merge eapply and >>> eapply_user? Or add some hook to PMS so that eapply_user isn't needed? >>> IOW, it'd be nice if every package was, by default, patchable by the user. >>> >> >> IMO, eapply_user should not be in the eclass and not in PMS. patches are >> something that can easily be done via PM hooks, if the PM has proper >> hooks support. >> > > The reason this was done was to give maintainers more control over > WHEN patches are applied, while still ensuring they are applyied. >
You can apply the patches post_unpack or post_src_prepare witht hooks. What's the problem? Maintainers might forget to add it, add it in the wrong order or the PM doesn't support it. So people who want consistent patch support already HAVE to use hooks and that will not change. So you could as well remove eapply_user/epatch_user altogether. > The other feature that is supposed to be in EAPI6 (I didn't read the > draft yet) is that the PM should refuse to install the package if > eapply is never called (ie src_prepare is overridden and the ebuild > didn't call eapply). It is required that all ebuilds call it once > unconditionally. That way users don't get inconsistent behavior from > package to package and be dependent on maintainers to fix it. > I hope that "feature" doesn't make it into EAPI6.
