On 02.06.21 23:27, Andrew Dunstan wrote:
<short-version-comments.patch>
It's not the fault of your patch, but the docs seem to be misleading in ways 
that your comments don't fix.  (Or perhaps my Windows knowledge is just too 
lacking to realize why these instructions are ok?)

Prior to where your patch makes changes, the docs say, "If you are building PostgreSQL for 
Microsoft Windows, read this chapter if you intend to build with MinGW or Cygwin".  I think 
going on to tell the users to use `su` is a bit odd.  Does that exist in standard MinGW and Cygwin 
environments?  I thought "run as" was the Windows option for this.


Yes, good point. We should fix that. Yes, "runas" is a sort of su.
There's no adduser either.

I think those instructions were written before "sudo" became common. Maybe we should update them a bit.


Reply via email to