Re: glibc-2.2.2 sys/time.h

[Andreas Jaeger]

address@hidden (Achim Gottinger) writes:

> Hi devs,
> 
> I tried to compile daemontools-0.7 with glibc-2.2.2 and it failed at
> tai64nlocal.c.
> I detected that this file includes sys/time.h because it requires
> localtime.
> locatime is defined in /usr/include/time.h and sys/time.h includes it. I
> changed tai64nlocale to include time.h instead
> of sys/time.h and it worked.
> I compared sys/time.h with the version from 2.2.1 and found that the new
> version defines __need_time.h before
> it includes time.h. If that var is defined time.h does not declare
> localtime.
> Is this a bug or a feature?

time.h and sys/time.h have been changed to follow some more standards
which specify this.

> Another thing. In the install docs stands that perl is used for make
> check if it is avaliable. I tried to compile glibc
> on a environment without perl and it falied at mtrace which is a perl
> script.
> So it whould be good to eighter change the docs or disable the test if
> perl is not installed.

I disabled the test with the appended patch,

Thanks,
Andreas

2001-02-19  Andreas Jaeger  <address@hidden>

        * malloc/Makefile (tests): Run mtrace only when perl is available.
        Reported by Achim Gottinger <address@hidden>.

============================================================
Index: malloc/Makefile
--- malloc/Makefile     2001/01/30 01:44:59     1.36
+++ malloc/Makefile     2001/02/19 12:36:05
@@ -100,9 +100,11 @@
 
 ifeq (no,$(cross-compiling))
 ifeq (yes,$(build-shared))
+ifneq ($(PERL),no)
 tests: $(objpfx)tst-mtrace.out
 $(objpfx)tst-mtrace.out: tst-mtrace.sh $(objpfx)tst-mtrace
        $(SHELL) -e $< $(common-objpfx)
+endif
 endif
 endif
 


-- 
 Andreas Jaeger
  SuSE Labs address@hidden
   private address@hidden
    http://www.suse.de/~aj