[dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[jerinj]

From: Jerin Jacob <jerinj@marvell.com>

Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
changed to TIMESTAMP [2]. Fixed the following warning[1] by providing
the correct configuration option based on doxygen version.

[1]
Warning: Tag 'HTML_TIMESTAMP' at line 115 of file 'doc/api/doxy-api.conf'
has become obsolete.

[2]
https://www.doxygen.nl/manual/config.html#cfg_timestamp

Cc: stable@dpdk.org

Signed-off-by: Jerin Jacob <jerinj@marvell.com>
---
 doc/api/doxy-api.conf.in | 2 +-
 doc/api/meson.build      | 6 ++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/doc/api/doxy-api.conf.in b/doc/api/doxy-api.conf.in
index 7e68e43c64..f5da1f2291 100644
--- a/doc/api/doxy-api.conf.in
+++ b/doc/api/doxy-api.conf.in
@@ -112,7 +112,7 @@ GENERATE_DEPRECATEDLIST = YES
 VERBATIM_HEADERS        = NO
 ALPHABETICAL_INDEX      = NO
 
-HTML_TIMESTAMP          = NO
+@TS_PREFIX@TIMESTAMP   = NO
 HTML_DYNAMIC_SECTIONS   = YES
 HTML_EXTRA_STYLESHEET   = @TOPDIR@/doc/api/custom.css
 SEARCHENGINE            = YES
diff --git a/doc/api/meson.build b/doc/api/meson.build
index 2876a78a7e..7fc5b064cd 100644
--- a/doc/api/meson.build
+++ b/doc/api/meson.build
@@ -41,6 +41,12 @@ if get_option('werror')
     cdata.set('WARN_AS_ERROR', 'YES')
 endif
 
+if doxygen.version().version_compare('>=1.9.7')
+    cdata.set('TS_PREFIX', '')
+else
+    cdata.set('TS_PREFIX', 'HTML_')
+endif
+
 doxy_conf = configure_file(input: 'doxy-api.conf.in',
         output: 'doxy-api.conf',
         configuration: cdata)
-- 
2.40.1

Re: [dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[Jerin Jacob]

On Wed, May 31, 2023 at 6:17 PM <jerinj@marvell.com> wrote:
>
> From: Jerin Jacob <jerinj@marvell.com>
>
> Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
> changed to TIMESTAMP [2]. Fixed the following warning[1] by providing
> the correct configuration option based on doxygen version.

loongarch-compilation reported a build issue  at
http://mails.dpdk.org/archives/test-report/2023-May/402940.html
https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
introduced in 0.62 version, so not sure how we can do version check
without that.

Another option is to just remove HTIML_TIMESTAMP from config file.
Since 1.8.9 doxgen version,
it is disabled by default.

Thoughts?


commit 3f2e8a3067712b025623e4420e6eb161febfd42b
Author: Dimitri van Heesch <dimitri@stack.nl>
Date:   Thu Mar 5 21:54:12 2015 +0100

    Bug 745439 - PATCH: please consider making doxygen output byte for
byte identical between individual runs by default

diff --git a/src/config.xml b/src/config.xml
index 747686031..3d81a21a2 100644
--- a/src/config.xml
+++ b/src/config.xml
@@ -1889,13 +1889,13 @@ hr.footer {
 ]]>
       </docs>
     </option>
-    <option type='bool' id='HTML_TIMESTAMP' defval='1' depends='GENERATE_HTML'>
+    <option type='bool' id='HTML_TIMESTAMP' defval='0' depends='GENERATE_HTML'>
       <docs>
 <![CDATA[
  If the \c HTML_TIMESTAMP tag is set to \c YES then the footer of
  each generated HTML page will contain the date and time when the page
- was generated. Setting this to \c NO can help when comparing the output of
- multiple runs.
+ was generated. Setting this to \c YES can help to show when doxygen
was last run
+ and thus if the documentation is up to date.
 ]]>
       </docs>
     </option>
[(HEAD detached at Release_1_9_5)]dell[doxygen] $ git describe
3f2e8a3067712b025623e4420e6eb161febfd42b
Release_1_8_9_1-64-g3f2e8a306

>
> [1]
> Warning: Tag 'HTML_TIMESTAMP' at line 115 of file 'doc/api/doxy-api.conf'
> has become obsolete.
>
> [2]
> https://www.doxygen.nl/manual/config.html#cfg_timestamp
>
> Cc: stable@dpdk.org
>
> Signed-off-by: Jerin Jacob <jerinj@marvell.com>
> ---
>  doc/api/doxy-api.conf.in | 2 +-
>  doc/api/meson.build      | 6 ++++++
>  2 files changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/doc/api/doxy-api.conf.in b/doc/api/doxy-api.conf.in
> index 7e68e43c64..f5da1f2291 100644
> --- a/doc/api/doxy-api.conf.in
> +++ b/doc/api/doxy-api.conf.in
> @@ -112,7 +112,7 @@ GENERATE_DEPRECATEDLIST = YES
>  VERBATIM_HEADERS        = NO
>  ALPHABETICAL_INDEX      = NO
>
> -HTML_TIMESTAMP          = NO
> +@TS_PREFIX@TIMESTAMP   = NO
>  HTML_DYNAMIC_SECTIONS   = YES
>  HTML_EXTRA_STYLESHEET   = @TOPDIR@/doc/api/custom.css
>  SEARCHENGINE            = YES
> diff --git a/doc/api/meson.build b/doc/api/meson.build
> index 2876a78a7e..7fc5b064cd 100644
> --- a/doc/api/meson.build
> +++ b/doc/api/meson.build
> @@ -41,6 +41,12 @@ if get_option('werror')
>      cdata.set('WARN_AS_ERROR', 'YES')
>  endif
>
> +if doxygen.version().version_compare('>=1.9.7')
> +    cdata.set('TS_PREFIX', '')
> +else
> +    cdata.set('TS_PREFIX', 'HTML_')
> +endif
> +
>  doxy_conf = configure_file(input: 'doxy-api.conf.in',
>          output: 'doxy-api.conf',
>          configuration: cdata)
> --
> 2.40.1
>

Re: [dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[Thomas Monjalon]

31/05/2023 15:21, Jerin Jacob:
> On Wed, May 31, 2023 at 6:17 PM <jerinj@marvell.com> wrote:
> >
> > From: Jerin Jacob <jerinj@marvell.com>
> >
> > Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
> > changed to TIMESTAMP [2]. Fixed the following warning[1] by providing
> > the correct configuration option based on doxygen version.
> 
> loongarch-compilation reported a build issue  at
> http://mails.dpdk.org/archives/test-report/2023-May/402940.html
> https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
> introduced in 0.62 version, so not sure how we can do version check
> without that.
> 
> Another option is to just remove HTIML_TIMESTAMP from config file.
> Since 1.8.9 doxgen version,
> it is disabled by default.
> 
> Thoughts?

No need to disable if it is already disabled by default.
What was the default of the old HTML_TIMESTAMP?

Re: [dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[Jerin Jacob]

On Wed, May 31, 2023 at 7:31 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> 31/05/2023 15:21, Jerin Jacob:
> > On Wed, May 31, 2023 at 6:17 PM <jerinj@marvell.com> wrote:
> > >
> > > From: Jerin Jacob <jerinj@marvell.com>
> > >
> > > Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
> > > changed to TIMESTAMP [2]. Fixed the following warning[1] by providing
> > > the correct configuration option based on doxygen version.
> >
> > loongarch-compilation reported a build issue  at
> > http://mails.dpdk.org/archives/test-report/2023-May/402940.html
> > https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
> > introduced in 0.62 version, so not sure how we can do version check
> > without that.
> >
> > Another option is to just remove HTIML_TIMESTAMP from config file.
> > Since 1.8.9 doxgen version,
> > it is disabled by default.
> >
> > Thoughts?
>
> No need to disable if it is already disabled by default.
> What was the default of the old HTML_TIMESTAMP?

Prior to doxygen Release_1_8_9_1-64-g3f2e8a306 it is enabled by default.

>
>

Re: [dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[Thomas Monjalon]

31/05/2023 16:07, Jerin Jacob:
> On Wed, May 31, 2023 at 7:31 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> >
> > 31/05/2023 15:21, Jerin Jacob:
> > > On Wed, May 31, 2023 at 6:17 PM <jerinj@marvell.com> wrote:
> > > >
> > > > From: Jerin Jacob <jerinj@marvell.com>
> > > >
> > > > Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
> > > > changed to TIMESTAMP [2]. Fixed the following warning[1] by providing
> > > > the correct configuration option based on doxygen version.
> > >
> > > loongarch-compilation reported a build issue  at
> > > http://mails.dpdk.org/archives/test-report/2023-May/402940.html
> > > https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
> > > introduced in 0.62 version, so not sure how we can do version check
> > > without that.
> > >
> > > Another option is to just remove HTIML_TIMESTAMP from config file.
> > > Since 1.8.9 doxgen version,
> > > it is disabled by default.
> > >
> > > Thoughts?
> >
> > No need to disable if it is already disabled by default.
> > What was the default of the old HTML_TIMESTAMP?
> 
> Prior to doxygen Release_1_8_9_1-64-g3f2e8a306 it is enabled by default.

Then it would be better to keep HTML_TIMESTAMP until doxygen 1.9
and don't add TIMESTAMP for new versions.

Re: [dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[Jerin Jacob]

On Wed, May 31, 2023 at 8:19 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> 31/05/2023 16:07, Jerin Jacob:
> > On Wed, May 31, 2023 at 7:31 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > >
> > > 31/05/2023 15:21, Jerin Jacob:
> > > > On Wed, May 31, 2023 at 6:17 PM <jerinj@marvell.com> wrote:
> > > > >
> > > > > From: Jerin Jacob <jerinj@marvell.com>
> > > > >
> > > > > Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
> > > > > changed to TIMESTAMP [2]. Fixed the following warning[1] by providing
> > > > > the correct configuration option based on doxygen version.
> > > >
> > > > loongarch-compilation reported a build issue  at
> > > > http://mails.dpdk.org/archives/test-report/2023-May/402940.html
> > > > https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
> > > > introduced in 0.62 version, so not sure how we can do version check
> > > > without that.
> > > >
> > > > Another option is to just remove HTIML_TIMESTAMP from config file.
> > > > Since 1.8.9 doxgen version,
> > > > it is disabled by default.
> > > >
> > > > Thoughts?
> > >
> > > No need to disable if it is already disabled by default.
> > > What was the default of the old HTML_TIMESTAMP?
> >
> > Prior to doxygen Release_1_8_9_1-64-g3f2e8a306 it is enabled by default.
>
> Then it would be better to keep HTML_TIMESTAMP until doxygen 1.9
> and don't add TIMESTAMP for new versions.

https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
introduced in 0.62 version. Thats is one issue to check doxgen version
in meson code to do something based
on version.



>
>
>

Re: [dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[Thomas Monjalon]

31/05/2023 16:54, Jerin Jacob:
> On Wed, May 31, 2023 at 8:19 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> >
> > 31/05/2023 16:07, Jerin Jacob:
> > > On Wed, May 31, 2023 at 7:31 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > > >
> > > > 31/05/2023 15:21, Jerin Jacob:
> > > > > On Wed, May 31, 2023 at 6:17 PM <jerinj@marvell.com> wrote:
> > > > > >
> > > > > > From: Jerin Jacob <jerinj@marvell.com>
> > > > > >
> > > > > > Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
> > > > > > changed to TIMESTAMP [2]. Fixed the following warning[1] by providing
> > > > > > the correct configuration option based on doxygen version.
> > > > >
> > > > > loongarch-compilation reported a build issue  at
> > > > > http://mails.dpdk.org/archives/test-report/2023-May/402940.html
> > > > > https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
> > > > > introduced in 0.62 version, so not sure how we can do version check
> > > > > without that.
> > > > >
> > > > > Another option is to just remove HTIML_TIMESTAMP from config file.
> > > > > Since 1.8.9 doxgen version,
> > > > > it is disabled by default.
> > > > >
> > > > > Thoughts?
> > > >
> > > > No need to disable if it is already disabled by default.
> > > > What was the default of the old HTML_TIMESTAMP?
> > >
> > > Prior to doxygen Release_1_8_9_1-64-g3f2e8a306 it is enabled by default.
> >
> > Then it would be better to keep HTML_TIMESTAMP until doxygen 1.9
> > and don't add TIMESTAMP for new versions.
> 
> https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
> introduced in 0.62 version. Thats is one issue to check doxgen version
> in meson code to do something based
> on version.

It should be possible to check version without using this new Meson feature.
Look at what is done for "binutils_ok" as an example.

[dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[jerinj]

From: Jerin Jacob <jerinj@marvell.com>

Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
changed to TIMESTAMP [2]. Fixed the following warning[1] by providing
the correct configuration option based on doxygen version.

[1]
Warning: Tag 'HTML_TIMESTAMP' at line 115 of file 'doc/api/doxy-api.conf'
has become obsolete.

[2]
https://www.doxygen.nl/manual/config.html#cfg_timestamp

Cc: stable@dpdk.org

Signed-off-by: Jerin Jacob <jerinj@marvell.com>
---
v2:
- use run_command() as https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
introduced in 0.62 version and DPDK min meson version is 0.53.2 

 doc/api/doxy-api.conf.in | 2 +-
 doc/api/meson.build      | 7 +++++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/doc/api/doxy-api.conf.in b/doc/api/doxy-api.conf.in
index d230a19e1f..60e582d8fc 100644
--- a/doc/api/doxy-api.conf.in
+++ b/doc/api/doxy-api.conf.in
@@ -111,7 +111,7 @@ GENERATE_DEPRECATEDLIST = YES
 VERBATIM_HEADERS        = NO
 ALPHABETICAL_INDEX      = NO
 
-HTML_TIMESTAMP          = NO
+@TS_PREFIX@TIMESTAMP   = NO
 HTML_DYNAMIC_SECTIONS   = YES
 HTML_EXTRA_STYLESHEET   = @TOPDIR@/doc/api/custom.css
 SEARCHENGINE            = YES
diff --git a/doc/api/meson.build b/doc/api/meson.build
index 2876a78a7e..4d1161d847 100644
--- a/doc/api/meson.build
+++ b/doc/api/meson.build
@@ -41,6 +41,13 @@ if get_option('werror')
     cdata.set('WARN_AS_ERROR', 'YES')
 endif
 
+doxygen_version = run_command('doxygen', '--version', check: true).stdout().strip()
+if doxygen_version.version_compare('>=1.9.7')
+    cdata.set('TS_PREFIX', '')
+else
+    cdata.set('TS_PREFIX', 'HTML_')
+endif
+
 doxy_conf = configure_file(input: 'doxy-api.conf.in',
         output: 'doxy-api.conf',
         configuration: cdata)
-- 
2.40.1

Re: [dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[Bruce Richardson]

On Wed, May 31, 2023 at 04:49:04PM +0200, Thomas Monjalon wrote:
> 31/05/2023 16:07, Jerin Jacob:
> > On Wed, May 31, 2023 at 7:31 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > >
> > > 31/05/2023 15:21, Jerin Jacob:
> > > > On Wed, May 31, 2023 at 6:17 PM <jerinj@marvell.com> wrote:
> > > > >
> > > > > From: Jerin Jacob <jerinj@marvell.com>
> > > > >
> > > > > Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
> > > > > changed to TIMESTAMP [2]. Fixed the following warning[1] by providing
> > > > > the correct configuration option based on doxygen version.
> > > >
> > > > loongarch-compilation reported a build issue  at
> > > > http://mails.dpdk.org/archives/test-report/2023-May/402940.html
> > > > https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
> > > > introduced in 0.62 version, so not sure how we can do version check
> > > > without that.
> > > >
> > > > Another option is to just remove HTIML_TIMESTAMP from config file.
> > > > Since 1.8.9 doxgen version,
> > > > it is disabled by default.
> > > >
> > > > Thoughts?
> > >
> > > No need to disable if it is already disabled by default.
> > > What was the default of the old HTML_TIMESTAMP?
> > 
> > Prior to doxygen Release_1_8_9_1-64-g3f2e8a306 it is enabled by default.
> 
> Then it would be better to keep HTML_TIMESTAMP until doxygen 1.9
> and don't add TIMESTAMP for new versions.
> 

Sorry for late arrival in the discussion.

I am not sure it's worth going to all that trouble. If the option is
disabled by default in all new versions of doxygen, then we don't have an
issue with the official DPDK docs on the website being any different if we
just drop the option from our config. All that would happen is that any
individual developer/user generating the docs with an older doxygen, will
get their HTML output timestampted in the footer.

Not a big deal, to my mind. Therefore I suggest just removing the option,
and being done with it. Once we put in the version check code, it's nigh on
impossible to remove it as it gets forgotten.

/Bruce

Re: [dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[Bruce Richardson]

On Fri, Jun 02, 2023 at 02:00:32PM +0530, jerinj@marvell.com wrote:
> From: Jerin Jacob <jerinj@marvell.com>
> 
> Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
> changed to TIMESTAMP [2]. Fixed the following warning[1] by providing
> the correct configuration option based on doxygen version.
> 
> [1]
> Warning: Tag 'HTML_TIMESTAMP' at line 115 of file 'doc/api/doxy-api.conf'
> has become obsolete.
> 
> [2]
> https://www.doxygen.nl/manual/config.html#cfg_timestamp
> 
> Cc: stable@dpdk.org
> 
> Signed-off-by: Jerin Jacob <jerinj@marvell.com>
> ---
> v2:
> - use run_command() as https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
> introduced in 0.62 version and DPDK min meson version is 0.53.2 
> 
>  doc/api/doxy-api.conf.in | 2 +-
>  doc/api/meson.build      | 7 +++++++
>  2 files changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/doc/api/doxy-api.conf.in b/doc/api/doxy-api.conf.in
> index d230a19e1f..60e582d8fc 100644
> --- a/doc/api/doxy-api.conf.in
> +++ b/doc/api/doxy-api.conf.in
> @@ -111,7 +111,7 @@ GENERATE_DEPRECATEDLIST = YES
>  VERBATIM_HEADERS        = NO
>  ALPHABETICAL_INDEX      = NO
>  
> -HTML_TIMESTAMP          = NO
> +@TS_PREFIX@TIMESTAMP   = NO

As explained in previous email, belately sent in reply to v1, I think we
should just drop this line, and not worry about version checks.

/Bruce

Re: [dpdk-dev] [PATCH v1] doc: remove warning with doxygen 1.9.7

[Jerin Jacob]

On Fri, Jun 2, 2023 at 2:11 PM Bruce Richardson
<bruce.richardson@intel.com> wrote:
>
> On Fri, Jun 02, 2023 at 02:00:32PM +0530, jerinj@marvell.com wrote:
> > From: Jerin Jacob <jerinj@marvell.com>
> >
> > Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
> > changed to TIMESTAMP [2]. Fixed the following warning[1] by providing
> > the correct configuration option based on doxygen version.
> >
> > [1]
> > Warning: Tag 'HTML_TIMESTAMP' at line 115 of file 'doc/api/doxy-api.conf'
> > has become obsolete.
> >
> > [2]
> > https://www.doxygen.nl/manual/config.html#cfg_timestamp
> >
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Jerin Jacob <jerinj@marvell.com>
> > ---
> > v2:
> > - use run_command() as https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
> > introduced in 0.62 version and DPDK min meson version is 0.53.2
> >
> >  doc/api/doxy-api.conf.in | 2 +-
> >  doc/api/meson.build      | 7 +++++++
> >  2 files changed, 8 insertions(+), 1 deletion(-)
> >
> > diff --git a/doc/api/doxy-api.conf.in b/doc/api/doxy-api.conf.in
> > index d230a19e1f..60e582d8fc 100644
> > --- a/doc/api/doxy-api.conf.in
> > +++ b/doc/api/doxy-api.conf.in
> > @@ -111,7 +111,7 @@ GENERATE_DEPRECATEDLIST = YES
> >  VERBATIM_HEADERS        = NO
> >  ALPHABETICAL_INDEX      = NO
> >
> > -HTML_TIMESTAMP          = NO
> > +@TS_PREFIX@TIMESTAMP   = NO
>
> As explained in previous email, belately sent in reply to v1, I think we
> should just drop this line, and not worry about version checks.

Yes. I think that is better. I will send new version to remove it.

>
> /Bruce
>

[dpdk-dev] [PATCH v3] doc: remove warning with doxygen 1.9.7

[jerinj]

From: Jerin Jacob <jerinj@marvell.com>

Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
changed to TIMESTAMP [2]. Fixed the following warning[1] by
removing TIMESTAMP options all together as all new versions(since 1.8.9)
of doxygen this option is disabled by default. Another option was
considered to have this configuration option based on version check, which
may not be worth having it, especially, external_program.version()
API introduced in 0.62.0 version of meson.

[1]
Warning: Tag 'HTML_TIMESTAMP' at line 115 of file 'doc/api/doxy-api.conf'
has become obsolete.

[2]
https://www.doxygen.nl/manual/config.html#cfg_timestamp

Cc: stable@dpdk.org

Signed-off-by: Jerin Jacob <jerinj@marvell.com>
---
v3:
- Based on Bruce's feedback, just removed the option all together to
  avoid special handling.
v2:
- use run_command() as
  https://mesonbuild.com/Reference-manual_returned_external_program.html#external_programversion
  introduced in 0.62 version and DPDK min meson version is 0.53.2.

 doc/api/doxy-api.conf.in | 1 -
 1 file changed, 1 deletion(-)

diff --git a/doc/api/doxy-api.conf.in b/doc/api/doxy-api.conf.in
index d230a19e1f..deea7b747d 100644
--- a/doc/api/doxy-api.conf.in
+++ b/doc/api/doxy-api.conf.in
@@ -111,7 +111,6 @@ GENERATE_DEPRECATEDLIST = YES
 VERBATIM_HEADERS        = NO
 ALPHABETICAL_INDEX      = NO
 
-HTML_TIMESTAMP          = NO
 HTML_DYNAMIC_SECTIONS   = YES
 HTML_EXTRA_STYLESHEET   = @TOPDIR@/doc/api/custom.css
 SEARCHENGINE            = YES
-- 
2.40.1

Re: [dpdk-dev] [PATCH v3] doc: remove warning with doxygen 1.9.7

[Bruce Richardson]

On Fri, Jun 02, 2023 at 04:36:59PM +0530, jerinj@marvell.com wrote:
> From: Jerin Jacob <jerinj@marvell.com>
> 
> Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
> changed to TIMESTAMP [2]. Fixed the following warning[1] by
> removing TIMESTAMP options all together as all new versions(since 1.8.9)
> of doxygen this option is disabled by default. Another option was
> considered to have this configuration option based on version check, which
> may not be worth having it, especially, external_program.version()
> API introduced in 0.62.0 version of meson.
> 
> [1]
> Warning: Tag 'HTML_TIMESTAMP' at line 115 of file 'doc/api/doxy-api.conf'
> has become obsolete.
> 
> [2]
> https://www.doxygen.nl/manual/config.html#cfg_timestamp
> 
> Cc: stable@dpdk.org
> 
> Signed-off-by: Jerin Jacob <jerinj@marvell.com>
> ---

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Re: [dpdk-dev] [PATCH v3] doc: remove warning with doxygen 1.9.7

[Thomas Monjalon]

02/06/2023 13:16, Bruce Richardson:
> On Fri, Jun 02, 2023 at 04:36:59PM +0530, jerinj@marvell.com wrote:
> > From: Jerin Jacob <jerinj@marvell.com>
> > 
> > Since doxygen 1.9.7 version, HTML_TIMESTAMP configuration option
> > changed to TIMESTAMP [2]. Fixed the following warning[1] by
> > removing TIMESTAMP options all together as all new versions(since 1.8.9)
> > of doxygen this option is disabled by default. Another option was
> > considered to have this configuration option based on version check, which
> > may not be worth having it, especially, external_program.version()
> > API introduced in 0.62.0 version of meson.
> > 
> > [1]
> > Warning: Tag 'HTML_TIMESTAMP' at line 115 of file 'doc/api/doxy-api.conf'
> > has become obsolete.
> > 
> > [2]
> > https://www.doxygen.nl/manual/config.html#cfg_timestamp
> > 
> > Cc: stable@dpdk.org
> > 
> > Signed-off-by: Jerin Jacob <jerinj@marvell.com>
> 
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Applied, thanks.