Re: [Qemu-devel] [PATCH] Remove extra -lrt switch

[Peter Portante]

On 04/27/2012 02:14 PM, Peter Maydell wrote:
> On 27 April 2012 18:43, Peter Portante<address@hidden>  wrote:
> > Remove the extra -lrt switch which might be there from the package
> > config check for gthreads.
> >
> > See also: e3c56761b465a4253871c32b06ebbc2d8b3fc3e1 for the extra
> > pthread switch removal.
> >
> > Signed-off-by: Peter Portante<address@hidden>
> > ---
> >   configure |   11 ++++++++++-
> >   1 files changed, 10 insertions(+), 1 deletions(-)
> >
> > diff --git a/configure b/configure
> > index 15bbc73..e748b06 100755
> > --- a/configure
> > +++ b/configure
> > @@ -2581,7 +2581,16 @@ EOF
> >   if compile_prog "" "" ; then
> >    :
> >   elif compile_prog "" "-lrt" ; then
> > -  LIBS="-lrt $LIBS"
> > +  found=no
> > +  for lib_entry in $LIBS; do
> > +    if test "$lib_entry" = "-lrt"; then
> > +      found=yes
> > +      break
> > +    fi
> > +  done
> > +  if test "$found" = "no"; then
> > +    LIBS="-lrt $LIBS"
> > +  fi
> >   fi
> If we need this more than once it looks like a candidate for
> a shell function, maybe:
>    add_to_libs -lrt
> ?

Great idea, thanks for pointing that out. I'll re-spin this patch as a 
re-factoring of the first, and then apply the second, by using a shell 
script as proposed.

-peter

> -- PMM