Re: [PATCH rtems-docs] Add option --build-manuals to build multiple specific manuals.

2022-02-13 Thread Chris Johns
On 12/2/22 3:37 am, Shashvat wrote: > Hi Gedare!! > > Is there a ticket associated with this, or any feature request? Or >> just something you thought of doing? >> > > I am sorry I should have mentioned the motive behind the option. > I was planning to work on ticket # which works on a specif

Re: [PATCH rtems-docs] Add option --build-manuals to build multiple specific manuals.

2022-02-11 Thread Joel Sherrill
A ticket would be nice on this since it is a user facing feature. Is there anything in the README for the docs that should highlight this? --joel On Fri, Feb 11, 2022 at 8:18 PM Gedare Bloom wrote: > > That looks better. My other comments still need to be addressed. You > should also make sure

Re: [PATCH rtems-docs] Add option --build-manuals to build multiple specific manuals.

2022-02-11 Thread Gedare Bloom
That looks better. My other comments still need to be addressed. You should also make sure the patch applies cleanly to a fresh repo. I think I saw some whitespace problems. It will be easier to review if you can figure out how to get git-send-email to work. On Fri, Feb 11, 2022 at 10:43 AM Shash

Re: [PATCH rtems-docs] Add option --build-manuals to build multiple specific manuals.

2022-02-11 Thread Shashvat
Here is the patch, I ran yapf on it and it didn't introduce any changes. Please take a look and let me know what you think. On Fri, 11 Feb 2022, 10:31 pm Gedare Bloom, wrote: > On Fri, Feb 11, 2022 at 9:38 AM Shashvat > wrote: > >> > >> > > > > Hi Gedare!! > > > >> Is there a ticket associated

Re: [PATCH rtems-docs] Add option --build-manuals to build multiple specific manuals.

2022-02-11 Thread Gedare Bloom
On Fri, Feb 11, 2022 at 9:38 AM Shashvat wrote: >> >> > > Hi Gedare!! > >> Is there a ticket associated with this, or any feature request? Or >> just something you thought of doing? > > > I am sorry I should have mentioned the motive behind the option. > I was planning to work on ticket # whic

Re: [PATCH rtems-docs] Add option --build-manuals to build multiple specific manuals.

2022-02-11 Thread Shashvat
> > > Hi Gedare!! Is there a ticket associated with this, or any feature request? Or > just something you thought of doing? > I am sorry I should have mentioned the motive behind the option. I was planning to work on ticket # which works on a specific posix-users manual afaik. I wanted waf to

Re: [PATCH rtems-docs] Add option --build-manuals to build multiple specific manuals.

2022-02-11 Thread Gedare Bloom
Hi Shashvat, On Fri, Feb 11, 2022 at 1:38 AM Shashvat wrote: > > Hello all!! > This patch adds the --build-manuals option enabling one to build specific > documentation manuals. I also updated the README. > Is there a ticket associated with this, or any feature request? Or just something you tho

[PATCH rtems-docs] Add option --build-manuals to build multiple specific manuals.

2022-02-11 Thread Shashvat
Hello all!! This patch adds the --build-manuals option enabling one to build specific documentation manuals. I also updated the README. --- README.txt | 10 +++--- common/waf.py | 9 + wscript | 9 + 3 files changed, 25 insertions(+), 3 deletions(-) diff --git a/README.txt b/