Flush stdout after "Opened a new pty: " message. #20

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

Contributor
vi commented Aug 16, 2012

Allows usage of reptry in scripts and other programs.

@vi vi Flush stdout after printing "Opened a new pty:" message
Allows usage of reptry in scripts and other programs
bfc86cd
Owner
nelhage commented Aug 16, 2012

Thanks, merged.

@nelhage nelhage closed this Aug 16, 2012
Contributor
vi commented Aug 16, 2012

Also pushed more extensive scripting support with fork/exec and REPTYR_* envionment variables.

Owner
nelhage commented Aug 16, 2012

I like the concept. Is there a reason to have the $REPTYR_REDIRECT
environment variable, instead of a command-line option?

Also, out of curiosity, what's your use case for this?

  • Nelson

On Thu, Aug 16, 2012 at 11:43 AM, Vitaly Shukela
notifications@github.com wrote:

Also pushed more extensive scripting support with fork/exec and REPTYR_*
envionment variables.


Reply to this email directly or view it on GitHub.

Contributor
vi commented Aug 16, 2012

Already pushed one more commit that makes '-L' alias to '-l' with REDIRECT.

In my programs I use environment variables for configuration more than non-positional arguments.

Use case: start programs that can clobber terminal or steal it's ownership.
Example: http://github.com/vi/dive
After starting it in usual mode the calling shell loses terminal owner status. But with "reptry -L" the problem is worked around.

Contributor
vi commented Aug 16, 2012

(Now thinking what to put to dive's documentation... "Use special vi's version of reptyr" or "Use reptry not less that v0.4")

Owner
nelhage commented Aug 16, 2012

What do you think of this: https://github.com/nelhage/reptyr/tree/subcommand

I'm happy to do an 0.4 release with this feature once we merge it.

On Thu, Aug 16, 2012 at 11:56 AM, Vitaly Shukela
notifications@github.comwrote:

(Now thinking what to put to dive's documentation... "Use special vi's
version of reptyr" or "Use reptry not less that v0.4")


Reply to this email directly or view it on GitHubhttps://github.com/nelhage/reptyr/pull/20#issuecomment-7789829.

Contributor
vi commented Aug 16, 2012

"dive" successfully works with "subcommand" version of reptyr.

/* Some area of improvement: it would be good if "reptyr -L bash" could work like "socat -,raw,echo=0 exec:bash,pty,setsid,stderr". The latter gives job control in the shell, the former gives normal terminal resizing. I want both. */

Owner
nelhage commented Aug 16, 2012

Just pushed a commit that I think will fix that. Let me know if it works.

  • Nelson

On Thu, Aug 16, 2012 at 12:38 PM, Vitaly Shukela
notifications@github.comwrote:

"dive" successfully works with "subcommand" version of reptyr.

/* Some area of improvement: it would be good if "reptyr -L bash" could
work like "socat -,raw,echo=0 exec:bash,pty,setsid,stderr". The latter
gives job control in the shell, the former gives normal terminal resizing.
I want both. */


Reply to this email directly or view it on GitHubhttps://github.com/nelhage/reptyr/pull/20#issuecomment-7791196.

Contributor
vi commented Aug 16, 2012

Works. Documented one more "reptyr" workaround in "dive" (in addition to that socat's).

That "Opened a new pty: /dev/pts/33" probably should not be displayed at all in "-L" mode.

Owner
nelhage commented Aug 16, 2012

Great. I've pushed to master and tagged 0.4.

  • Nelson

On Thu, Aug 16, 2012 at 1:13 PM, Vitaly Shukela notifications@github.comwrote:

Works. Documented one more "reptyr" workaround in "dive" (in addition to
that socat's).


Reply to this email directly or view it on GitHubhttps://github.com/nelhage/reptyr/pull/20#issuecomment-7792230.

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