Hello,
Barry Scott, on mar. 21 nov. 2017 20:14:10 +0000, wrote:
> patches applied my fedora attached.
> diff --git a/Drivers/Speech/SpeechDispatcher/speech.c
> b/Drivers/Speech/SpeechDispatcher/speech.c
> index d73250a..df92708 100644
> --- a/Drivers/Speech/SpeechDispatcher/speech.c
> +++ b/Drivers/Speech/SpeechDispatcher/speech.c
> @@ -34,7 +34,7 @@ typedef enum {
>
> #include "spk_driver.h"
>
> -#include <libspeechd.h>
> +#include <speech-dispatcher/libspeechd.h>
Indeed, upstream has moved the header. We should however detect which
one is correct, though, to avoid breaking building on older systems.
> diff --git a/Bindings/Java/Native.java b/Bindings/Java/Native.java
> index 8a3b003..c404bf2 100644
> --- a/Bindings/Java/Native.java
> +++ b/Bindings/Java/Native.java
> @@ -21,7 +21,15 @@ package org.a11y.BrlAPI;
>
> public class Native {
> static {
> - System.loadLibrary("brlapi_java");
> + try {
> + System.load( "/usr/lib64/brltty/libbrlapi_java.so" );
> + } catch( UnsatisfiedLinkError e ) {
> + try {
> + System.load( "/usr/lib/brltty/libbrlapi_java.so" );
This, however, rather looks like a packaging issue in Fedora. It should
set JAVA_JNI_DIR to the place where JNI libraries can be loaded, and
that brltty's make install should thus install to.
Hardcoding /usr/libsomething is really not something we want :)
Samuel
_______________________________________________
This message was sent via the BRLTTY mailing list.
To post a message, send an e-mail to: [email protected]
For general information, go to: http://brltty.com/mailman/listinfo/brltty
