* Re: [dpdk-stable] [scripts] README: clean DPDK clone and main branch to generate list [not found] <1619589811-9458-1-git-send-email-xuemingl@nvidia.com> @ 2021-04-28 6:40 ` Christian Ehrhardt 2021-05-06 13:21 ` Xueming(Steven) Li [not found] ` <1620307153-12649-1-git-send-email-xuemingl@nvidia.com> 2021-05-10 13:23 ` Xueming Li 2 siblings, 1 reply; 6+ messages in thread From: Christian Ehrhardt @ 2021-04-28 6:40 UTC (permalink / raw) To: Xueming Li; +Cc: dpdk stable, Luca Boccassi On Wed, Apr 28, 2021 at 8:03 AM Xueming Li <xuemingl@nvidia.com> wrote: > > Tags of "dirty" repository confuse git-log-fixes.sh. > The script also expects current branch to contains end tag of range. > > > Signed-off-by: Xueming Li <xuemingl@nvidia.com> > --- > README | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/README b/README > index 70f4320..911f169 100644 > --- a/README > +++ b/README > @@ -26,11 +26,15 @@ as tmp dir for the patches and mails. > 1.A > --- > A list of commits to backport should be generated. This can be generated > -from "git-log-fixes.sh" script from DPDK. Normally the start of range is > +from "git-log-fixes.sh" script from DPDK. The DPDK repository should be > +clean to avoid additional tags confuse script. Normally the start of range is I agree with the overall hint being useful, but dirty/clean have a very specific meaning in git and I wonder if that makes this confusing. How about: "The repository this is executed in needs to be free of conflicting tags - since developers accumulate remote repositories, branches and tags a fresh git clone is the most reliable source to use." ^^ This (using a new clone) also matches your extended example that you've added > the end tag of last generated range, branch tag if working on the first > stable release. The end of range is latest release tag. Assuming v17.11-rc1 > -is out you can prepare a commit list for v17.08.1 stable release with: > +is out, you should checkout a branch that contains end of range, normally > +main branch. Hmm, you are right it is needed to be on a branch containing those. I never realized that and got lucky by accident it seems. I tried a different branch and it indeed returns an empty list. It is the function commit_version and therein the --merged that otherwise fails. One could provide a commit to --merged, but then how would we reliably know how the repository and branches are called. I agree that asking for the main branch to be checked out is a simple solution until someone wants to improve git-log-fixes.sh in that regard. Thereby ack to this section. > Then prepare a commit list for v17.08.1 stable release with: > > + $ git clone http://dpdk.org/git/dpdk > + $ cd dpdk Ack to this, while everyone can do it differently - this example works "for sure" > $ ./devtools/git-log-fixes.sh v17.08..v17.11-rc1 > /tmp/list > > > -- > 2.25.1 > -- Christian Ehrhardt Staff Engineer, Ubuntu Server Canonical Ltd ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-stable] [scripts] README: clean DPDK clone and main branch to generate list 2021-04-28 6:40 ` [dpdk-stable] [scripts] README: clean DPDK clone and main branch to generate list Christian Ehrhardt @ 2021-05-06 13:21 ` Xueming(Steven) Li 0 siblings, 0 replies; 6+ messages in thread From: Xueming(Steven) Li @ 2021-05-06 13:21 UTC (permalink / raw) To: Christian Ehrhardt; +Cc: dpdk stable, Luca Boccassi > -----Original Message----- > From: Christian Ehrhardt <christian.ehrhardt@canonical.com> > Sent: Wednesday, April 28, 2021 2:40 PM > To: Xueming(Steven) Li <xuemingl@nvidia.com> > Cc: dpdk stable <stable@dpdk.org>; Luca Boccassi <bluca@debian.org> > Subject: Re: [scripts] README: clean DPDK clone and main branch to generate list > > On Wed, Apr 28, 2021 at 8:03 AM Xueming Li <xuemingl@nvidia.com> wrote: > > > > Tags of "dirty" repository confuse git-log-fixes.sh. > > The script also expects current branch to contains end tag of range. > > > > > > Signed-off-by: Xueming Li <xuemingl@nvidia.com> > > --- > > README | 8 ++++++-- > > 1 file changed, 6 insertions(+), 2 deletions(-) > > > > diff --git a/README b/README > > index 70f4320..911f169 100644 > > --- a/README > > +++ b/README > > @@ -26,11 +26,15 @@ as tmp dir for the patches and mails. > > 1.A > > --- > > A list of commits to backport should be generated. This can be > > generated -from "git-log-fixes.sh" script from DPDK. Normally the > > start of range is > > +from "git-log-fixes.sh" script from DPDK. The DPDK repository should > > +be clean to avoid additional tags confuse script. Normally the start > > +of range is > > I agree with the overall hint being useful, but dirty/clean have a very specific meaning in git and I wonder if that makes this confusing. > How about: "The repository this is executed in needs to be free of conflicting tags - since developers accumulate remote repositories, > branches and tags a fresh git clone is the most reliable source to use." > ^^ This (using a new clone) also matches your extended example that you've added Thanks, new version sent. > > > the end tag of last generated range, branch tag if working on the > > first stable release. The end of range is latest release tag. > > Assuming v17.11-rc1 -is out you can prepare a commit list for v17.08.1 stable release with: > > +is out, you should checkout a branch that contains end of range, > > +normally main branch. > > Hmm, you are right it is needed to be on a branch containing those. > I never realized that and got lucky by accident it seems. > I tried a different branch and it indeed returns an empty list. > It is the function commit_version and therein the --merged that otherwise fails. > One could provide a commit to --merged, but then how would we reliably know how the repository and branches are called. > I agree that asking for the main branch to be checked out is a simple solution until someone wants to improve git-log-fixes.sh in that > regard I'm wondering is there some mature scripts from popular open source community could be borrowed. > Thereby ack to this section. > > > Then prepare a commit list for v17.08.1 stable release with: > > > > + $ git clone http://dpdk.org/git/dpdk > > + $ cd dpdk > > Ack to this, while everyone can do it differently - this example works "for sure" > > > $ ./devtools/git-log-fixes.sh v17.08..v17.11-rc1 > /tmp/list > > > > > > -- > > 2.25.1 > > > > > -- > Christian Ehrhardt > Staff Engineer, Ubuntu Server > Canonical Ltd ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <1620307153-12649-1-git-send-email-xuemingl@nvidia.com>]
* Re: [dpdk-stable] [scripts v2] README: fresh DPDK clone and main branch to generate list [not found] ` <1620307153-12649-1-git-send-email-xuemingl@nvidia.com> @ 2021-05-06 16:05 ` Luca Boccassi 0 siblings, 0 replies; 6+ messages in thread From: Luca Boccassi @ 2021-05-06 16:05 UTC (permalink / raw) To: Xueming Li, stable; +Cc: Christian Ehrhardt On Thu, 2021-05-06 at 13:19 +0000, Xueming Li wrote: > Appends notes of: > - git-log-fixes.sh needs to be free of conflicting tags. > - The script also expects current branch to contains end tag of range. > > > Signed-off-by: Xueming Li <xuemingl@nvidia.com> > --- > README | 16 +++++++++++----- > 1 file changed, 11 insertions(+), 5 deletions(-) > > diff --git a/README b/README > index 70f4320..260e46e 100644 > --- a/README > +++ b/README > @@ -26,11 +26,17 @@ as tmp dir for the patches and mails. > 1.A > --- > A list of commits to backport should be generated. This can be generated > -from "git-log-fixes.sh" script from DPDK. Normally the start of range is > -the end tag of last generated range, branch tag if working on the first > -stable release. The end of range is latest release tag. Assuming v17.11-rc1 > -is out you can prepare a commit list for v17.08.1 stable release with: > - > +from "git-log-fixes.sh" script from DPDK. The repository this is executed > +in needs to be free of conflicting tags - since developers accumulate remote > +repositories, branches and tags a fresh git clone is the most reliable source > +to use. Normally the start of range is the end tag of last generated range, > +branch tag if working on the first stable release. The end of range is latest > +release tag. Assuming v17.11-rc1 is out, you should checkout a branch that > +contains end of range, normally main branch. Then prepare a commit list for > +v17.08.1 stable release with: > + > + $ git clone http://dpdk.org/git/dpdk > + $ cd dpdk > $ ./devtools/git-log-fixes.sh v17.08..v17.11-rc1 > /tmp/list Acked-by: Luca Boccassi <bluca@debian.org> -- Kind regards, Luca Boccassi ^ permalink raw reply [flat|nested] 6+ messages in thread
* [dpdk-stable] [scripts v2] README: fresh DPDK clone and main branch to generate list [not found] <1619589811-9458-1-git-send-email-xuemingl@nvidia.com> 2021-04-28 6:40 ` [dpdk-stable] [scripts] README: clean DPDK clone and main branch to generate list Christian Ehrhardt [not found] ` <1620307153-12649-1-git-send-email-xuemingl@nvidia.com> @ 2021-05-10 13:23 ` Xueming Li 2021-05-10 13:27 ` Xueming(Steven) Li 2 siblings, 1 reply; 6+ messages in thread From: Xueming Li @ 2021-05-10 13:23 UTC (permalink / raw) To: stable, Luca Boccassi; +Cc: Christian Ehrhardt, xuemingl Appends notes of: - git-log-fixes.sh needs to be free of conflicting tags. - The script also expects current branch to contains end tag of range. Signed-off-by: Xueming Li <xuemingl@nvidia.com> --- README | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/README b/README index 70f4320..260e46e 100644 --- a/README +++ b/README @@ -26,11 +26,17 @@ as tmp dir for the patches and mails. 1.A --- A list of commits to backport should be generated. This can be generated -from "git-log-fixes.sh" script from DPDK. Normally the start of range is -the end tag of last generated range, branch tag if working on the first -stable release. The end of range is latest release tag. Assuming v17.11-rc1 -is out you can prepare a commit list for v17.08.1 stable release with: - +from "git-log-fixes.sh" script from DPDK. The repository this is executed +in needs to be free of conflicting tags - since developers accumulate remote +repositories, branches and tags a fresh git clone is the most reliable source +to use. Normally the start of range is the end tag of last generated range, +branch tag if working on the first stable release. The end of range is latest +release tag. Assuming v17.11-rc1 is out, you should checkout a branch that +contains end of range, normally main branch. Then prepare a commit list for +v17.08.1 stable release with: + + $ git clone http://dpdk.org/git/dpdk + $ cd dpdk $ ./devtools/git-log-fixes.sh v17.08..v17.11-rc1 > /tmp/list -- 2.25.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-stable] [scripts v2] README: fresh DPDK clone and main branch to generate list 2021-05-10 13:23 ` Xueming Li @ 2021-05-10 13:27 ` Xueming(Steven) Li 2021-05-20 19:59 ` Christian Ehrhardt 0 siblings, 1 reply; 6+ messages in thread From: Xueming(Steven) Li @ 2021-05-10 13:27 UTC (permalink / raw) To: stable, Luca Boccassi; +Cc: Christian Ehrhardt Hi Luca, Sorry that the previous email only delivered to individual instead of maillist due to some internal tech issue. Resend now. Best Regards, Xueming > -----Original Message----- > From: Xueming(Steven) Li <xuemingl@nvidia.com> > Sent: Monday, May 10, 2021 9:23 PM > To: stable@dpdk.org; Luca Boccassi <bluca@debian.org> > Cc: Christian Ehrhardt <christian.ehrhardt@canonical.com>; Xueming(Steven) Li <xuemingl@nvidia.com> > Subject: [scripts v2] README: fresh DPDK clone and main branch to generate list > > Appends notes of: > - git-log-fixes.sh needs to be free of conflicting tags. > - The script also expects current branch to contains end tag of range. > > > Signed-off-by: Xueming Li <xuemingl@nvidia.com> > --- > README | 16 +++++++++++----- > 1 file changed, 11 insertions(+), 5 deletions(-) > > diff --git a/README b/README > index 70f4320..260e46e 100644 > --- a/README > +++ b/README > @@ -26,11 +26,17 @@ as tmp dir for the patches and mails. > 1.A > --- > A list of commits to backport should be generated. This can be generated -from "git-log-fixes.sh" script from DPDK. Normally the start > of range is -the end tag of last generated range, branch tag if working on the first -stable release. The end of range is latest release tag. > Assuming v17.11-rc1 -is out you can prepare a commit list for v17.08.1 stable release with: > - > +from "git-log-fixes.sh" script from DPDK. The repository this is > +executed in needs to be free of conflicting tags - since developers > +accumulate remote repositories, branches and tags a fresh git clone is > +the most reliable source to use. Normally the start of range is the end > +tag of last generated range, branch tag if working on the first stable > +release. The end of range is latest release tag. Assuming v17.11-rc1 is > +out, you should checkout a branch that contains end of range, normally > +main branch. Then prepare a commit list for > +v17.08.1 stable release with: > + > + $ git clone http://dpdk.org/git/dpdk > + $ cd dpdk > $ ./devtools/git-log-fixes.sh v17.08..v17.11-rc1 > /tmp/list > > > -- > 2.25.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-stable] [scripts v2] README: fresh DPDK clone and main branch to generate list 2021-05-10 13:27 ` Xueming(Steven) Li @ 2021-05-20 19:59 ` Christian Ehrhardt 0 siblings, 0 replies; 6+ messages in thread From: Christian Ehrhardt @ 2021-05-20 19:59 UTC (permalink / raw) To: Xueming(Steven) Li; +Cc: stable, Luca Boccassi On Mon, May 10, 2021 at 3:27 PM Xueming(Steven) Li <xuemingl@nvidia.com> wrote: > > Hi Luca, > > Sorry that the previous email only delivered to individual instead of maillist due to some internal tech issue. > Resend now. No problem, v2 is here and Acked. Thanks for your help, I've pushed it now. > Best Regards, > Xueming > > > -----Original Message----- > > From: Xueming(Steven) Li <xuemingl@nvidia.com> > > Sent: Monday, May 10, 2021 9:23 PM > > To: stable@dpdk.org; Luca Boccassi <bluca@debian.org> > > Cc: Christian Ehrhardt <christian.ehrhardt@canonical.com>; Xueming(Steven) Li <xuemingl@nvidia.com> > > Subject: [scripts v2] README: fresh DPDK clone and main branch to generate list > > > > Appends notes of: > > - git-log-fixes.sh needs to be free of conflicting tags. > > - The script also expects current branch to contains end tag of range. > > > > > > Signed-off-by: Xueming Li <xuemingl@nvidia.com> > > --- > > README | 16 +++++++++++----- > > 1 file changed, 11 insertions(+), 5 deletions(-) > > > > diff --git a/README b/README > > index 70f4320..260e46e 100644 > > --- a/README > > +++ b/README > > @@ -26,11 +26,17 @@ as tmp dir for the patches and mails. > > 1.A > > --- > > A list of commits to backport should be generated. This can be generated -from "git-log-fixes.sh" script from DPDK. Normally the start > > of range is -the end tag of last generated range, branch tag if working on the first -stable release. The end of range is latest release tag. > > Assuming v17.11-rc1 -is out you can prepare a commit list for v17.08.1 stable release with: > > - > > +from "git-log-fixes.sh" script from DPDK. The repository this is > > +executed in needs to be free of conflicting tags - since developers > > +accumulate remote repositories, branches and tags a fresh git clone is > > +the most reliable source to use. Normally the start of range is the end > > +tag of last generated range, branch tag if working on the first stable > > +release. The end of range is latest release tag. Assuming v17.11-rc1 is > > +out, you should checkout a branch that contains end of range, normally > > +main branch. Then prepare a commit list for > > +v17.08.1 stable release with: > > + > > + $ git clone http://dpdk.org/git/dpdk > > + $ cd dpdk > > $ ./devtools/git-log-fixes.sh v17.08..v17.11-rc1 > /tmp/list > > > > > > -- > > 2.25.1 > -- Christian Ehrhardt Staff Engineer, Ubuntu Server Canonical Ltd ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-05-20 19:59 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <1619589811-9458-1-git-send-email-xuemingl@nvidia.com> 2021-04-28 6:40 ` [dpdk-stable] [scripts] README: clean DPDK clone and main branch to generate list Christian Ehrhardt 2021-05-06 13:21 ` Xueming(Steven) Li [not found] ` <1620307153-12649-1-git-send-email-xuemingl@nvidia.com> 2021-05-06 16:05 ` [dpdk-stable] [scripts v2] README: fresh " Luca Boccassi 2021-05-10 13:23 ` Xueming Li 2021-05-10 13:27 ` Xueming(Steven) Li 2021-05-20 19:59 ` Christian Ehrhardt
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).