Skip to content

Fixes init validation for Termux #260

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

Closed
pizcadesaber wants to merge 3 commits into from
Closed

Fixes init validation for Termux #260

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
5287940
Fixes init validation for Termux
pizcadesaber Feb 21, 2025
e532e64
Reverts INI directive check
pizcadesaber Feb 24, 2025
f60916a
Merge branch 'master' into termux-support
jcupitt Apr 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion src/FFI.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,11 @@ private static function init(): void
throw new Exception("zend.max_allowed_stack_size not set to '-1'");
}

$vips_libname = self::libraryName("libvips", 42);
// Use "libvips.so" on Termux, otherwise use the standard versioned library
$vips_libname = getenv('PREFIX') === '/data/data/com.termux/files/usr'

@jcupitt jcupitt Feb 21, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very specific hacks like this feel bad. Perhaps we should try to open libvips.so.42 first, then fall back to libvips.so if that fails?

Off-topic, but do you know why termux doesn't include API numbers in library names? I'm curious.

@pizcadesaber pizcadesaber Feb 21, 2025 •
edited
Loading

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's a good idea to check if .42 exists first before the other one. However, I'm not sure how to write it without breaking the code.

To be honest, in this commit, I just added conditions that worked for me to make this package function properly. It's a dependency of a package I use, and it was only failing to validate these conditions in Termux. It hasn't caused me any issues since.

Termux always updates libraries to the latest compatible version. I haven't seen any ways to downgrade a package. I guess that's the reason why it omits version numbers in its files.

@kleisauke kleisauke Feb 21, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like Termux needs to compile libvips with the TERMUX_MESON_ENABLE_SOVERSION=1 env set.

termux_step_pre_configure() {
	export TERMUX_MESON_ENABLE_SOVERSION=1
}

See for example:
https://github.com/termux/termux-packages/blob/38379b121bff426393df599b7b08554dcaea1c5f/packages/libmpdclient/build.sh#L15-L17

@kleisauke kleisauke Apr 4, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... I just opened PR termux/termux-packages#24153 for this.

? "libvips.so"
: self::libraryName("libvips", 42);

if (PHP_OS_FAMILY === "Windows") {
$glib_libname = self::libraryName("libglib-2.0", 0);
$gobject_libname = self::libraryName("libgobject-2.0", 0);
Expand Down