[PATCH v1] Add steps to test Newlib patch.

Gedare Bloom gedare at rtems.org
Fri Aug 16 14:46:07 UTC 2019


Vaibhav,

The use of the -vX is to differentiate patches that are sent to review
on the mailing list. If this is not clear in our contributing
guidelines, we probably need to improve the description...

On Fri, Aug 16, 2019 at 8:32 AM Vaibhav Gupta <vaibhavgupta40 at gmail.com> wrote:
>
> Sure,
> Actually made this patch on different branch so used -v1.
> Sending a new one with -v2.
>
> -Vaibhav
>
> On Fri, Aug 16, 2019, 7:58 PM Gedare Bloom <gedare at rtems.org> wrote:
>>
>> Then this should be sent with -v2.
>>
>> On Thu, Aug 15, 2019 at 1:33 PM Vaibhav Gupta <vaibhavgupta40 at gmail.com> wrote:
>> >
>> >
>> >
>> > On Thu, Aug 15, 2019 at 8:51 PM Gedare Bloom <gedare at rtems.org> wrote:
>> >>
>> >> Is this a duplicate of the other "patch v1 Add steps to test Newlib patch." or this one replaces it?
>> >
>> > This one replaces it. Actually I replaced all the previous ones with mine.
>> > The commands, steps I have written are based on what worked for me for number of times.
>> > I build the toolchain twice when RSB got updated, but my changes were not pushed so had to
>> > use patches in RSB. And these steps worked perfectly both times.
>> >
>> > -Vaibhav
>> >>
>> >>
>> >> On Wed, Aug 14, 2019 at 11:45 PM Vaibhav Gupta <vaibhavgupta40 at gmail.com> wrote:
>> >>>
>> >>> Update the checksum to be used for the Newlib patches.
>> >>> Earlier it was msd5, but it is depreciated for security
>> >>> reasons. Now RSB accepts sha512.
>> >>> ---
>> >>>  user/rsb/project-sets.rst | 41 +++++++++++++++++++++++++++++++++------
>> >>>  1 file changed, 35 insertions(+), 6 deletions(-)
>> >>>
>> >>> diff --git a/user/rsb/project-sets.rst b/user/rsb/project-sets.rst
>> >>> index 5ffce26..b01857e 100644
>> >>> --- a/user/rsb/project-sets.rst
>> >>> +++ b/user/rsb/project-sets.rst
>> >>> @@ -261,17 +261,46 @@ in the ``source-builder/config`` template configuration files.
>> >>>  To test a patch simply copy it to your local ``patches`` directory. The RSB
>> >>>  will see the patch is present and will not attempt to download it. Once you are
>> >>>  happy with the patch submit it to the project and a core developer will review
>> >>> -it and add it to the RTEMS Tools git repository.  For example, to test a local
>> >>> -patch for newlib, add the following two lines to the .cfg file in
>> >>> -``rtems/config/tools/`` that is included by the bset you use:
>> >>> +it and add it to the RTEMS Tools git repository.
>> >>> +
>> >>> +Testing a Newlib Patch
>> >>> +~~~~~~~~~~~~~~~~~~~~~~
>> >>> +
>> >>> +To test a local patch for newlib, you need to add the following
>> >>> +two lines to the ``.cfg`` file in ``rsb/rtems/config/tools/`` that is included
>> >>> +by the bset you use:
>> >>> +
>> >>> +.. topic:: Steps:
>> >>> +
>> >>> +  1. Create patches for the changes you want to test. (Note: For RSB, before
>> >>> +     creating Newlib patch, you must run ``autoreconf -fvi`` in the required
>> >>> +     directory after you make changes to the code. This is not required when
>> >>> +     you create patch to send to ``newlib-devel``. But if you want RSB to
>> >>> +     address your changes, your patch should also include regenerated files.)
>> >>> +
>> >>> +  2. Calculate ``sha512`` of your patch.
>> >>> +
>> >>> +  3. Place the patches in ``rsb/rtems/patches`` directory.
>> >>> +
>> >>> +  4. Open the ``.bset`` file used by your BSP in ``rsb/rtems/config``.
>> >>> +     For example, for ``rtems5``, ``SPARC``, the file will be
>> >>> +     ``rsb/rtems/config/5/rtems-sparc.bset``.
>> >>> +
>> >>> +  5. Inside it you will find the name of ``.cfg`` file for Newlib, used by
>> >>> +     your BSP.
>> >>> +     For example, I found ``tools/rtems-gcc-7.4.0-newlib-1d35a003f``.
>> >>> +
>> >>> +  6. Edit your ``.cfg`` file. In my case it will be,
>> >>> +     ``rsb/rtems/config/tools/rtems-gcc-7.4.0-newlib-1d35a003f.cfg``. And
>> >>> +     add the information about your patch as mentioned below.
>> >>>
>> >>>  .. code-block:: spec
>> >>>
>> >>> -    %patch add newlib file://0001-this-is-a-newlib-patch.patch   <1>
>> >>> -    %hash md5 0001-this-is-a-newlib-patch.diff 77d070878112783292461bd6e7db17fb <2>
>> >>> +    %patch add newlib -p1 file://0001-Port-ndbm.patch <1>
>> >>> +    %hash sha512 0001-Port-ndbm.patch 7d999ceeea4f3dc82e8e0aadc09d983a7a68b44470da8a3d61ab6fc558fdba6f2c2de3acc2f32c0b0b97fcc9ab799c27e87afe046544a69519881f947e7881d1 <2>
>> >>>
>> >>>  .. topic:: Items:
>> >>>
>> >>>    1. The diff file prepended with ``file://`` to tell RSB this is a local file.
>> >>>
>> >>> -  2. The output from md5sum on the diff file.
>> >>> +  2. The output from sha512sum on the patch file.
>> >>>
>> >>> --
>> >>> 2.21.0
>> >>> _______________________________________________
>> >>> devel mailing list
>> >>> devel at rtems.org
>> >>> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list