[PATCH v1] Add steps to test Newlib patch.

Vaibhav Gupta vaibhavgupta40 at gmail.com
Sun Aug 18 06:47:16 UTC 2019


Awesome, thanks!

On Fri, Aug 16, 2019 at 8:52 PM Joel Sherrill <joel at rtems.org> wrote:

> Ignoring the v1 vs v2, Jeff Johnson sent me a script to bootstrap newlib
> from the top. I updated it for the addition of fenv.
>
> And you need specifically automake 1.11.6 and autoconf 2.68 per him.
>
> --joel
>
> On Fri, Aug 16, 2019 at 10:03 AM Vaibhav Gupta <vaibhavgupta40 at gmail.com>
> wrote:
> >
> >
> >
> > 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
> >
> > _______________________________________________
> > 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/20190818/9a76334b/attachment-0002.html>


More information about the devel mailing list