Re: pg_upgrade test mods for Windows/Mingw

On 09/02/2012 11:53 PM, Gurjeet Singh wrote:
> On Sun, Sep 2, 2012 at 11:29 PM, Andrew Dunstan <andrew@dunslane.net 
> <mailto:andrew@dunslane.net>> wrote:
>
>     The attached patch is what I had to do to get pg_upgrade's "make
>     check" to run on Windows under Mingw. Mostly the changes have to
>     do with getting paths right between Windows and MSys, or calling
>     generated .bat files instead of shell scripts.
>
>
> When reading shell script code like this
>
> `uname -a | sed 's/.* //'` = Msys
>
> and
>
> sed -i -e 's,/,\\,g' -e 's,\\s\\q ,/s/q ,' delete_old_cluster.bat 
> 2>/dev/null
>
> I find it easier to understand and maintain if the comments also 
> describe what is the original string format that  this 
> pattern-matching expects, like:
>
> # We expect `uname -a` output like:
> #  Windows_NT4.0 Msys
>
> and
>
> # We expect lines of the format:
> #   abc/xyz/def/
> # and we convert them to
> #  abc\xyz\def
>
>
> BTW, would `uname -o` eliminate the need of pattern matching in the 
> first snippet? The Wikipedia [1] article suggests so.
>
> [1] http://en.wikipedia.org/wiki/Uname
>
>


Yeah it would. This wasn't intended as a final patch anyway, just as 
notice of what I actually had working in case anyone else wanted to try.

cheers

andrew