unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#29819] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins.
@ 2017-12-22 22:13 Roel Janssen
  2017-12-23 21:30 ` Danny Milosavljevic
  2018-05-07  9:55 ` Ludovic Courtès
  0 siblings, 2 replies; 7+ messages in thread
From: Roel Janssen @ 2017-12-22 22:13 UTC (permalink / raw)
  To: 29819

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

Dear Guix,

I'd like to squeeze the most performance out of virtuoso, as well as add
some of the VAD packages to extend its functionality.

I attached the patch, but I have a question about the -m64 flag.  My
impression is that this flag will force a 64-bit build.  Since only
64-bit architectures are supported for this package anyway, I think this
should be fine.  Now, my question is: Does this flag have any effect on
the build?

Thanks for your time!

Kind regards,
Roel Janssen


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-virtuoso-ose-Enable-fct-isparql-and-ods-plugins.patch --]
[-- Type: text/x-patch, Size: 1516 bytes --]

From ffab36dc96d7c17ab699b1e5fa641386ae87f681 Mon Sep 17 00:00:00 2001
From: Roel Janssen <roel@gnu.org>
Date: Fri, 22 Dec 2017 23:03:36 +0100
Subject: [PATCH] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins.

* gnu/packages/databases.scm (virtuoso-ose): Enable fct, isparql, and ods
  plugins.  Also compile with optimization level 2, and compile 64-bit only.
---
 gnu/packages/databases.scm | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index a6a7c3178..d802a35c7 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -1962,10 +1962,16 @@ implementation for Python.")
         (base32 "12dqam1gc1v93l0bj0vlpvjqppki6y1hqrlznywxnw0rrz9pb002"))))
     (build-system gnu-build-system)
     (arguments
-     `(#:tests? #f)) ; Tests require a network connection.
+     `(#:tests? #f ; Tests require a network connection.
+       #:configure-flags '("--with-readline"
+                           "--enable-fct-vad"
+                           "--enable-isparql-vad"
+                           "--enable-ods-vad"
+                           "CFLAGS=-O2 -m64")))
     (inputs
      `(("openssl" ,openssl)
-       ("net-tools" ,net-tools)))
+       ("net-tools" ,net-tools)
+       ("readline" ,readline)))
     (home-page "http://vos.openlinksw.com/owiki/wiki/VOS/")
     (synopsis "Multi-model database system")
     (description "Virtuoso is a scalable cross-platform server that combines
-- 
2.15.1


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

* [bug#29819] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins.
  2017-12-22 22:13 [bug#29819] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins Roel Janssen
@ 2017-12-23 21:30 ` Danny Milosavljevic
  2018-01-11  9:44   ` Ludovic Courtès
  2018-05-07  9:55 ` Ludovic Courtès
  1 sibling, 1 reply; 7+ messages in thread
From: Danny Milosavljevic @ 2017-12-23 21:30 UTC (permalink / raw)
  To: Roel Janssen; +Cc: 29819

In my limited understanding "-m64" and "-m32" are for multiarch gccs and we don't support that anyway in Guix gcc, so I don't think it does anything for us.

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

* [bug#29819] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins.
  2017-12-23 21:30 ` Danny Milosavljevic
@ 2018-01-11  9:44   ` Ludovic Courtès
  0 siblings, 0 replies; 7+ messages in thread
From: Ludovic Courtès @ 2018-01-11  9:44 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: 29819

Danny Milosavljevic <dannym@scratchpost.org> skribis:

> In my limited understanding "-m64" and "-m32" are for multiarch gccs and we don't support that anyway in Guix gcc, so I don't think it does anything for us.

Indeed.  Using -m32 on x86_64-linux won't work with our toolchain.

Ludo’.

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

* [bug#29819] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins.
  2017-12-22 22:13 [bug#29819] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins Roel Janssen
  2017-12-23 21:30 ` Danny Milosavljevic
@ 2018-05-07  9:55 ` Ludovic Courtès
  2018-05-07 12:44   ` Roel Janssen
  1 sibling, 1 reply; 7+ messages in thread
From: Ludovic Courtès @ 2018-05-07  9:55 UTC (permalink / raw)
  To: Roel Janssen; +Cc: 29819

Hi Roel,

Roel Janssen <roel@gnu.org> skribis:

>>From ffab36dc96d7c17ab699b1e5fa641386ae87f681 Mon Sep 17 00:00:00 2001
> From: Roel Janssen <roel@gnu.org>
> Date: Fri, 22 Dec 2017 23:03:36 +0100
> Subject: [PATCH] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins.
>
> * gnu/packages/databases.scm (virtuoso-ose): Enable fct, isparql, and ods
>   plugins.  Also compile with optimization level 2, and compile 64-bit only.
> ---
>  gnu/packages/databases.scm | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
> index a6a7c3178..d802a35c7 100644
> --- a/gnu/packages/databases.scm
> +++ b/gnu/packages/databases.scm
> @@ -1962,10 +1962,16 @@ implementation for Python.")
>          (base32 "12dqam1gc1v93l0bj0vlpvjqppki6y1hqrlznywxnw0rrz9pb002"))))
>      (build-system gnu-build-system)
>      (arguments
> -     `(#:tests? #f)) ; Tests require a network connection.
> +     `(#:tests? #f ; Tests require a network connection.
> +       #:configure-flags '("--with-readline"
> +                           "--enable-fct-vad"
> +                           "--enable-isparql-vad"
> +                           "--enable-ods-vad"
> +                           "CFLAGS=-O2 -m64")))
>      (inputs
>       `(("openssl" ,openssl)
> -       ("net-tools" ,net-tools)))
> +       ("net-tools" ,net-tools)
> +       ("readline" ,readline)))

This patch without the CFLAGS bit would be fine, if you still need it.

Thoughts?

Ludo’.

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

* [bug#29819] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins.
  2018-05-07  9:55 ` Ludovic Courtès
@ 2018-05-07 12:44   ` Roel Janssen
  2018-05-07 16:06     ` Ludovic Courtès
  0 siblings, 1 reply; 7+ messages in thread
From: Roel Janssen @ 2018-05-07 12:44 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 29819


Ludovic Courtès <ludo@gnu.org> writes:

> Hi Roel,
>
> Roel Janssen <roel@gnu.org> skribis:
>
>>>From ffab36dc96d7c17ab699b1e5fa641386ae87f681 Mon Sep 17 00:00:00 2001
>> From: Roel Janssen <roel@gnu.org>
>> Date: Fri, 22 Dec 2017 23:03:36 +0100
>> Subject: [PATCH] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins.
>>
>> * gnu/packages/databases.scm (virtuoso-ose): Enable fct, isparql, and ods
>>   plugins.  Also compile with optimization level 2, and compile 64-bit only.
>> ---
>>  gnu/packages/databases.scm | 10 ++++++++--
>>  1 file changed, 8 insertions(+), 2 deletions(-)
>>
>> diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
>> index a6a7c3178..d802a35c7 100644
>> --- a/gnu/packages/databases.scm
>> +++ b/gnu/packages/databases.scm
>> @@ -1962,10 +1962,16 @@ implementation for Python.")
>>          (base32 "12dqam1gc1v93l0bj0vlpvjqppki6y1hqrlznywxnw0rrz9pb002"))))
>>      (build-system gnu-build-system)
>>      (arguments
>> -     `(#:tests? #f)) ; Tests require a network connection.
>> +     `(#:tests? #f ; Tests require a network connection.
>> +       #:configure-flags '("--with-readline"
>> +                           "--enable-fct-vad"
>> +                           "--enable-isparql-vad"
>> +                           "--enable-ods-vad"
>> +                           "CFLAGS=-O2 -m64")))
>>      (inputs
>>       `(("openssl" ,openssl)
>> -       ("net-tools" ,net-tools)))
>> +       ("net-tools" ,net-tools)
>> +       ("readline" ,readline)))
>
> This patch without the CFLAGS bit would be fine, if you still need it.
>
> Thoughts?

In commit 6d09a7b5e I added --with-readline.  I couldn't find the bug
report, but I believe the extra plugins (ODS and facetted browser
(FCT)) cannot be activated due to a bug in Virtuoso's plugin loading
function.  This bug has been resolved, but the fix isn't present in this
released version.

So I guess we can close this one and wait for the next release to
re-evaluate whether we can build these extra plugins.

WDYT?

Kind regards,
Roel Janssen

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

* [bug#29819] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins.
  2018-05-07 12:44   ` Roel Janssen
@ 2018-05-07 16:06     ` Ludovic Courtès
  2018-05-10 20:32       ` bug#29819: " Roel Janssen
  0 siblings, 1 reply; 7+ messages in thread
From: Ludovic Courtès @ 2018-05-07 16:06 UTC (permalink / raw)
  To: Roel Janssen; +Cc: 29819

Roel Janssen <roel@gnu.org> skribis:

> So I guess we can close this one and wait for the next release to
> re-evaluate whether we can build these extra plugins.
>
> WDYT?

Sounds good.  I just wanted to make sure we weren’t forgetting about
it.  :-)

Ludo’.

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

* bug#29819: gnu: virtuoso-ose: Enable fct, isparql, and ods plugins.
  2018-05-07 16:06     ` Ludovic Courtès
@ 2018-05-10 20:32       ` Roel Janssen
  0 siblings, 0 replies; 7+ messages in thread
From: Roel Janssen @ 2018-05-10 20:32 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 29819-done


Ludovic Courtès <ludo@gnu.org> writes:

> Roel Janssen <roel@gnu.org> skribis:
>
>> So I guess we can close this one and wait for the next release to
>> re-evaluate whether we can build these extra plugins.
>>
>> WDYT?
>
> Sounds good.  I just wanted to make sure we weren’t forgetting about
> it.  :-)
>
> Ludo’.

Marked this as done as all *working* changes that I wanted to apply here
have been applied in another patch.

Thanks for bringing this to my attention!

Kind regards,
Roel Janssen

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

end of thread, other threads:[~2018-05-10 20:34 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-22 22:13 [bug#29819] gnu: virtuoso-ose: Enable fct, isparql, and ods plugins Roel Janssen
2017-12-23 21:30 ` Danny Milosavljevic
2018-01-11  9:44   ` Ludovic Courtès
2018-05-07  9:55 ` Ludovic Courtès
2018-05-07 12:44   ` Roel Janssen
2018-05-07 16:06     ` Ludovic Courtès
2018-05-10 20:32       ` bug#29819: " Roel Janssen

Code repositories for project(s) associated with this public inbox

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

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