Re: [PATCH] git-remote-testgit: fix race when spawning fast-import

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Pete Wyckoff <pw@xxxxxxxx> writes:

> Fix this by setting stdin to unbuffered.
> Signed-off-by: Pete Wyckoff <pw@xxxxxxxx>
> ---
> This is the one that might be suitable for upstream.  It fixes
> the problem with this particular remote helper.
>     |    7 +++++++
>  t/ |   11 +++++++++++
>  2 files changed, 18 insertions(+)
> diff --git a/ b/
> index 3dc4851..5f3ebd2 100644
> --- a/
> +++ b/
> @@ -22,6 +22,7 @@ except ImportError:
>      _digest =
>  import sys
>  import os
> +import time
>  sys.path.insert(0, os.getenv("GITPYTHONLIB","."))
>  from git_remote_helpers.util import die, debug, warn
> @@ -204,6 +205,11 @@ def read_one_line(repo):
>      """Reads and processes one command.
>      """
> +    sleepy = os.environ.get("GIT_REMOTE_TESTGIT_SLEEPY")
> +    if sleepy:
> +        debug("Sleeping %d sec before readline" % int(sleepy))
> +        time.sleep(int(sleepy))
> +

If I understand your explanation correctly, the primary purpose of the
remote-testgit is to test the parts of the system that talk to remote
helpers that are used in production in the t/t5800 script, and this
"sleep" is to make it easier to trigger the particular bug you fixed 
in *this* script.  The bug is _not_ in the parts of the system being
tested, but is in this test scaffolding.

If that is the case, then it should not be enabled unconditionally.
When somebody wants to see if remote-testgit was broken again (perhaps
after observing occassional hangs), the environment should be set when
running the test, but not in t5800.
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at

[Newbies FAQ]     [Linux Kernel Development]     [Free Online Dating]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Free Online Dating]     [Photo]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]     [Linux Resources]

Add to Google