matplotlib-devel

[matplotlib-devel] basedir

[Benjamin D. <bd...@ub...>]

Hi,

setupext.py contains a basedir directory. Please add there
'linux2-alpha', 'linux2-hppa', 'linux2-mips', 'linux2-sparc',
'gnukfreebsd7', 'gnukfreebsd8'.

It would be nice, if matplotlib had an option to disable '/usr/local'.
This would be useful for distributions.

The Debian patch attached (does both).

Cheers,
Benjamin

Re: [matplotlib-devel] basedir

[Eric F. <ef...@ha...>]

Benjamin Drung wrote:
> Hi,
> 
> setupext.py contains a basedir directory. Please add there
> 'linux2-alpha', 'linux2-hppa', 'linux2-mips', 'linux2-sparc',
> 'gnukfreebsd7', 'gnukfreebsd8'.
> 
> It would be nice, if matplotlib had an option to disable '/usr/local'.
> This would be useful for distributions.
> 
> The Debian patch attached (does both).

I'm not sure I understand this.  The patch seems to remove /usr/local 
from the list.  Why do you want to do this?  I am assuming the original 
idea was that if a user wants to build a version of a library and put it 
in /usr/local, then that is what mpl should use; if it is not found 
there, then use the distro version in /usr.  This seems to me like the 
right strategy, so I am reluctant to apply your patch as-is.

Eric

> 
> Cheers,
> Benjamin
> 
> 
> ------------------------------------------------------------------------
> 
> ------------------------------------------------------------------------------
> Come build with us! The BlackBerry® Developer Conference in SF, CA
> is the only developer event you need to attend this year. Jumpstart your
> developing skills, take BlackBerry mobile applications to market and stay 
> ahead of the curve. Join us from November 9-12, 2009. Register now!
> http://p.sf.net/sfu/devconf
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Matplotlib-devel mailing list
> Mat...@li...
> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

Re: [matplotlib-devel] basedir

[Benjamin D. <bd...@ub...>]

Am Montag, den 05.10.2009, 12:50 -1000 schrieb Eric Firing:
> Benjamin Drung wrote:
> > Hi,
> > 
> > setupext.py contains a basedir directory. Please add there
> > 'linux2-alpha', 'linux2-hppa', 'linux2-mips', 'linux2-sparc',
> > 'gnukfreebsd7', 'gnukfreebsd8'.
> > 
> > It would be nice, if matplotlib had an option to disable '/usr/local'.
> > This would be useful for distributions.
> > 
> > The Debian patch attached (does both).
> 
> I'm not sure I understand this.  The patch seems to remove /usr/local 
> from the list.  Why do you want to do this?  I am assuming the original 
> idea was that if a user wants to build a version of a library and put it 
> in /usr/local, then that is what mpl should use; if it is not found 
> there, then use the distro version in /usr.  This seems to me like the 
> right strategy, so I am reluctant to apply your patch as-is.

Ok, attached a patch which adds support for other architectures.

The Debian/Ubuntu package should be build against the distro versions of
the libraries. The Debian/Ubuntu package should not link to packages
in /usr/local. Therefore we want to disable it for building the package.

It would be nice, if you can add an --disable-usr-local for distro
maintainers. With this option set, '/usr/local' will be stripped from
basedir.

Cheers,
Benjamin

Re: [matplotlib-devel] basedir

[Eric F. <ef...@ha...>]

Benjamin Drung wrote:
> Am Montag, den 05.10.2009, 12:50 -1000 schrieb Eric Firing:
>> Benjamin Drung wrote:
>>> Hi,
>>>
>>> setupext.py contains a basedir directory. Please add there
>>> 'linux2-alpha', 'linux2-hppa', 'linux2-mips', 'linux2-sparc',
>>> 'gnukfreebsd7', 'gnukfreebsd8'.
>>>
>>> It would be nice, if matplotlib had an option to disable '/usr/local'.
>>> This would be useful for distributions.
>>>
>>> The Debian patch attached (does both).
>> I'm not sure I understand this.  The patch seems to remove /usr/local 
>> from the list.  Why do you want to do this?  I am assuming the original 
>> idea was that if a user wants to build a version of a library and put it 
>> in /usr/local, then that is what mpl should use; if it is not found 
>> there, then use the distro version in /usr.  This seems to me like the 
>> right strategy, so I am reluctant to apply your patch as-is.
> 
> Ok, attached a patch which adds support for other architectures.

Applied, thank you.

> 
> The Debian/Ubuntu package should be build against the distro versions of
> the libraries. The Debian/Ubuntu package should not link to packages
> in /usr/local. Therefore we want to disable it for building the package.
> 
> It would be nice, if you can add an --disable-usr-local for distro
> maintainers. With this option set, '/usr/local' will be stripped from
> basedir.

It looks like it would be very easy to add this as an option to be set 
in setup.cfg; is this adequate?

I don't know how to do it easily via a command-line option because those 
options are processed by distutils.core.setup(), so as far as I can see, 
they are not directly available to setup.py or setupext.py, short of 
adding our own layer of command-line processing.

Eric


> 
> Cheers,
> Benjamin
> 
> 
> ------------------------------------------------------------------------
> 
> ------------------------------------------------------------------------------
> Come build with us! The BlackBerry® Developer Conference in SF, CA
> is the only developer event you need to attend this year. Jumpstart your
> developing skills, take BlackBerry mobile applications to market and stay 
> ahead of the curve. Join us from November 9-12, 2009. Register now!
> http://p.sf.net/sfu/devconf
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Matplotlib-devel mailing list
> Mat...@li...
> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

Re: [matplotlib-devel] basedir

[Benjamin D. <bd...@ub...>]

Am Montag, den 05.10.2009, 13:36 -1000 schrieb Eric Firing:
> Benjamin Drung wrote:
> > Am Montag, den 05.10.2009, 12:50 -1000 schrieb Eric Firing:
> >> Benjamin Drung wrote:
> >>> Hi,
> >>>
> >>> setupext.py contains a basedir directory. Please add there
> >>> 'linux2-alpha', 'linux2-hppa', 'linux2-mips', 'linux2-sparc',
> >>> 'gnukfreebsd7', 'gnukfreebsd8'.
> >>>
> >>> It would be nice, if matplotlib had an option to disable '/usr/local'.
> >>> This would be useful for distributions.
> >>>
> >>> The Debian patch attached (does both).
> >> I'm not sure I understand this.  The patch seems to remove /usr/local 
> >> from the list.  Why do you want to do this?  I am assuming the original 
> >> idea was that if a user wants to build a version of a library and put it 
> >> in /usr/local, then that is what mpl should use; if it is not found 
> >> there, then use the distro version in /usr.  This seems to me like the 
> >> right strategy, so I am reluctant to apply your patch as-is.
> > 
> > Ok, attached a patch which adds support for other architectures.
> 
> Applied, thank you.

Thanks.

> > The Debian/Ubuntu package should be build against the distro versions of
> > the libraries. The Debian/Ubuntu package should not link to packages
> > in /usr/local. Therefore we want to disable it for building the package.
> > 
> > It would be nice, if you can add an --disable-usr-local for distro
> > maintainers. With this option set, '/usr/local' will be stripped from
> > basedir.
> 
> It looks like it would be very easy to add this as an option to be set 
> in setup.cfg; is this adequate?

Yes, that's equivalent.

> I don't know how to do it easily via a command-line option because those 
> options are processed by distutils.core.setup(), so as far as I can see, 
> they are not directly available to setup.py or setupext.py, short of 
> adding our own layer of command-line processing.

Implement what you like more.