[ovs-dev] [PATCH ovn v1 1/5] manpages.mk: fix dependencies path
Dumitru Ceara
dceara at redhat.com
Thu Nov 18 14:16:36 UTC 2021
On 11/9/21 10:08 PM, Numan Siddique wrote:
>>>> +Numan
>>>>
>>>> In general, I'm not completely sure about why sodepends.py needs to be
>>>> part of OVN and why we can't use the OVS version?
>>> OVS does not have this problem as everything is under the root tree so I
>>> assumed it would be preferred to implement it in OVN rather than add
>>> stuff to OVS that will not be used by OVS.
>>> But I don't have a strong opinion on this. If you prefer to implement
>>> this in OVS and use it from OVN, I'll remove this patch from the series,
>>> send it to OVS and send a patch to OVN that uses it.
>>>
>> +Mark
>>
>> In any case, this doesn't really block the rest of the series, I think
>> patches 2-5 can be applied already to main branch if maintainers agree.
> Thanks Adrian for this patch series and Dumitru for the reviews.
>
> I applied the patches 2-5 to the main branch.
>
> For this patch
> Acked-by: Numan Siddique <numans at ovn.org>
>
> My 2 cents - I'm fine modifying the sodepends.py in OVN. I'd personally prefer
> to move the OVS bits into OVN for man page generation so that we can change
> the code (in future) if required as per OVN's requirements.
>
OK, makes sense to me then.
Acked-by: Dumitru Ceara <dceara at redhat.com>
Regards,
Dumitru
More information about the dev
mailing list