Re: [libvirt] [PATCH 0/2] Revert use of weak symbols

On 07/12/2017 12:14 PM, Daniel P. Berrange wrote: > This series reverts the use of the __weak__ attribute which we did > to fix the test suite with CLang. It cause unintended problems > elsewhere which are not practical to fix. See the second patch > for details. > > Daniel P. Berrange (2): > Revert "internal: don't use weak symbols for Win32 platform" > Revert "Prevent more compiler optimization of mockable functions" > So I thought about this too and I guess my question is - why are we not dynamically linking with the mock libraries? What we do is use LD_PRELOAD at runtime. However, if say virpcitest would link with virpcimock at compile time (dynamically), we can throw away LD_PRELOAD handling code.