All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/c-periphery: Bump version to 2.1.0
@ 2020-01-08 19:27 Joris Offouga
  2020-01-08 19:58 ` Ryan Barnett
  2020-01-08 19:59 ` Pierre-Jean Texier
  0 siblings, 2 replies; 5+ messages in thread
From: Joris Offouga @ 2020-01-08 19:27 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Joris Offouga <offougajoris@gmail.com>
---
 package/c-periphery/c-periphery.hash | 4 ++--
 package/c-periphery/c-periphery.mk   | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/package/c-periphery/c-periphery.hash b/package/c-periphery/c-periphery.hash
index 8f3bd8a3da..244f8f60f1 100644
--- a/package/c-periphery/c-periphery.hash
+++ b/package/c-periphery/c-periphery.hash
@@ -1,3 +1,3 @@
 # Locally calculated
-sha256 50665a4f298b30e4cc8f7dbd38362fd860cc3746803d00325cb04806674e3166  c-periphery-1.1.3.tar.gz
-sha256 6b5669a90260126f0600aae98cb10fb5e219269dc9cbd0137f63927b96d5c31d  LICENSE
+sha256 1c455c42b26818d199ac227cd19714f265623e15d363fdfdf47a2416a4165654  c-periphery-2.1.0.tar.gz
+sha256 e7b9464cb8511f19ddb40f9cf52fe86b3dfe38b2faed9d7bbc3d5707677ffbe3  LICENSE
diff --git a/package/c-periphery/c-periphery.mk b/package/c-periphery/c-periphery.mk
index e62d3e05ca..bb50434efa 100644
--- a/package/c-periphery/c-periphery.mk
+++ b/package/c-periphery/c-periphery.mk
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-C_PERIPHERY_VERSION = 1.1.3
+C_PERIPHERY_VERSION = 2.1.0
 C_PERIPHERY_SITE = $(call github,vsergeev,c-periphery,v$(C_PERIPHERY_VERSION))
 C_PERIPHERY_INSTALL_STAGING = YES
 # only a static library
-- 
2.20.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] package/c-periphery: Bump version to 2.1.0
  2020-01-08 19:27 [Buildroot] [PATCH] package/c-periphery: Bump version to 2.1.0 Joris Offouga
@ 2020-01-08 19:58 ` Ryan Barnett
  2020-01-08 20:35   ` Thomas Petazzoni
  2020-01-08 19:59 ` Pierre-Jean Texier
  1 sibling, 1 reply; 5+ messages in thread
From: Ryan Barnett @ 2020-01-08 19:58 UTC (permalink / raw)
  To: buildroot

Joris,

Is there any need to update the build environment to support the new
cross compiling method that has been incorporated?

https://github.com/vsergeev/c-periphery/commit/742d983e76546d4253df6e89604d7ebaa2507de0

Thanks,
-Ryan

On Wed, Jan 8, 2020 at 1:33 PM Joris Offouga <offougajoris@gmail.com> wrote:
>
> Signed-off-by: Joris Offouga <offougajoris@gmail.com>
> ---
>  package/c-periphery/c-periphery.hash | 4 ++--
>  package/c-periphery/c-periphery.mk   | 2 +-
>  2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/package/c-periphery/c-periphery.hash b/package/c-periphery/c-periphery.hash
> index 8f3bd8a3da..244f8f60f1 100644
> --- a/package/c-periphery/c-periphery.hash
> +++ b/package/c-periphery/c-periphery.hash
> @@ -1,3 +1,3 @@
>  # Locally calculated
> -sha256 50665a4f298b30e4cc8f7dbd38362fd860cc3746803d00325cb04806674e3166  c-periphery-1.1.3.tar.gz
> -sha256 6b5669a90260126f0600aae98cb10fb5e219269dc9cbd0137f63927b96d5c31d  LICENSE
> +sha256 1c455c42b26818d199ac227cd19714f265623e15d363fdfdf47a2416a4165654  c-periphery-2.1.0.tar.gz
> +sha256 e7b9464cb8511f19ddb40f9cf52fe86b3dfe38b2faed9d7bbc3d5707677ffbe3  LICENSE
> diff --git a/package/c-periphery/c-periphery.mk b/package/c-periphery/c-periphery.mk
> index e62d3e05ca..bb50434efa 100644
> --- a/package/c-periphery/c-periphery.mk
> +++ b/package/c-periphery/c-periphery.mk
> @@ -4,7 +4,7 @@
>  #
>  ################################################################################
>
> -C_PERIPHERY_VERSION = 1.1.3
> +C_PERIPHERY_VERSION = 2.1.0
>  C_PERIPHERY_SITE = $(call github,vsergeev,c-periphery,v$(C_PERIPHERY_VERSION))
>  C_PERIPHERY_INSTALL_STAGING = YES
>  # only a static library

^ permalink raw reply	[flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] package/c-periphery: Bump version to 2.1.0
  2020-01-08 19:27 [Buildroot] [PATCH] package/c-periphery: Bump version to 2.1.0 Joris Offouga
  2020-01-08 19:58 ` Ryan Barnett
@ 2020-01-08 19:59 ` Pierre-Jean Texier
  2020-01-13 23:03   ` Arnout Vandecappelle
  1 sibling, 1 reply; 5+ messages in thread
From: Pierre-Jean Texier @ 2020-01-08 19:59 UTC (permalink / raw)
  To: buildroot

Hi Joris,

Le 08/01/2020 ? 20:27, Joris Offouga a ?crit?:
> Signed-off-by: Joris Offouga <offougajoris@gmail.com>
> ---
>   package/c-periphery/c-periphery.hash | 4 ++--
>   package/c-periphery/c-periphery.mk   | 2 +-
>   2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/package/c-periphery/c-periphery.hash b/package/c-periphery/c-periphery.hash
> index 8f3bd8a3da..244f8f60f1 100644
> --- a/package/c-periphery/c-periphery.hash
> +++ b/package/c-periphery/c-periphery.hash
> @@ -1,3 +1,3 @@
>   # Locally calculated
> -sha256 50665a4f298b30e4cc8f7dbd38362fd860cc3746803d00325cb04806674e3166  c-periphery-1.1.3.tar.gz
> -sha256 6b5669a90260126f0600aae98cb10fb5e219269dc9cbd0137f63927b96d5c31d  LICENSE
> +sha256 1c455c42b26818d199ac227cd19714f265623e15d363fdfdf47a2416a4165654  c-periphery-2.1.0.tar.gz
> +sha256 e7b9464cb8511f19ddb40f9cf52fe86b3dfe38b2faed9d7bbc3d5707677ffbe3  LICENSE
> diff --git a/package/c-periphery/c-periphery.mk b/package/c-periphery/c-periphery.mk
> index e62d3e05ca..bb50434efa 100644
> --- a/package/c-periphery/c-periphery.mk
> +++ b/package/c-periphery/c-periphery.mk
> @@ -4,7 +4,7 @@
>   #
>   ################################################################################
>   
> -C_PERIPHERY_VERSION = 1.1.3
> +C_PERIPHERY_VERSION = 2.1.0
>   C_PERIPHERY_SITE = $(call github,vsergeev,c-periphery,v$(C_PERIPHERY_VERSION))
>   C_PERIPHERY_INSTALL_STAGING = YES
>   # only a static library
> 

I just tested your patch and I have the following
issues:

                              br-arm-full [1/6]: OK
                   br-arm-cortex-a9-glibc [2/6]: OK
                    br-arm-cortex-m4-full [3/6]: OK
                           br-x86-64-musl [4/6]: OK
                       br-arm-full-static [5/6]: FAILED
                             sourcery-arm [6/6]: FAILED
6 builds, 0 skipped, 2 build failed, 0 legal-info failed

The output log is:

src/gpio.c:21:24: fatal error: linux/gpio.h: No such file or directory
  #include <linux/gpio.h>

Thanks!
-- 
Pierre-Jean Texier
Embedded Linux Engineer
https://koncepto.io

^ permalink raw reply	[flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] package/c-periphery: Bump version to 2.1.0
  2020-01-08 19:58 ` Ryan Barnett
@ 2020-01-08 20:35   ` Thomas Petazzoni
  0 siblings, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2020-01-08 20:35 UTC (permalink / raw)
  To: buildroot

On Wed, 8 Jan 2020 13:58:00 -0600
Ryan Barnett <ryan.barnett@rockwellcollins.com> wrote:

> Joris,
> 
> Is there any need to update the build environment to support the new
> cross compiling method that has been incorporated?
> 
> https://github.com/vsergeev/c-periphery/commit/742d983e76546d4253df6e89604d7ebaa2507de0

This just add a $(CROSS) variable in front of $(CC), which is not so
useful in the context of Buildroot, because for target packages, we
anyway already pass CC=$(TARGET_CC).

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

^ permalink raw reply	[flat|nested] 5+ messages in thread
* [Buildroot] [PATCH] package/c-periphery: Bump version to 2.1.0
  2020-01-08 19:59 ` Pierre-Jean Texier
@ 2020-01-13 23:03   ` Arnout Vandecappelle
  0 siblings, 0 replies; 5+ messages in thread
From: Arnout Vandecappelle @ 2020-01-13 23:03 UTC (permalink / raw)
  To: buildroot



On 08/01/2020 20:59, Pierre-Jean Texier wrote:
> Hi Joris,
> 
> Le 08/01/2020 ? 20:27, Joris Offouga a ?crit?:
>> Signed-off-by: Joris Offouga <offougajoris@gmail.com>
>> ---
>> ? package/c-periphery/c-periphery.hash | 4 ++--
>> ? package/c-periphery/c-periphery.mk?? | 2 +-
>> ? 2 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/package/c-periphery/c-periphery.hash
>> b/package/c-periphery/c-periphery.hash
>> index 8f3bd8a3da..244f8f60f1 100644
>> --- a/package/c-periphery/c-periphery.hash
>> +++ b/package/c-periphery/c-periphery.hash
>> @@ -1,3 +1,3 @@
>> ? # Locally calculated
>> -sha256 50665a4f298b30e4cc8f7dbd38362fd860cc3746803d00325cb04806674e3166?
>> c-periphery-1.1.3.tar.gz
>> -sha256 6b5669a90260126f0600aae98cb10fb5e219269dc9cbd0137f63927b96d5c31d? LICENSE
>> +sha256 1c455c42b26818d199ac227cd19714f265623e15d363fdfdf47a2416a4165654?
>> c-periphery-2.1.0.tar.gz
>> +sha256 e7b9464cb8511f19ddb40f9cf52fe86b3dfe38b2faed9d7bbc3d5707677ffbe3? LICENSE
>> diff --git a/package/c-periphery/c-periphery.mk
>> b/package/c-periphery/c-periphery.mk
>> index e62d3e05ca..bb50434efa 100644
>> --- a/package/c-periphery/c-periphery.mk
>> +++ b/package/c-periphery/c-periphery.mk
>> @@ -4,7 +4,7 @@
>> ? #
>> ?
>> ################################################################################
>> ? -C_PERIPHERY_VERSION = 1.1.3
>> +C_PERIPHERY_VERSION = 2.1.0
>> ? C_PERIPHERY_SITE = $(call github,vsergeev,c-periphery,v$(C_PERIPHERY_VERSION))
>> ? C_PERIPHERY_INSTALL_STAGING = YES
>> ? # only a static library
>>
> 
> I just tested your patch and I have the following
> issues:
> 
> ???????????????????????????? br-arm-full [1/6]: OK
> ????????????????? br-arm-cortex-a9-glibc [2/6]: OK
> ?????????????????? br-arm-cortex-m4-full [3/6]: OK
> ????????????????????????? br-x86-64-musl [4/6]: OK
> ????????????????????? br-arm-full-static [5/6]: FAILED
> ??????????????????????????? sourcery-arm [6/6]: FAILED
> 6 builds, 0 skipped, 2 build failed, 0 legal-info failed
> 
> The output log is:
> 
> src/gpio.c:21:24: fatal error: linux/gpio.h: No such file or directory
> ?#include <linux/gpio.h>

 That header was introduced in 4.6, and the failing toolchains have headers 4.4
and 3.13 respectively. So c-periphery should either get a dependency on
BR2_TOOLCHAIN_HEADERS_AT_LEAST_4_6, or it should be patched to revert to the
sysfs API for older kernels.


 Regards,
 Arnout

^ permalink raw reply	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-01-13 23:03 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-01-08 19:27 [Buildroot] [PATCH] package/c-periphery: Bump version to 2.1.0 Joris Offouga
2020-01-08 19:58 ` Ryan Barnett
2020-01-08 20:35   ` Thomas Petazzoni
2020-01-08 19:59 ` Pierre-Jean Texier
2020-01-13 23:03   ` Arnout Vandecappelle

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.