On Sun, Apr 5, 2020 at 11:07 AM Cláudio Maia <cl...@isep.ipp.pt> wrote:
>
> Hello everyone,
>
> I have reviewed the file user/rsb/configuration.rst and I want to send a 
> patch with my proposed changes. However, before doing that I would like to 
> clarify some aspects before sending the patch, namely:
>
> - In the section 13.5.7 it is mentioned the following "The Device Tree 
> Compiler source code can be downloaded from http://www.jdl.com/software.";. 
> However this web site is offline and I'm not sure if it is supposed to be 
> online or it the went down between the time the documentation was produced 
> and now. Should we correct the link (if so, which should be the new one?) or 
> leave it as it is for example purposes and put a note on the document stating 
> that it is offline?
>
The RSB recipe for dtc now uses https://www.kernel.org/pub/software/utils/dtc/
I guess the example should be updated, consistent with what is
available in the RSB tree. Can you open a ticket with milestone 6.1?
patches welcome :)

> - In section 13.5.1.1, the "xy" compression format is mentioned, however a 
> search on google revealed no such format. Is this a typo or does this format 
> really exist?
>
xz, patch sent.

> - In section 13.5.2.1, the following text appears"See <<X1,``_configdir``>> 
> variable for details.". What is "<<X1,``_configdir``>>" supposed to mean? 
> Should it be a cross reference to some other place?
>
That's an xref leftover before conversion of this doc from asciidoc to
sphinx/rst. I believe it intends to orient the reader to the
definition of _configdir provided at
https://docs.rtems.org/branches/master/user/rsb/configuration.html#configuration

I just sent a patch that updates the link.

> - In section 13.5.7.4., it is mentioned a "|DESTDIR|" and a "DISTDIR". I 
> believe there is a typo on the second one, as it is not mentioned anywhere 
> else on the text in the page. Can someone please confirm if my judgement is 
> correct?
>
This is the dtc example, add to the ticket.  And yes, it appears to be a typo.

> - In section 13.5.8.2, a different notation is used using + signs, as for 
> instance "+%prep+" and "+%source+" which I also believe it is a typo and 
> these should be using this notation``%prep``. Again, can please someone 
> confirm this?
>
Another holdover from the conversion to sphinx/rst. Patch sent.

> Regards,
> Cláudio

Thanks for your review,
Gedare

> _______________________________________________
> devel mailing list
> devel@rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to