[postgis-devel] PostGIS 1.5 beta2

[Paragon Corporation]

Paul,

This technically is not #311 related, since I haven't been able to make
check PostGIS in 1.5 without hacking the make file.

So I propose we close out #311 and create another ticket for MingW issue
make check.  The #311 was fixed after the last change Mark made.

I'll do all that. 

-----Original Message-----
From: postgis-devel-bounces at postgis.refractions.net
[mailto:postgis-devel-bounces at postgis.refractions.net] On Behalf Of Paul
Ramsey
Sent: Wednesday, January 20, 2010 6:36 PM
To: PostGIS Development Discussion
Subject: Re: [postgis-devel] PostGIS 1.5 beta2

Great, commit it and close ticket #311, noting the revision of your commit.

Thanks!

P

On Wed, Jan 20, 2010 at 12:28 PM, Nicklas Avén <nicklas.aven at jordogskog.no>
wrote:
> Ok, it seems to work :-)
>
>
> new patch attached
>
> But it didn't work at first, and I don't know what I changed ( I 
> downloaded fresh config.ac and regress/makefile.in)
>
> But it works to me now
>
> /Nicklas
>
> 2010-01-20 Mark Cave-Ayland wrote:
>
> Nicklas Avén wrote:
>>
>>> Mark
>>>
>>> After a lot of trial and error I have a patch that works for me in 
>>> MingW
>>> :-)
>>>
>>> first I tried to pass "pwd -W" as is, to makefile.in and let it make 
>>> the path from there. I thought that was what you meant. But I didn't 
>>> success so in the patch the path is made earlier so PWDREGRESS holds 
>>> the path in itself instead of pwd-W Is that ok?
>>>
>>> If someone wants to try it:
>>> you have to run autogen.sh after applying the patch.
>>>
>>>
>>> /Nicklas
>>
>>I don't think we should make use of $(pwd) in this way, since as 
>>implemented by your patch then PWDREGRESS is evaluated to the current 
>>path at configure time, rather than at runtime which I suspect will 
>>cause some confusion further down the line.
>>
>>The configure.ac part is mostly correct, but I would expect it to look 
>>like this, as per the original version:
>>
>>dnl
>>dnl MingW requires use of pwd -W to give proper Windows (not MingW) 
>>paths dnl for in-place regression tests dnl
>>
>>case $host_os in
>> *mingw*)
>> PWDREGRESS="pwd -W"
>> ;;
>> *)
>> PWDREGRESS="pwd"
>> ;;
>>esac
>>
>>
>>AC_DEFINE_UNQUOTED([PWDREGRESS])
>>AC_SUBST([PWDREGRESS])
>>
>>After this, the changes to regress/Makefile.in should then be 
>>reasonably simple, e.g.:
>>
>># Where we put our regression installation srcdir=$(shell 
>>@PWDREGRESS@) REGRESS_INSTALLDIR=$(srcdir)/00-regress-install
>>
>>
>>HTH,
>>
>>Mark.
>>
>>--
>>Mark Cave-Ayland - Senior Technical Architect PostgreSQL - PostGIS 
>>Sirius Corporation plc - control through freedom 
>>http://www.siriusit.co.uk
>>t: +44 870 608 0063
>>
>>Sirius Labs: http://www.siriusit.co.uk/labs 
>>_______________________________________________
>>postgis-devel mailing list
>>postgis-devel at postgis.refractions.net
>>http://postgis.refractions.net/mailman/listinfo/postgis-devel
>>
>>
> _______________________________________________
> postgis-devel mailing list
> postgis-devel at postgis.refractions.net
> http://postgis.refractions.net/mailman/listinfo/postgis-devel
>
>
_______________________________________________
postgis-devel mailing list
postgis-devel at postgis.refractions.net
http://postgis.refractions.net/mailman/listinfo/postgis-devel