[Buildroot] [PATCH v2 1/1] package/xerces: add enable network option
Matthew Weber
matthew.weber at rockwellcollins.com
Wed May 20 22:30:34 UTC 2020
Thomas,
On Sat, May 9, 2020 at 3:05 PM Thomas Petazzoni
<thomas.petazzoni at bootlin.com> wrote:
>
> On Mon, 4 May 2020 15:25:58 -0500
> Jared Bents <jared.bents at rockwellcollins.com> wrote:
>
> > Update to add the option to compile xerces with network
> > enabled by default so it can be unselected to compile
> > without network support.
> >
> > Signed-off-by: Jared Bents <jared.bents at rockwellcollins.com>
>
> Could you expand the commit log with an explantion of the *why* we want
> to add this new option? Size impact? If so, what is the size impact?
> Some other reason?
>
> > +ifeq ($(BR2_PACKAGE_XERCES_ENABLE_NETWORK),y)
> > ifeq ($(BR2_PACKAGE_LIBCURL),y)
> > XERCES_CONF_OPTS += -Dnetwork-accessor=curl
> > XERCES_DEPENDENCIES += libcurl
> > else
> > XERCES_CONF_OPTS += -Dnetwork-accessor=socket
> > endif
> > +else
> > +XERCES_CONF_OPTS += -Dnetwork:BOOL=OFF
>
> We don't use this :BOOL type specification anywhere in CMake packages
> in Buildroot, could you keep it consistent with the rest of the package?
v3
http://patchwork.ozlabs.org/project/buildroot/patch/20200520222850.55150-1-matthew.weber@rockwellcollins.com/
Best Regards,
Matt
More information about the buildroot
mailing list