[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
wrong.
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:
POLLRDNORM
Equivalent to POLLIN.
So, you need to #define _XOPEN_SOURCE instead.
> -C_PERIPHERY_VERSION = 2.1.0
> +C_PERIPHERY_VERSION = 2.2.0
> C_PERIPHERY_SITE = $(call github,vsergeev,c-periphery,v$(C_PERIPHERY_VERSION))
> C_PERIPHERY_INSTALL_STAGING = YES
> # 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
$(TARGET_DIR).
Could you adjust this and send a new version ?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list