basedir

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

10_build_fix.dpatch (1.34 KB)

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
Matplotlib-devel@lists.sourceforge.net
matplotlib-devel List Signup and Options

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

archs.patch (927 Bytes)

···

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.

Benjamin Drung wrote:

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

···

Am Montag, den 05.10.2009, 12:50 -1000 schrieb Eric Firing:

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
Matplotlib-devel@lists.sourceforge.net
matplotlib-devel List Signup and Options

Benjamin Drung wrote:
>> 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.

···

Am Montag, den 05.10.2009, 13:36 -1000 schrieb Eric Firing:

> Am Montag, den 05.10.2009, 12:50 -1000 schrieb Eric Firing: