Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] strace: fix program_invocation_name uClibc declaration mismatch
@ 2017-09-08  9:23 Baruch Siach
  2017-09-08  9:23 ` [Buildroot] [PATCH 2/2] strace: update homepage link Baruch Siach
  2017-09-09 20:16 ` [Buildroot] [PATCH 1/2] strace: fix program_invocation_name uClibc declaration mismatch Thomas Petazzoni
  0 siblings, 2 replies; 4+ messages in thread
From: Baruch Siach @ 2017-09-08  9:23 UTC (permalink / raw)
  To: buildroot

The local program_invocation_name declaration conflicts with the uClibc
one. Add a patch making this declaration depend on
!HAVE_PROGRAM_INVOCATION_NAME.

Fixes:
http://autobuild.buildroot.net/results/5f0/5f0852f3ffb46f8fb2b4c9318652c5ab3ab5e97d/

Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
 ...s-fix-program_invocation_name-type-confli.patch | 45 ++++++++++++++++++++++
 1 file changed, 45 insertions(+)
 create mode 100644 package/strace/0001-error_prints-fix-program_invocation_name-type-confli.patch

diff --git a/package/strace/0001-error_prints-fix-program_invocation_name-type-confli.patch b/package/strace/0001-error_prints-fix-program_invocation_name-type-confli.patch
new file mode 100644
index 000000000000..b144bdd78c50
--- /dev/null
+++ b/package/strace/0001-error_prints-fix-program_invocation_name-type-confli.patch
@@ -0,0 +1,45 @@
+From 96ad51b91c6ab7ff8bac58f90f199324baed07af Mon Sep 17 00:00:00 2001
+From: Baruch Siach <baruch@tkos.co.il>
+Date: Fri, 8 Sep 2017 12:08:13 +0300
+Subject: [PATCH] error_prints: fix program_invocation_name type conflict
+
+uClibc-ng declares program_invocation_name as const. This causes the
+build failure below:
+
+error_prints.c:40:14: error: conflicting types for ?program_invocation_name?
+ extern char *program_invocation_name;
+              ^~~~~~~~~~~~~~~~~~~~~~~
+In file included from error_prints.c:32:0:
+.../output/host/x86_64-buildroot-linux-uclibc/sysroot/usr/include/errno.h:54:20: note: previous declaration of ?program_invocation_name? was here
+ extern const char *program_invocation_name, *program_invocation_short_name;
+                    ^~~~~~~~~~~~~~~~~~~~~~~
+Makefile:3856: recipe for target 'strace-error_prints.o' failed
+
+Declare program_invocation_name only when the C library does not provide
+one.
+
+Signed-off-by: Baruch Siach <baruch@tkos.co.il>
+---
+Upstream status: sent upstream; waiting for moderator approval to appear at
+https://sourceforge.net/p/strace/mailman/strace-devel/.
+
+ error_prints.c | 2 ++
+ 1 file changed, 2 insertions(+)
+
+diff --git a/error_prints.c b/error_prints.c
+index 8519b9246698..710e99b04ab9 100644
+--- a/error_prints.c
++++ b/error_prints.c
+@@ -37,7 +37,9 @@
+ 
+ #include "error_prints.h"
+ 
++#ifndef HAVE_PROGRAM_INVOCATION_NAME
+ extern char *program_invocation_name;
++#endif
+ 
+ static void
+ verror_msg(int err_no, const char *fmt, va_list p)
+-- 
+2.14.1
+
-- 
2.14.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH 2/2] strace: update homepage link
  2017-09-08  9:23 [Buildroot] [PATCH 1/2] strace: fix program_invocation_name uClibc declaration mismatch Baruch Siach
@ 2017-09-08  9:23 ` Baruch Siach
  2017-09-09 20:17   ` Thomas Petazzoni
  2017-09-09 20:16 ` [Buildroot] [PATCH 1/2] strace: fix program_invocation_name uClibc declaration mismatch Thomas Petazzoni
  1 sibling, 1 reply; 4+ messages in thread
From: Baruch Siach @ 2017-09-08  9:23 UTC (permalink / raw)
  To: buildroot

strace moved to a new homepage as upstream commit 2bba131575878 (Update
homepage URL) indicates.

Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
 package/strace/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/strace/Config.in b/package/strace/Config.in
index 501c9646d6f0..b84b3fd179ea 100644
--- a/package/strace/Config.in
+++ b/package/strace/Config.in
@@ -7,7 +7,7 @@ config BR2_PACKAGE_STRACE
 	  Allows you to track what system calls a program makes
 	  while it is running.
 
-	  http://sourceforge.net/projects/strace/
+	  https://strace.io
 
 comment "strace needs a toolchain w/ headers >= 4.0 on nios2"
 	depends on BR2_nios2
-- 
2.14.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH 1/2] strace: fix program_invocation_name uClibc declaration mismatch
  2017-09-08  9:23 [Buildroot] [PATCH 1/2] strace: fix program_invocation_name uClibc declaration mismatch Baruch Siach
  2017-09-08  9:23 ` [Buildroot] [PATCH 2/2] strace: update homepage link Baruch Siach
@ 2017-09-09 20:16 ` Thomas Petazzoni
  1 sibling, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2017-09-09 20:16 UTC (permalink / raw)
  To: buildroot

Hello,

On Fri,  8 Sep 2017 12:23:48 +0300, Baruch Siach wrote:
> The local program_invocation_name declaration conflicts with the uClibc
> one. Add a patch making this declaration depend on
> !HAVE_PROGRAM_INVOCATION_NAME.
> 
> Fixes:
> http://autobuild.buildroot.net/results/5f0/5f0852f3ffb46f8fb2b4c9318652c5ab3ab5e97d/
> 
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
>  ...s-fix-program_invocation_name-type-confli.patch | 45 ++++++++++++++++++++++
>  1 file changed, 45 insertions(+)
>  create mode 100644 package/strace/0001-error_prints-fix-program_invocation_name-type-confli.patch

Applied to master, thanks. I've updated the upstream status in the
patch, since it has been merged upstream. Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH 2/2] strace: update homepage link
  2017-09-08  9:23 ` [Buildroot] [PATCH 2/2] strace: update homepage link Baruch Siach
@ 2017-09-09 20:17   ` Thomas Petazzoni
  0 siblings, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2017-09-09 20:17 UTC (permalink / raw)
  To: buildroot

Hello,

On Fri,  8 Sep 2017 12:23:49 +0300, Baruch Siach wrote:
> strace moved to a new homepage as upstream commit 2bba131575878 (Update
> homepage URL) indicates.
> 
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
>  package/strace/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-09-09 20:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-08  9:23 [Buildroot] [PATCH 1/2] strace: fix program_invocation_name uClibc declaration mismatch Baruch Siach
2017-09-08  9:23 ` [Buildroot] [PATCH 2/2] strace: update homepage link Baruch Siach
2017-09-09 20:17   ` Thomas Petazzoni
2017-09-09 20:16 ` [Buildroot] [PATCH 1/2] strace: fix program_invocation_name uClibc declaration mismatch Thomas Petazzoni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox