patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [PATCH stable-scripts 1/2] Update version mangling
@ 2020-03-17 10:19 luca.boccassi
  2020-03-17 10:19 ` [dpdk-stable] [PATCH stable-scripts 2/2] Make failed_list and known_issues optional luca.boccassi
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: luca.boccassi @ 2020-03-17 10:19 UTC (permalink / raw)
  To: stable; +Cc: ktraynor

From: Luca Boccassi <luca.boccassi@microsoft.com>

The version has moved to a VERSION file, and the spec file has been
removed.

Signed-off-by: Luca Boccassi <luca.boccassi@microsoft.com>
---
 5-make-release-commit | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/5-make-release-commit b/5-make-release-commit
index a73efa7..4926ab1 100755
--- a/5-make-release-commit
+++ b/5-make-release-commit
@@ -11,8 +11,12 @@ change_version()
 		-i $DPDK_DIR/lib/librte_eal/common/include/rte_version.h
 	sed "s/#define RTE_VER_RELEASE .*/#define RTE_VER_RELEASE 16/" \
 		-i $DPDK_DIR/lib/librte_eal/common/include/rte_version.h
-	sed "s/Version: .*/Version: ${stable_release}/" -i $DPDK_DIR/pkg/dpdk.spec
-	if [ -e $DPDK_DIR/meson.build ]; then
+	if [ -e $DPDK_DIR/pkg/dpdk.spec ]; then
+		sed "s/Version: .*/Version: ${stable_release}/" -i $DPDK_DIR/pkg/dpdk.spec
+	fi
+	if [ -e $DPDK_DIR/VERSION ]; then
+		echo "${stable_release}" > $DPDK_DIR/VERSION
+	elif [ -e $DPDK_DIR/meson.build ]; then
 		sed "s/version: '.*',/version: '${stable_release}',/" -i $DPDK_DIR/meson.build
 	fi
 }
-- 
2.20.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [dpdk-stable] [PATCH stable-scripts 2/2] Make failed_list and known_issues optional
  2020-03-17 10:19 [dpdk-stable] [PATCH stable-scripts 1/2] Update version mangling luca.boccassi
@ 2020-03-17 10:19 ` luca.boccassi
  2020-03-18 12:08   ` Kevin Traynor
  2020-03-18 12:15 ` [dpdk-stable] [PATCH stable-scripts 1/2] Update version mangling Kevin Traynor
  2020-03-18 12:47 ` Kevin Traynor
  2 siblings, 1 reply; 5+ messages in thread
From: luca.boccassi @ 2020-03-17 10:19 UTC (permalink / raw)
  To: stable; +Cc: ktraynor

From: Luca Boccassi <luca.boccassi@microsoft.com>

Some releases don't have them
---
 5-make-release-commit | 26 ++++++++++++++------------
 6-announce-release    | 19 ++++++++++---------
 2 files changed, 24 insertions(+), 21 deletions(-)

diff --git a/5-make-release-commit b/5-make-release-commit
index 4926ab1..832e8e4 100755
--- a/5-make-release-commit
+++ b/5-make-release-commit
@@ -43,19 +43,21 @@ update_release_note()
 
 	echo "$(cat $stable_release/validation)"
 
-        echo
-        echo "$stable_release Known Issues"
-        echo "~~~~~~~~~~~~~~~~~~~~"
-        echo
+	if [ -e $stable_release/known_issues ]; then
+		echo
+		echo "$stable_release Known Issues"
+		echo "~~~~~~~~~~~~~~~~~~~~"
+		echo
+		echo "$(cat $stable_release/known_issues)"
+	fi
 
-	echo "$(cat $stable_release/known_issues)"
-
-        echo
-        echo "$stable_release Fixes skipped and status unresolved"
-        echo "~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"
-        echo
-
-	echo "$(cat $stable_release/failed_list)"
+	if [ -e $stable_release/failed_list ]; then
+		echo
+		echo "$stable_release Fixes skipped and status unresolved"
+		echo "~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"
+		echo
+		echo "$(cat $stable_release/failed_list)"
+	fi
 
 	} >> $DPDK_DIR/doc/guides/rel_notes/release_${RTE_VER_MAJOR/./_}.rst
 }
diff --git a/6-announce-release b/6-announce-release
index 0a7a4fc..6d9fb86 100755
--- a/6-announce-release
+++ b/6-announce-release
@@ -2,13 +2,11 @@
 
 source ./lib.sh
 
-if [ $# -lt 1 ] || [ ! -f "$1" ]
+if [ $# -ge 1 ]
 then
-	echo "Usage: $0 failed_list"
-	exit 1
+	failed_list="/tmp/.list-$$"
+	grep -v "^#" $1 > $failed_list
 fi
-failed_list="/tmp/.list-$$"
-grep -v "^#" $1 > $failed_list
 
 get_diffs()
 {
@@ -44,7 +42,9 @@ EOF
 
 compose_release_note
 
-cat << EOF
+if [ $# -ge 1 ]
+then
+	cat << EOF
 ---[ List of missing patches ]---
 The following is the accumulated list of skipped patches. Authors/maintainers
 have not yet responded to backport requests. Actions to take per each patch,
@@ -53,6 +53,7 @@ have not yet responded to backport requests. Actions to take per each patch,
 	b) send out a backport to stable@dpdk.org
 
 EOF
-while read commit patch; do
-	describe_commit $commit
-done < $failed_list | sort -k 2
+	while read commit patch; do
+		describe_commit $commit
+	done < $failed_list | sort -k 2
+fi
-- 
2.20.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-stable] [PATCH stable-scripts 2/2] Make failed_list and known_issues optional
  2020-03-17 10:19 ` [dpdk-stable] [PATCH stable-scripts 2/2] Make failed_list and known_issues optional luca.boccassi
@ 2020-03-18 12:08   ` Kevin Traynor
  0 siblings, 0 replies; 5+ messages in thread
From: Kevin Traynor @ 2020-03-18 12:08 UTC (permalink / raw)
  To: luca.boccassi, stable

On 17/03/2020 10:19, luca.boccassi@gmail.com wrote:
> From: Luca Boccassi <luca.boccassi@microsoft.com>
> 
> Some releases don't have them

Lucky you!

Acked-by: Kevin Traynor <ktraynor@redhat.com>

> ---
>  5-make-release-commit | 26 ++++++++++++++------------
>  6-announce-release    | 19 ++++++++++---------
>  2 files changed, 24 insertions(+), 21 deletions(-)
> 
> diff --git a/5-make-release-commit b/5-make-release-commit
> index 4926ab1..832e8e4 100755
> --- a/5-make-release-commit
> +++ b/5-make-release-commit
> @@ -43,19 +43,21 @@ update_release_note()
>  
>  	echo "$(cat $stable_release/validation)"
>  
> -        echo
> -        echo "$stable_release Known Issues"
> -        echo "~~~~~~~~~~~~~~~~~~~~"
> -        echo
> +	if [ -e $stable_release/known_issues ]; then
> +		echo
> +		echo "$stable_release Known Issues"
> +		echo "~~~~~~~~~~~~~~~~~~~~"
> +		echo
> +		echo "$(cat $stable_release/known_issues)"
> +	fi
>  
> -	echo "$(cat $stable_release/known_issues)"
> -
> -        echo
> -        echo "$stable_release Fixes skipped and status unresolved"
> -        echo "~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"
> -        echo
> -
> -	echo "$(cat $stable_release/failed_list)"
> +	if [ -e $stable_release/failed_list ]; then
> +		echo
> +		echo "$stable_release Fixes skipped and status unresolved"
> +		echo "~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"
> +		echo
> +		echo "$(cat $stable_release/failed_list)"
> +	fi
>  
>  	} >> $DPDK_DIR/doc/guides/rel_notes/release_${RTE_VER_MAJOR/./_}.rst
>  }
> diff --git a/6-announce-release b/6-announce-release
> index 0a7a4fc..6d9fb86 100755
> --- a/6-announce-release
> +++ b/6-announce-release
> @@ -2,13 +2,11 @@
>  
>  source ./lib.sh
>  
> -if [ $# -lt 1 ] || [ ! -f "$1" ]
> +if [ $# -ge 1 ]
>  then
> -	echo "Usage: $0 failed_list"
> -	exit 1
> +	failed_list="/tmp/.list-$$"
> +	grep -v "^#" $1 > $failed_list
>  fi
> -failed_list="/tmp/.list-$$"
> -grep -v "^#" $1 > $failed_list
>  
>  get_diffs()
>  {
> @@ -44,7 +42,9 @@ EOF
>  
>  compose_release_note
>  
> -cat << EOF
> +if [ $# -ge 1 ]
> +then
> +	cat << EOF
>  ---[ List of missing patches ]---
>  The following is the accumulated list of skipped patches. Authors/maintainers
>  have not yet responded to backport requests. Actions to take per each patch,
> @@ -53,6 +53,7 @@ have not yet responded to backport requests. Actions to take per each patch,
>  	b) send out a backport to stable@dpdk.org
>  
>  EOF
> -while read commit patch; do
> -	describe_commit $commit
> -done < $failed_list | sort -k 2
> +	while read commit patch; do
> +		describe_commit $commit
> +	done < $failed_list | sort -k 2
> +fi
> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-stable] [PATCH stable-scripts 1/2] Update version mangling
  2020-03-17 10:19 [dpdk-stable] [PATCH stable-scripts 1/2] Update version mangling luca.boccassi
  2020-03-17 10:19 ` [dpdk-stable] [PATCH stable-scripts 2/2] Make failed_list and known_issues optional luca.boccassi
@ 2020-03-18 12:15 ` Kevin Traynor
  2020-03-18 12:47 ` Kevin Traynor
  2 siblings, 0 replies; 5+ messages in thread
