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

Add support for Android Emacs using Termux #230

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

johanwiden
Copy link

Add support for pdf-tools-install in Android Emacs using Termux.
The dependencies for android in autobuild can not be installed in pdf-tools-install, they have to be installed manually in Termux. But the dependencies are checked when running autobuild.

pdf-tools-install successfully installs epdfinfo in Android Emacs. One can then view pdf files.

The shebang in autobuild does not work in Termux, I therefore had to create a separate script "autobuild.android" that in turn executes autobuild. The problem with the shebang is that "env" in Termux is not located at "/usr/bin/env".

pdf-tools/Makefile does currently not work in Android Emacs, but it is not needed for running pdf-tools-install. Fixing this will have to be done in a separate pull request, if deemed necessary. Reason: There is yet no consensus for how to implement a command line emacs, to be run in batch mode, in Android Emacs.

I have successfully run "make test" in Ubuntu, in the repo feature branch.
Running "make server-test" in the master branch fails, so I have not tested this in the feature branch.

@johanwiden
Copy link
Author

This is linked to Issue #231

@vedang vedang added the pending review Maintainer needs to review the changes label Aug 8, 2023
@vedang
Copy link
Owner

vedang commented Aug 8, 2023

Thanks! I'll review this and let you know.

The change looks fine. I'll test it and perhaps add a disclaimer that this is not an officially supported platform.

@johanwiden
Copy link
Author

johanwiden commented Aug 8, 2023 via email

@gbarnich
Copy link

Exactly what I was looking for ! The fork lets me built epdfinfo server correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending review Maintainer needs to review the changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants