diff mbox

[V3] xfstests: fix common filter include in filter.btrfs

Message ID 51746D8A.60707@sandeen.net (mailing list archive)
State Not Applicable
Headers show

Commit Message

Eric Sandeen April 21, 2013, 10:51 p.m. UTC
Not sure how this happened, but filter.btrfs including
itself leads to immense sadness for any file that includes it.
(I got a segfault when I tried to run 307)

It should be including ./common/filter not ./common/filter.btrfs

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---

cripes, I'm sorry that took 3 times to get right :/



--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Rich Johnston April 22, 2013, 1:45 p.m. UTC | #1
On 04/21/2013 05:51 PM, Eric Sandeen wrote:
> Not sure how this happened, but filter.btrfs including
> itself leads to immense sadness for any file that includes it.
> (I got a segfault when I tried to run 307)
>
> It should be including ./common/filter not ./common/filter.btrfs
>
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
> ---
>
> cripes, I'm sorry that took 3 times to get right :/
>

Third time is the charm ;) Looks good.

Reviewed-by: Rich Johnston <rjohnston@sgi.com>

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/common/filter.btrfs b/common/filter.btrfs
index b1aa733..e9a2bc2 100644
--- a/common/filter.btrfs
+++ b/common/filter.btrfs
@@ -1,6 +1,6 @@ 
 # Filters for btrfs command output
 
-. ./common/filter.btrfs
+. ./common/filter
 
 # Some, but not all, commands emit "Btrfs <version>"
 _filter_btrfs_version()