search

389ds / 389-ds-base

The enterprise-class Open Source LDAP server for Linux
https://www.port389.org/
Other
213 stars 94 forks source link

Issue 6397 - Remove deprecated setting for HR time stamps in logs #6398

Closed mreynolds389 closed 2 weeks ago

mreynolds389 commented 2 weeks ago

Remove the code for checking is CLOCK exists and the logging_hr_timestamps config settings

Also fixed some compiler warnings

relates: https://github.com/389ds/389-ds-base/issues/6397

progier389 commented 2 weeks ago

Looks like all #if(n)def HAVE_CLOCK_GETTIME have been removed except the back-ldbm/vlv.c one. Was it forgotten ?

mreynolds389 commented 2 weeks ago

Looks like all #if(n)def HAVE_CLOCK_GETTIME have been removed except the back-ldbm/vlv.c one. Was it forgotten ?

I don't see HAVE_CLOCK_GETTIME anywhere in the code anymore. I did add some last minute commits - maybe you need to refresh the PR?

mreynolds389 commented 2 weeks ago

Couple of related lines in healthcheck_test were missed:

Fixed

mreynolds389 commented 2 weeks ago

Couple of related lines in healthcheck_test were missed:

Fixed

All the tests as passing now, can you review this @droideck ?