[PATCH] trivial README file changes!

[PATCH] trivial README file changes!

[malahal]

# HG changeset patch
# User malahal@us.ibm.com
# Node ID eddad6028a4cd4a73775aa85773a5a6237051ab9
# Parent  9b73afea874e2b6d8c4e56f504aa591ac30be5b6

Changed package to program, added kernel build step in the README file

diff -r 1a7383f84989 -r bf3c01c4eac4 docs/check_pkgs

diff -r 9b73afea874e -r eddad6028a4c docs/check_pkgs
--- a/docs/check_pkgs	Fri Jul  8 15:35:43 2005
+++ b/docs/check_pkgs	Fri Jul  8 17:52:39 2005
@@ -4,7 +4,7 @@
         if ! `which $1 1>/dev/null 2>/dev/null`; then
                 echo "*************************************************"
                 echo "*************************************************"
-                echo "* WARNING: Package '$1' is required"
+                echo "* WARNING: Program '$1' is required"
                 echo "*          to build Xen documentation"
                 echo "*************************************************"
                 echo "*************************************************"
diff -r 9b73afea874e -r eddad6028a4c README
--- a/README	Fri Jul  8 15:35:43 2005
+++ b/README	Fri Jul  8 17:52:39 2005
@@ -108,8 +108,8 @@
     # make world
     # make install
 
-   This will create the directories linux-2.6.11-dom0/ and
-   linux-2.6.11-domU/ after first cleaning everything. It will
+   This will create the directories linux-2.6.11-xen0/ and
+   linux-2.6.11-xenU/ after first cleaning everything. It will
    create and install into the dist/ directory which is the
    default install location. It will build the xen binary
    (xen.gz), the boot images for dom0 and an unpriviledged
@@ -127,6 +127,7 @@
     # cd linux-2.6.11-xen0         # or linux-2.6.11-xenU
     # make ARCH=xen menuconfig     # or xconfig
     # cd ..
+    # make ARCH=xen
     # make dist
     # make install

RE: [PATCH] trivial README file changes!

[Ian Pratt]

 

> -----Original Message-----
> From: xen-devel-bounces@lists.xensource.com 
> [mailto:xen-devel-bounces@lists.xensource.com] On Behalf Of 
> malahal@us.ibm.com
> Sent: 08 July 2005 22:35
> To: xen-devel@lists.xensource.com
> Subject: [Xen-devel] [PATCH] trivial README file changes!
> 
> # HG changeset patch
> # User malahal@us.ibm.com
> # Node ID eddad6028a4cd4a73775aa85773a5a6237051ab9
> # Parent  9b73afea874e2b6d8c4e56f504aa591ac30be5b6
> 
> Changed package to program, added kernel build step in the README file
> 
> diff -r 1a7383f84989 -r bf3c01c4eac4 docs/check_pkgs
> 
> diff -r 9b73afea874e -r eddad6028a4c docs/check_pkgs
> --- a/docs/check_pkgs	Fri Jul  8 15:35:43 2005
> +++ b/docs/check_pkgs	Fri Jul  8 17:52:39 2005
> @@ -4,7 +4,7 @@
>          if ! `which $1 1>/dev/null 2>/dev/null`; then
>                  echo 
> "*************************************************"
>                  echo 
> "*************************************************"
> -                echo "* WARNING: Package '$1' is required"
> +                echo "* WARNING: Program '$1' is required"
>                  echo "*          to build Xen documentation"
>                  echo 
> "*************************************************"
>                  echo 
> "*************************************************"
> diff -r 9b73afea874e -r eddad6028a4c README
> --- a/README	Fri Jul  8 15:35:43 2005
> +++ b/README	Fri Jul  8 17:52:39 2005
> @@ -108,8 +108,8 @@
>      # make world
>      # make install
>  
> -   This will create the directories linux-2.6.11-dom0/ and
> -   linux-2.6.11-domU/ after first cleaning everything. It will
> +   This will create the directories linux-2.6.11-xen0/ and
> +   linux-2.6.11-xenU/ after first cleaning everything. It will
>     create and install into the dist/ directory which is the
>     default install location. It will build the xen binary
>     (xen.gz), the boot images for dom0 and an unpriviledged 
> @@ -127,6 +127,7 @@
>      # cd linux-2.6.11-xen0         # or linux-2.6.11-xenU
>      # make ARCH=xen menuconfig     # or xconfig
>      # cd ..
> +    # make ARCH=xen

What's the intention of this step? It isn't needed.

Ian

Re: [PATCH] trivial README file changes!

[malahal]

The following statement from the README file indicates that "ARCH=xen'
must be specified while building the kernel. But the example did not use
'ARCH=xen' while building the kernel. I could change "make dist" to
"make ARCH=xen dist", but I chose to add an extra line. Is this
statement from the README file really true?

"Include the ARCH=xen directive for all make commands when building the
kernels."

Ian Pratt [m+Ian.Pratt@cl.cam.ac.uk] wrote:
> >      # cd linux-2.6.11-xen0         # or linux-2.6.11-xenU
> >      # make ARCH=xen menuconfig     # or xconfig
> >      # cd ..
> > +    # make ARCH=xen
> 
> What's the intention of this step? It isn't needed.
> 
> Ian
> 

Re: [PATCH] trivial README file changes!

[malahal]

Got it. Looked at the wrong makefile. The target 'dist' is the default
target. Here is a corrected patch file. Thanks, Malahal.

# HG changeset patch
# User malahal@us.ibm.com
# Node ID 5e62f64b20ff6d211ba6fb557b2a4248e26be6e0
# Parent  9b73afea874e2b6d8c4e56f504aa591ac30be5b6

Changed package to program, added kernel build step in the README file

diff -r 9b73afea874e -r 5e62f64b20ff docs/check_pkgs
--- a/docs/check_pkgs	Fri Jul  8 15:35:43 2005
+++ b/docs/check_pkgs	Mon Jul 11 20:50:22 2005
@@ -4,7 +4,7 @@
         if ! `which $1 1>/dev/null 2>/dev/null`; then
                 echo "*************************************************"
                 echo "*************************************************"
-                echo "* WARNING: Package '$1' is required"
+                echo "* WARNING: Program '$1' is required"
                 echo "*          to build Xen documentation"
                 echo "*************************************************"
                 echo "*************************************************"
diff -r 9b73afea874e -r 5e62f64b20ff README
--- a/README	Fri Jul  8 15:35:43 2005
+++ b/README	Mon Jul 11 20:50:22 2005
@@ -108,8 +108,8 @@
     # make world
     # make install
 
-   This will create the directories linux-2.6.11-dom0/ and
-   linux-2.6.11-domU/ after first cleaning everything. It will
+   This will create the directories linux-2.6.11-xen0/ and
+   linux-2.6.11-xenU/ after first cleaning everything. It will
    create and install into the dist/ directory which is the
    default install location. It will build the xen binary
    (xen.gz), the boot images for dom0 and an unpriviledged
@@ -127,7 +127,7 @@
     # cd linux-2.6.11-xen0         # or linux-2.6.11-xenU
     # make ARCH=xen menuconfig     # or xconfig
     # cd ..
-    # make dist
+    # make ARCH=xen dist
     # make install
 
    You can copy your own config into linux-2.6.11-xen0 first.

Re: [PATCH] trivial README file changes!

[Keir Fraser]

On 11 Jul 2005, at 21:53, malahal@us.ibm.com wrote:

> Got it. Looked at the wrong makefile. The target 'dist' is the default
> target. Here is a corrected patch file. Thanks, Malahal.

You don;t need 'ARCH=xen' when building in the root of the repository. 
The build system will automatically add it when it enters the Linux 
subdirs.

  -- Keir