Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [rfc] Fix header in gdb.gdbtk
@ 2008-02-18 17:24 Markus Deuling
  2008-02-18 17:40 ` Daniel Jacobowitz
  0 siblings, 1 reply; 2+ messages in thread
From: Markus Deuling @ 2008-02-18 17:24 UTC (permalink / raw)
  To: GDB Patches

[-- Attachment #1: Type: text/plain, Size: 398 bytes --]

Hi,

some tests in gdb.gdbtk/ fail on x86 because some headers are missing. This patch
adds those.

Btw, why is there a separate ChangeLog? 

Ok to commit?


ChangeLog:

	* simple.c: Inlude <stdlib.h> and <string.h> for declarations of
	malloc and strncpy.
	* stack1.c: Inlucde <stdlib.h> for a declaration of exit.


-- 
  Markus Deuling
  GNU Toolchain for Linux on Cell BE
  deuling@de.ibm.com


[-- Attachment #2: fix-gdbtk-test --]
[-- Type: text/plain, Size: 721 bytes --]

diff -urpN src/gdb/testsuite/gdb.gdbtk/simple.c dev/gdb/testsuite/gdb.gdbtk/simple.c
--- src/gdb/testsuite/gdb.gdbtk/simple.c	2002-09-10 04:13:16.000000000 +0200
+++ dev/gdb/testsuite/gdb.gdbtk/simple.c	2008-02-18 18:16:04.000000000 +0100
@@ -1,3 +1,6 @@
+#include <stdlib.h>
+#include <string.h>
+
 int
 main(int argc, char * argv[])
 {
diff -urpN src/gdb/testsuite/gdb.gdbtk/stack1.c dev/gdb/testsuite/gdb.gdbtk/stack1.c
--- src/gdb/testsuite/gdb.gdbtk/stack1.c	2000-02-07 01:19:45.000000000 +0100
+++ dev/gdb/testsuite/gdb.gdbtk/stack1.c	2008-02-18 18:13:54.000000000 +0100
@@ -1,3 +1,5 @@
+#include <stdlib.h>
+
 /* Functions defined in other files */
 extern void extern_func1_1 (int a, char *b, unsigned long c);
 

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

* Re: [rfc] Fix header in gdb.gdbtk
  2008-02-18 17:24 [rfc] Fix header in gdb.gdbtk Markus Deuling
@ 2008-02-18 17:40 ` Daniel Jacobowitz
  0 siblings, 0 replies; 2+ messages in thread
From: Daniel Jacobowitz @ 2008-02-18 17:40 UTC (permalink / raw)
  To: Markus Deuling; +Cc: GDB Patches

On Mon, Feb 18, 2008 at 06:21:59PM +0100, Markus Deuling wrote:
> Hi,
>
> some tests in gdb.gdbtk/ fail on x86 because some headers are missing. This patch
> adds those.
>
> Btw, why is there a separate ChangeLog? 

Same reason that you need to send mail to the insight@ list about
this; that directory is part of Insight, which is built on top of
GDB.

-- 
Daniel Jacobowitz
CodeSourcery


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

end of thread, other threads:[~2008-02-18 17:40 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-02-18 17:24 [rfc] Fix header in gdb.gdbtk Markus Deuling
2008-02-18 17:40 ` Daniel Jacobowitz

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