[Buildroot] package/aravis: new package

[Thomas Petazzoni]

Hello,

On Thu, 6 Jan 2022 10:24:53 +0100
Romain Naour <romain.naour at smile.fr> wrote:

> > diff --git a/package/aravis/Config.in b/package/aravis/Config.in
> > new file mode 100644
> > index 0000000000..02c71c44da
> > --- /dev/null
> > +++ b/package/aravis/Config.in
> > @@ -0,0 +1,16 @@
> > +config BR2_PACKAGE_ARAVIS
> > +    bool "aravis - GigE Camera"

Just:

	 bool "aravis"

> > +    depends on BR2_USE_WCHAR
> > +    depends on BR2_TOOLCHAIN_HAS_THREADS_NPTL
> > +    depends on BR2_PACKAGE_PCRE_16
> > +    depends on BR2_PACKAGE_PCRE_32

Please use "select" for pcre.

> > +ARAVIS_LICENSE_FILES = COPYING
> > +ARAVIS_LDFLAGS = $(TARGET_LDFLAGS) $(TARGET_NLS_LIBS)  
> 
> Is gettext a mandatory dependency ?
> At least gettext should be in ARAVIS_DEPENDENCIES when NLS support is enabled.

No, don't add gettext directly, use $(TARGET_NLS_DEPENDENCIES), which
will automatically add (or not) host-gettext and/or gettext to the
dependencies, depending on when it's needed or not.

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com