patches for DPDK stable branches
 help / color / Atom feed
* [dpdk-stable] [PATCH] doc: in doc-clean removed the folder not containing rst files
@ 2020-03-19 13:27 Muhammad Bilal
  2020-03-20  8:02 ` [dpdk-stable] [PATCH v2 18.11 17.11] " Muhammad Bilal
  0 siblings, 1 reply; 7+ messages in thread
From: Muhammad Bilal @ 2020-03-19 13:27 UTC (permalink / raw)
  To: thomas, john.mcnamara, bluca; +Cc: Muhammad Bilal, stable

in doc-clean removing the folder which contain
the text files and no .rst files,
they cause problem while switching from newer branches

Bugzilla ID: 364
Cc: Thomas Monjalon <thomas@monjalon.net>
Cc: John McNamara <john.mcnamara@intel.com>
Cc: M: Luca Boccassi <bluca@debian.org>
Cc: stable@dpdk.org
Signed-off-by: Muhammad Bilal <m.bilal@emumba.com>
---
 mk/rte.sdkdoc.mk | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/mk/rte.sdkdoc.mk b/mk/rte.sdkdoc.mk
index c44db6447..a80250ce8 100644
--- a/mk/rte.sdkdoc.mk
+++ b/mk/rte.sdkdoc.mk
@@ -37,7 +37,7 @@ help:
 all: api-html guides-html guides-pdf
 
 .PHONY: clean
-clean: api-html-clean guides-html-clean guides-pdf-clean guides-man-clean
+clean: api-html-clean guides-html-clean guides-pdf-clean guides-man-clean guide-table-clean
 
 .PHONY: api-html
 api-html: $(API_EXAMPLES)
@@ -56,7 +56,7 @@ api-html: $(API_EXAMPLES)
 .PHONY: api-html-clean
 api-html-clean:
 	$(Q)rm -f $(API_EXAMPLES)
-	$(Q)rm -f $(RTE_OUTPUT)/doc/html/api/*
+	$(Q)rm -rf $(RTE_OUTPUT)/doc/html/api/*
 	$(Q)rmdir -p --ignore-fail-on-non-empty $(RTE_OUTPUT)/doc/html/api 2>&- || true
 
 $(API_EXAMPLES): api-html-clean
@@ -67,6 +67,10 @@ guides-pdf-clean: guides-pdf-img-clean
 guides-pdf-img-clean:
 	$(Q)rm -f $(RTE_SDK)/doc/guides/*/img/*.pdf
 
+guide-table-clean: guide-overview_feature_table-clean
+guide-overview_feature_table-clean:
+	$(Q)rm -rf $(shell find $(RTE_SDK)/doc/guides/ -maxdepth 1 -type d '!' -exec test -e "{}/index.rst" ';' -print)
+
 guides-%-clean:
 	$(Q)rm -rf $(RTE_OUTPUT)/doc/$*/guides
 	$(Q)rmdir -p --ignore-fail-on-non-empty $(RTE_OUTPUT)/doc/$* 2>&- || true
-- 
2.17.1


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [dpdk-stable] [PATCH v2 18.11 17.11] doc: in doc-clean removed the folder not containing rst files
  2020-03-19 13:27 [dpdk-stable] [PATCH] doc: in doc-clean removed the folder not containing rst files Muhammad Bilal
@ 2020-03-20  8:02 ` " Muhammad Bilal
  2020-03-20  9:39   ` Luca Boccassi
  0 siblings, 1 reply; 7+ messages in thread
From: Muhammad Bilal @ 2020-03-20  8:02 UTC (permalink / raw)
  To: thomas, john.mcnamara, bluca; +Cc: Muhammad Bilal, dev, stable

in doc-clean removing the folder which contain
the text files and no .rst files,
they cause problem When switching branches and compiling
the documentation for older releases (for example 
building a documentation on 18.11 or 17.11, with a
documentation on master branch, built before.

Bugzilla ID: 364
Cc: Thomas Monjalon <thomas@monjalon.net>
Cc: John McNamara <john.mcnamara@intel.com>
Cc: M: Luca Boccassi <bluca@debian.org>
Cc: dev@dpdk.org
Cc: stable@dpdk.org
Signed-off-by: Muhammad Bilal <m.bilal@emumba.com>
---
v2:
* Added Details of branches for which patch is used.
* This patch is intended for v18.11 and v17.11 branches

 mk/rte.sdkdoc.mk | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/mk/rte.sdkdoc.mk b/mk/rte.sdkdoc.mk
index c44db6447..a80250ce8 100644
--- a/mk/rte.sdkdoc.mk
+++ b/mk/rte.sdkdoc.mk
@@ -37,7 +37,7 @@ help:
 all: api-html guides-html guides-pdf
 
 .PHONY: clean
-clean: api-html-clean guides-html-clean guides-pdf-clean guides-man-clean
+clean: api-html-clean guides-html-clean guides-pdf-clean guides-man-clean guide-table-clean
 
 .PHONY: api-html
 api-html: $(API_EXAMPLES)
@@ -56,7 +56,7 @@ api-html: $(API_EXAMPLES)
 .PHONY: api-html-clean
 api-html-clean:
 	$(Q)rm -f $(API_EXAMPLES)
-	$(Q)rm -f $(RTE_OUTPUT)/doc/html/api/*
+	$(Q)rm -rf $(RTE_OUTPUT)/doc/html/api/*
 	$(Q)rmdir -p --ignore-fail-on-non-empty $(RTE_OUTPUT)/doc/html/api 2>&- || true
 
 $(API_EXAMPLES): api-html-clean
@@ -67,6 +67,10 @@ guides-pdf-clean: guides-pdf-img-clean
 guides-pdf-img-clean:
 	$(Q)rm -f $(RTE_SDK)/doc/guides/*/img/*.pdf
 
+guide-table-clean: guide-overview_feature_table-clean
+guide-overview_feature_table-clean:
+	$(Q)rm -rf $(shell find $(RTE_SDK)/doc/guides/ -maxdepth 1 -type d '!' -exec test -e "{}/index.rst" ';' -print)
+
 guides-%-clean:
 	$(Q)rm -rf $(RTE_OUTPUT)/doc/$*/guides
 	$(Q)rmdir -p --ignore-fail-on-non-empty $(RTE_OUTPUT)/doc/$* 2>&- || true
-- 
2.17.1


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-stable] [PATCH v2 18.11 17.11] doc: in doc-clean removed the folder not containing rst files
  2020-03-20  8:02 ` [dpdk-stable] [PATCH v2 18.11 17.11] " Muhammad Bilal
