Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fails on make libs at libbluray-1.1.2 #155

Open
sgreene820 opened this issue May 9, 2023 · 13 comments
Open

Fails on make libs at libbluray-1.1.2 #155

sgreene820 opened this issue May 9, 2023 · 13 comments

Comments

@sgreene820
Copy link

Attempting to build the android client, 0.33. It fails on building libbluray-1.1.2 without giving a meaningful hint (that I can see).

libtool: compile: /home/sgreene/Android/android-ndk/toolchains/llvm/prebuilt/linux-x86_64/bin/armv7a-linux-androideabi29-clang -DHAVE_CONFIG_H -I. -D_ISOC99_SOURCE -D_POSIX_C_SOURCE=200809L -D_REENTRANT -I./src -I./src/libbluray -I./src/libbluray -I/home/sgreene/MythAndroid/packaging/android/libs/libbluray-1.1.2/jni -I/home/sgreene/MythAndroid/packaging/android/libs/libbluray-1.1.2/jni/linux -I./contrib/libudfread/src/ -I/home/sgreene/MythAndroid/packaging/android/libsinstall/include/libxml2 -I/home/sgreene/MythAndroid/packaging/android/libsinstall/include/freetype2 -std=c99 -isysroot /home/sgreene/Android/android-ndk/toolchains/llvm/prebuilt/linux-x86_64/sysroot -isystem /home/sgreene/MythAndroid/packaging/android/libsinstall/include -D__ANDROID_API__=29 -Wall -Wdisabled-optimization -Wpointer-arith -Wredundant-decls -Wcast-qual -Wwrite-strings -Wtype-limits -Wundef -Wmissing-prototypes -Wshadow -Werror=implicit-function-declaration -Wextra -Winline -O3 -fomit-frame-pointer -MT contrib/libudfread/src/libbluray_la-udfread.lo -MD -MP -MF contrib/libudfread/src/.deps/libbluray_la-udfread.Tpo -c contrib/libudfread/src/udfread.c -o contrib/libudfread/src/libbluray_la-udfread.o >/dev/null 2>&1
mv -f src/libbluray/hdmv/.deps/libbluray_la-hdmv_vm.Tpo src/libbluray/hdmv/.deps/libbluray_la-hdmv_vm.Plo
mv -f src/libbluray/decoders/.deps/libbluray_la-graphics_controller.Tpo src/libbluray/decoders/.deps/libbluray_la-graphics_controller.Plo
mv -f contrib/libudfread/src/.deps/libbluray_la-udfread.Tpo contrib/libudfread/src/.deps/libbluray_la-udfread.Plo
mv -f src/libbluray/.deps/libbluray_la-bluray.Tpo src/libbluray/.deps/libbluray_la-bluray.Plo
make[2]: Leaving directory '/home/sgreene/MythAndroid/packaging/android/libs/libbluray-1.1.2'
make[1]: *** [Makefile:1032: all] Error 2
make[1]: Leaving directory '/home/sgreene/MythAndroid/packaging/android/libs/libbluray-1.1.2'
~/MythAndroid/packaging/android/libs ~/MythAndroid/packaging/android
make: *** [Makefile:53: libs] Error 2
@nietu
Copy link

nietu commented Dec 30, 2023

Had you find solution for this? I had same issue when trying to build mythtv for Nvidia shield android.
In my setup, issue is in javac version. I had tested build w/ openjdk javac 11, 13

in 11 it failed due this:

