I already noticed this locally, but termux-tools needs to use env variables that will be exported in Termux app v0.119.0, and build time placeholders only need to be used as fallback. Also TERMUX_ root scope may be different so scripts being added need to be used instead. Same for other placeholders. But this can be merged for now for initial fix.

Also TERMUX_APP_PACKAGE is for the app package name, but forks may be using a different class package name than com.termux.app, so separate variable needs to be added in properties.sh and env for it.

I'm not sure I understood it right; like this?

(turns out I do have a fork suffixing app package name, but not the class names)

No no, just use @TERMUX_APP_PACKAGE@.app for now. Class package name support needs to be added in termux-packages repo, I'll do it later.

OK, reverted.

Thanks for the pull.

Had to fix termux-reset for changes made previously:

• The /.app.TermuxService had to be replaced with /@TERMUX_APP_PACKAGE@.app, otherwise it would have expanded to fork app package name instead of termux classes package name. Even that is not technically correct as termux package name may not be the same as its namespace in a fork.
• The com.termux.termuxam.Am must not be replaced with @TERMUX_CLASS_NAME@.termuxam.Am, as that class does not belong in termux app, but in a different repo and has a different namespace/package name than com.termux. It should only be changed if some fork moves the classes to a different package name.
  • https://github.com/termux/TermuxAm/blob/v0.8.0/app/src/main/java/com/termux/termuxam/Am.java
  • https://developer.android.com/build/configure-app-module#set-namespace

Additional fixes will be done later, read the commit message e50eb26

Whoever updates the termux-tools package, should either ping me or check the paths themselves in the built deb as I haven't tested.