[Buildroot] [PATCH] support/download: further silence the git helper
Yann E. MORIN
yann.morin.1998 at free.fr
Mon Jan 5 17:40:55 UTC 2015
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 at 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 at free-electrons.com>
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> > Cc: Thomas Petazzoni <thomas.petazzoni at 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 at gmail.com>
> Tested-by: Fabio Porcedda <fabio.porcedda at 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. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list