linux-um archives
 help / color / mirror / Atom feed
* [uml-devel] [PATCH] um: Fix gcc 7 sys/sysmacros.h warning
@ 2017-07-06 20:47 Thomas Meyer
  2017-07-07  9:13 ` Richard Weinberger
  2017-07-07 21:02 ` [uml-devel] [PATCH] um: v2: " Thomas Meyer
  0 siblings, 2 replies; 5+ messages in thread
From: Thomas Meyer @ 2017-07-06 20:47 UTC (permalink / raw)
  To: user-mode-linux-devel

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---
 arch/um/os-Linux/file.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/um/os-Linux/file.c b/arch/um/os-Linux/file.c
index 2db18cbbb0ea..d1901de8ce9f 100644
--- a/arch/um/os-Linux/file.c
+++ b/arch/um/os-Linux/file.c
@@ -14,6 +14,7 @@
 #include <sys/stat.h>
 #include <sys/un.h>
 #include <sys/types.h>
+#include <sys/sysmacros.h>
 #include <os.h>
 
 static void copy_stat(struct uml_stat *dst, const struct stat64 *src)
-- 
2.13.0


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel


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

* Re: [uml-devel] [PATCH] um: Fix gcc 7 sys/sysmacros.h warning
  2017-07-06 20:47 [uml-devel] [PATCH] um: Fix gcc 7 sys/sysmacros.h warning Thomas Meyer
@ 2017-07-07  9:13 ` Richard Weinberger
  2017-07-07 10:01   ` Thomas Meyer
  2017-07-07 21:02 ` [uml-devel] [PATCH] um: v2: " Thomas Meyer
  1 sibling, 1 reply; 5+ messages in thread
From: Richard Weinberger @ 2017-07-07  9:13 UTC (permalink / raw)
  To: Thomas Meyer; +Cc: user-mode-linux-devel@lists.sourceforge.net

On Thu, Jul 6, 2017 at 10:47 PM, Thomas Meyer <thomas@m3y3r.de> wrote:
> Signed-off-by: Thomas Meyer <thomas@m3y3r.de>

When fixing such a problem, always include die compiler warning.
This has to wait for -rc2.

-- 
Thanks,
//richard

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel


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

* Re: [uml-devel] [PATCH] um: Fix gcc 7 sys/sysmacros.h warning
  2017-07-07  9:13 ` Richard Weinberger
@ 2017-07-07 10:01   ` Thomas Meyer
  2017-07-07 10:12     ` Richard Weinberger
  0 siblings, 1 reply; 5+ messages in thread
From: Thomas Meyer @ 2017-07-07 10:01 UTC (permalink / raw)
  To: Richard Weinberger; +Cc: user-mode-linux-devel

[-- Attachment #1: Type: text/html, Size: 57 bytes --]

[-- Attachment #2: Type: text/plain, Size: 202 bytes --]

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot

[-- Attachment #3: Type: text/plain, Size: 194 bytes --]

_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

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

* Re: [uml-devel] [PATCH] um: Fix gcc 7 sys/sysmacros.h warning
  2017-07-07 10:01   ` Thomas Meyer
@ 2017-07-07 10:12     ` Richard Weinberger
  0 siblings, 0 replies; 5+ messages in thread
From: Richard Weinberger @ 2017-07-07 10:12 UTC (permalink / raw)
  To: Thomas Meyer, Richard Weinberger; +Cc: user-mode-linux-devel

Am 07.07.2017 um 12:01 schrieb Thomas Meyer:
> Should I resend with warning as v2?

Yes, please.

Thanks,
//richard

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel


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

* [uml-devel] [PATCH] um: v2: Fix gcc 7 sys/sysmacros.h warning
  2017-07-06 20:47 [uml-devel] [PATCH] um: Fix gcc 7 sys/sysmacros.h warning Thomas Meyer
  2017-07-07  9:13 ` Richard Weinberger
@ 2017-07-07 21:02 ` Thomas Meyer
  1 sibling, 0 replies; 5+ messages in thread
From: Thomas Meyer @ 2017-07-07 21:02 UTC (permalink / raw)
  To: user-mode-linux-devel

Original warning was:

arch/um/os-Linux/file.c: In Funktion »os_major«:
arch/um/os-Linux/file.c:600:13: Warnung: In the GNU C Library, "major" is defined
 by <sys/sysmacros.h>. For historical compatibility, it is
 currently defined by <sys/types.h> as well, but we plan to
 remove this soon. To use "major", include <sys/sysmacros.h>
 directly. If you did not intend to use a system-defined macro
 "major", you should undefine it after including <sys/types.h>.
  return major(dev);

Signed-off-by: Thomas Meyer <thomas@m3y3r.de>
---
 arch/um/os-Linux/file.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/um/os-Linux/file.c b/arch/um/os-Linux/file.c
index 2db18cbbb0ea..d1901de8ce9f 100644
--- a/arch/um/os-Linux/file.c
+++ b/arch/um/os-Linux/file.c
@@ -14,6 +14,7 @@
 #include <sys/stat.h>
 #include <sys/un.h>
 #include <sys/types.h>
+#include <sys/sysmacros.h>
 #include <os.h>
 
 static void copy_stat(struct uml_stat *dst, const struct stat64 *src)
-- 
2.13.0


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

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

end of thread, other threads:[~2017-07-07 21:02 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-07-06 20:47 [uml-devel] [PATCH] um: Fix gcc 7 sys/sysmacros.h warning Thomas Meyer
2017-07-07  9:13 ` Richard Weinberger
2017-07-07 10:01   ` Thomas Meyer
2017-07-07 10:12     ` Richard Weinberger
2017-07-07 21:02 ` [uml-devel] [PATCH] um: v2: " Thomas Meyer

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