All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] btrfs/202: fix golden output
@ 2020-01-08 17:45 Johannes Thumshirn
  2020-01-09  0:14 ` Qu Wenruo
  0 siblings, 1 reply; 2+ messages in thread
From: Johannes Thumshirn @ 2020-01-08 17:45 UTC (permalink / raw)
  To: Eryu Guan; +Cc: linux-btrfs, fstests, Johannes Thumshirn

From: Johannes Thumshirn <johannes.thumshirn@wdc.com>

The golden output of btrfs/202 contains the sequence number 201 instead of
202, fix this.

Signed-off-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
---
 tests/btrfs/202.out | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/btrfs/202.out b/tests/btrfs/202.out
index 938870cf..7f33d49f 100644
--- a/tests/btrfs/202.out
+++ b/tests/btrfs/202.out
@@ -1,4 +1,4 @@
-QA output created by 201
+QA output created by 202
 Create subvolume 'SCRATCH_MNT/a'
 Create subvolume 'SCRATCH_MNT/a/b'
 Create a snapshot of 'SCRATCH_MNT/a' in 'SCRATCH_MNT/c'
-- 
2.16.4


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] btrfs/202: fix golden output
  2020-01-08 17:45 [PATCH] btrfs/202: fix golden output Johannes Thumshirn
@ 2020-01-09  0:14 ` Qu Wenruo
  0 siblings, 0 replies; 2+ messages in thread
From: Qu Wenruo @ 2020-01-09  0:14 UTC (permalink / raw)
  To: Johannes Thumshirn, Eryu Guan; +Cc: linux-btrfs, fstests, Johannes Thumshirn


[-- Attachment #1.1: Type: text/plain, Size: 1157 bytes --]



On 2020/1/9 上午1:45, Johannes Thumshirn wrote:
> From: Johannes Thumshirn <johannes.thumshirn@wdc.com>
> 
> The golden output of btrfs/202 contains the sequence number 201 instead of
> 202, fix this.
> 
> Signed-off-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
> ---
>  tests/btrfs/202.out | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/btrfs/202.out b/tests/btrfs/202.out
> index 938870cf..7f33d49f 100644
> --- a/tests/btrfs/202.out
> +++ b/tests/btrfs/202.out
> @@ -1,4 +1,4 @@
> -QA output created by 201
> +QA output created by 202

This really makes people laughing.

But this also makes me wonder, do we really need the test number in
golden output?

If the diff is from `check`, we have context showing which test case we're.
If the diff is from manually diffing golden and result, then we have the
test number in the path.

The test number in golden output looks duplicated to me.

Eryu, can we just remove it?

Thanks,
Qu

>  Create subvolume 'SCRATCH_MNT/a'
>  Create subvolume 'SCRATCH_MNT/a/b'
>  Create a snapshot of 'SCRATCH_MNT/a' in 'SCRATCH_MNT/c'
> 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-01-09  0:15 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-01-08 17:45 [PATCH] btrfs/202: fix golden output Johannes Thumshirn
2020-01-09  0:14 ` Qu Wenruo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.