[Buildroot] [PATCH] support/download: further silence the git helper

[Buildroot] [PATCH] support/download: further silence the git helper

[Yann E. MORIN]

We expresely call printf in the git helper, calls which were not
addresed in the previous silent-build patchset.

Just redirect stdout to oblivion when being silent.

Reported-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 support/download/git | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/support/download/git b/support/download/git
index 7f1801f..73cbc0f 100755
--- a/support/download/git
+++ b/support/download/git
@@ -14,7 +14,7 @@ set -e
 verbose=-v
 while getopts :q OPT; do
     case "${OPT}" in
-    q)  verbose=-q;;
+    q)  verbose=-q; exec >/dev/null;;
     \?) printf "unknown option '%s'\n" "${OPTARG}" >&2; exit 1;;
     esac
 done
-- 
1.9.1

[Buildroot] [PATCH] support/download: further silence the git helper

[Fabio Porcedda]

On Sun, Jan 4, 2015 at 7:10 PM, Yann E. MORIN <yann.morin.1998@free.fr> wrote:
> We expresely call printf in the git helper, calls which were not
> addresed in the previous silent-build patchset.
>
> Just redirect stdout to oblivion when being silent.
>
> Reported-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>  support/download/git | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/support/download/git b/support/download/git
> index 7f1801f..73cbc0f 100755
> --- a/support/download/git
> +++ b/support/download/git
> @@ -14,7 +14,7 @@ set -e
>  verbose=-v
>  while getopts :q OPT; do
>      case "${OPT}" in
> -    q)  verbose=-q;;
> +    q)  verbose=-q; exec >/dev/null;;

As general rule It's best to use "&&" instead of ";" so any error can
be caught, otherwise is:
Acked-by: Fabio Porcedda <fabio.porcedda@gmail.com>
Tested-by: Fabio Porcedda <fabio.porcedda@gmail.com>

BR
-- 
Fabio Porcedda

[Buildroot] [PATCH] support/download: further silence the git helper

[Thomas Petazzoni]

Dear Yann E. MORIN,

On Sun,  4 Jan 2015 19:10:14 +0100, Yann E. MORIN wrote:
> We expresely call printf in the git helper, calls which were not
> addresed in the previous silent-build patchset.
> 
> Just redirect stdout to oblivion when being silent.
> 
> Reported-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Applied, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH] support/download: further silence the git helper

[Yann E. MORIN]

Fabio, All,

On 2015-01-05 12:35 +0100, Fabio Porcedda spake thusly:
> On Sun, Jan 4, 2015 at 7:10 PM, Yann E. MORIN <yann.morin.1998@free.fr> wrote:
> > We expresely call printf in the git helper, calls which were not
> > addresed in the previous silent-build patchset.
> >
> > Just redirect stdout to oblivion when being silent.
> >
> > Reported-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> > ---
> >  support/download/git | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/support/download/git b/support/download/git
> > index 7f1801f..73cbc0f 100755
> > --- a/support/download/git
> > +++ b/support/download/git
> > @@ -14,7 +14,7 @@ set -e
> >  verbose=-v
> >  while getopts :q OPT; do
> >      case "${OPT}" in
> > -    q)  verbose=-q;;
> > +    q)  verbose=-q; exec >/dev/null;;
> 
> As general rule It's best to use "&&" instead of ";" so any error can
> be caught,

That is really not necessary, because the backend scripts are all
'set -e', so they exit on the first error that occurs.

Besides, even if they were not 'set -e', then the '&&' construct would
not catch the error at all, and the script would continue without
redirecting.

Furthermore, in that case, we're doing two things:
  - set a variable
  - redirect stdout to /dev/null

If either were to fail, you would have a hell of a bigger problem:
  - unable to set a variable: big problem in the shell
  - not able to redirect: /dev/null does not exist, or you do not have
    the rights to write to it.

In either case, this would be a really, *really* big problem, which I
doubt we should care about. ;-)

> otherwise is:
> Acked-by: Fabio Porcedda <fabio.porcedda@gmail.com>
> Tested-by: Fabio Porcedda <fabio.porcedda@gmail.com>

Thanks!

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'