[dpdk-stable] [PATCH stable-scripts 1/2] 3-request-backport: suppress pushd/popd output

[dpdk-stable] [PATCH stable-scripts 1/2] 3-request-backport: suppress pushd/popd output

[Luca Boccassi]

Otherwise it ends up in the email's text

Signed-off-by: Luca Boccassi <bluca@debian.org>
---
 3-request-backport | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/3-request-backport b/3-request-backport
index 5fa814d..8bdf2a4 100755
--- a/3-request-backport
+++ b/3-request-backport
@@ -6,7 +6,7 @@ list=""
 if [ $# -lt 1 ] && [ "$GIT_AM_PAUSE_ON_FAIL" = "yes" ]
 then
 	_scripts_dir="$PWD"
-	pushd $STABLE_DIR
+	pushd $STABLE_DIR &>/dev/null
 	list=$(cat $_scripts_dir/$stable_release/tmp_import/list | tr '\n' ' ')
 	_parent_commit_id=$(cat $_scripts_dir/$stable_release/parent_commit_id)
 	_applied=$(git log --oneline $_parent_commit_id..HEAD | cut -d\  -f2-)
@@ -15,7 +15,7 @@ then
 		_sub=$(grep Subject $_file | cut -d\  -f3-)
 		echo "$_applied" | grep -qF "$_sub" || (head -n1 $_file | awk '{print $2}' >> /tmp/.list-$$)
 	done
-	popd
+	popd &>/dev/null
 else
 	list=$1
 	[ -f "$list" ] || {
-- 
2.20.1

[dpdk-stable] [PATCH stable-scripts 2/2] 3-request-backport: use BCC for stable@dpdk.org to bypass filters

[Luca Boccassi]

It has been reported that the backport request mail gets caught by filters
and thus many developers do not see it.
Use BCC instead of CC for the list, and use it in Reply-To, to bypass the
filters.

Signed-off-by: Luca Boccassi <bluca@debian.org>
---
 3-request-backport | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/3-request-backport b/3-request-backport
index 8bdf2a4..62f6bf3 100755
--- a/3-request-backport
+++ b/3-request-backport
@@ -43,7 +43,8 @@ get_cc_list()
 	rm $cc_list
 }
 
-echo "To: dpdk stable <stable@dpdk.org>"
+echo "Bcc: dpdk stable <stable@dpdk.org>"
+echo "Reply-To: dpdk stable <stable@dpdk.org>"
 get_cc_list
 echo "Subject: please help backporting some patches to $(stable_or_lts) release $stable_release"
 
-- 
2.20.1

Re: [dpdk-stable] [PATCH stable-scripts 2/2] 3-request-backport: use BCC for stable@dpdk.org to bypass filters

[Yongseok Koh]

> On Feb 24, 2019, at 4:24 AM, Luca Boccassi <bluca@debian.org> wrote:
> 
> It has been reported that the backport request mail gets caught by filters
> and thus many developers do not see it.
> Use BCC instead of CC for the list, and use it in Reply-To, to bypass the
> filters.
> 
> Signed-off-by: Luca Boccassi <bluca@debian.org>
> ---

Acked-by: Yongseok Koh <yskoh@mellanox.com>
 
Thanks

> 3-request-backport | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/3-request-backport b/3-request-backport
> index 8bdf2a4..62f6bf3 100755
> --- a/3-request-backport
> +++ b/3-request-backport
> @@ -43,7 +43,8 @@ get_cc_list()
> 	rm $cc_list
> }
> 
> -echo "To: dpdk stable <stable@dpdk.org>"
> +echo "Bcc: dpdk stable <stable@dpdk.org>"
> +echo "Reply-To: dpdk stable <stable@dpdk.org>"
> get_cc_list
> echo "Subject: please help backporting some patches to $(stable_or_lts) release $stable_release"
> 
> -- 
> 2.20.1
> 

Re: [dpdk-stable] [PATCH stable-scripts 1/2] 3-request-backport: suppress pushd/popd output

[Yongseok Koh]

> On Feb 24, 2019, at 4:24 AM, Luca Boccassi <bluca@debian.org> wrote:
> 
> Otherwise it ends up in the email's text
> 
> Signed-off-by: Luca Boccassi <bluca@debian.org>
> ---

Acked-by: Yongseok Koh <yskoh@mellanox.com>
 
Thanks

> 3-request-backport | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/3-request-backport b/3-request-backport
> index 5fa814d..8bdf2a4 100755
> --- a/3-request-backport
> +++ b/3-request-backport
> @@ -6,7 +6,7 @@ list=""
> if [ $# -lt 1 ] && [ "$GIT_AM_PAUSE_ON_FAIL" = "yes" ]
> then
> 	_scripts_dir="$PWD"
> -	pushd $STABLE_DIR
> +	pushd $STABLE_DIR &>/dev/null
> 	list=$(cat $_scripts_dir/$stable_release/tmp_import/list | tr '\n' ' ')
> 	_parent_commit_id=$(cat $_scripts_dir/$stable_release/parent_commit_id)
> 	_applied=$(git log --oneline $_parent_commit_id..HEAD | cut -d\  -f2-)
> @@ -15,7 +15,7 @@ then
> 		_sub=$(grep Subject $_file | cut -d\  -f3-)
> 		echo "$_applied" | grep -qF "$_sub" || (head -n1 $_file | awk '{print $2}' >> /tmp/.list-$$)
> 	done
> -	popd
> +	popd &>/dev/null
> else
> 	list=$1
> 	[ -f "$list" ] || {
> -- 
> 2.20.1
>