@ 2020-03-20  9:39   ` Luca Boccassi
  2020-03-20  9:53     ` Muhammad Bilal
  0 siblings, 1 reply; 7+ messages in thread
From: Luca Boccassi @ 2020-03-20  9:39 UTC (permalink / raw)
  To: Muhammad Bilal, thomas, john.mcnamara; +Cc: dev, stable

On Fri, 2020-03-20 at 13:02 +0500, Muhammad Bilal wrote:
> in doc-clean removing the folder which contain
> the text files and no .rst files,
> they cause problem When switching branches and compiling
> the documentation for older releases (for example 
> building a documentation on 18.11 or 17.11, with a
> documentation on master branch, built before.
> 
> Bugzilla ID: 364
> Cc: Thomas Monjalon <thomas@monjalon.net>
> Cc: John McNamara <john.mcnamara@intel.com>
> Cc: M: Luca Boccassi <bluca@debian.org>
> Cc: dev@dpdk.org
> Cc: stable@dpdk.org
> Signed-off-by: Muhammad Bilal <m.bilal@emumba.com>
> ---
> v2:
> * Added Details of branches for which patch is used.
> * This patch is intended for v18.11 and v17.11 branches

Thanks for the patch, but 17.11 is EOL and will no longer receive
updates.

-- 
Kind regards,
Luca Boccassi

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-stable] [PATCH v2 18.11 17.11] doc: in doc-clean removed the folder not containing rst files
  2020-03-20  9:39   ` Luca Boccassi
@ 2020-03-20  9:53     ` Muhammad Bilal
  2020-03-20 10:02       ` Kevin Traynor
  0 siblings, 1 reply; 7+ messages in thread
From: Muhammad Bilal @ 2020-03-20  9:53 UTC (permalink / raw)
  To: Luca Boccassi; +Cc: thomas, john.mcnamara, dev, stable

On Fri, Mar 20, 2020 at 2:39 PM Luca Boccassi <bluca@debian.org> wrote:
>
> On Fri, 2020-03-20 at 13:02 +0500, Muhammad Bilal wrote:
> > in doc-clean removing the folder which contain
> > the text files and no .rst files,
> > they cause problem When switching branches and compiling
> > the documentation for older releases (for example
> > building a documentation on 18.11 or 17.11, with a
> > documentation on master branch, built before.
> >
> > Bugzilla ID: 364
> > Cc: Thomas Monjalon <thomas@monjalon.net>
> > Cc: John McNamara <john.mcnamara@intel.com>
> > Cc: M: Luca Boccassi <bluca@debian.org>
> > Cc: dev@dpdk.org
> > Cc: stable@dpdk.org
> > Signed-off-by: Muhammad Bilal <m.bilal@emumba.com>
> > ---
> > v2:
> > * Added Details of branches for which patch is used.
> > * This patch is intended for v18.11 and v17.11 branches
>
> Thanks for the patch, but 17.11 is EOL and will no longer receive
> updates.
>
This bug also occurs on 18.11
So this patch is also valid for 18.11.
> --
> Kind regards,
> Muhammad Bilal

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-stable] [PATCH v2 18.11 17.11] doc: in doc-clean removed the folder not containing rst files
  2020-03-20  9:53     ` Muhammad Bilal
@ 2020-03-20 10:02       ` Kevin Traynor
  2020-03-24  7:04         ` Muhammad Bilal
  0 siblings, 1 reply; 7+ messages in thread
From: Kevin Traynor @ 2020-03-20 10:02 UTC (permalink / raw)
  To: Muhammad Bilal, Luca Boccassi; +Cc: thomas, john.mcnamara, dev, stable

On 20/03/2020 09:53, Muhammad Bilal wrote:
> On Fri, Mar 20, 2020 at 2:39 PM Luca Boccassi <bluca@debian.org> wrote:
>>
>> On Fri, 2020-03-20 at 13:02 +0500, Muhammad Bilal wrote:
>>> in doc-clean removing the folder which contain
>>> the text files and no .rst files,
>>> they cause problem When switching branches and compiling
>>> the documentation for older releases (for example
>>> building a documentation on 18.11 or 17.11, with a
>>> documentation on master branch, built before.
>>>
>>> Bugzilla ID: 364
>>> Cc: Thomas Monjalon <thomas@monjalon.net>
>>> Cc: John McNamara <john.mcnamara@intel.com>
>>> Cc: M: Luca Boccassi <bluca@debian.org>
>>> Cc: dev@dpdk.org
>>> Cc: stable@dpdk.org
>>> Signed-off-by: Muhammad Bilal <m.bilal@emumba.com>
>>> ---
>>> v2:
>>> * Added Details of branches for which patch is used.
>>> * This patch is intended for v18.11 and v17.11 branches
>>
>> Thanks for the patch, but 17.11 is EOL and will no longer receive
>> updates.
>>
> This bug also occurs on 18.11
> So this patch is also valid for 18.11.

Thanks for the fix. This patch is for dpdk master as well, right?

If so, will backport to 18.11 as part of normal rounds of backporting
from master to 18.11.

>> --
>> Kind regards,
>> Muhammad Bilal
> 


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-stable] [PATCH v2 18.11 17.11] doc: in doc-clean removed the folder not containing rst files
  2020-03-20 10:02       ` Kevin Traynor
@ 2020-03-24  7:04         ` Muhammad Bilal
  2020-03-24 12:57           ` Kevin Traynor
  0 siblings, 1 reply; 7+ messages in thread
From: Muhammad Bilal @ 2020-03-24  7:04 UTC (permalink / raw)
  To: Kevin Traynor; +Cc: Luca Boccassi, thomas, john.mcnamara, dev, stable

On Fri, Mar 20, 2020 at 3:02 PM Kevin Traynor <ktraynor@redhat.com> wrote:
>
> On 20/03/2020 09:53, Muhammad Bilal wrote:
> > On Fri, Mar 20, 2020 at 2:39 PM Luca Boccassi <bluca@debian.org> wrote:
> >>
> >> On Fri, 2020-03-20 at 13:02 +0500, Muhammad Bilal wrote:
> >>> in doc-clean removing the folder which contain
> >>> the text files and no .rst files,
> >>> they cause problem When switching branches and compiling
> >>> the documentation for older releases (for example
> >>> building a documentation on 18.11 or 17.11, with a
> >>> documentation on master branch, built before.
> >>>
> >>> Bugzilla ID: 364
> >>> Cc: Thomas Monjalon <thomas@monjalon.net>
> >>> Cc: John McNamara <john.mcnamara@intel.com>
> >>> Cc: M: Luca Boccassi <bluca@debian.org>
> >>> Cc: dev@dpdk.org
> >>> Cc: stable@dpdk.org
> >>> Signed-off-by: Muhammad Bilal <m.bilal@emumba.com>
> >>> ---
> >>> v2:
> >>> * Added Details of branches for which patch is used.
> >>> * This patch is intended for v18.11 and v17.11 branches
> >>
> >> Thanks for the patch, but 17.11 is EOL and will no longer receive
> >> updates.
> >>
> > This bug also occurs on 18.11
> > So this patch is also valid for 18.11.
>
>No, This patch is not for dpdk master, it is intended only for 18.11 branch
>when we switch to 18.11 branch (after making documentation in master or newer branch ) an error occurs in making documentation on 18.11 branch.
>So This patch is intended to solve this problem.
> >> --
> >> Kind regards,
> >> Muhammad Bilal
> >
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-stable] [PATCH v2 18.11 17.11] doc: in doc-clean removed the folder not containing rst files
  2020-03-24  7:04         ` Muhammad Bilal
@ 2020-03-24 12:57           ` Kevin Traynor
  0 siblings, 0 replies; 7+ messages in thread
From: Kevin Traynor @ 2020-03-24 12:57 UTC (permalink / raw)
  To: Muhammad Bilal, john.mcnamara, Marko Kovacevic
  Cc: Luca Boccassi, thomas, dev, stable

On 24/03/2020 07:04, Muhammad Bilal wrote:
> On Fri, Mar 20, 2020 at 3:02 PM Kevin Traynor <ktraynor@redhat.com> wrote:
>>
>> On 20/03/2020 09:53, Muhammad Bilal wrote:
>>> On Fri, Mar 20, 2020 at 2:39 PM Luca Boccassi <bluca@debian.org> wrote:
>>>>
>>>> On Fri, 2020-03-20 at 13:02 +0500, Muhammad Bilal wrote:
>>>>> in doc-clean removing the folder which contain
>>>>> the text files and no .rst files,
>>>>> they cause problem When switching branches and compiling
>>>>> the documentation for older releases (for example
>>>>> building a documentation on 18.11 or 17.11, with a
>>>>> documentation on master branch, built before.
>>>>>
>>>>> Bugzilla ID: 364
>>>>> Cc: Thomas Monjalon <thomas@monjalon.net>
>>>>> Cc: John McNamara <john.mcnamara@intel.com>
>>>>> Cc: M: Luca Boccassi <bluca@debian.org>
>>>>> Cc: dev@dpdk.org
>>>>> Cc: stable@dpdk.org
>>>>> Signed-off-by: Muhammad Bilal <m.bilal@emumba.com>
>>>>> ---
>>>>> v2:
>>>>> * Added Details of branches for which patch is used.
>>>>> * This patch is intended for v18.11 and v17.11 branches
>>>>
>>>> Thanks for the patch, but 17.11 is EOL and will no longer receive
>>>> updates.
>>>>
>>> This bug also occurs on 18.11
>>> So this patch is also valid for 18.11.
>>
>> No, This patch is not for dpdk master, it is intended only for 18.11 branch
>> when we switch to 18.11 branch (after making documentation in master or newer branch ) an error occurs in making documentation on 18.11 branch.
>> So This patch is intended to solve this problem.

The line to remove the search dir from commit [1] fixes the api build
for me.

My pdf build is not working atm and I'm not sure about the paths. Can
doc maintainers review this please.

[1]
commit 92d6c874d33dbbd3ace7c06a3490a61ff6641d61
Author: Aideen McLoughlin <aideen.mcloughlin@intel.com>
Date:   Thu Jun 27 14:58:19 2019 +0100

    doc: add search engine for API

>>>> --
>>>> Kind regards,
>>>> Muhammad Bilal
>>>
>>
> 


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, back to index

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-19 13:27 [dpdk-stable] [PATCH] doc: in doc-clean removed the folder not containing rst files Muhammad Bilal
2020-03-20  8:02 ` [dpdk-stable] [PATCH v2 18.11 17.11] " Muhammad Bilal
2020-03-20  9:39   ` Luca Boccassi
2020-03-20  9:53     ` Muhammad Bilal
2020-03-20 10:02       ` Kevin Traynor
2020-03-24  7:04         ` Muhammad Bilal
2020-03-24 12:57           ` Kevin Traynor

patches for DPDK stable branches

Archives are clonable:
	git clone --mirror http://inbox.dpdk.org/stable/0 stable/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 stable stable/ http://inbox.dpdk.org/stable \
		stable@dpdk.org
	public-inbox-index stable


Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.stable


AGPL code for this site: git clone https://public-inbox.org/ public-inbox