public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [GOLD][PATCH] Fix sys/times.h include in timer.cc
@ 2010-05-20 20:07 Viktor Kutuzov
  2010-05-26  0:13 ` Ian Lance Taylor
  0 siblings, 1 reply; 2+ messages in thread
From: Viktor Kutuzov @ 2010-05-20 20:07 UTC (permalink / raw)
  To: Ian Lance Taylor, binutils

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

Hello Ian,

please find attached a patch that fixes an including of sys/times.h when
if HAVE_TIMES is not defined. It prevents some compilation errors on the
system, which don't have this file.

-- 
-Viktor

[-- Attachment #2: binutils-gold-fix-times-include.patch --]
[-- Type: text/x-patch, Size: 374 bytes --]

Index: timer.cc
===================================================================
RCS file: /cvs/src/src/gold/timer.cc,v
retrieving revision 1.1
diff -u -r1.1 timer.cc
--- timer.cc	17 Dec 2009 16:02:03 -0000	1.1
+++ timer.cc	19 May 2010 23:13:39 -0000
@@ -22,7 +22,9 @@
 
 #include "gold.h"
 
+#ifdef HAVE_TIMES
 #include <sys/times.h>
+#endif
 
 #include "libiberty.h"
 

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

* Re: [GOLD][PATCH] Fix sys/times.h include in timer.cc
  2010-05-20 20:07 [GOLD][PATCH] Fix sys/times.h include in timer.cc Viktor Kutuzov
@ 2010-05-26  0:13 ` Ian Lance Taylor
  0 siblings, 0 replies; 2+ messages in thread
From: Ian Lance Taylor @ 2010-05-26  0:13 UTC (permalink / raw)
  To: vkutuzov; +Cc: binutils

Viktor Kutuzov <vkutuzov@accesssoftek.com> writes:

> please find attached a patch that fixes an including of sys/times.h when
> if HAVE_TIMES is not defined. It prevents some compilation errors on the
> system, which don't have this file.

Thanks.  Approved and applied with this ChangeLog entry.

2010-05-25  Viktor Kutuzov  <vkutuzov@accesssoftek.com>

	* timer.cc: Only #include <sys/times.h> if HAVE_TIMES is defined.

Ian

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

end of thread, other threads:[~2010-05-26  0:13 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-05-20 20:07 [GOLD][PATCH] Fix sys/times.h include in timer.cc Viktor Kutuzov
2010-05-26  0:13 ` Ian Lance Taylor

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).