Re: [PATCH] tests/xenstore: link in librt

[Juergen Gross <jgross@xxxxxxxx>]

On 13.08.21 14:35, Jan Beulich wrote:
> On 13.08.2021 13:09, Ian Jackson wrote:
> > Jan Beulich writes ("Re: [PATCH] tests/xenstore: link in librt"):
> > > On 12.08.2021 13:24, Ian Jackson wrote:
> > > > > +LDFLAGS += -lrt
> > > > Don't this unconditionally is definitely not right.
> > > Assuming you meant "Doing this ..." - why? If the concern is that
> > > librt.so may needlessly get recorded in a DT_NEEDED entry, then I
> > > can replace the early addition with a late
> > >
> > > LDFLAGS += -Wl,--as-needed -lc -lrt
> > >
> > > one.
> > librt might not exist at all on some platforms.
> Hmm, indeed. Do you have any suggestion then? Adding a ./configure
> check is going to be beyond what I'd we willing to spend time on ...
libxl is using (probably due to the same problem):

LDUSELIBS-$(CONFIG_Linux) += -lrt

So I guess the same construct is possible here, too.


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature