Thanks for the patch. A couple of thoughts/questions:
(1) Notice the patch uses/assumes pkg-config is present
which is not always the case. Have you tried adding
PkgConfig as a dependency and using that directly
from the Makefile.PL? That should work for SPP and
a pre-configured ASperl.
(2) Re LDLOADLIBS, is that required for MSVC builds
and so may need to be conditionalized?
(1) Notice the patch uses/assumes pkg-config is present
which is not always the case. Have you tried adding
PkgConfig as a dependency and using that directly
from the Makefile.PL? That should work for SPP and
a pre-configured ASperl.
With the new strawberry perl 5.20.x all these methods will work:
- pkg-config command
- PkgConfig module
-ExtUtils::PkgConfig module
I am not sure whether it is a good idea to make PkgConfig a dependency but
we can do something like this:
my $libs;
if (eval {require PkgConfig}) {
$libs = PkgConfig->find('lapack')->get_ldflags;
}
if (!$libs && eval {require ExtUtils::PkgConfig}) {
$libs = ExtUtils::PkgConfig->libs('lapack');
}
if (!$libs && pkg-config lapack --modversion}) {
$libs = pkg-config lapack --libs;
}
if (!$libs) {
$libs = '-llapack -lblas -lgfortran'
}
(2) Re LDLOADLIBS, is that required for MSVC builds
and so may need to be conditionalized?
I am not sure but EU::MM throws some warnings saying that it does not know
about LDLOADLIBS
--
kmx
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Status: open Group: Labels: LinearAlgebra Created: Wed May 21, 2014 09:35 AM UTC by kmx Last Updated: Thu Dec 25, 2014 10:28 PM UTC Owner: Chris Marshall
Hi,
please find enclosed patch with modified lapack&blas lib detection in a
way that works with straberry perl PDL/portable.
Status: open Group: Labels: LinearAlgebra Created: Wed May 21, 2014 09:35 AM UTC by kmx Last Updated: Thu Dec 25, 2014 10:28 PM UTC Owner: Chris Marshall
Hi,
please find enclosed patch with modified lapack&blas lib detection in a
way that works with straberry perl PDL/portable.
Status: open Group: Labels: LinearAlgebra Created: Wed May 21, 2014 09:35 AM UTC by kmx Last Updated: Thu Dec 25, 2014 10:28 PM UTC Owner: Chris Marshall
Hi,
please find enclosed patch with modified lapack&blas lib detection in a
way that works with straberry perl PDL/portable.
Thanks for the patch. A couple of thoughts/questions:
(1) Notice the patch uses/assumes pkg-config is present
which is not always the case. Have you tried adding
PkgConfig as a dependency and using that directly
from the Makefile.PL? That should work for SPP and
a pre-configured ASperl.
(2) Re LDLOADLIBS, is that required for MSVC builds
and so may need to be conditionalized?
--Chris
On Wed, May 21, 2014 at 5:35 AM, kmx kmx1@users.sf.net wrote:
Related
Patches: #77
With the new strawberry perl 5.20.x all these methods will work:
- pkg-config command
- PkgConfig module
-ExtUtils::PkgConfig module
I am not sure whether it is a good idea to make PkgConfig a dependency but
we can do something like this:
my $libs;
if (eval {require PkgConfig}) {
$libs = PkgConfig->find('lapack')->get_ldflags;
}
if (!$libs && eval {require ExtUtils::PkgConfig}) {
$libs = ExtUtils::PkgConfig->libs('lapack');
}
if (!$libs &&
pkg-config lapack --modversion}) {$libs =
pkg-config lapack --libs;}
if (!$libs) {
$libs = '-llapack -lblas -lgfortran'
}
I am not sure but EU::MM throws some warnings saying that it does not know
about LDLOADLIBS
--
kmx
Or more compressed form:
--
kmx
Last edit: kmx 2014-05-21
Here is updated patch addressing both (1) and (2)
ping
Applied to git and soon to appear in CHM/PDL-LinearAlgebra-0.08_03.tar.gz on CPAN.
I confirm PDL-LinearAlgebra-0.08_03.tar.gz works, you can close this ticket.
Chris,
you probably forgot to "git push" the latest changes to
PDL::LinearAlgebra's repo
--
kmx
Thanks for the info. I'll update tomorrow.
On Sun, Jan 4, 2015 at 8:49 AM, kmx kmx1@users.sf.net wrote:
Related
Patches: #77
Pushed.
On Sun, Jan 4, 2015 at 12:50 PM, Chris Marshall marshallch@users.sf.net
wrote:
Related
Patches: #77