Re: [PATCH v3 00/10] Fix git's test suite to pass when the path contains spaces

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

 



Bryan Donlan schrieb:
> This patch series fixes bugs in git and git's testsuite to allow all tests
> to pass when the working directory contains whitespace and/or shell
> metacharacters.

Thanks a lot, it's appreciated.

> Bryan Donlan (10):
>   git-rebase.sh: Fix --merge --abort failures when path contains
>     whitespace
>   config.c: Escape backslashes in section names properly
>   git-send-email.perl: Handle shell metacharacters in $EDITOR properly
>   test-lib.sh: Add a test_set_editor function to safely set $VISUAL
>   Use test_set_editor in t9001-send-email.sh
>   test-lib.sh: Fix some missing path quoting
>   lib-git-svn.sh: Fix quoting issues with paths containing shell
>     metacharacters
>   Don't use the export NAME=value form in the test scripts.
>   Fix tests breaking when checkout path contains shell metacharacters
>   Rename the test trash directory to contain spaces.

The last patch breaks t9400-git-cvsserver-server.sh on my system:

* expecting success: cat request-anonymous | git-cvsserver --strict-paths
pserver $SERVERDIR >log 2>&1 &&
   sed -ne \$p log | grep "^I LOVE YOU$"
* FAIL 9: req_Root (strict paths)
        cat request-anonymous | git-cvsserver --strict-paths pserver
$SERVERDIR >log 2>&1 &&
           sed -ne \$p log | grep "^I LOVE YOU$"

D-Quoting $SERVERDIR helps. ;)

Please post an incremental diff when you resend it. Feel free to add:

Tested-by: Johannes Sixt <johannes.sixt@xxxxxxxxxx>

-- Hannes

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux