Re: [PATCH] disable t4016-diff-quote.sh on some filesystems

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

 



"Alex Riesen" <raa.lkml@xxxxxxxxx> writes:

> because the filesystems (most typically FAT and NTFS) do not support
> HT nor LF in filenames.
>
> Signed-off-by: Alex Riesen <raa.lkml@xxxxxxxxx>
>
> ---
> God help me...
> 
>  t/t4016-diff-quote.sh |    4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)

Buddha help me.

You are still lacking the "good stuff" in what I'd end up saving
in a separate file and applying, which is the attachment part...

> From 0edbc6df977e7d954032eaf72720c93b86ab4c0f Mon Sep 17 00:00:00 2001
> From: Alex Riesen <raa.lkml@xxxxxxxxx>
> Date: Tue, 20 Feb 2007 10:04:32 +0100
> Subject: [PATCH] disable t4016-diff-quote.sh on some filesystems
>
> ---
>  t/t4016-diff-quote.sh |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)

... but that is a minor detail.  I can survive.

> +if ! test : 2>/dev/null >"$P1" ; then
> +	echo >&2 'Filesystem does not support tabs in names'
> +	test_done
> +fi

I do not understand what this test-colon is doing?  Did you mean

	if ! : >"$P1" 2>&1
        then
		...
	fi

I would have written it like this, to check if it really
succeeded creating and also we can remove it.

	: >"$P1" 2>&1 && test -f "$P1" && rm -f "$P1" || {
		...
	}

-
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]