FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Gentoo > Gentoo Development

 
 
LinkBack Thread Tools
 
Old 03-20-2012, 03:29 AM
Luca Barbato
 
Default Fix spurious dep to eselect-python

Hi, I tried to avoid depending on eselect-python if the useflag is disabled.

Please test and review.

lu

--

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero
 
Old 03-20-2012, 03:41 PM
Arfrever Frehtes Taifersar Arahesis
 
Default Fix spurious dep to eselect-python

2012-03-20 05:29:20 Luca Barbato napisał(a):
> Hi, I tried to avoid depending on eselect-python if the useflag is disabled.
>
> Please test and review.

The proper fix is to make python.eclass add dependency on app-admin/eselect-python only when
${CATEGORY}/${PN} is dev-lang/python, dev-java/jython or dev-python/pypy. See bug #341037.

--
Arfrever Frehtes Taifersar Arahesis
 
Old 03-20-2012, 03:51 PM
Mike Gilbert
 
Default Fix spurious dep to eselect-python

On Tue, Mar 20, 2012 at 12:41 PM, Arfrever Frehtes Taifersar Arahesis
<arfrever@gentoo.org> wrote:
> 2012-03-20 05:29:20 Luca Barbato napisał(a):
>> Hi, I tried to avoid depending on eselect-python if the useflag is disabled.
>>
>> Please test and review.
>
> The proper fix is to make python.eclass add dependency on app-admin/eselect-python only when
> ${CATEGORY}/${PN} is dev-lang/python, dev-java/jython or dev-python/pypy. See bug #341037.
>

Could you provide a patch against gentoo-x86 python.eclass for that? I
think you may have already done that, but things said in IRC get
forgotten.
 
Old 03-20-2012, 03:53 PM
Michał Górny
 
Default Fix spurious dep to eselect-python

On Tue, 20 Mar 2012 17:41:39 +0100
Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org> wrote:

> 2012-03-20 05:29:20 Luca Barbato napisał(a):
> > Hi, I tried to avoid depending on eselect-python if the useflag is
> > disabled.
> >
> > Please test and review.
>
> The proper fix is to make python.eclass add dependency on
> app-admin/eselect-python only when ${CATEGORY}/${PN} is
> dev-lang/python, dev-java/jython or dev-python/pypy. See bug #341037.

Couldn't we just push that dependency to the specific ebuilds?

--
Best regards,
Michał Górny
 
Old 03-20-2012, 06:35 PM
Arfrever Frehtes Taifersar Arahesis
 
Default Fix spurious dep to eselect-python

2012-03-20 17:53:56 Michał Górny napisał(a):
> On Tue, 20 Mar 2012 17:41:39 +0100
> Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org> wrote:
>
> > 2012-03-20 05:29:20 Luca Barbato napisał(a):
> > > Hi, I tried to avoid depending on eselect-python if the useflag is
> > > disabled.
> > >
> > > Please test and review.
> >
> > The proper fix is to make python.eclass add dependency on
> > app-admin/eselect-python only when ${CATEGORY}/${PN} is
> > dev-lang/python, dev-java/jython or dev-python/pypy. See bug #341037.
>
> Couldn't we just push that dependency to the specific ebuilds?

We want to control required version (>=20091230 in case of app-admin/eselect-python) in 1 place.

--
Arfrever Frehtes Taifersar Arahesis
 
Old 03-20-2012, 07:57 PM
Alexis Ballier
 
Default Fix spurious dep to eselect-python

On Tue, 20 Mar 2012 20:35:17 +0100
Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org> wrote:

> 2012-03-20 17:53:56 Michał Górny napisał(a):
> > On Tue, 20 Mar 2012 17:41:39 +0100
> > Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org> wrote:
> >
> > > 2012-03-20 05:29:20 Luca Barbato napisał(a):
> > > > Hi, I tried to avoid depending on eselect-python if the useflag
> > > > is disabled.
> > > >
> > > > Please test and review.
> > >
> > > The proper fix is to make python.eclass add dependency on
> > > app-admin/eselect-python only when ${CATEGORY}/${PN} is
> > > dev-lang/python, dev-java/jython or dev-python/pypy. See bug
> > > #341037.
> >
> > Couldn't we just push that dependency to the specific ebuilds?
>
> We want to control required version (>=20091230 in case of
> app-admin/eselect-python) in 1 place.
>

this can be achieved by setting a variable that said ebuilds will append
to (R)DEPEND; no need for complex 'if then else' on CAT/PN in an eclass
for this.
 
Old 03-20-2012, 08:05 PM
Alexis Ballier
 
Default Fix spurious dep to eselect-python

On Tue, 20 Mar 2012 17:57:29 -0300
Alexis Ballier <aballier@gentoo.org> wrote:

> On Tue, 20 Mar 2012 20:35:17 +0100
> Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org> wrote:
>
> > 2012-03-20 17:53:56 Michał Górny napisał(a):
> > > On Tue, 20 Mar 2012 17:41:39 +0100
> > > Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org> wrote:
> > >
> > > > 2012-03-20 05:29:20 Luca Barbato napisał(a):
> > > > > Hi, I tried to avoid depending on eselect-python if the
> > > > > useflag is disabled.
> > > > >
> > > > > Please test and review.
> > > >
> > > > The proper fix is to make python.eclass add dependency on
> > > > app-admin/eselect-python only when ${CATEGORY}/${PN} is
> > > > dev-lang/python, dev-java/jython or dev-python/pypy. See bug
> > > > #341037.
> > >
> > > Couldn't we just push that dependency to the specific ebuilds?
> >
> > We want to control required version (>=20091230 in case of
> > app-admin/eselect-python) in 1 place.
> >
>
> this can be achieved by setting a variable that said ebuilds will
> append to (R)DEPEND; no need for complex 'if then else' on CAT/PN in
> an eclass for this.
>

or also by adding a new python-implementation.eclass instead of
polluting an eclass used by hundreds of packages for only 3 special
ones...
 
Old 03-20-2012, 08:09 PM
Arfrever Frehtes Taifersar Arahesis
 
Default Fix spurious dep to eselect-python

2012-03-20 22:05:17 Alexis Ballier napisał(a):
> On Tue, 20 Mar 2012 17:57:29 -0300
> Alexis Ballier <aballier@gentoo.org> wrote:
>
> > On Tue, 20 Mar 2012 20:35:17 +0100
> > Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org> wrote:
> >
> > > 2012-03-20 17:53:56 Michał Górny napisał(a):
> > > > On Tue, 20 Mar 2012 17:41:39 +0100
> > > > Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org> wrote:
> > > >
> > > > > 2012-03-20 05:29:20 Luca Barbato napisał(a):
> > > > > > Hi, I tried to avoid depending on eselect-python if the
> > > > > > useflag is disabled.
> > > > > >
> > > > > > Please test and review.
> > > > >
> > > > > The proper fix is to make python.eclass add dependency on
> > > > > app-admin/eselect-python only when ${CATEGORY}/${PN} is
> > > > > dev-lang/python, dev-java/jython or dev-python/pypy. See bug
> > > > > #341037.
> > > >
> > > > Couldn't we just push that dependency to the specific ebuilds?
> > >
> > > We want to control required version (>=20091230 in case of
> > > app-admin/eselect-python) in 1 place.
> > >
> >
> > this can be achieved by setting a variable that said ebuilds will
> > append to (R)DEPEND; no need for complex 'if then else' on CAT/PN in
> > an eclass for this.
> >
>
> or also by adding a new python-implementation.eclass instead of
> polluting an eclass used by hundreds of packages for only 3 special
> ones...

Some functions in python.eclass need to know if they are called in ebuild of a Python implementation.

--
Arfrever Frehtes Taifersar Arahesis
 
Old 03-20-2012, 08:09 PM
Mike Gilbert
 
Default Fix spurious dep to eselect-python

On Tue, Mar 20, 2012 at 5:05 PM, Alexis Ballier <aballier@gentoo.org> wrote:
> On Tue, 20 Mar 2012 17:57:29 -0300
> Alexis Ballier <aballier@gentoo.org> wrote:
>
>> On Tue, 20 Mar 2012 20:35:17 +0100
>> Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org> wrote:
>>
>> > 2012-03-20 17:53:56 Michał Górny napisał(a):
>> > > On Tue, 20 Mar 2012 17:41:39 +0100
>> > > Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org> wrote:
>> > >
>> > > > 2012-03-20 05:29:20 Luca Barbato napisał(a):
>> > > > > Hi, I tried to avoid depending on eselect-python if the
>> > > > > useflag is disabled.
>> > > > >
>> > > > > Please test and review.
>> > > >
>> > > > The proper fix is to make python.eclass add dependency on
>> > > > app-admin/eselect-python only when ${CATEGORY}/${PN} is
>> > > > dev-lang/python, dev-java/jython or dev-python/pypy. See bug
>> > > > #341037.
>> > >
>> > > Couldn't we just push that dependency to the specific ebuilds?
>> >
>> > We want to control required version (>=20091230 in case of
>> > app-admin/eselect-python) in 1 place.
>> >
>>
>> this can be achieved by setting a variable that said ebuilds will
>> append to (R)DEPEND; no need for complex 'if then else' on CAT/PN in
>> an eclass for this.
>>
>
> or also by adding a new python-implementation.eclass instead of
> polluting an eclass used by hundreds of packages for only 3 special
> ones...
>

A four-way if-then statement is not what I would call "complex". This
was already present in the eclass anyway, so there is no additional
"pollution".

I have already committed Arfrever's patch. If anyone wants to move the
dependency into the ebuilds, you are welcome to do so; patching the
eclass was just faster/easier.
 
Old 03-20-2012, 08:20 PM
Alexis Ballier
 
Default Fix spurious dep to eselect-python

On Tue, 20 Mar 2012 17:09:55 -0400
Mike Gilbert <floppym@gentoo.org> wrote:

> On Tue, Mar 20, 2012 at 5:05 PM, Alexis Ballier <aballier@gentoo.org>
> wrote:
> > On Tue, 20 Mar 2012 17:57:29 -0300
> > Alexis Ballier <aballier@gentoo.org> wrote:
> >
> >> On Tue, 20 Mar 2012 20:35:17 +0100
> >> Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org> wrote:
> >>
> >> > 2012-03-20 17:53:56 Michał Górny napisał(a):
> >> > > On Tue, 20 Mar 2012 17:41:39 +0100
> >> > > Arfrever Frehtes Taifersar Arahesis <arfrever@gentoo.org>
> >> > > wrote:
> >> > >
> >> > > > 2012-03-20 05:29:20 Luca Barbato napisał(a):
> >> > > > > Hi, I tried to avoid depending on eselect-python if the
> >> > > > > useflag is disabled.
> >> > > > >
> >> > > > > Please test and review.
> >> > > >
> >> > > > The proper fix is to make python.eclass add dependency on
> >> > > > app-admin/eselect-python only when ${CATEGORY}/${PN} is
> >> > > > dev-lang/python, dev-java/jython or dev-python/pypy. See bug
> >> > > > #341037.
> >> > >
> >> > > Couldn't we just push that dependency to the specific ebuilds?
> >> >
> >> > We want to control required version (>=20091230 in case of
> >> > app-admin/eselect-python) in 1 place.
> >> >
> >>
> >> this can be achieved by setting a variable that said ebuilds will
> >> append to (R)DEPEND; no need for complex 'if then else' on CAT/PN
> >> in an eclass for this.
> >>
> >
> > or also by adding a new python-implementation.eclass instead of
> > polluting an eclass used by hundreds of packages for only 3 special
> > ones...
> >
>
> A four-way if-then statement is not what I would call "complex". This
> was already present in the eclass anyway, so there is no additional
> "pollution".

a four-way if-then with pattern matching / strcmp is slower than a
constant variable assignment; it could matter when this is done
everytime an ebuild inheriting it is sourced. it probably doesnt, but
imho, its good practice to keep global scope code in eclasses as simple
as possible.
 

Thread Tools




All times are GMT. The time now is 08:15 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org