Skip to content

Conversation

@probonopd
Copy link
Member

@probonopd probonopd commented Nov 28, 2024

Closes #92

@mehmetakbulut @lalten does this fix the issue?

@github-actions
Copy link

Build for testing:
artifacts x86_64
artifacts i686
artifacts armhf
artifacts aarch64
Use at your own risk.

@probonopd
Copy link
Member Author

probonopd commented Nov 28, 2024

Still get

sudo strace -f -e waitpid,fork,execve ./test-1-x86_64.AppImage
(...)
[pid   933] execve("/usr/bin/python3", ["python3", "/tmp/.mount_appimagkjGMD/AppRun"], 0x7ffe345736a8 /* 18 vars */) = 0
Hello world!
[pid   933] +++ exited with 0 +++
[pid   937] --- SIGPIPE {si_signo=SIGPIPE, si_code=SI_USER, si_pid=936, si_uid=0} ---
[pid   937] --- SIGTERM {si_signo=SIGTERM, si_code=SI_USER, si_pid=936, si_uid=0} ---
[pid   937] +++ exited with 0 +++
^Cstrace: Process 936 detached

@probonopd probonopd marked this pull request as draft November 28, 2024 13:58
@probonopd
Copy link
Member Author

Maybe, just maybe, we also need opts.idle_timeout_secs = 1;?

@github-actions
Copy link

Build for testing:
artifacts x86_64
artifacts i686
artifacts armhf
artifacts aarch64
Use at your own risk.

@probonopd
Copy link
Member Author

[pid   194] execve("/usr/bin/python3", ["python3", "/tmp/.mount_appimaICnnNe/AppRun"], 0x7fff521778d8 /* 18 vars */) = 0
Hello world!
[pid   194] +++ exited with 0 +++
[pid   198] --- SIGPIPE {si_signo=SIGPIPE, si_code=SI_USER, si_pid=197, si_uid=0} ---
[pid   198] --- SIGTERM {si_signo=SIGTERM, si_code=SI_USER, si_pid=197, si_uid=0} ---
[pid   198] +++ exited with 0 +++
--- SIGALRM {si_signo=SIGALRM, si_code=SI_KERNEL} ---
+++ exited with 0 +++

Seems to have done the trick!

@probonopd probonopd marked this pull request as ready for review November 29, 2024 10:43
@mehmetakbulut
Copy link

I tested the x86_64 build in my setup, worked great!

@github-actions
Copy link

github-actions bot commented Dec 3, 2024

Build for testing:
artifacts x86_64
artifacts i686
artifacts armhf
artifacts aarch64
Use at your own risk.

@probonopd probonopd merged commit 92caa2e into main Dec 3, 2024
18 checks passed
@probonopd probonopd deleted the auto_unmount branch December 3, 2024 19:10
@Samueru-sama Samueru-sama mentioned this pull request Mar 8, 2025
probonopd added a commit that referenced this pull request Oct 21, 2025
This reverts commit 92caa2e.
@probonopd probonopd mentioned this pull request Oct 21, 2025
TheAssassin pushed a commit that referenced this pull request Nov 7, 2025
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.

AppImage orphan survives main process exit

4 participants