On Sun, Sep 2, 2012 at 11:29 PM, Andrew Dunstan <and...@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

Best regards,
-- 
Gurjeet Singh

Reply via email to