On Fri, Dec 01, 2023 at 11:33:25AM +0000, Ferruh Yigit wrote: > On 12/1/2023 11:13 AM, Bruce Richardson wrote: > > On Fri, Dec 01, 2023 at 09:04:10AM +0100, Thomas Monjalon wrote: > >> 30/11/2023 19:33, Patrick Robb: > >>> On Thu, Nov 30, 2023 at 4:24 AM David Marchand <david.march...@redhat.com> > >>> wrote: > >>> > >>>> What it means: > >>>> - for the https://dpdk.org/git/dpdk repository, all the branches and > >>>> tags are mirrored to https://github.com/DPDK/dpdk as it was done so > >>>> far, > >>>> - for the https://git.dpdk.org/next/dpdk-next-* repositories, only > >>>> branches named "main", "staging" or "for-*" are mirrored to > >>>> https://github.com/DPDK/dpdk with a prefix. > >>> > >>> Thank you David for clearing some of this up on the CI testing meeting. I > >>> think the final loose end was you were wondering which branches within the > >>> next-* repos we were running from. I'll paste that below: > >>> > >>> dpdk-next-crypto: for-main > >>> dpdk-next-eventdev: for-main > >>> dpdk-next-net: main > >>> dpdk-next-net-brcm: main > >>> dpdk-next-net-intel: main > >>> dpdk-next-net-mlx: main > >>> dpdk-next-net-mrvl: for-next-net > >>> dpdk-next-virtio: main > >>> dpdk-next-baseband: for-main > >> > >> We should test patches on top of the branch which is validated > >> by the tree maintainer and ready to pull. > >> This is the default branch (HEAD) of its repository on dpdk.org. > >> This is the list of equivalent GitHub branches to use for testing: > >> > >> main > >> next-baseband-for-main > >> next-crypto-for-main > >> next-eventdev-for-main > >> next-net-for-main-repo > > > > The (slight) inconsistency here is curious. Is there a reason why this > > branch has "repo" on the end and none of the others don't? > > > > No specific reason, it started like that in the past. If the consensus > is to go with 'for-main', I can update it. > In all likelihood it's not a big deal.
I'm just thinking that for any future automation or tracking, having a very consistent naming would be best. For example, in the one-liner I posted for generating the graph showing the inter-tree flow, I needed a special regex to strip off the "-repo", so that we didn't have a separate target called "main-repo" and another called "main". /Bruce
