diff mbox

xfstests and ext4

Message ID CAH2r5mvi2LijxWia2sCf+em7rfPtBn_fg6WKU=aftVUTuzXMgw@mail.gmail.com
State Accepted, archived
Headers show

Commit Message

Steve French Nov. 14, 2014, 1:20 a.m. UTC
On Thu, Nov 13, 2014 at 6:02 PM, Dave Chinner <david@fromorbit.com> wrote:
> On Thu, Nov 13, 2014 at 05:38:53PM -0600, Steve French wrote:
>> To get xfstests to build on Ubuntu 14.10 and run (after adding various
>> packages I could get all but 11 of the tests to build and run) I had
>> to install the following packages:
>>
>> sudo apt-get install xfslibs-dev uuid-dev libtool e2fsprogs automake
>> libuuid1 libuuidm-ocaml-dev attr libattr1-dev libacl1-dev libaio-dev
>> xfsprogs libgdbm-dev gawk fio dbench
>
> Documentation patch for the README file?
>
> Cheers,
>
> Dave.

Patch attached



[PATCH] common: update README to list build dependencies

Add sample list of packages (dependencies) required to build XFSTESTS
and add a step to the instructions to mention creating the test
user needed for some of the ACL tests.

Signed-off-by: Steve French <smfrench@gmail.com>
---
 README            | 6 ++++++

Comments

Dave Chinner Nov. 17, 2014, 8:38 p.m. UTC | #1
On Thu, Nov 13, 2014 at 05:20:34PM -0800, Steve French wrote:
> On Thu, Nov 13, 2014 at 6:02 PM, Dave Chinner <david@fromorbit.com> wrote:
> > On Thu, Nov 13, 2014 at 05:38:53PM -0600, Steve French wrote:
> >> To get xfstests to build on Ubuntu 14.10 and run (after adding various
> >> packages I could get all but 11 of the tests to build and run) I had
> >> to install the following packages:
> >>
> >> sudo apt-get install xfslibs-dev uuid-dev libtool e2fsprogs automake
> >> libuuid1 libuuidm-ocaml-dev attr libattr1-dev libacl1-dev libaio-dev
> >> xfsprogs libgdbm-dev gawk fio dbench
> >
> > Documentation patch for the README file?
> >
> > Cheers,
> >
> > Dave.
> 
> Patch attached

There are two copies of it in this email - one pasted in the middle
of the rest of the quoted email text and so is whitespace damaged,
the other as a base64 encoded attachment so can't be quoted and
replied to easily. In future, can you send patches as separate
emails according to:

https://www.kernel.org/doc/Documentation/email-clients.txt

> [PATCH] common: update README to list build dependencies
> 
> Add sample list of packages (dependencies) required to build XFSTESTS
> and add a step to the instructions to mention creating the test
> user needed for some of the ACL tests.
> 
> Signed-off-by: Steve French <smfrench@gmail.com>
> ---
>  README            | 6 ++++++
> 
> 
> diff --git a/README b/README
> index 8a362bd..8db7efe 100644
> --- a/README
> +++ b/README
> @@ -4,8 +4,14 @@ _______________________
> 
>  Building Linux:
>      - cd into the xfstests directory
> +    - install prerequisite packages
> +      For example, for Ubuntu:
> +        "sudo apt-get install xfslibs-dev uuid-dev libtool e2fsprogs
> +         automake gcc libuuid1 libuuidm-ocaml-dev attr libattr1-dev
                                  ^^^^^^^^^^^^^^^^^^

What in xfstests is dependent on ocaml?

Cheers,

Dave.
Steve French Nov. 18, 2014, 2:35 a.m. UTC | #2
On Mon, Nov 17, 2014 at 2:38 PM, Dave Chinner <david@fromorbit.com> wrote:

> the other as a base64 encoded attachment so can't be quoted and
> replied to easily. In future, can you send patches as separate
> emails according to:

Yes.  WIll do.  It is a pain in test  VMs to setup git with gmail two
phase authentication
but doable if I set it up to send mail from a different host.