From: Kevin Traynor @ 2020-03-18 12:15 UTC (permalink / raw)
  To: luca.boccassi, stable

On 17/03/2020 10:19, luca.boccassi@gmail.com wrote:
> From: Luca Boccassi <luca.boccassi@microsoft.com>
> 
> The version has moved to a VERSION file, and the spec file has been
> removed.
> 

Acked-by: Kevin Traynor <ktraynor@redhat.com>

This is good for final release, but some changes are also needed if we
want to keep updating for RCs.

> Signed-off-by: Luca Boccassi <luca.boccassi@microsoft.com>
> ---
>  5-make-release-commit | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/5-make-release-commit b/5-make-release-commit
> index a73efa7..4926ab1 100755
> --- a/5-make-release-commit
> +++ b/5-make-release-commit
> @@ -11,8 +11,12 @@ change_version()
>  		-i $DPDK_DIR/lib/librte_eal/common/include/rte_version.h
>  	sed "s/#define RTE_VER_RELEASE .*/#define RTE_VER_RELEASE 16/" \
>  		-i $DPDK_DIR/lib/librte_eal/common/include/rte_version.h
> -	sed "s/Version: .*/Version: ${stable_release}/" -i $DPDK_DIR/pkg/dpdk.spec
> -	if [ -e $DPDK_DIR/meson.build ]; then
> +	if [ -e $DPDK_DIR/pkg/dpdk.spec ]; then
> +		sed "s/Version: .*/Version: ${stable_release}/" -i $DPDK_DIR/pkg/dpdk.spec
> +	fi
> +	if [ -e $DPDK_DIR/VERSION ]; then
> +		echo "${stable_release}" > $DPDK_DIR/VERSION
> +	elif [ -e $DPDK_DIR/meson.build ]; then
>  		sed "s/version: '.*',/version: '${stable_release}',/" -i $DPDK_DIR/meson.build
>  	fi
>  }
> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-stable] [PATCH stable-scripts 1/2] Update version mangling
  2020-03-17 10:19 [dpdk-stable] [PATCH stable-scripts 1/2] Update version mangling luca.boccassi
  2020-03-17 10:19 ` [dpdk-stable] [PATCH stable-scripts 2/2] Make failed_list and known_issues optional luca.boccassi
  2020-03-18 12:15 ` [dpdk-stable] [PATCH stable-scripts 1/2] Update version mangling Kevin Traynor
@ 2020-03-18 12:47 ` Kevin Traynor
  2 siblings, 0 replies; 5+ messages in thread
From: Kevin Traynor @ 2020-03-18 12:47 UTC (permalink / raw)
  To: luca.boccassi, stable

On 17/03/2020 10:19, luca.boccassi@gmail.com wrote:
> From: Luca Boccassi <luca.boccassi@microsoft.com>
> 
> The version has moved to a VERSION file, and the spec file has been
> removed.
> 
> Signed-off-by: Luca Boccassi <luca.boccassi@microsoft.com>
> ---
>  5-make-release-commit | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/5-make-release-commit b/5-make-release-commit
> index a73efa7..4926ab1 100755
> --- a/5-make-release-commit
> +++ b/5-make-release-commit
> @@ -11,8 +11,12 @@ change_version()
>  		-i $DPDK_DIR/lib/librte_eal/common/include/rte_version.h
>  	sed "s/#define RTE_VER_RELEASE .*/#define RTE_VER_RELEASE 16/" \
>  		-i $DPDK_DIR/lib/librte_eal/common/include/rte_version.h
> -	sed "s/Version: .*/Version: ${stable_release}/" -i $DPDK_DIR/pkg/dpdk.spec
> -	if [ -e $DPDK_DIR/meson.build ]; then
> +	if [ -e $DPDK_DIR/pkg/dpdk.spec ]; then
> +		sed "s/Version: .*/Version: ${stable_release}/" -i $DPDK_DIR/pkg/dpdk.spec
> +	fi
> +	if [ -e $DPDK_DIR/VERSION ]; then
> +		echo "${stable_release}" > $DPDK_DIR/VERSION
> +	elif [ -e $DPDK_DIR/meson.build ]; then
>  		sed "s/version: '.*',/version: '${stable_release}',/" -i $DPDK_DIR/meson.build
>  	fi
>  }
> 

Applied series, thanks.


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-03-18 12:47 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-17 10:19 [dpdk-stable] [PATCH stable-scripts 1/2] Update version mangling luca.boccassi
2020-03-17 10:19 ` [dpdk-stable] [PATCH stable-scripts 2/2] Make failed_list and known_issues optional luca.boccassi
2020-03-18 12:08   ` Kevin Traynor
2020-03-18 12:15 ` [dpdk-stable] [PATCH stable-scripts 1/2] Update version mangling Kevin Traynor
2020-03-18 12:47 ` Kevin Traynor

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).