[PATCH v1] Add steps to test Newlib patch.

Vaibhav Gupta vaibhavgupta40 at gmail.com
Fri Aug 16 15:03:16 UTC 2019


On Fri, Aug 16, 2019 at 8:16 PM Gedare Bloom <gedare at rtems.org> wrote:

> Vaibhav,
>
> The use of the -vX is to differentiate patches that are sent to review
> on the mailing list.

Yeah, actually I assumed that working on different branch may
allow us to send patches like this.

> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20190816/368dd6da/attachment-0002.html>


More information about the devel mailing list