Re: [PATCH] send-email: improve recipients_cmd()

From: Junio C Hamano <gitster@pobox.com>
Date: Apr 29, 2013, 05:24 UTC
Message-ID: <7vd2tdoqna.fsf@alter.siamese.dyndns.org>
In-Reply-To: <CAMP44s0x8fK-6vbY20eKA9QjWqhiOCvx+KFOgKOZ8q58bwk8Tw@mail.gmail.com>

Felipe Contreras <felipe.contreras@gmail.com> writes:

>>         open $fh, "-|", qw(sh -c), $cmd, @args
>
> That doesn't seem to work for me.

My fault.  It needs to form a command line like this:

	sh -c 'cccmd --frotz --nitfol "$@"' - a r g s

[jc: goes and tries
	$ sh -c 'echo X Y "$@"' - a r g s
	X Y a r g s
]

so if we want to get rid of \Q\E, it would be:

	open $fh, '-|', qw(sh -c), "$cmd " . '"$@"', '-', @args