Re: [PATCH] dts: fix doxygen API link

[Paul Szczepanek <paul.szczepanek@arm.com>]

On 24/10/2024 10:12, David Marchand wrote:
>>
>> Looking again while applying, something is fishy in doc/api/meson.build:
>>
>> if not doxygen.found()
>>   # process DTS API doc build even if DPDK API doc build can't be done
>>   cdata = configuration_data()
>>   subdir('dts')
>>   subdir_done()
>> endif
>>
>> Should this cdata (unused afaics) object be removed too?

Yes, I'm removing this.

> 
> It seems there are more issues with the doc generation.
> 
> Looking at dpdk.org documentation (https://doc.dpdk.org/api/), there
> is a dead link to DTS at the bottom of the page:
> tests: [DTS]()

That's what the original patch addressed.

> 
> The documentation for dts should be generated to dpdk_build_root
> /doc/api/html/dts

Yes, my V2 will fix this and place dts inside API so the website docs work.

> And I think the link should be a relative one, like
> cdata.set('DTS_API_MAIN_PAGE', join_paths('dts', 'index.html'))
> 
> Please have a look.
> 

Thanks. Will send out V2 as soon as it clears local checks.