termux / termux-api

Termux add-on app which exposes device functionality as API to command line programs.
https://f-droid.org/en/packages/com.termux.api/
2.22k stars 442 forks source link

v0.50.1: termux-notification custom icon not working anymore #483

Open seffparker opened 2 years ago

seffparker commented 2 years ago

The custom notification icons set using --icon option is no longer working after updating from 0.49 to 0.50. Downgraded to 0.49 and confirmed the bug in the version 0.50

Link to Reddit post: https://www.reddit.com/r/termux/comments/ru09h4/termuxnotification_icon_is_not_showing_it_worked

Grimler91 commented 2 years ago

What is your output of termux-info?

seffparker commented 2 years ago
Application version:
0.117
Packages CPU architecture:
aarch64
Subscribed repositories:
# sources.list
deb https://packages-cf.termux.org/apt/termux-main/ stable main
# root-repo (sources.list.d/root.list)
deb https://packages.termux.org/apt/termux-root root stable
# x11-repo (sources.list.d/x11.list)
deb https://packages.termux.org/apt/termux-x11 x11 main
Updatable packages:
All packages up to date
Android version:
10
Kernel build information:
Linux batmobile 4.9.186-perf-g10af704 #1 SMP PREEMPT Sun Dec 27 14:28:21 WIB 2020 aarch64 Android
Device manufacturer:
Xiaomi
Device model:
POCO F1
intika commented 2 years ago

Working with Termux-api v0.49 and not on v0.50.0 nor v0.50.1... downgraded the api while keeping all the rest updated...

AgentConDier commented 2 years ago

I just installed the latest termux + termux-api APKs from GitHub Actions, and notification icons are working again! ~If this is not due to some difference between debug and fdroid builds it should be fixed in the next release.~ Edit: It seems icons only work in debug builds currently Tested with termux 0.118.0+1488bb0 and termux-api 0.50.1+df3cb50

Neo-Oli commented 2 years ago

Can we get a release then? Or is something blocking it?

agnostic-apollo commented 2 years ago

Main issue is that termux-am-socket support added will only be available when termux-app release gets out. I did plan to refactor Termux:API stuff since its a mess and wanted to add a proper api convention, but can optionally be done after release, I am busy with termux-app release anyways. There may be some issues as well that may need fixing, or PRs merged by others may need looking into, like useless flags being passed, etc.

mizzunet commented 1 year ago

I'm also having the issue

Application version:
0.118.0
Packages CPU architecture:
aarch64
Subscribed repositories:
# sources.list
deb https://packages.termux.org/apt/termux-main/ stable main
Updatable packages:
apt/stable 2.7.3 aarch64 [upgradable from: 2.3.13-3]
bash/stable 5.2.15-1 aarch64 [upgradable from: 5.1.12-2]
ca-certificates/stable 1:2023.05.30 all [upgradable from: 1:2021-10-26-0]
command-not-found/stable 2.2.0-11 aarch64 [upgradable from: 1.71]
coreutils/stable 9.3 aarch64 [upgradable from: 9.0]
curl/stable 8.2.1 aarch64 [upgradable from: 7.81.0]
dash/stable 0.5.12 aarch64 [upgradable from: 0.5.11.5]
debianutils/stable 5.8 aarch64 [upgradable from: 5.5]
dialog/stable 1.3-20230209-0 aarch64 [upgradable from: 1.3-20211214-0]
diffutils/stable 3.10 aarch64 [upgradable from: 3.8]
dos2unix/stable 7.5.0 aarch64 [upgradable from: 7.4.2]
dpkg/stable 1.21.22 aarch64 [upgradable from: 1.21.1]
ed/stable 1.19 aarch64 [upgradable from: 1.17-4]
findutils/stable 4.9.0-2 aarch64 [upgradable from: 4.8.0]
gawk/stable 5.2.2 aarch64 [upgradable from: 5.1.1]
gpgv/stable 2.4.3 aarch64 [upgradable from: 2.3.4]
grep/stable 3.11 aarch64 [upgradable from: 3.7-2]
gzip/stable 1.12-1 aarch64 [upgradable from: 1.11-3]
inetutils/stable 2.4-1 aarch64 [upgradable from: 1.9.4-12]
less/stable 633-1 aarch64 [upgradable from: 590]
libandroid-support/stable 28-3 aarch64 [upgradable from: 28-2]
libassuan/stable 2.5.6 aarch64 [upgradable from: 2.5.5]
libc++/stable 25c aarch64 [upgradable from: 23b-3]
libcap-ng/stable 2:0.8.3 aarch64 [upgradable from: 1:0.8.3-pre1-0]
libcrypt/stable 0.2-5 aarch64 [upgradable from: 0.2-3]
libcurl/stable 8.2.1 aarch64 [upgradable from: 7.81.0]
libevent/stable 2.1.12-2 aarch64 [upgradable from: 2.1.12]
libexpat/stable 2.5.0-1 aarch64 [upgradable from: 2.4.2]
libgcrypt/stable 1.10.2 aarch64 [upgradable from: 1.9.4]
libgmp/stable 6.2.1-2 aarch64 [upgradable from: 6.2.1]
libgnutls/stable 3.8.0-1 aarch64 [upgradable from: 3.6.16-1]
libgpg-error/stable 1.47 aarch64 [upgradable from: 1.43]
libiconv/stable 1.17 aarch64 [upgradable from: 1.16-3]
libidn2/stable 2.3.4 aarch64 [upgradable from: 2.3.2]
liblz4/stable 1.9.4 aarch64 [upgradable from: 1.9.3]
liblzma/stable 5.4.4 aarch64 [upgradable from: 5.2.5-1]
libmpfr/stable 4.2.0-p9-0 aarch64 [upgradable from: 4.1.0]
libnettle/stable 3.9.1 aarch64 [upgradable from: 3.7.3]
libnghttp2/stable 1.55.1 aarch64 [upgradable from: 1.46.0]
libssh2/stable 1.11.0 aarch64 [upgradable from: 1.10.0]
libtirpc/stable 1.3.3 aarch64 [upgradable from: 1.3.2]
libunistring/stable 1.1 aarch64 [upgradable from: 0.9.10-5]
lsof/stable 4.98.0 aarch64 [upgradable from: 4.94.0-1]
nano/stable 7.2 aarch64 [upgradable from: 6.0]
openssl/stable 1:3.1.2 aarch64 [upgradable from: 1.1.1l]
pcre2/stable 10.42 aarch64 [upgradable from: 10.39-2]
procps/stable 3.3.17-2 aarch64 [upgradable from: 3.3.17-1]
psmisc/stable 23.6-1 aarch64 [upgradable from: 23.4]
readline/stable 8.2.1 aarch64 [upgradable from: 8.1.1]
sed/stable 4.9-1 aarch64 [upgradable from: 4.8-2]
tar/stable 1.35 aarch64 [upgradable from: 1.34]
termux-am-socket/stable 1.5.0 aarch64 [upgradable from: 1.02]
termux-exec/stable 1:1.0 aarch64 [upgradable from: 1:0.9]
termux-keyring/stable 3.11 all [upgradable from: 2.4]
termux-licenses/stable 2.0-3 all [upgradable from: 2.0-1]
termux-tools/stable 1.38.5 all [upgradable from: 0.155]
unbound/stable 1.17.1-2 aarch64 [upgradable from: 1.13.2-1]
unzip/stable 6.0-9 aarch64 [upgradable from: 6.0-7]
util-linux/stable 2.39.1 aarch64 [upgradable from: 2.37.2-1]
xxhash/stable 0.8.2 aarch64 [upgradable from: 0.8.1]
xz-utils/stable 5.4.4 aarch64 [upgradable from: 5.2.5-1]
zlib/stable 1.2.13 aarch64 [upgradable from: 1.2.11-5]
Android version:
13
Kernel build information:
Linux localhost 4.14.302-ab23410010 #1 SMP PREEMPT Tue Jun 13 22:16:21 UTC 2023 aarch64 Android
Device manufacturer:
Google
Device model:
Pixel 4a
ollietues commented 3 months ago

I am also still having this issue.

AgentConDier commented 1 month ago

I feel like this might be due to an issue in the build process, maybe the icons are considered unused because they are never referenced in the code directly and get removed from release builds. I just built both release and debug signed APKs using Android Studio (of 65da1770a044e09cd9d0cf32e916706146099efb). If I install the release version, icons don't work. If I install debug instead, the icons show up as expected.

elig0n commented 4 weeks ago

Same on Termux 0.118.1, Termux-API 0.50.1 from F-Droid on Samsung OneUI 4.1.

event_note icon is always shown despite trying several other --icon names like: terminal, Terminal, search, Search.

Also, when multiple notifications are sent the event_note icon is only displayed for the Termux:API notifications container itself and not for each note.