/src/libbluray/bdj/java-j2se/java/io/BDFileSystemImpl.java:21: error: BDFileSystemImpl is not abstract and does not override abstract method isInvalid(File) in FileSystem
    [javac] class BDFileSystemImpl extends BDFileSystem {
    [javac] ^
    [javac] Note: Some input files use or override a deprecated API.
    [javac] Note: Recompile with -Xlint:deprecation for details.
    [javac] Note: Some input files use unchecked or unsafe operations.
    [javac] Note: Recompile with -Xlint:unchecked for details.
    [javac] 1 error
    [javac] 4 warnings

BUILD FAILED
/home/nietu/projects/mythtv/packaging/android/libs64/libbluray-1.1.2/src/libbluray/bdj/build.xml:32: Compile failed; see the compiler error output for details.

in 13 it failed due this:

libtool: compile:  /home/nietu/Android/android-ndk/toolchains/llvm/prebuilt/linux-x86_64/bin/aarch64-linux-android29-clang -DHAVE_CONFIG_H -I. -D_ISOC99_SOURCE -D_POSIX_C_SOURCE=200809L -D_REENTRANT -I./src -I./src/libbluray -I./src/libbluray -I/home/nietu/projects/mythtv/packaging/android/libs64/libbluray-1.1.2/jni -I/home/nietu/projects/mythtv/packaging/android/libs64/libbluray-1.1.2/jni/linux -I./contrib/libudfread/src/ -I/home/nietu/projects/mythtv/packaging/android/libsinstall64/include/libxml2 -I/home/nietu/projects/mythtv/packaging/android/libsinstall64/include/freetype2 -std=c99 -isysroot /home/nietu/Android/android-ndk/toolchains/llvm/prebuilt/linux-x86_64/sysroot -isystem /home/nietu/projects/mythtv/packaging/android/libsinstall64/include -D__ANDROID_API__=29 -Wall -Wdisabled-optimization -Wpointer-arith -Wredundant-decls -Wcast-qual -Wwrite-strings -Wtype-limits -Wundef -Wmissing-prototypes -Wshadow -Werror=implicit-function-declaration -Wextra -Winline -O3 -fomit-frame-pointer -MT src/libbluray/bd    [javac] warning: [options] bootstrap class path not set in conjunction with -source 6
j/native/libbluray_la-org_videolan_Logger.lo -MD -MP -MF src/libbluray/bdj/native/.deps/libbluray_la-org_videolan_Logger.Tpo -c src/libbluray/bdj/native/org_videolan_Logger.c  -fPIC -DPIC -o src/libbluray/bdj/native/.libs/libbluray_la-org_videolan_Logger.o
    [javac] error: Source option 6 is no longer supported. Use 7 or later.
    [javac] error: Target option 6 is no longer supported. Use 7 or later.
libtool: compile:  /home/nietu/Android/android-ndk/toolchains/llvm/prebuilt/linux-x86_64/bin/aarch64-linux-android29-clang -DHAVE_CONFIG_H -I. -D_ISOC99_SOURCE -D_POSIX_C_SOURCE=200809L -D_REENTRANT -I./src -I./src/libbluray -I./src/libbluray -I/home/nietu/projects/mythtv/packaging/android/libs64/libbluray-1.1.2/jni -I/home/nietu/projects/mythtv/packaging/android/libs64/libbluray-1.1.2/jni/linux -I./contrib/libudfread/src/ -I/home/nietu/projects/mythtv/packaging/android/libsinstall64/include/libxml2 -I/home/nietu/projects/mythtv/packaging/android/libsinstall64/include/freetype2 -std=c99 -isysroot /home/nietu/Android/android-ndk/toolchains/llvm/prebuilt/linux-x86_64/sysroot -isystem /home/nietu/projects/mythtv/packaging/android/libsinstall64/include -D__ANDROID_API__=29 -Wall -Wdisabled-optimization -Wpointer-arith -Wredundant-decls -Wcast-qual -Wwrite-strings -Wtype-limits -Wundef -Wmissing-prototypes -Wshadow -Werror=implicit-function-declaration -Wextra -Winline -O3 -fomit-frame-pointer -MT src/libbluray/bdj/native/libbluray_la-register_native.lo -MD -MP -MF src/libbluray/bdj/native/.deps/libbluray_la-register_native.Tpo -c src/libbluray/bdj/native/register_native.c  -fPIC -DPIC -o src/libbluray/bdj/native/.libs/libbluray_la-register_native.o

BUILD FAILED
/home/nietu/projects/mythtv/packaging/android/libs64/libbluray-1.1.2/src/libbluray/bdj/build.xml:26: Compile failed; see the compiler error output for details.```

@jpoet
Copy link
Member

jpoet commented Dec 30, 2023

The last time I tried to build for Android, I used the attached patch to get past that part. It pulls the latest version of libbluray which then seems to build.

android-libbluray-master.txt

@sgreene820
Copy link
Author

sgreene820 commented Dec 30, 2023 via email •
edited
Loading

@sgreene820
Copy link
Author

sgreene820 commented Dec 30, 2023 via email •
edited
Loading

@jpoet
Copy link
Member

jpoet commented Dec 30, 2023

Is that patch to be run from "packaging/android"? I'd like to give it a try. Steve
…
On Sat, Dec 30, 2023 at 12:19 PM John Poet @.> wrote: The last time I tried to build for Android, I used the attached patch to get past that part. It pulls the latest version of libbluray which then seems to build. android-libbluray-master.txt https://github.com/MythTV/packaging/files/13798756/android-libbluray-master.txt — Reply to this email directly, view it on GitHub <#155 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASQW2SZKJC27E6Q24UILYDDYMBELXAVCNFSM6AAAAAAX3MF7MKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZSGU3DQNRUGQ . You are receiving this because you authored the thread.Message ID: @.>

I believe it was to be applied just outside of the android directory, or inside the android directory if you use "-p1", but that is from memory. I don't currently have the code in place to test. You could try various values to patch -p# < patchfile until it applies.

@nietu
Copy link

nietu commented Dec 30, 2023

@jpoet Thx that pach fix the issue
@sgreene820 run in root of mythtv/packaging repo
$ patch -p1 < android-libbluray-master.txt

@sgreene820
Copy link
Author

Still fails here.

`compile:
[javac] /home/sgreene/MythAndroid/packaging/android/libs/libbluray/src/libbluray/bdj/build.xml:26: warning: 'includeantruntime' was not set, defaulting to build.sysclasspath=last; set to false for repeatable builds
[javac] Compiling 33 source files to /home/sgreene/MythAndroid/packaging/android/libs/libbluray/src/libbluray/bdj/build
[javac] warning: [options] bootstrap class path not set in conjunction with -source 5
[javac] error: Source option 5 is no longer supported. Use 6 or later.
[javac] error: Target option 1.5 is no longer supported. Use 1.6 or later.

BUILD FAILED
/home/sgreene/MythAndroid/packaging/android/libs/libbluray/src/libbluray/bdj/build.xml:26: Compile failed; see the compiler error output for details.

Total time: 0 seconds
make[1]: *** [Makefile:2902: all-local] Error 1
make[1]: Leaving directory '/home/sgreene/MythAndroid/packaging/android/libs/libbluray'
make: *** [Makefile:1055: all] Error 2`

@nietu
Copy link

nietu commented Dec 31, 2023

@sgreene820 I did overcome this issue

[javac] error: Source option 5 is no longer supported. Use 6 or later.
[javac] error: Target option 1.5 is no longer supported. Use 1.6 or later.

by using openjdk version 11 in ubuntu. Any more fresh java version failed

@sgreene820
Copy link
Author

I'll try that tomorrow. Thanks for the suggestion.

@sgreene820
Copy link
Author

sgreene820 commented Jan 2, 2024 via email

@mspieth
Copy link
Contributor

mspieth commented Jan 2, 2024

Ill look at this in the next few days.

@sgreene820
Copy link
Author

sgreene820 commented Jan 3, 2024 via email •
edited
Loading

@IcoDeveloper
Copy link

I've just spent some time getting the compile for fixes/34 to work in the docker environment;
ran into the same build error there; fixed by switching to latest libbluray in makelibs.sh (LIBBLURAYVER=1.3.4)

@IcoDeveloper IcoDeveloper mentioned this issue Mar 1, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@jpoet @mspieth @nietu @IcoDeveloper @sgreene820