Re: [Buildroot] [PATCH] support/scripts/check-host-rpath: send readelf error output to oblivion

["Yann E. MORIN" <yann.morin.1998@free.fr>]

Thomas, All,

On 2022-07-27 19:42 +0200, Thomas Petazzoni via buildroot spake thusly:
> On Wed, 27 Jul 2022 18:40:57 +0200
> "Yann E. MORIN" <yann.morin.1998@free.fr> wrote:
> > > Of course, it would be nicer to only filter out those warnings, but
> > > filtering the error output without merging the error output into the
> > > standard output is tricky, so let's keep things simple. If there is
> > > really an error, readelf will abort.
> > Applied to master, thanks.
> Thanks! I must say I was kind of hoping you would come up with a way of
> keeping the errors displayed, while filtering the warnings :-)

That's going to be a little bit convoluted...

> But another issue in this script is that it doesn't have set -o
> pipefail, so if readelf fails, because the output is piped into another
> process, the script will not bork. Perhaps something to fix ;-) (hint hint!)

Using -o pipefail would not help, because the pipe is in a process
substitution:

    some-command < <( readelf blabla |sed foobar )

If we set -o pipefail, then the pipe will indeed fail, but the process
substitution will not fail.

So, meh.

However, check-host-rpath is slow as hell; it would highly benefit from
a makeover.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot