Re: [PATCH 7/8] Use test_set_editor in t9001-send-email.sh | KernelTrap

Re: [PATCH 7/8] Use test_set_editor in t9001-send-email.sh

view
thread

!MAILaRCHIVE_VOTE_RePLACE

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

[view in full thread]

From: Johannes Sixt <j.sixt@...>

To: Bryan Donlan <bdonlan@...>

Cc: <git@...>, Adam Roben <aroben@...>, <gitster@...>

Subject: Re: [PATCH 7/8] Use test_set_editor in t9001-send-email.sh

Date: Wednesday, April 9, 2008 - 3:01 am

Bryan Donlan schrieb:
quoted text> diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh
> index 030f66c..0a65785 100755
> --- a/t/t9001-send-email.sh
> +++ b/t/t9001-send-email.sh
> @@ -144,10 +144,7 @@ test_expect_success 'setup fake editor' '
>  	chmod +x fake-editor
>  '
>  
> -FAKE_EDITOR="$(pwd)/fake-editor"
> -export FAKE_EDITOR
> -GIT_EDITOR='"$FAKE_EDITOR"'
> -export GIT_EDITOR
> +test_set_editor "$(pwd)/fake-editor"
>  
>  test_expect_success '--compose works' '
>  	clean_fake_sendmail &&

I'd squash this into 5/8 or 8/8. Dunno.

-- Hannes

--
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 message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:

[PATCH 0/8] Fix git's test suite to pass when the path conta..., Bryan Donlan, (Tue Apr 8, 9:29 pm)

[PATCH v2 00/10] Fix git's test suite to pass when the path ..., Bryan Donlan, (Thu Apr 10, 2:50 am)

[PATCH v3 00/10] Fix git's test suite to pass when the path ..., Bryan Donlan, (Sun May 4, 1:37 am)

Re: [PATCH v3 00/10] Fix git's test suite to pass when the p..., Johannes Sixt, (Mon May 5, 3:03 am)

Re: [PATCH v3 00/10] Fix git's test suite to pass when the p..., Bryan Donlan, (Mon May 5, 3:59 am)

Re: [PATCH v3 00/10] Fix git's test suite to pass when the p..., Johannes Sixt, (Mon May 5, 4:19 am)

[PATCH v3 01/10] git-rebase.sh: Fix --merge --abort failures..., Bryan Donlan, (Sun May 4, 1:37 am)

[PATCH v3 02/10] config.c: Escape backslashes in section nam..., Bryan Donlan, (Sun May 4, 1:37 am)

[PATCH v3 03/10] git-send-email.perl: Handle shell metachara..., Bryan Donlan, (Sun May 4, 1:37 am)

[PATCH v3 04/10] test-lib.sh: Add a test_set_editor function..., Bryan Donlan, (Sun May 4, 1:37 am)

[PATCH v3 05/10] Use test_set_editor in t9001-send-email.sh, Bryan Donlan, (Sun May 4, 1:37 am)

[PATCH v3 06/10] test-lib.sh: Fix some missing path quoting, Bryan Donlan, (Sun May 4, 1:37 am)

[PATCH v3 07/10] lib-git-svn.sh: Fix quoting issues with pat..., Bryan Donlan, (Sun May 4, 1:37 am)

[PATCH v3 08/10] Don't use the export NAME=value form in the..., Bryan Donlan, (Sun May 4, 1:37 am)

[PATCH v2 01/10] git-rebase.sh: Fix --merge --abort failures..., Bryan Donlan, (Thu Apr 10, 2:50 am)

Re: [PATCH v2 01/10] git-rebase.sh: Fix --merge --abort fail..., Junio C Hamano, (Thu Apr 10, 3:45 am)

Re: [PATCH v2 01/10] git-rebase.sh: Fix --merge --abort fail..., Bryan Donlan, (Thu Apr 10, 4:48 am)

[PATCH v2 02/10] config.c: Escape backslashes in section nam..., Bryan Donlan, (Thu Apr 10, 2:50 am)

[PATCH v2 03/10] git-send-email.perl: Handle shell metachara..., Bryan Donlan, (Thu Apr 10, 2:50 am)

[PATCH v2 04/10] test-lib.sh: Add a test_set_editor function..., Bryan Donlan, (Thu Apr 10, 2:50 am)

[PATCH v2 05/10] Use test_set_editor in t9001-send-email.sh, Bryan Donlan, (Thu Apr 10, 2:50 am)

[PATCH v2 06/10] test-lib.sh: Fix some missing path quoting, Bryan Donlan, (Thu Apr 10, 2:50 am)

