[Buildroot] [PATCH 1/1] postgresql --with-libxml configure option added

[Thomas Petazzoni]

Hello,

On Fri, 27 Jul 2018 17:39:57 -0400, Adam Duskett wrote:

> Another nitpick I would have is that there should probably be a
> Config.in entry for this as well.

No, probably not. As a general rule in Buildroot, we prefer automatic
handling of optional dependencies, in order to avoid a proliferation of
gazillion of Config.in options. In addition, postgresql.mk already
handles all its optional dependencies this way.

The only reason to not do this in this case would be if the XML2
support in postgresql increases significantly the size of the
postgresql binary.

But again, in general, automatic handling of optional dependencies is
preferred.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com