public inbox for newlib@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] newlib: posix: use local includes for local headers
@ 2022-02-17  4:51 Mike Frysinger
  2022-02-18 10:00 ` Corinna Vinschen
  0 siblings, 1 reply; 2+ messages in thread
From: Mike Frysinger @ 2022-02-17  4:51 UTC (permalink / raw)
  To: newlib

These headers aren't installed, so use "" includes instead of <> so
we don't search system header paths.  This matches the style used
elsewhere in the tree for these local headers, and makes it work
w/out explicit -I flags (as needed with non-recursive make).
---
 newlib/libc/posix/collate.c  | 4 ++--
 newlib/libc/posix/rune.h     | 2 +-
 newlib/libc/time/gettzinfo.c | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/newlib/libc/posix/collate.c b/newlib/libc/posix/collate.c
index 8bb04ef0d088..a2f62c978f93 100644
--- a/newlib/libc/posix/collate.c
+++ b/newlib/libc/posix/collate.c
@@ -28,13 +28,13 @@
 #include <sys/cdefs.h>
 
 #include "namespace.h"
-#include <rune.h>
+#include "rune.h"
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <errno.h>
 #include <unistd.h>
-#include <sysexits.h>
+#include "sysexits.h"
 #include "un-namespace.h"
 
 #include "collate.h"
diff --git a/newlib/libc/posix/rune.h b/newlib/libc/posix/rune.h
index 6966d33750be..888e6fb0423b 100644
--- a/newlib/libc/posix/rune.h
+++ b/newlib/libc/posix/rune.h
@@ -36,7 +36,7 @@
 #ifndef	_RUNE_H_
 #define	_RUNE_H_
 
-#include <runetype.h>
+#include "runetype.h"
 #include <stdio.h>
 
 #define	_PATH_LOCALE	"/usr/share/locale"
diff --git a/newlib/libc/time/gettzinfo.c b/newlib/libc/time/gettzinfo.c
index 54c9bc2f0af7..c28e72f0a858 100644
--- a/newlib/libc/time/gettzinfo.c
+++ b/newlib/libc/time/gettzinfo.c
@@ -1,5 +1,5 @@
 #include <sys/types.h>
-#include <local.h>
+#include "local.h"
 
 /* Shared timezone information for libc/time functions.  */
 static __tzinfo_type tzinfo = {1, 0,
-- 
2.34.1


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

* Re: [PATCH] newlib: posix: use local includes for local headers
  2022-02-17  4:51 [PATCH] newlib: posix: use local includes for local headers Mike Frysinger
@ 2022-02-18 10:00 ` Corinna Vinschen
  0 siblings, 0 replies; 2+ messages in thread
From: Corinna Vinschen @ 2022-02-18 10:00 UTC (permalink / raw)
  To: newlib

On Feb 16 23:51, Mike Frysinger wrote:
> These headers aren't installed, so use "" includes instead of <> so
> we don't search system header paths.  This matches the style used
> elsewhere in the tree for these local headers, and makes it work
> w/out explicit -I flags (as needed with non-recursive make).
> ---
>  newlib/libc/posix/collate.c  | 4 ++--
>  newlib/libc/posix/rune.h     | 2 +-
>  newlib/libc/time/gettzinfo.c | 2 +-
>  3 files changed, 4 insertions(+), 4 deletions(-)

GTG

Thx,
Corinna


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

end of thread, other threads:[~2022-02-18 10:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-17  4:51 [PATCH] newlib: posix: use local includes for local headers Mike Frysinger
2022-02-18 10:00 ` Corinna Vinschen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).