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.

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

[Bug]: libandroid: What is this package for? #16902

Closed
xtkoba opened this issue Jun 15, 2023 · 11 comments · Fixed by #16903
Closed

[Bug]: libandroid: What is this package for? #16902

xtkoba opened this issue Jun 15, 2023 · 11 comments · Fixed by #16903
Labels
bug report Something is not working properly

Comments

@xtkoba
Copy link
Contributor

xtkoba commented Jun 15, 2023

Problem description

Please explain why this package is needed. I'm going to remove this if no explanation is provided.

What steps will reproduce the bug?

Install ffmpeg package which now depends on libandroid package.

What is the expected behavior?

libandroid package should not be needed.

System information

termux-info:

Termux Variables:
TERMUX_APK_RELEASE=F_DROID
TERMUX_APP_PACKAGE_MANAGER=apt
TERMUX_APP_PID=5693
TERMUX_IS_DEBUGGABLE_BUILD=0
TERMUX_MAIN_PACKAGE_FORMAT=debian
TERMUX_VERSION=0.118.0
Packages CPU architecture:
aarch64
Subscribed repositories:
# sources.list
deb https://packages-cf.termux.dev/apt/termux-main/ stable main
# x11-repo (sources.list.d/x11.list)
deb https://packages-cf.termux.dev/apt/termux-x11/ x11 main
Updatable packages:
ffmpeg/stable 6.0-3 aarch64 [upgradable from: 6.0-2]
fluidsynth/stable 2.3.3 aarch64 [upgradable from: 2.3.2]
libde265/stable 1.0.12 aarch64 [upgradable from: 1.0.11]
libjxl/stable 0.8.2 aarch64 [upgradable from: 0.8.1]
termux-tools version:
1.38.3
Android version:
12
Kernel build information:
Linux localhost 4.19.191-g234d1747e22a #1 SMP PREEMPT Sat Mar 25 18:12:54 CST 2023 aarch64 Android
Device manufacturer:
motorola
Device model:
moto e32(s)
LD Variables:
LD_LIBRARY_PATH=
LD_PRELOAD=/data/data/com.termux/files/usr/lib/libtermux-exec.so
@xtkoba xtkoba added bug report Something is not working properly untriaged labels Jun 15, 2023
@truboxl
Copy link
Contributor

truboxl commented Jun 15, 2023

#16680 (comment)

If you have better ideas feel free to break the package

@xtkoba
Copy link
Contributor Author

xtkoba commented Jun 15, 2023

I'm not familiar with Termux Docker, but is it true that it does not have /system/lib(64)?/libandroid.so?

@xtkoba
Copy link
Contributor Author

xtkoba commented Jun 15, 2023

Note that there are other packages (e.g. mpv, libgnustep-base) that link against libandroid.so. They also won't work on Termux Docker without libandroid package, if ffmpeg doesn't. We should be consistent in that.

@truboxl
Copy link
Contributor

truboxl commented Jun 15, 2023

I'm not familiar with Termux Docker, but is it true that it does not have /system/lib(64)?/libandroid.so?

Yes and I suggest you to use it if you can fix this in better ways

@xtkoba
Copy link
Contributor Author

xtkoba commented Jun 15, 2023

What do you mean by "it" if it does not exist?

@truboxl
Copy link
Contributor

truboxl commented Jun 15, 2023

@xtkoba
Copy link
Contributor Author

xtkoba commented Jun 15, 2023

Then my opinion is Termux Docker is broken and it should be fixed so that it includes system libandroid.so. libandroid.so is assumed to exist on every Android environment.

@xtkoba
Copy link
Contributor Author

xtkoba commented Jun 15, 2023

And normal Termux environment should not require libandroid package.

@xtkoba
Copy link
Contributor Author

xtkoba commented Jun 15, 2023

FYI, system libs that are assumed to exist:

libEGL.so
libGLESv1_CM.so
libGLESv2.so
libGLESv3.so
libOpenMAXAL.so
libOpenSLES.so
libandroid.so
libc.so
libcamera2ndk.so
libdl.so
libjnigraphics.so
liblog.so
libm.so
libmediandk.so
libstdc++.so
libvulkan.so
libz.so

@xtkoba
Copy link
Contributor Author

xtkoba commented Jun 15, 2023

IIUC Termux Docker is not an officially supported environment. So IIUC no problem if the dependency of a package is specified so that it may be insufficient for Termux Docker.

I understand libandroid package itself is useful for some environment and I won't remove it from our repo. Please manually install libandroid package if necessary.

@xtkoba
Copy link
Contributor Author

xtkoba commented Jun 15, 2023

(Honestly I don't want to check every package if it depends on libandroid.so.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Something is not working properly
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants