all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] gnu: clucene: Enable the contrib-libs.
@ 2014-10-06 11:30 John Darrington
  2014-10-06 16:02 ` Ludovic Courtès
  0 siblings, 1 reply; 5+ messages in thread
From: John Darrington @ 2014-10-06 11:30 UTC (permalink / raw)
  To: guix-devel; +Cc: John Darrington

* gnu/packages/rdf.scm (clucene): Enable the contribs-lib directory.
---
 gnu/packages/patches/clucene-contribs-lib.patch |   43 +++++++++++++++++++++++
 gnu/packages/rdf.scm                            |    5 ++-
 2 files changed, 47 insertions(+), 1 deletion(-)
 create mode 100644 gnu/packages/patches/clucene-contribs-lib.patch

diff --git a/gnu/packages/patches/clucene-contribs-lib.patch b/gnu/packages/patches/clucene-contribs-lib.patch
new file mode 100644
index 0000000..be11551
--- /dev/null
+++ b/gnu/packages/patches/clucene-contribs-lib.patch
@@ -0,0 +1,43 @@
+# Source: http://sourceforge.net/p/clucene/bugs/200/
+diff -NaurpBb clucene-core-2.3.3.4/CMakeLists.txt clucene-core-2.3.3.4-mod/CMakeLists.txt
+--- clucene-core-2.3.3.4/CMakeLists.txt	2011-03-17 03:21:07.000000000 +0300
++++ clucene-core-2.3.3.4-mod/CMakeLists.txt	2011-08-16 16:56:55.968268152 +0400
+@@ -163,7 +163,7 @@ IF ( BUILD_CONTRIBS )
+   SET(BUILD_CONTRIBS_LIB 1)
+ ENDIF ( BUILD_CONTRIBS )
+ IF ( BUILD_CONTRIBS_LIB )
+-  ADD_SUBDIRECTORY (src/contribs-lib EXCLUDE_FROM_ALL)
++  ADD_SUBDIRECTORY (src/contribs-lib)
+ ENDIF ( BUILD_CONTRIBS_LIB )
+ 
+ 
+diff -NaurpBb clucene-core-2.3.3.4/src/contribs-lib/CMakeLists.txt clucene-core-2.3.3.4-mod/src/contribs-lib/CMakeLists.txt
+--- clucene-core-2.3.3.4/src/contribs-lib/CMakeLists.txt	2011-03-17 03:21:07.000000000 +0300
++++ clucene-core-2.3.3.4-mod/src/contribs-lib/CMakeLists.txt	2011-08-16 17:14:13.499275499 +0400
+@@ -106,9 +106,26 @@ add_library(clucene-contribs-lib SHARED
+ )
+ TARGET_LINK_LIBRARIES(clucene-contribs-lib ${clucene_contrib_extra_libs})
+ 
++#install public headers.
++FOREACH(file ${HEADERS})
++	get_filename_component(apath ${file} PATH)
++	get_filename_component(aname ${file} NAME)
++	file(RELATIVE_PATH relpath ${CMAKE_SOURCE_DIR}/src/contribs-lib ${apath})
++	IF ( NOT aname MATCHES "^_.*" )
++		install(FILES ${file} 
++		        DESTINATION include/${relpath}
++		        COMPONENT development)
++	ENDIF ( NOT aname MATCHES "^_.*" )
++ENDFOREACH(file)
++
+ #set properties on the libraries
+ SET_TARGET_PROPERTIES(clucene-contribs-lib PROPERTIES
+     VERSION ${CLUCENE_VERSION}
+     SOVERSION ${CLUCENE_SOVERSION}
+     COMPILE_DEFINITIONS_DEBUG _DEBUG
+ )
++
++#and install library
++install(TARGETS clucene-contribs-lib 
++        DESTINATION ${LIB_DESTINATION}  
++        COMPONENT runtime )
diff --git a/gnu/packages/rdf.scm b/gnu/packages/rdf.scm
index 19a68be..e03f3d9 100644
--- a/gnu/packages/rdf.scm
+++ b/gnu/packages/rdf.scm
@@ -81,13 +81,16 @@ HTML and JSON.")
              (sha256
               (base32
                "1arffdwivig88kkx685pldr784njm0249k0rb1f1plwavlrw9zfx"))
-             (patches (list (search-patch "clucene-pkgconfig.patch")))))
+             (patches (list 
+                       (search-patch "clucene-pkgconfig.patch")
+                       (search-patch "clucene-contribs-lib.patch")))))
     (build-system cmake-build-system)
     (inputs
      `(("boost" ,boost) ; could also use bundled copy
        ("zlib" ,zlib)))
     (arguments
      `(#:test-target "cl_test"
+       #:configure-flags '("-DBUILD_CONTRIBS_LIB=ON")
        #:tests? #f)) ; Tests do not compile, as TestIndexSearcher.cpp uses
                      ; undeclared usleep. After fixing this, one needs to run
                      ; "make test" in addition to "make cl_test", then
-- 
1.7.10.4

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

* Re: [PATCH] gnu: clucene: Enable the contrib-libs.
  2014-10-06 11:30 [PATCH] gnu: clucene: Enable the contrib-libs John Darrington
@ 2014-10-06 16:02 ` Ludovic Courtès
  2014-10-06 16:26   ` Andreas Enge
  0 siblings, 1 reply; 5+ messages in thread
From: Ludovic Courtès @ 2014-10-06 16:02 UTC (permalink / raw)
  To: John Darrington; +Cc: guix-devel

John Darrington <jmd@gnu.org> skribis:

> * gnu/packages/rdf.scm (clucene): Enable the contribs-lib directory.

LGTM.  (Andreas is our Clucene expert now ;-), so you could leave him a
one-day delay before committing just in case.)

Ludo’.

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

* Re: [PATCH] gnu: clucene: Enable the contrib-libs.
  2014-10-06 16:02 ` Ludovic Courtès
@ 2014-10-06 16:26   ` Andreas Enge
  2014-10-06 17:21     ` John Darrington
  0 siblings, 1 reply; 5+ messages in thread
From: Andreas Enge @ 2014-10-06 16:26 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel, John Darrington

On Mon, Oct 06, 2014 at 06:02:02PM +0200, Ludovic Courtès wrote:
> LGTM.  (Andreas is our Clucene expert now ;-), so you could leave him a
> one-day delay before committing just in case.)

I would not claim so. Maybe you could add a quick explanation at the top
of the patches on where they are from (copied from another distribution or
added by yourself?), and what they are supposed to do. I trust they solve
a problem you encountered!

Andreas

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

* Re: [PATCH] gnu: clucene: Enable the contrib-libs.
  2014-10-06 16:26   ` Andreas Enge
@ 2014-10-06 17:21     ` John Darrington
  2014-10-06 17:56       ` Andreas Enge
  0 siblings, 1 reply; 5+ messages in thread
From: John Darrington @ 2014-10-06 17:21 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel, John Darrington

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

On Mon, Oct 06, 2014 at 06:26:46PM +0200, Andreas Enge wrote:
     On Mon, Oct 06, 2014 at 06:02:02PM +0200, Ludovic Courtès wrote:
     > LGTM.  (Andreas is our Clucene expert now ;-), so you could leave him a
     > one-day delay before committing just in case.)
     
     I would not claim so. Maybe you could add a quick explanation at the top
     of the patches on where they are from (copied from another distribution or
     added by yourself?), and what they are supposed to do. 

I thought I did that.

J'
     

-- 
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://sks-keyservers.net or any PGP keyserver for public key.


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

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

* Re: [PATCH] gnu: clucene: Enable the contrib-libs.
  2014-10-06 17:21     ` John Darrington
@ 2014-10-06 17:56       ` Andreas Enge
  0 siblings, 0 replies; 5+ messages in thread
From: Andreas Enge @ 2014-10-06 17:56 UTC (permalink / raw)
  To: John Darrington; +Cc: guix-devel, John Darrington

On Mon, Oct 06, 2014 at 07:21:56PM +0200, John Darrington wrote:
> I thought I did that.

Sorry, my mistake!

Andreas

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

end of thread, other threads:[~2014-10-06 17:56 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-06 11:30 [PATCH] gnu: clucene: Enable the contrib-libs John Darrington
2014-10-06 16:02 ` Ludovic Courtès
2014-10-06 16:26   ` Andreas Enge
2014-10-06 17:21     ` John Darrington
2014-10-06 17:56       ` Andreas Enge

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.