Re: [PATCH] docs: Remove dependency on external resources

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/20/21 19:45, Andrea Bolognani wrote:
> On Fri, Dec 10, 2021 at 03:07:10AM -0800, Andrea Bolognani wrote:
>> On Fri, Dec 10, 2021 at 11:41:29AM +0100, Peter Krempa wrote:
>>>>  docs/event_loop_simple.png       | Bin 0 -> 16043 bytes
>>>>  docs/event_loop_simple.svg       | 398 +++++++++++++++++
>>>>  docs/event_loop_worker.png       | Bin 0 -> 29696 bytes
>>>>  docs/event_loop_worker.svg       | 708 +++++++++++++++++++++++++++++++
>>>
>>> Shouldn't these files ...
>>>
>>>>  docs/internals/eventloop.html.in |   4 +-
>>>
>>> .. be inside the 'internals' directory?
>>
>> For that to work, I'd have to replicate the docs_assets machinery
>> found in docs/meson.build to docs/internals/meson.build.
>>
>> I'm not absolutely against it, and we already have a copy in
>> docs/logos/meson.build after all, but using relative paths seemed
>> preferable.
> 
> Any further thoughts? I'd like to get this merged, be it in one shape
> or another :)
> 

I'd say it's okay. It's just a bunch of pictures.

Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

Michal




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux