Gentoo Logo
Gentoo Spaceship




Note: Due to technical difficulties, the Archives are currently not up to date. GMANE provides an alternative service for most mailing lists.
c.f. bug 424647
List Archive: gentoo-pms
Navigation:
Lists: gentoo-pms: < Prev By Thread Next > < Prev By Date Next >
Headers:
To: gentoo-pms@g.o
From: Ciaran McCreesh <ciaran.mccreesh@...>
Subject: Re: apply_user_patches (was: EAPI 5 development branch)
Date: Mon, 7 May 2012 17:28:09 +0100
On Mon, 7 May 2012 17:17:10 +0200
Ulrich Mueller <ulm@g.o> wrote:
> >>>>> On Mon, 7 May 2012, Ciaran McCreesh wrote:
> > On Mon, 7 May 2012 02:12:14 +0200
> > Ulrich Mueller <ulm@g.o> wrote:
> >> | Furthermore, for these EAPIs, if the function is overridden, it
> >> | shall be a fatal error if the apply_user_patches command has not
> >> | been called at least once by the end of the phase.
> >> 
> >> Wouldn't it make more sense to call apply_user_patches implicitly
> >> at the end of the phase, if it hasn't been called before?
> >> 
> >> Otherwise, a call to that function would have to be added to every
> >> ebuild that defines src_prepare.
> 
> > That was the point. The discussion on gentoo-dev suggested that "at
> > the end" is often the wrong place to put it, due to eautoreconf etc.
> > We need people to be explicit about where it goes.
> 
> Yes, so apply_user_patches gives ebuilds the possibility to specify
> the exact place. I still think that a fallback to calling it at the
> end of the phase would be better than aborting with a fatal error.

Why? That error will only happen at most once, and users will never
see it.

> After all, this functionality is just a stop-gap measure for users to
> apply quick bug fixes, so I don't expect that it will be used very
> often. Even fewer cases will require that eautoreconf is called. Do we
> really want to force developers to put this function call into every
> ebuild? That would be out of proportion, IMHO.

"It not being used very often" is the key issue: developers are likely
to forget about it if they're not forced to remember. It's especially
complicated when eclasses are involved, so even if developers do
remember, they may not be sure whether they have to specify it in a
particular case.

We can avoid a huge pile of "blah doesn't work properly with user
patches!" bugs here, essentially for free.

-- 
Ciaran McCreesh
Attachment:
signature.asc (PGP signature)
References:
EAPI 5 development branch
-- Ciaran McCreesh
apply_user_patches (was: EAPI 5 development branch)
-- Ulrich Mueller
Re: apply_user_patches (was: EAPI 5 development branch)
-- Ciaran McCreesh
Re: apply_user_patches (was: EAPI 5 development branch)
-- Ulrich Mueller
Navigation:
Lists: gentoo-pms: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
Re: apply_user_patches
Next by thread:
Re: EAPI 5 development branch
Previous by date:
Re: apply_user_patches
Next by date:
Re: apply_user_patches


Updated Jul 18, 2012

Summary: Archive of the gentoo-pms mailing list.

Donate to support our development efforts.

Copyright 2001-2013 Gentoo Foundation, Inc. Questions, Comments? Contact us.