[PATCH v2 07/10] lib-git-svn.sh: Fix quoting issues with pat..., Bryan Donlan, (Thu Apr 10, 2:50 am)

[PATCH v2 08/10] Don't use the export NAME=value form in the..., Bryan Donlan, (Thu Apr 10, 2:50 am)

[PATCH v2 09/10] Fix tests breaking when checkout path conta..., Bryan Donlan, (Thu Apr 10, 2:50 am)

[PATCH v2 10/10] Rename the test trash directory to contain ..., Bryan Donlan, (Thu Apr 10, 2:50 am)

[PATCH 1/8] git-rebase.sh: Fix --merge --abort failures when..., Bryan Donlan, (Tue Apr 8, 9:29 pm)

Re: [PATCH 1/8] git-rebase.sh: Fix --merge --abort failures ..., Johannes Sixt, (Wed Apr 9, 2:55 am)

Re: [PATCH 1/8] git-rebase.sh: Fix --merge --abort failures ..., Bryan Donlan, (Wed Apr 9, 10:37 am)

Re: [PATCH 1/8] git-rebase.sh: Fix --merge --abort failures ..., Johannes Sixt, (Wed Apr 9, 10:51 am)

Re: [PATCH 1/8] git-rebase.sh: Fix --merge --abort failures ..., Bryan Donlan, (Wed Apr 9, 11:02 am)

Re: [PATCH 1/8] git-rebase.sh: Fix --merge --abort failures ..., Johannes Sixt, (Thu Apr 10, 2:10 am)

[PATCH 2/8] config.c: Escape backslashes in section names pr..., Bryan Donlan, (Tue Apr 8, 9:29 pm)

Re: [PATCH 2/8] config.c: Escape backslashes in section name..., Johannes Sixt, (Wed Apr 9, 2:31 am)

Re: [PATCH 2/8] config.c: Escape backslashes in section name..., Bryan Donlan, (Wed Apr 9, 10:13 am)

Re: [PATCH 2/8] config.c: Escape backslashes in section name..., Johannes Sixt, (Wed Apr 9, 10:25 am)

[PATCH 3/8] git-send-email.perl: Handle shell metacharacters..., Bryan Donlan, (Tue Apr 8, 9:29 pm)

Re: [PATCH 3/8] git-send-email.perl: Handle shell metacharac..., Johannes Sixt, (Wed Apr 9, 2:31 am)

Re: [PATCH 3/8] git-send-email.perl: Handle shell metacharac..., Junio C Hamano, (Wed Apr 9, 11:39 pm)

Re: [PATCH 3/8] git-send-email.perl: Handle shell metacharac..., Bryan Donlan, (Wed Apr 9, 10:28 am)

[PATCH 4/8] test-lib.sh: Fix some missing path quoting, Bryan Donlan, (Tue Apr 8, 9:30 pm)

[PATCH 5/8] test-lib.sh: Add a test_set_editor function to s..., Bryan Donlan, (Tue Apr 8, 9:30 pm)

Re: [PATCH 5/8] test-lib.sh: Add a test_set_editor function ..., Johannes Sixt, (Wed Apr 9, 2:36 am)

[PATCH 6/8] lib-git-svn.sh: Fix quoting issues with paths co..., Bryan Donlan, (Tue Apr 8, 9:30 pm)

[PATCH 7/8] Use test_set_editor in t9001-send-email.sh, Bryan Donlan, (Tue Apr 8, 9:30 pm)

Re: [PATCH 7/8] Use test_set_editor in t9001-send-email.sh, Johannes Sixt, (Wed Apr 9, 3:01 am)

Re: [PATCH 7/8] Use test_set_editor in t9001-send-email.sh, Bryan Donlan, (Wed Apr 9, 10:16 am)

[PATCH 8/8] Fix tests breaking when checkout path contains s..., Bryan Donlan, (Tue Apr 8, 9:30 pm)

Navigation

Popular discussions


linux-kernel:
Tarkan Erimer	Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3
Greg Kroah-Hartman	[PATCH 004/196] Chinese: add translation of SubmittingPatches
Artem Bityutskiy	[PATCH 18/44 take 2] [UBI] build unit implementation
James Morris	Re: LSM conversion to static interface
git:

linux-netdev:
Paul Jackson	[PATCH] cpuset sched_load_balance kmalloc fix
Gerrit Renker	[PATCH 15/37] dccp: Set per-connection CCIDs via socket options
Jarek Poplawski	[PATCH] pkt_sched: Destroy gen estimators under rtnl_lock().
Linus Torvalds	Re: [GIT]: Networking
openbsd-misc:

Colocation donated by:

Who's online

There are currently 1 user and 508 guests online.

Online users

naomiwisdom42

Syndicate