这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@TomJo2000
Copy link
Member

@TomJo2000 TomJo2000 requested a review from Grimler91 as a code owner April 8, 2025 20:39
@TomJo2000 TomJo2000 changed the title Libxml2 rebuilds: Part 4 - Misc. X11 repo package Libxml2 rebuilds: Part 4 - Misc. X11 repo packages Apr 8, 2025
@robertkirkman
Copy link
Member

2025-04-09T00:02:56.9529040Z /home/builder/.termux-build/xournalpp/src/src/core/plugin/luapi_application.h:1987:9: error: use of undeclared identifier 'lua_isinteger'; did you mean 'lua_pushinteger'?
2025-04-09T00:02:56.9530223Z  1987 |     if (lua_isinteger(L, 2)) {
2025-04-09T00:02:56.9530621Z       |         ^~~~~~~~~~~~~
2025-04-09T00:02:56.9531089Z       |         lua_pushinteger
2025-04-09T00:02:56.9531801Z /data/data/com.termux/files/usr/include/lua5.2/lua.h:207:22: note: 'lua_pushinteger' declared here
2025-04-09T00:02:56.9532797Z   207 | LUA_API void        (lua_pushinteger) (lua_State *L, lua_Integer n);
2025-04-09T00:02:56.9533350Z       |                      ^

prefix pollution liblua52->xournalpp.

it can be resolved like this:

--- a/x11-packages/xournalpp/build.sh
+++ b/x11-packages/xournalpp/build.sh
@@ -4,12 +4,14 @@ TERMUX_PKG_LICENSE="GPL-2.0"
 TERMUX_PKG_LICENSE_FILE="LICENSE, copyright.txt"
 TERMUX_PKG_MAINTAINER="@termux"
 TERMUX_PKG_VERSION="1.2.6"
-TERMUX_PKG_REVISION=1
+TERMUX_PKG_REVISION=2
 TERMUX_PKG_SRCURL=https://github.com/xournalpp/xournalpp/archive/refs/tags/v${TERMUX_PKG_VERSION}.tar.gz
 TERMUX_PKG_SHA256=dce483e01e267b0d20afb88c59bf53b8ca1bd8518a31f98ef5061a334d6dc4eb
 TERMUX_PKG_AUTO_UPDATE=true
 TERMUX_PKG_DEPENDS="gdk-pixbuf, glib, gtk3, libandroid-execinfo, libc++, libcairo, librsvg, libsndfile, libx11, libxi, libxml2, libzip, pango, poppler, portaudio, zlib"
 TERMUX_PKG_REPLACES="xournal"
+# Lua 5.4 would be a dependency if plugins were wanted
 TERMUX_PKG_EXTRA_CONFIGURE_ARGS="
 -DHELP2MAN=NO
+-DENABLE_PLUGINS=OFF
 "

@TomJo2000
Copy link
Member Author

Thanks, will include that fix.

@TomJo2000 TomJo2000 force-pushed the libxml2-rebuilds4 branch 2 times, most recently from fcc7467 to b534155 Compare April 13, 2025 22:27
@TomJo2000 TomJo2000 merged commit 5d8e3f1 into termux:master Apr 14, 2025
4 of 9 checks passed
@TomJo2000 TomJo2000 deleted the libxml2-rebuilds4 branch June 8, 2025 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants