Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Add detection for Termux terminal #1923

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

Conversation

easyaspi314
Copy link

@easyaspi314 easyaspi314 commented Sep 29, 2021

Description

Adds detection for Termux (Termux sets the $TERMUX_VERSION environment variable, but doesn't expose its PID).

Screenshot_20210929-120452

Termux sets the TERMUX_VERSION environment variable.
Catches something like running xfce4-terminal in an X server
@easyaspi314
Copy link
Author

Changed the Termux detection to be after the PPID check. Termux doesn't show its own PID, so this catches an x11 terminal running off Termux.

@hykilpikonna hykilpikonna mentioned this pull request Jul 31, 2022
hykilpikonna added a commit to hykilpikonna/hyfetch that referenced this pull request Jul 31, 2022
…inal

Upstream PR: dylanaraps#1923
Thanks to @easyaspi314

Co-authored-by: easyaspi314 (Devin) <easyaspi314@users.noreply.github.com>
hykilpikonna added a commit to hykilpikonna/hyfetch that referenced this pull request Jul 31, 2022
@hykilpikonna
Copy link

This shows the terminal as "com.termux" for termux 0.118...

I fixed it by adding a special check

image

image

@hykilpikonna
Copy link

Thank you for your contribution!

This PR is merged into hyfetch since this repo (dylanaraps/neofetch) seems no longer maintained.

HyFetch is a fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.

Read the "Running Updated Original Neofetch" section for more info!

image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants