* [PATCH v4] Improve Windows build documentation @ 2022-07-25 7:42 Martin Bonner 2022-07-25 7:46 ` Bin Meng 2022-07-27 13:29 ` Heinrich Schuchardt 0 siblings, 2 replies; 10+ messages in thread From: Martin Bonner @ 2022-07-25 7:42 UTC (permalink / raw) To: u-boot, Bin Meng * Add three more modules that are required. * Remove the version numbers (because they are hard to keep in sync with the latest MSYS2 versions) * Add a pacman command line to install everything. Signed-off-by: Martin Bonner <martingreybeard@gmail.com> --- doc/build/tools.rst | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/doc/build/tools.rst b/doc/build/tools.rst index c06f915274..5f8a04a31b 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 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 -- Martin Bonner ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v4] Improve Windows build documentation 2022-07-25 7:42 [PATCH v4] Improve Windows build documentation Martin Bonner @ 2022-07-25 7:46 ` Bin Meng 2022-07-27 13:29 ` Heinrich Schuchardt 1 sibling, 0 replies; 10+ messages in thread From: Bin Meng @ 2022-07-25 7:46 UTC (permalink / raw) To: Martin Bonner; +Cc: U-Boot Mailing List On Mon, Jul 25, 2022 at 3:42 PM Martin Bonner <martingreybeard@gmail.com> wrote: > > * Add three more modules that are required. > * Remove the version numbers (because they are hard to keep in sync > with the latest MSYS2 versions) > * Add a pacman command line to install everything. > > Signed-off-by: Martin Bonner <martingreybeard@gmail.com> > --- > doc/build/tools.rst | 22 ++++++++++++++-------- > 1 file changed, 14 insertions(+), 8 deletions(-) > Reviewed-by: Bin Meng <bmeng.cn@gmail.com> ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4] Improve Windows build documentation 2022-07-25 7:42 [PATCH v4] Improve Windows build documentation Martin Bonner 2022-07-25 7:46 ` Bin Meng @ 2022-07-27 13:29 ` Heinrich Schuchardt 2022-07-27 13:51 ` Heinrich Schuchardt 2022-07-27 15:07 ` Martin Bonner 1 sibling, 2 replies; 10+ messages in thread From: Heinrich Schuchardt @ 2022-07-27 13:29 UTC (permalink / raw) To: Martin Bonner; +Cc: Bin Meng, u-boot On 7/25/22 09:42, Martin Bonner wrote: > * Add three more modules that are required. > * Remove the version numbers (because they are hard to keep in sync > with the latest MSYS2 versions) > * Add a pacman command line to install everything. > > Signed-off-by: Martin Bonner <martingreybeard@gmail.com> Your mail is not a valid patch: $ git am /tmp/0.patch Applying: Improve Windows build documentation error: corrupt patch at line 10 Patch failed at 0001 Improve Windows build documentation Please, use 'git send-email' for sending patches. Best regards Heinrich > --- > doc/build/tools.rst | 22 ++++++++++++++-------- > 1 file changed, 14 insertions(+), 8 deletions(-) > > diff --git a/doc/build/tools.rst b/doc/build/tools.rst > index c06f915274..5f8a04a31b 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 > 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 > -- > Martin Bonner ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4] Improve Windows build documentation 2022-07-27 13:29 ` Heinrich Schuchardt @ 2022-07-27 13:51 ` Heinrich Schuchardt 2022-07-27 13:56 ` Heinrich Schuchardt 2022-07-27 15:07 ` Martin Bonner 1 sibling, 1 reply; 10+ messages in thread From: Heinrich Schuchardt @ 2022-07-27 13:51 UTC (permalink / raw) To: Martin Bonner; +Cc: Bin Meng, u-boot On 7/27/22 15:29, Heinrich Schuchardt wrote: > On 7/25/22 09:42, Martin Bonner wrote: >> * Add three more modules that are required. >> * Remove the version numbers (because they are hard to keep in sync >> with the latest MSYS2 versions) >> * Add a pacman command line to install everything. >> >> Signed-off-by: Martin Bonner <martingreybeard@gmail.com> > > Your mail is not a valid patch: > > $ git am /tmp/0.patch > Applying: Improve Windows build documentation > error: corrupt patch at line 10 > Patch failed at 0001 Improve Windows build documentation > > Please, use 'git send-email' for sending patches. > > Best regards > > Heinrich > >> --- >> doc/build/tools.rst | 22 ++++++++++++++-------- >> 1 file changed, 14 insertions(+), 8 deletions(-) >> >> diff --git a/doc/build/tools.rst b/doc/build/tools.rst >> index c06f915274..5f8a04a31b 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 This should not be pre-formatted text but simply a list. >> + >> +You probably want ``git`` as well. You can install all these with:: Allow for syntax highlighting: .. code-block:: bash >> + >> + $ pacman -S gcc make bison diffutils openssl-devel flex >> libgnutls-devel libuuid-devel git The line should be limited to 80 characters. Use \. Please, remove the leading $. Best regards Heinrich >> + >> +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 >> -- >> Martin Bonner ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4] Improve Windows build documentation 2022-07-27 13:51 ` Heinrich Schuchardt @ 2022-07-27 13:56 ` Heinrich Schuchardt 2022-07-27 14:15 ` Tom Rini 0 siblings, 1 reply; 10+ messages in thread From: Heinrich Schuchardt @ 2022-07-27 13:56 UTC (permalink / raw) To: Martin Bonner; +Cc: Bin Meng, u-boot On 7/27/22 15:51, Heinrich Schuchardt wrote: > On 7/27/22 15:29, Heinrich Schuchardt wrote: >> On 7/25/22 09:42, Martin Bonner wrote: >>> * Add three more modules that are required. >>> * Remove the version numbers (because they are hard to keep in sync >>> with the latest MSYS2 versions) >>> * Add a pacman command line to install everything. >>> >>> Signed-off-by: Martin Bonner <martingreybeard@gmail.com> >> >> Your mail is not a valid patch: >> >> $ git am /tmp/0.patch >> Applying: Improve Windows build documentation >> error: corrupt patch at line 10 >> Patch failed at 0001 Improve Windows build documentation >> >> Please, use 'git send-email' for sending patches. >> >> Best regards >> >> Heinrich >> >>> --- >>> doc/build/tools.rst | 22 ++++++++++++++-------- >>> 1 file changed, 14 insertions(+), 8 deletions(-) >>> >>> diff --git a/doc/build/tools.rst b/doc/build/tools.rst >>> index c06f915274..5f8a04a31b 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 > > This should not be pre-formatted text but simply a list. > >>> + >>> +You probably want ``git`` as well. You can install all these with:: > > Allow for syntax highlighting: > > .. code-block:: bash > >>> + >>> + $ pacman -S gcc make bison diffutils openssl-devel flex >>> libgnutls-devel libuuid-devel git > > The line should be limited to 80 characters. Use \. > Please, remove the leading $. > > Best regards > > Heinrich > >>> + >>> +The MSYS2 installer tested is >>> http://repo.msys2.org/distrib/x86_64/msys2-x86_64-20190524.exe. Do you really suggest to download a 3 year old version? Shouldn't we simply remove this sentence? Best regards Heinrich >>> >>> There are 3 MSYS subsystems installed: MSYS2, MinGW32 and MinGW64. >>> Each >>> -- >>> Martin Bonner > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4] Improve Windows build documentation 2022-07-27 13:56 ` Heinrich Schuchardt @ 2022-07-27 14:15 ` Tom Rini 2022-07-27 15:15 ` Martin Bonner 0 siblings, 1 reply; 10+ messages in thread From: Tom Rini @ 2022-07-27 14:15 UTC (permalink / raw) To: Heinrich Schuchardt; +Cc: Martin Bonner, Bin Meng, u-boot [-- Attachment #1: Type: text/plain, Size: 2896 bytes --] On Wed, Jul 27, 2022 at 03:56:07PM +0200, Heinrich Schuchardt wrote: > On 7/27/22 15:51, Heinrich Schuchardt wrote: > > On 7/27/22 15:29, Heinrich Schuchardt wrote: > > > On 7/25/22 09:42, Martin Bonner wrote: > > > > * Add three more modules that are required. > > > > * Remove the version numbers (because they are hard to keep in sync > > > > with the latest MSYS2 versions) > > > > * Add a pacman command line to install everything. > > > > > > > > Signed-off-by: Martin Bonner <martingreybeard@gmail.com> > > > > > > Your mail is not a valid patch: > > > > > > $ git am /tmp/0.patch > > > Applying: Improve Windows build documentation > > > error: corrupt patch at line 10 > > > Patch failed at 0001 Improve Windows build documentation > > > > > > Please, use 'git send-email' for sending patches. > > > > > > Best regards > > > > > > Heinrich > > > > > > > --- > > > > doc/build/tools.rst | 22 ++++++++++++++-------- > > > > 1 file changed, 14 insertions(+), 8 deletions(-) > > > > > > > > diff --git a/doc/build/tools.rst b/doc/build/tools.rst > > > > index c06f915274..5f8a04a31b 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 > > > > This should not be pre-formatted text but simply a list. > > > > > > + > > > > +You probably want ``git`` as well. You can install all these with:: > > > > Allow for syntax highlighting: > > > > .. code-block:: bash > > > > > > + > > > > + $ pacman -S gcc make bison diffutils openssl-devel flex > > > > libgnutls-devel libuuid-devel git > > > > The line should be limited to 80 characters. Use \. > > Please, remove the leading $. > > > > Best regards > > > > Heinrich > > > > > > + > > > > +The MSYS2 installer tested is > > > > http://repo.msys2.org/distrib/x86_64/msys2-x86_64-20190524.exe. > > Do you really suggest to download a 3 year old version? It should be kept in-sync with what .azure-pipeline.yml does which is a newer version than that, yes. -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4] Improve Windows build documentation 2022-07-27 14:15 ` Tom Rini @ 2022-07-27 15:15 ` Martin Bonner 2022-07-27 15:24 ` Tom Rini 0 siblings, 1 reply; 10+ messages in thread From: Martin Bonner @ 2022-07-27 15:15 UTC (permalink / raw) To: Heinrich Schuchardt, u-boot, Tom Rini Martin On Wed, 27 Jul 2022 at 15:15, Tom Rini <trini@konsulko.com> wrote: > On Wed, Jul 27, 2022 at 03:56:07PM +0200, Heinrich Schuchardt wrote: > > On 7/27/22 15:51, Heinrich Schuchardt wrote: > > > On 7/27/22 15:29, Heinrich Schuchardt wrote: > > > > On 7/25/22 09:42, Martin Bonner wrote: > > > > > * Add three more modules that are required. > > > > > * Remove the version numbers (because they are hard to keep in sync > > > > > with the latest MSYS2 versions) > > > > > * Add a pacman command line to install everything. > > > > > > > > > > Signed-off-by: Martin Bonner <martingreybeard@gmail.com> > > > > > > > > Your mail is not a valid patch: > > > > > > > > $ git am /tmp/0.patch > > > > Applying: Improve Windows build documentation > > > > error: corrupt patch at line 10 > > > > Patch failed at 0001 Improve Windows build documentation > > > > > > > > Please, use 'git send-email' for sending patches. > > > > > > > > Best regards > > > > > > > > Heinrich > > > > > > > > > --- > > > > > doc/build/tools.rst | 22 ++++++++++++++-------- > > > > > 1 file changed, 14 insertions(+), 8 deletions(-) > > > > > > > > > > diff --git a/doc/build/tools.rst b/doc/build/tools.rst > > > > > index c06f915274..5f8a04a31b 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 > > > > > > This should not be pre-formatted text but simply a list. > I don't understand what you mean. That _is_ a (bulleted) list (or have I misunderstood? - very possible) > > > > > > > > + > > > > > +You probably want ``git`` as well. You can install all these > with:: > > > > > > Allow for syntax highlighting: > > > > > > .. code-block:: bash > There's no syntax to highlight! > > > > > > > > + > > > > > + $ pacman -S gcc make bison diffutils openssl-devel flex > > > > > libgnutls-devel libuuid-devel git > > > > > > The line should be limited to 80 characters. Use \. > The line length is about 94 characters. I don't think that is excessive. I believe Tom has said u-boot has given up on a strict 80-character limit. > > Please, remove the leading $. > I find it helpful to indicate the text is a command. > > > > > > Best regards > > > > > > Heinrich > > > > > > > > + > > > > > +The MSYS2 installer tested is > > > > > http://repo.msys2.org/distrib/x86_64/msys2-x86_64-20190524.exe. > > > > Do you really suggest to download a 3 year old version? > > It should be kept in-sync with what .azure-pipeline.yml does which is a > newer version than that, yes. > I can't find a file of that name in the repo. Wouldn't it be better to just do as Heinrich suggests, and drop the line entirely? ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4] Improve Windows build documentation 2022-07-27 15:15 ` Martin Bonner @ 2022-07-27 15:24 ` Tom Rini 0 siblings, 0 replies; 10+ messages in thread From: Tom Rini @ 2022-07-27 15:24 UTC (permalink / raw) To: Martin Bonner; +Cc: Heinrich Schuchardt, u-boot [-- Attachment #1: Type: text/plain, Size: 4722 bytes --] On Wed, Jul 27, 2022 at 04:15:52PM +0100, Martin Bonner wrote: > Martin > > > On Wed, 27 Jul 2022 at 15:15, Tom Rini <trini@konsulko.com> wrote: > > > On Wed, Jul 27, 2022 at 03:56:07PM +0200, Heinrich Schuchardt wrote: > > > On 7/27/22 15:51, Heinrich Schuchardt wrote: > > > > On 7/27/22 15:29, Heinrich Schuchardt wrote: > > > > > On 7/25/22 09:42, Martin Bonner wrote: > > > > > > * Add three more modules that are required. > > > > > > * Remove the version numbers (because they are hard to keep in sync > > > > > > with the latest MSYS2 versions) > > > > > > * Add a pacman command line to install everything. > > > > > > > > > > > > Signed-off-by: Martin Bonner <martingreybeard@gmail.com> > > > > > > > > > > Your mail is not a valid patch: > > > > > > > > > > $ git am /tmp/0.patch > > > > > Applying: Improve Windows build documentation > > > > > error: corrupt patch at line 10 > > > > > Patch failed at 0001 Improve Windows build documentation > > > > > > > > > > Please, use 'git send-email' for sending patches. > > > > > > > > > > Best regards > > > > > > > > > > Heinrich > > > > > > > > > > > --- > > > > > > doc/build/tools.rst | 22 ++++++++++++++-------- > > > > > > 1 file changed, 14 insertions(+), 8 deletions(-) > > > > > > > > > > > > diff --git a/doc/build/tools.rst b/doc/build/tools.rst > > > > > > index c06f915274..5f8a04a31b 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 > > > > > > > > This should not be pre-formatted text but simply a list. > > > > I don't understand what you mean. That _is_ a (bulleted) list (or have I > misunderstood? - very possible) It's a matter, I think, of how you need to write the rST so that it renders nicely. Leading spaces lead to one way, no leading spaces (what Heinrich asks for) renders another way. This, I think, is spelled out in the Linux kernel docs about writing docs, which we now do link to. > > > > > > +You probably want ``git`` as well. You can install all these > > with:: > > > > > > > > Allow for syntax highlighting: > > > > > > > > .. code-block:: bash > > > There's no syntax to highlight! > > > > > > > > > > > + > > > > > > + $ pacman -S gcc make bison diffutils openssl-devel flex > > > > > > libgnutls-devel libuuid-devel git > > > > > > > > The line should be limited to 80 characters. Use \. > > > The line length is about 94 characters. I don't think that is excessive. > I believe Tom has said u-boot has given up on a strict 80-character limit. > > > > Please, remove the leading $. > > > > I find it helpful to indicate the text is a command. These three comments at least I think go together. Have you done a "make htmldocs" and reviewed the output? In a code-block tagged as bash thing should look nice and then it's a general style guide to not start a command with '$' when there's no sample output to distinguish it from. And I could go back and forth on if I think that's right but we're consistent in our docs, and consistency matters more in this regard I believe. > > > > > > + > > > > > > +The MSYS2 installer tested is > > > > > > http://repo.msys2.org/distrib/x86_64/msys2-x86_64-20190524.exe. > > > > > > Do you really suggest to download a 3 year old version? > > > > It should be kept in-sync with what .azure-pipeline.yml does which is a > > newer version than that, yes. > > > I can't find a file of that name in the repo. Wouldn't it be better to > just do as Heinrich suggests, and drop the line entirely? That too, along with making sure we have appropriate overall wording to make sure it's clear MSYS2 is to be used, is a better still idea, yes. -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4] Improve Windows build documentation 2022-07-27 13:29 ` Heinrich Schuchardt 2022-07-27 13:51 ` Heinrich Schuchardt @ 2022-07-27 15:07 ` Martin Bonner 2022-07-27 16:21 ` Heinrich Schuchardt 1 sibling, 1 reply; 10+ messages in thread From: Martin Bonner @ 2022-07-27 15:07 UTC (permalink / raw) To: Heinrich Schuchardt; +Cc: u-boot Martin On Wed, 27 Jul 2022 at 14:29, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote: > On 7/25/22 09:42, Martin Bonner wrote: > > * Add three more modules that are required. > > * Remove the version numbers (because they are hard to keep in sync > > with the latest MSYS2 versions) > > * Add a pacman command line to install everything. > > > > Signed-off-by: Martin Bonner <martingreybeard@gmail.com> > > Your mail is not a valid patch: > > $ git am /tmp/0.patch > Applying: Improve Windows build documentation > error: corrupt patch at line 10 > Patch failed at 0001 Improve Windows build documentation > > Please, use 'git send-email' for sending patches. > I'm afraid I can't do that. The machine that the patches are on is at work, and behind a firewall that doesn't allow SMTP through. In addition, I don't think gmail likes SMTP either. Do you have any suggestions how to work round this? > > Best regards > > Heinrich > > > --- > > doc/build/tools.rst | 22 ++++++++++++++-------- > > 1 file changed, 14 insertions(+), 8 deletions(-) > > > > diff --git a/doc/build/tools.rst b/doc/build/tools.rst > > index c06f915274..5f8a04a31b 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 > > 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 > > -- > > Martin Bonner > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v4] Improve Windows build documentation 2022-07-27 15:07 ` Martin Bonner @ 2022-07-27 16:21 ` Heinrich Schuchardt 0 siblings, 0 replies; 10+ messages in thread From: Heinrich Schuchardt @ 2022-07-27 16:21 UTC (permalink / raw) To: Martin Bonner; +Cc: u-boot On 7/27/22 17:07, Martin Bonner wrote: > > Martin > > > On Wed, 27 Jul 2022 at 14:29, Heinrich Schuchardt <xypron.glpk@gmx.de > <mailto:xypron.glpk@gmx.de>> wrote: > > On 7/25/22 09:42, Martin Bonner wrote: > > * Add three more modules that are required. > > * Remove the version numbers (because they are hard to keep in sync > > with the latest MSYS2 versions) > > * Add a pacman command line to install everything. > > > > Signed-off-by: Martin Bonner <martingreybeard@gmail.com > <mailto:martingreybeard@gmail.com>> > > Your mail is not a valid patch: > > $ git am /tmp/0.patch > Applying: Improve Windows build documentation > error: corrupt patch at line 10 > Patch failed at 0001 Improve Windows build documentation > > Please, use 'git send-email' for sending patches. > > > I'm afraid I can't do that. The machine that the patches are on is at > work, and behind a firewall that doesn't allow SMTP through. Then you will have to send the patches from home. > In addition, I don't think gmail likes SMTP either. See Check Gmail through other email platforms https://support.google.com/mail/answer/7126229?hl=en Best regards Heinrich > > Do you have any suggestions how to work round this? > > > Best regards > > Heinrich > > > --- > > doc/build/tools.rst | 22 ++++++++++++++-------- > > 1 file changed, 14 insertions(+), 8 deletions(-) > > > > diff --git a/doc/build/tools.rst b/doc/build/tools.rst > > index c06f915274..5f8a04a31b 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 > <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 > > libgnutls-devel libuuid-devel git > > + > > +The MSYS2 installer tested is > > http://repo.msys2.org/distrib/x86_64/msys2-x86_64-20190524.exe > <http://repo.msys2.org/distrib/x86_64/msys2-x86_64-20190524.exe>. > > > > There are 3 MSYS subsystems installed: MSYS2, MinGW32 and > MinGW64. Each > > -- > > Martin Bonner > ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-07-27 16:21 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-07-25 7:42 [PATCH v4] Improve Windows build documentation Martin Bonner 2022-07-25 7:46 ` Bin Meng 2022-07-27 13:29 ` Heinrich Schuchardt 2022-07-27 13:51 ` Heinrich Schuchardt 2022-07-27 13:56 ` Heinrich Schuchardt 2022-07-27 14:15 ` Tom Rini 2022-07-27 15:15 ` Martin Bonner 2022-07-27 15:24 ` Tom Rini 2022-07-27 15:07 ` Martin Bonner 2022-07-27 16:21 ` Heinrich Schuchardt
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.