50 lines
2.1 KiB
Diff
50 lines
2.1 KiB
Diff
|
From c1c2000c35ff39b09cb70fbdf66a107d3b17a674 Mon Sep 17 00:00:00 2001
|
|||
|
|
From: Stephan Bergmann <sbergman@redhat.com>
|
|||
|
|
Date: Wed, 12 Oct 2022 08:40:49 +0200
|
|||
|
|
Subject: [PATCH] Fix missing #include <time.h>
|
|||
|
|
MIME-Version: 1.0
|
|||
|
|
Content-Type: text/plain; charset=UTF-8
|
|||
|
|
Content-Transfer-Encoding: 8bit
|
|||
|
|
|
|||
|
|
At least on recent Fedora 37 beta, building now failed with
|
|||
|
|
|
|||
|
|
> CLucene/document/DateTools.cpp:26:19: error: ‘gmtime’ was not declared in this scope
|
|||
|
|
> 26 | tm *ptm = gmtime(&secs);
|
|||
|
|
> | ^~~~~~
|
|||
|
|
|
|||
|
|
etc.
|
|||
|
|
|
|||
|
|
As it turns out, after 22f9d40320e3deeaa8d6aaa7a770077c20a21dae "git-svn-id:
|
|||
|
|
https://clucene.svn.sourceforge.net/svnroot/clucene/branches/lucene2_3_2@2672
|
|||
|
|
20ef185c-fe11-0410-a618-ba9304b01011" on 2008-06-26 had commented out
|
|||
|
|
_CL_TIME_WITH_SYS_TIME in clucene-config.h.cmake as "not actually used for
|
|||
|
|
anything", then cceccfb52917b5f4da447f1cf20c135952d41442 "Presenting DateTools
|
|||
|
|
and deprecating DateField. DateTools still requires some testing and its own
|
|||
|
|
unit testing" on 2008-06-29 had introduced this use of it (into then
|
|||
|
|
src/CLucene/document/DateTools.H). And apparently most build environments have
|
|||
|
|
silently been happy ever since when the dead leading check for
|
|||
|
|
_CL_TIME_WITH_SYS_TIME didn't include both <sys/time.h> and <time.h>, but the
|
|||
|
|
following check for _CL_HAVE_SYS_TIME_H only included <sys/time.h> but not
|
|||
|
|
<time.h>.
|
|||
|
|
---
|
|||
|
|
src/shared/CLucene/clucene-config.h.cmake | 3 +--
|
|||
|
|
1 file changed, 1 insertion(+), 2 deletions(-)
|
|||
|
|
|
|||
|
|
diff --git a/src/shared/CLucene/clucene-config.h.cmake b/src/shared/CLucene/clucene-config.h.cmake
|
|||
|
|
index bd8683a5..6fe0f92b 100644
|
|||
|
|
--- a/src/shared/CLucene/clucene-config.h.cmake
|
|||
|
|
+++ b/src/shared/CLucene/clucene-config.h.cmake
|
|||
|
|
@@ -100,8 +100,7 @@ ${SYMBOL__T}
|
|||
|
|
//#cmakedefine _CL_STAT_MACROS_BROKEN
|
|||
|
|
|
|||
|
|
/* Define to 1 if you can safely include both <sys/time.h> and <time.h>. */
|
|||
|
|
-//not actually used for anything...
|
|||
|
|
-//#cmakedefine _CL_TIME_WITH_SYS_TIME 1
|
|||
|
|
+#cmakedefine _CL_TIME_WITH_SYS_TIME 1
|
|||
|
|
|
|||
|
|
/* Define that we will be using -fvisibility=hidden, and
|
|||
|
|
* make public classes visible using __attribute__ ((visibility("default")))
|
|||
|
|
--
|
|||
|
|
2.37.3
|
|||
|
|
|