On Thu, Jun 7, 2018 at 6:11 PM, Thomas Munro
<thomas.munro@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