<div dir="auto">I ran the commands and copied and pasted wherever there were mismatches especially with respect to the version numbers.<div dir="auto"><br></div><div dir="auto">Should I resend the patchset with a better commit message?</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 16 Mar 2021, 6:50 pm Gedare Bloom, <<a href="mailto:gedare@rtems.org" target="_blank" rel="noreferrer">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Ida,<br>
<br>
Since the patches now indicate rtems-docs, you don't need to include<br>
that in the commit message. Instead, it is recommended to put the<br>
manual directory name that is modified as the first part of the<br>
commit, e.g., "user: bump version 5 to 6 in start/tools.rst"<br>
<br>
More below:<br>
<br>
On Tue, Mar 16, 2021 at 2:16 AM Ida Delphine <<a href="mailto:idadelm@gmail.com" rel="noreferrer noreferrer" target="_blank">idadelm@gmail.com</a>> wrote:<br>
><br>
> Edit edit sample output of tool suite installation.<br>
> Edit command to check if C cross compiler works and sample output of this command.<br>
> ---<br>
>  user/start/tools.rst | 26 +++++++++++++-------------<br>
>  1 file changed, 13 insertions(+), 13 deletions(-)<br>
><br>
> diff --git a/user/start/tools.rst b/user/start/tools.rst<br>
> index baa2387..9a915ab 100644<br>
> --- a/user/start/tools.rst<br>
> +++ b/user/start/tools.rst<br>
> @@ -12,7 +12,7 @@ Install the Tool Suite<br>
><br>
>  You have chosen an installation prefix, the BSP to build, the tool's<br>
>  architecure and prepared the source for the RSB in the previous sections.  We<br>
> -have chosen :file:`$HOME/quick-start/rtems/5` as the installation prefix, the<br>
> +have chosen :file:`$HOME/quick-start/rtems/6` as the installation prefix, the<br>
>  ``erc32`` BSP and the SPARC architecture name of ``sparc-rtems5``, and unpacked<br>
>  the RSB source in :file:`$HOME/quick-start/src`.<br>
><br>
> @@ -25,7 +25,7 @@ Build and install the tool suite:<br>
>  .. code-block:: none<br>
><br>
>      cd $HOME/quick-start/src/rsb/rtems<br>
> -    ../source-builder/sb-set-builder --prefix=$HOME/quick-start/rtems/5 5/rtems-sparc<br>
> +    ../source-builder/sb-set-builder --prefix=$HOME/quick-start/rtems/6 6/rtems-sparc<br>
><br>
>  This command should output something like this (omitted lines are denoted by<br>
>  ...). The build host appears as part of the name of the package being<br>
> @@ -33,19 +33,19 @@ built. The name you see may vary depending on the host you are using:<br>
><br>
>  .. code-block:: none<br>
><br>
> -    RTEMS Source Builder - Set Builder, 5.1.0<br>
> -    Build Set: 5/rtems-sparc<br>
> +    RTEMS Source Builder - Set Builder, 6<br>
> +    Build Set: 6/rtems-sparc<br>
>      ...<br>
>      config: tools/rtems-binutils-2.34.cfg<br>
> -    package: sparc-rtems5-binutils-2.34-x86_64-freebsd12.1-1<br>
> -    building: sparc-rtems5-binutils-2.34-x86_64-freebsd12.1-1<br>
> -    sizes: sparc-rtems5-binutils-2.34-x86_64-freebsd12.1-1: 305.866MB (installed: 29.966MB)<br>
> -    cleaning: sparc-rtems5-binutils-2.34-x86_64-freebsd12.1-1<br>
> +    package: sparc-rtems6=-binutils-2.34-x86_64-freebsd12.1-1<br>
> +    building: sparc-rtems6-binutils-2.34-x86_64-freebsd12.1-1<br>
> +    sizes: sparc-rtems6-binutils-2.34-x86_64-freebsd12.1-1: 305.866MB (installed: 29.966MB)<br>
> +    cleaning: sparc-rtems6-binutils-2.34-x86_64-freebsd12.1-1<br>
>      reporting: tools/rtems-binutils-2.34.cfg -> sparc-rtems5-binutils-2.34-x86_64-freebsd12.1-1.txt<br>
>      reporting: tools/rtems-binutils-2.34.cfg -> sparc-rtems5-binutils-2.34-x86_64-freebsd12.1-1.xml<br>
>      config: tools/rtems-gcc-7.5.0-newlib-fbaa096.cfg<br>
> -    package: sparc-rtems5-gcc-7.5.0-newlib-fbaa096-x86_64-freebsd12.1-1<br>
> -    building: sparc-rtems5-gcc-7.5.0-newlib-fbaa096-x86_64-freebsd12.1-1<br>
> +    package: sparc-rtems6-gcc-7.5.0-newlib-fbaa096-x86_64-freebsd12.1-1<br>
> +    building: sparc-rtems6-gcc-7.5.0-newlib-fbaa096-x86_64-freebsd12.1-1<br>
>      ....<br>
>      Build Sizes: usage: 5.684GB total: 1.112GB (sources: 143.803MB, patches: 21.348KB, installed 995.188MB)<br>
>      Build Set: Time 0:21:35.626294<br>
<br>
Did you run the command and capture the sample output, or do you edit<br>
the sample output manually?<br>
<br>
It would be better to copy-paste the actual sample output. Making this<br>
version change update automatic could be a nice small scripting<br>
exercise.<br>
<br>
> @@ -55,7 +55,7 @@ works with the following command:<br>
><br>
>  .. code-block:: none<br>
><br>
> -    $HOME/quick-start/rtems/5/bin/sparc-rtems5-gcc --version<br>
> +    $HOME/quick-start/rtems/6/bin/sparc-rtems6-gcc --version<br>
><br>
>  This command should output something like below.  The version informtion helps<br>
Unrelated, but can you fix this typo: information<br>
<br>
>  you to identify the exact sources used to build the cross compiler of your<br>
> @@ -69,8 +69,8 @@ source code used.<br>
><br>
>  .. code-block:: none<br>
><br>
> -    sparc-rtems5-gcc (GCC) 7.5.0 20191114 (RTEMS 5, RSB 5.1.0, Newlib fbaa096)<br>
> -    Copyright (C) 2017 Free Software Foundation, Inc.<br>
> +    sparc-rtems6-gcc (GCC) 10.2.1 20210309 (RTEMS 6, RSB 5e449fb5c2cb6812a238f9f9764fd339cbbf05c2, Newlib d10d0d9)<br>
> +    Copyright (C) 2020 Free Software Foundation, Inc.<br>
>      This is free software; see the source for copying conditions.  There is NO<br>
>      warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.<br>
><br>
> --<br>
> 2.25.1<br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" rel="noreferrer noreferrer" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>