# t7610-mergetool.sh -q isn't quiet

2 messages from 2010-08-20 to 2010-08-20. Participants: Brian Gernhardt, Ævar Arnfjörð Bjarmason.
Thread: https://gitlist.dev/t/24798

## Brian Gernhardt, 2010-08-20 06:50

Subject: t7610-mergetool.sh -q isn't quiet
Message-ID: <E395511F-ACF9-4390-A081-4AC4C072C7CC@gernhardtsoftware.com>
URL: https://gitlist.dev/e/E395511F-ACF9-4390-A081-4AC4C072C7CC%40gernhardtsoftware.com

```
I'm just posting this to the list because I should really already be in bed...

Running t7610-mergetool with -q isn't quiet.  This is also noticeable when running the full test suite in a TAP harness (e.g. prove).

$ ./t7610-mergetool.sh -q
Normal merge conflict for 'file1':
  {local}: modified
  {remote}: modified
Normal merge conflict for 'file2':
  {local}: created
  {remote}: created
Normal merge conflict for 'subdir/file3':
  {local}: modified
  {remote}: modified
# passed all 7 test(s)
1..7

All the output seems to be from the last test.

Also, some of the test style seems odd.  For example:

test_expect_success 'mergetool on file in parent dir' '
    cd subdir && (
    ( yes "" | git mergetool ../file1 >/dev/null 2>&1 ) &&
    ( yes "" | git mergetool ../file2 >/dev/null 2>&1 ) &&
    test "$(cat ../file1)" = "master updated" &&
    test "$(cat ../file2)" = "master new" &&
    git commit -m "branch1 resolved with mergetool - subdir") &&
    cd ..
'

Shouldn't that just be :

    (
     cd subdir &&
     yadda &&
     yadda
    )

If nobody else gets to it, hopefully I'll find time tomorrow.  If not, I should get a fresh supply of tuits over the weekend.

~~ Brian
```

## Ævar Arnfjörð Bjarmason, 2010-08-20 21:03

Subject: Re: t7610-mergetool.sh -q isn't quiet
Message-ID: <AANLkTink1HaOWCYsr5+Q+JAi_zA_gEHmOU1J3fvq8jnd@mail.gmail.com>
URL: https://gitlist.dev/e/AANLkTink1HaOWCYsr5%2BQ%2BJAi_zA_gEHmOU1J3fvq8jnd%40mail.gmail.com
In-Reply-To: <E395511F-ACF9-4390-A081-4AC4C072C7CC@gernhardtsoftware.com>

```
On Fri, Aug 20, 2010 at 06:50, Brian Gernhardt
<brian@gernhardtsoftware.com> wrote:

>    test "$(cat ../file1)" = "master updated" &&
>    test "$(cat ../file2)" = "master new" &&

That should also use test_cmp.

```