On Wed, Nov 11, 2020 at 11:01:57AM +1300, David Rowley wrote: > I'm still working through some small differences in some of the > .vcxproj files. I've been comparing these by copying *.vcxproj out to > another directory with patched and unpatched then diffing the > directory. See attached txt file with those diffs. Here's a summary of > some of them:
Thanks. It would be good to not have those diffs if not necessary. > 1. There are a few places that libpq gets linked where it previously did not. It seems to me that your patch is doing the right thing for adminpack and that its Makefile has no need to include a reference to libpq source path, no? For dblink and postgres_fdw, the duplication comes from PG_CPPFLAGS. It does not matter much in practice, but it would be nice to not have unnecessary data in the project files. One thing that could be done is to make Project.pm more aware of the uniqueness of the elements included. But, do we really need -I$(libpq_srcdir) there anyway? From what I can see, we have all the paths in -I we'd actually need with or without USE_PGXS. > 2. REFINT_VERBOSE gets defined in a few more places than it did > previously. This makes it closer to what happens on Linux anyway, if > you look at the Make output from contrib/spi/Makefile you'll see > -DREFINT_VERBOSE in there for autoinc. Indeed. > 3. LOWER_NODE gets defined in ltree now where it wasn't before. It's > defined on Linux. Unsure why it wasn't before on Windows. Your patch is grabbing the value of PG_CPPFLAGS from ltree's Makefile, which is fine. We may be able to remove this flag and rely on pg_tolower() instead in the long run? I am not sure about FLG_CANLOOKSIGN() though. -- Michael
signature.asc
Description: PGP signature
