On Thu, Jun 7, 2018 at 6:11 PM, Thomas Munro <thomas.mu...@enterprisedb.com> wrote: >> Cool! Is there any reason that your patch for Travis and AppVeyor >> integration is not just committed to master? > > I think that's a good idea and I know that some others are in favour.
One problem is that was discussed at PGCon it commits us to one particular build configuration i.e. one set of --with-whatever options to configure. It's not bad to provide a reasonable set of defaults, but it means that patches which are best tested with some other set of values will have to modify the file (I guess?). Patches that need to be tested with multiple sets of flags are ... maybe just out of luck? I really don't understand the notion of putting the build script inside the source tree. It's all fine if there's One Way To Do It but often TMTOWTDII. If the build configurations are described outside the source tree then you can have as many of them as you need. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company