patches for DPDK stable branches
 help / color / mirror / Atom feed
* 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

* 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

patches for DPDK stable branches

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/stable/0 stable/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 stable stable/ https://inbox.dpdk.org/stable \
		stable@dpdk.org
	public-inbox-index stable

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.stable


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git