git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] git p4 test: Check ignore files with client spec

From
PWPete Wyckoff <pw@padd.com>
Date
Jul 20, 2013, 13:48 UTC
Message-ID
<20130720134837.GB18479@padd.com>
In-Reply-To
<1374188693-547-1-git-send-email-vitor.hda@gmail.com>
vitor.hda@gmail.com wrote on Fri, 19 Jul 2013 00:04 +0100:
> This test confirms that a file can be ignored during git p4 sync if if is
> excluded in P4 client specification.
This is a good check to have, and I'm glad it happens to work.  :)

I'd forgotten during your conversation with Matthieu that we did indeed have tests for detect-branches with use-client-spec. This test sure seems like it should cover that situation though.

Acked-by: Pete Wyckoff <pw@padd.com>
Show 53 quoted lines
> ---
>  t/t9801-git-p4-branch.sh | 23 ++++++++++++++++++++---
>  1 file changed, 20 insertions(+), 3 deletions(-)
> 
> diff --git a/t/t9801-git-p4-branch.sh b/t/t9801-git-p4-branch.sh
> index 9730821..2bf142d 100755
> --- a/t/t9801-git-p4-branch.sh
> +++ b/t/t9801-git-p4-branch.sh
> @@ -469,9 +469,11 @@ test_expect_success 'use-client-spec detect-branches skips branches setup' '
>  		View: //depot/usecs/b1/... //depot/usecs/b3/...
>  		EOF
>  
> -		echo b3/b3-file3 >b3/b3-file3 &&
> -		p4 add b3/b3-file3 &&
> -		p4 submit -d "b3/b3-file3"
> +		echo b3/b3-file3_1 >b3/b3-file3_1 &&
> +		echo b3/b3-file3_2 >b3/b3-file3_2 &&
> +		p4 add b3/b3-file3_1 &&
> +		p4 add b3/b3-file3_2 &&
> +		p4 submit -d "b3/b3-file3_1 b3/b3-file3_2"
>  	)
>  '
>  
> @@ -487,6 +489,21 @@ test_expect_success 'use-client-spec detect-branches skips branches' '
>  	)
>  '
>  
> +test_expect_success 'use-client-spec detect-branches skips files in branches' '
> +	client_view "//depot/usecs/... //client/..." \
> +		    "-//depot/usecs/b3/b3-file3_1 //client/b3/b3-file3_1" &&
> +	test_when_finished cleanup_git &&
> +	test_create_repo "$git" &&
> +	(
> +		cd "$git" &&
> +		git p4 sync --detect-branches --use-client-spec //depot/usecs@all &&
> +		git checkout -b master p4/usecs/b3 &&
> +		test_path_is_file b1-file1 &&
> +		test_path_is_file b3-file3_2 &&
> +		test_path_is_missing b3-file3_1
> +	)
> +'
> +
>  test_expect_success 'kill p4d' '
>  	kill_p4d
>  '
> -- 
> 1.8.3.2
> 
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
Previous: Vitor AntunesNext: Vitor Antunes
Message 2 of 3 in “git p4 test: Check ignore files with client spec”
  1. git p4 test: Check ignore files with client specVitor Antunes, Jul 18, 2013
  2. Pete WyckoffJul 20, 2013
  3. Vitor AntunesJul 20, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.