>>  Building Linux:
>>      - cd into the xfstests directory
>> +    - install prerequisite packages
>> +      For example, for Ubuntu:
>> +        "sudo apt-get install xfslibs-dev uuid-dev libtool e2fsprogs
>> +         automake gcc libuuid1 libuuidm-ocaml-dev attr libattr1-dev
>                                   ^^^^^^^^^^^^^^^^^^
>
> What in xfstests is dependent on ocaml?

That libuuidm-ocaml-dev looks unneeded now (I ended up pulling in one
of the tools a different
way so whatever that pulled in looks superfluous now) - in an earlier
try at this I built a few
of the tools. But there is one missing. I missed in the cut-and-pastes
an obvious very
important dependency when I was transcribing the dependency list -
"quota"   I will add
that to the list and remove the libuuidm-ocaml-dev off now that I have
confirmed the dependencies
list again today on a fresh Ubuntu install.
Dave Chinner Nov. 18, 2014, 8:28 p.m. UTC | #3
On Mon, Nov 17, 2014 at 08:35:39PM -0600, Steve French wrote:
> On Mon, Nov 17, 2014 at 2:38 PM, Dave Chinner <david@fromorbit.com> wrote:
> 
> > the other as a base64 encoded attachment so can't be quoted and
> > replied to easily. In future, can you send patches as separate
> > emails according to:
> 
> Yes.  WIll do.  It is a pain in test  VMs to setup git with gmail two
> phase authentication
> but doable if I set it up to send mail from a different host.

No need for that. Keep the xfstests source tree on your
laptop/workstation and do all your edits there. When you've done
that, simply rsync the tree to the test VM(s) and run "make; ./check
..." from the shell you are running in the test VM.

That way you can manage the code multiple test VMs run from a single
source tree, all in one location, and you don't have to worry about
setting up for mail, backups, losing changes because a test machine
crash ate your recent changes, accidentally sending changes are
root, etc...

Cheers,

Dave.
Christoph Hellwig Nov. 19, 2014, 7:07 p.m. UTC | #4
On Wed, Nov 19, 2014 at 07:28:18AM +1100, Dave Chinner wrote:
> No need for that. Keep the xfstests source tree on your
> laptop/workstation and do all your edits there. When you've done
> that, simply rsync the tree to the test VM(s) and run "make; ./check
> ..." from the shell you are running in the test VM.
> 
> That way you can manage the code multiple test VMs run from a single
> source tree, all in one location, and you don't have to worry about
> setting up for mail, backups, losing changes because a test machine
> crash ate your recent changes, accidentally sending changes are
> root, etc...

Or commit them to a local git branch that you can pull from the VMs
instead of the rsync.  The effect is the same, though.

--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" 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

From 1822e733684f031b13392782e2a7eef1ab102d17 Mon Sep 17 00:00:00 2001
From: Steve French <smfrench@gmail.com>
Date: Thu, 13 Nov 2014 17:07:27 -0800
Subject: [PATCH] common: update README to list build dependencies

Add sample list of packages (dependencies) required to build XFSTESTS
and add a step to the instructions to mention creating the test
user needed for some of the ACL tests.

Signed-off-by: Steve French <smfrench@gmail.com>
---
 README | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/README b/README
index 8a362bd..8db7efe 100644
--- a/README
+++ b/README
@@ -4,8 +4,14 @@  _______________________
 
 Building Linux:
 	- cd into the xfstests directory
+	- install prerequisite packages
+	  For example, for Ubuntu:
+		"sudo apt-get install xfslibs-dev uuid-dev libtool e2fsprogs
+		 automake gcc libuuid1 libuuidm-ocaml-dev attr libattr1-dev
+		 libacl1-dev libaio-dev xfsprogs libgdbm-dev gawk fio dbench"
 	- run make
 	- run make install
+	- create fsgqa test user ("sudo useradd fsgqa")
 	
 Building IRIX:
 	- cd into the xfstests directory 
-- 
2.1.0