Skip to content

Conversation

@miminar
Copy link
Contributor

@miminar miminar commented Nov 13, 2014

Fixes: #5547

This PR just changes description of --sig-proxy which does work only in
non-TTY mode (--tty=false).

It obsoletes #7567 which I find superior to this because it fixes the
issue correctly.

Signal proxy does work only in non-TTY mode (--tty=false). Man pages and
commands should not lie about it.

Signed-off-by: Michal Minar <[email protected]>
@cevich
Copy link

cevich commented Nov 13, 2014

Doc fix is okay, but not addressing the problem smells ripe for more and more and more documentation needed for all the other corner-cases we simply haven't run into yet 😖 I'm a fan of applying #7567 instead.

@SvenDowideit
Copy link
Contributor

LGTM - this makes the documentation correct wrt what is currently implemented

@fredlf @jamtur01

@jamtur01
Copy link
Contributor

Docs LGTM

@SvenDowideit
Copy link
Contributor

@vieux @jfrazelle @tianon

@jessfraz
Copy link
Contributor

LGTM

@tianon
Copy link
Member

tianon commented Nov 15, 2014

LGTM, though doesn't the zsh tab completion have the flag text too?

@SvenDowideit
Copy link
Contributor

@miminar can you update the PR to also change the contrib/completion/* files - git grep sig-proxy will tell you what they are :)

@jessfraz jessfraz added this to the 1.4 milestone Nov 18, 2014
@fredlf
Copy link
Contributor

fredlf commented Nov 18, 2014

Docs LGTM once the change is propagated throughout as per @SvenDowideit

FWIW, I agree it would be better to fix the issue, but we always have to document reality, even if we don't like it!

@cevich
Copy link

cevich commented Nov 18, 2014

always have to document reality

Yes for sure, the more docs the better (underlying fix or not) 😄

@SvenDowideit
Copy link
Contributor

@miminar ping?

@miminar
Copy link
Contributor Author

miminar commented Nov 19, 2014

@SvenDowideit: Done.

@SvenDowideit
Copy link
Contributor

awesome, as they're really obvious identical changes, I'm going to treat everyone's review as done.

So I'm mergeing, thankyou very much @miminar

SvenDowideit added a commit that referenced this pull request Nov 19, 2014
Corrected description of --sig-proxy
@SvenDowideit SvenDowideit merged commit c7e6ad8 into moby:master Nov 19, 2014
@miminar miminar deleted the sigproxy_tty branch January 14, 2015 08:51
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.

man page inaccuracy about --sig-proxy and --tty

7 participants