public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] xfstests 250: remove unwanted new line in 250.out
@ 2012-10-19  3:44 Eryu Guan
  2012-10-19 15:21 ` Carlos Maiolino
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Eryu Guan @ 2012-10-19  3:44 UTC (permalink / raw)
  To: xfs; +Cc: Eryu Guan

Additional new line at the end of 250.out will fail the test, remove it.

Signed-off-by: Eryu Guan <eguan@redhat.com>
---
 250.out | 1 -
 1 file changed, 1 deletion(-)

diff --git a/250.out b/250.out
index 2225d6e..acb042a 100644
--- a/250.out
+++ b/250.out
@@ -12,4 +12,3 @@ realtime =RDEV extsz=XXX blocks=XXX, rtextents=XXX
 *** unmount loop filesystem
 *** check loop filesystem
 *** done
-
-- 
1.7.11.7

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

* Re: [PATCH] xfstests 250: remove unwanted new line in 250.out
  2012-10-19  3:44 [PATCH] xfstests 250: remove unwanted new line in 250.out Eryu Guan
@ 2012-10-19 15:21 ` Carlos Maiolino
  2012-10-23 12:27 ` Christoph Hellwig
  2012-10-29 14:58 ` Rich Johnston
  2 siblings, 0 replies; 4+ messages in thread
From: Carlos Maiolino @ 2012-10-19 15:21 UTC (permalink / raw)
  To: xfs

On Fri, Oct 19, 2012 at 11:44:44AM +0800, Eryu Guan wrote:
> Additional new line at the end of 250.out will fail the test, remove it.
> 
> Signed-off-by: Eryu Guan <eguan@redhat.com>
> ---
>  250.out | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/250.out b/250.out
> index 2225d6e..acb042a 100644
> --- a/250.out
> +++ b/250.out
> @@ -12,4 +12,3 @@ realtime =RDEV extsz=XXX blocks=XXX, rtextents=XXX
>  *** unmount loop filesystem
>  *** check loop filesystem
>  *** done
> -
> -- 
> 1.7.11.7
> 
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs

Looks Good

Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
-- 
--Carlos

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

* Re: [PATCH] xfstests 250: remove unwanted new line in 250.out
  2012-10-19  3:44 [PATCH] xfstests 250: remove unwanted new line in 250.out Eryu Guan
  2012-10-19 15:21 ` Carlos Maiolino
@ 2012-10-23 12:27 ` Christoph Hellwig
  2012-10-29 14:58 ` Rich Johnston
  2 siblings, 0 replies; 4+ messages in thread
From: Christoph Hellwig @ 2012-10-23 12:27 UTC (permalink / raw)
  To: Eryu Guan; +Cc: xfs

On Fri, Oct 19, 2012 at 11:44:44AM +0800, Eryu Guan wrote:
> Additional new line at the end of 250.out will fail the test, remove it.
> 
> Signed-off-by: Eryu Guan <eguan@redhat.com>

Looks good,

Reviewed-by: Christoph Hellwig <hch@lst.de>

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

* Re: xfstests 250: remove unwanted new line in 250.out
  2012-10-19  3:44 [PATCH] xfstests 250: remove unwanted new line in 250.out Eryu Guan
  2012-10-19 15:21 ` Carlos Maiolino
  2012-10-23 12:27 ` Christoph Hellwig
@ 2012-10-29 14:58 ` Rich Johnston
  2 siblings, 0 replies; 4+ messages in thread
From: Rich Johnston @ 2012-10-29 14:58 UTC (permalink / raw)
  To: Eryu Guan; +Cc: xfs

On 10/18/2012 10:44 PM, Eryu Guan wrote:
> Additional new line at the end of 250.out will fail the test, remove it.
>
> Signed-off-by: Eryu Guan <eguan@redhat.com>
> Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
> Reviewed-by: Christoph Hellwig <hch@lst.de>
>
> ---
> 250.out | 1 -
>   1 file changed, 1 deletion(-)
>
> diff --git a/250.out b/250.out
> index 2225d6e..acb042a 100644
> --- a/250.out
> +++ b/250.out
> @@ -12,4 +12,3 @@ realtime =RDEV extsz=XXX blocks=XXX, rtextents=XXX
>   *** unmount loop filesystem
>   *** check loop filesystem
>   *** done
> -
>

Eryu,

This patch has been committed to git://oss.sgi.com/xfs/cmds/xfstests 
master branch, commit d1ffc87.

Thanks
--Rich

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

end of thread, other threads:[~2012-10-29 14:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-19  3:44 [PATCH] xfstests 250: remove unwanted new line in 250.out Eryu Guan
2012-10-19 15:21 ` Carlos Maiolino
2012-10-23 12:27 ` Christoph Hellwig
2012-10-29 14:58 ` Rich Johnston

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox