Kupfer process will stop if loading from background #5

Open
iven opened this Issue Jun 5, 2013 · 5 comments

Comments

Projects
None yet
4 participants
@iven

iven commented Jun 5, 2013

The commit ad9a064 introduces an regression that, when loading kupfer from background, say, loading from a desktop file, the process stops.

This is because interactive bash processes could not be run in background. You should prefix the command with 'setsid' to fix.

@SanskritFritz

This comment has been minimized.

Show comment
Hide comment
@SanskritFritz

SanskritFritz Jun 6, 2013

Contributor

I start Kupfer from the openbox autostart file and am hit by this bug. This workaround works well, thank you!

Contributor

SanskritFritz commented Jun 6, 2013

I start Kupfer from the openbox autostart file and am hit by this bug. This workaround works well, thank you!

@ikn

This comment has been minimized.

Show comment
Hide comment
@ikn

ikn Jun 6, 2013

Contributor

Wait, if a background process (a process without a controlling terminal, I guess) runs 'bash -i', the parent process (of the 'bash -i' process) dies? Or am I misunderstanding this?

And how does setsid work? If I do setsid bash -i & (or setsid bash -c "bash -i" &), I get some strange errors and the process I started dies.

Contributor

ikn commented Jun 6, 2013

Wait, if a background process (a process without a controlling terminal, I guess) runs 'bash -i', the parent process (of the 'bash -i' process) dies? Or am I misunderstanding this?

And how does setsid work? If I do setsid bash -i & (or setsid bash -c "bash -i" &), I get some strange errors and the process I started dies.

@iven

This comment has been minimized.

Show comment
Hide comment
@iven

iven Jun 7, 2013

It stops during launching, that is, you must call fg to let it go on.

Run bash -i -c ls and fg for test, also setsid bash -i -c ls.

iven commented Jun 7, 2013

It stops during launching, that is, you must call fg to let it go on.

Run bash -i -c ls and fg for test, also setsid bash -i -c ls.

@ikn

This comment has been minimized.

Show comment
Hide comment
@ikn

ikn Jun 7, 2013

Contributor

Ah, I still get the weird error messages, but it does work. Thanks for clarifying.

Contributor

ikn commented Jun 7, 2013

Ah, I still get the weird error messages, but it does work. Thanks for clarifying.

@KarolBedkowski

This comment has been minimized.

Show comment
Hide comment
@KarolBedkowski

KarolBedkowski Jun 9, 2013

Owner

I don't see this problem in any of my environments.
Anyway I added setsid in ee5dcb2.
Thanks.

Owner

KarolBedkowski commented Jun 9, 2013

I don't see this problem in any of my environments.
Anyway I added setsid in ee5dcb2.
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment