On 4/14/2023 10:05 AM, Guo, Junfeng wrote: > Hi Ferruh, > > Here is the summary of the dependency of related patches: > > P1: necessary headers update > https://patchwork.dpdk.org/project/dpdk/list/?series=27647&state=* > P2: minor update for base code > https://patchwork.dpdk.org/project/dpdk/list/?series=27653&state=* > P3: gve PMD enhancement > https://patchwork.dpdk.org/project/dpdk/list/?series=27687&state=* > P4: add new AdminQ cmd to verify drv cap > https://patchwork.dpdk.org/project/dpdk/list/?series=27703&state=* > > The merging order could be: P1 > P2 > P3 > P4, due to the dependency. > Just to inform you about this here. Thanks for the review! >
Thanks Junfeng, this is useful. Btw, we have a defined syntax for this [1], having it in commit log may help automated CI to parse it and apply in correct order (when it is fully supported in CI :) ) [1] https://doc.dpdk.org/guides/contributing/patches.html#patch-dependencies Depends-on: series-NNNNN ("Title of the series") or Depends-on: patch-NNNNN ("Title of the patch") And I just recognized that syntax is not clear for multiple dependency case, we should define it better.
