Re: [PATCH 1/4] x86, build: Fact out fdimage/isoimage generation commands to standalone script

From: Ingo Molnar
Date: Sun Nov 05 2017 - 04:32:20 EST



A few spelling fixes:

in the title:

s/Fact out
/Factor out

* changbin.du@xxxxxxxxx <changbin.du@xxxxxxxxx> wrote:

> From: Changbin Du <changbin.du@xxxxxxxxx>
>
> The build message for fdimage/isoimage are pretty unstructured. The raw
> shell command blocks are printed. We can improve them as regular build
> system messages. Besides, writing commands in shell script is much more
> easier than in a Makefile.

s/much more easier
/much more easy

>
> See Ingo's suggestion here https://lkml.org/lkml/2017/10/31/124.
>
> This patch fact out the commands used for fdimage/isoimage generation from
> arch/x86/boot/Makefile to new script arch/x86/boot/genimage.sh. Then add a
> new kbuild command 'genimage' which invokes the new script. All
> fdimages/isoimage now is generated by call to 'genimage' with different
> parameters.

s/fact out
/factors out

s/to new script
to a new script

s/Then add
/Then it adds

s/a new kbuild command 'genimage'
/the new 'genimage' kbuild command

s/All fdimages/isoimage now is generated by call to
/All fdimage/isoimage files are now generated by a call to

> +# $3 - kernel bzImage file
> +# $4 - mtool configuration file
> +# $5 - kernel cmdline
> +# $6 - inird image file
> +#

The new script is much easier to read!

Thanks,

Ingo