[PATCH] Improve Window's build documentation

Bin Meng bmeng.cn at gmail.com
Wed Jul 20 14:33:39 CEST 2022


On Wed, Jul 20, 2022 at 2:50 PM Martin Bonner <martingreybeard at gmail.com> wrote:
>
> I have recently build the u-boot tools on Windows and ran into
> difficulties.  This patch to the documentation would have avoided
> those difficulties.  Contentious points:
>
> * While I have run the document through a rst previewer, I haven't run
> the full test suite.  I know, I know, everyone thinks _they_ don't
> need to run the tests because reasons, but I think it's true this
> time.
> * I have removed the version numbers from the list of packages.  I
> don't think they are helpful, and are just maintenance overhead.  (I
> don't think I got those versions for example).
>
> I'd be grateful if @Bin Meng could review this, because `git blame`
> says the file I am changing is all his.
>
> Are there any other formalities I need to complete before this can be
> accepted into u-boot?

Thanks for the patch!

Please refer to the Linux kernel documentation on how to document your
changes in the commit message:
https://www.kernel.org/doc/html/latest/process/submitting-patches.html

>
> -----------------------------------------------------------------------
> diff --git a/doc/build/tools.rst b/doc/build/tools.rst
> index c06f915274..95f1246617 100644
> --- a/doc/build/tools.rst
> +++ b/doc/build/tools.rst
> @@ -24,14 +24,20 @@ you can use MSYS2, a software distro and building
> platform for Windows.
>  Download the MSYS2 installer from https://www.msys2.org. Make sure you have
>  installed all required packages below in order to build these host tools::
>
> -   * gcc (9.1.0)
> -   * make (4.2.1)
> -   * bison (3.4.2)
> -   * diffutils (3.7)
> -   * openssl-devel (1.1.1.d)
> -
> -Note the version numbers in these parentheses above are the package versions
> -at the time being when writing this document. The MSYS2 installer tested is
> +   * gcc
> +   * make
> +   * bison
> +   * diffutils
> +   * openssl-devel
> +   * flex
> +   * libgnutls-devel
> +   * libuuid-devel
> +
> +You probably want ``git`` as well.  You can install all these with::
> +
> +    $ pacman -S gcc make bison diffutils openssl=devel flex

s/=/-

> libgnutls-devel libuuid-devel git
> +
> +The MSYS2 installer tested is
>  http://repo.msys2.org/distrib/x86_64/msys2-x86_64-20190524.exe.
>
>  There are 3 MSYS subsystems installed: MSYS2, MinGW32 and MinGW64. Each
>

Regards,
Bin


More information about the U-Boot mailing list