[Buildroot] [PATCH V2 1/2] package/c-periphery: bump version to 2.2.0

Thomas Petazzoni thomas.petazzoni at bootlin.com
Fri May 29 21:06:25 UTC 2020

Hello Joris,

On Fri, 29 May 2020 16:20:12 +0200
Joris Offouga <offougajoris at gmail.com> wrote:

> Since commit "cmake: add cmake build support"
> (https://github.com/vsergeev/c-periphery/commit/952e1e906a5d65b78932128af24b7dbb8cce2e9dvsergeev/c-periphery@d0a973c),
> c-periphery implement cmake build, so change generic-package instead of cmake-package

It's the opposite you're using: you' replacing generic-package by
cmake-package, so the wording here is confusing.

> + #include <unistd.h>
> + #include <fcntl.h>
> ++
> ++#define __USE_XOPEN

Defining a __<something> define from <features.h> is almost always

According to man poll(2):

       When  compiling with _XOPEN_SOURCE defined, one also has the following,
       which convey no further information beyond the bits listed above:

              Equivalent to POLLIN.

So, you need to #define _XOPEN_SOURCE instead.

>  C_PERIPHERY_SITE = $(call github,vsergeev,c-periphery,v$(C_PERIPHERY_VERSION))
>  # only a static library
> @@ -12,16 +12,4 @@ C_PERIPHERY_INSTALL_TARGET = NO

This is not longer true: with the CMake build system, a shared library
is built and installed. So with your change the package is broken: any
application that links against c-periphery will be linked against the
shared library, but the shared library is not installed in

Could you adjust this and send a new version ?


Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering

More information about the buildroot mailing list