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

Conversation

@Auxilus
Copy link
Contributor

@Auxilus Auxilus commented Sep 25, 2017

Endprocess string modified to check for ~/hushlogout if exists then exit the terminal without echoing exitDescription

int exitCode = (Integer) msg.obj;
mShellExitStatus = exitCode;
mShellPid = -1;
mTerminalToProcessIOQueue.close(); mProcessToTerminalIOQueue.close();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm guessing this shouldn't be here twice?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solved..

@Grimler91
Copy link
Member

Grimler91 commented Sep 25, 2017

Wouldn't this sort of undo the very thing that the exit message is suppose to do?
If a user breaks his environment then there would be no way to launch a failsafe terminal if the terminal exists immediately.

Maybe it's possible to check for /sdcard/hushlogout instead, as @Neo-Oli suggested here

@Auxilus
Copy link
Contributor Author

Auxilus commented Sep 25, 2017

Hmm, that sounds like a problem, but I just made temp changes..
Should I add something like warning when used tries touch ~/hushlogout ?

@Auxilus
Copy link
Contributor Author

Auxilus commented Sep 25, 2017

I'll just replace ~/hushlogout by /sdcard/hushlogout

@cgarz
Copy link

cgarz commented Feb 15, 2018

@Grimler91 True, but the exit message wont protect you from everything anyway. Such as killall com.termux in your bashrc. A way to properly close termux panes programatically is sorely needed.
Many, including myself have said as much on various issues such as Issue 56

Edit: woops. Just saw that you've already saw Issue 56 and referenced this from there.

@fornwall
Copy link
Member

fornwall commented Jan 7, 2020

Not relevant after #56.

@fornwall fornwall closed this Jan 7, 2020
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.

4 participants