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: Ulrich Mueller <ulm@g.o>
Subject: Re: apply_user_patches (was: EAPI 5 development branch)
Date: Mon, 7 May 2012 17:17:10 +0200
>>>>> 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.

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.

Ulrich


Replies:
Re: apply_user_patches (was: EAPI 5 development branch)
-- Ciaran McCreesh
Re: apply_user_patches
-- Michael Orlitzky
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
Navigation:
Lists: gentoo-pms: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
Re: apply_user_patches (was: EAPI 5 development branch)
Next by thread:
Re: apply_user_patches
Previous by date:
Re: apply_user_patches (was: EAPI 5 development branch)
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.