Re: [PATCH] tests/meson: fix mock library link on MacOS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 8/31/20 3:36 PM, Daniel Henrique Barboza wrote:


On 8/28/20 4:35 PM, Scott Shambarger wrote:
MacOS can not pre-load modules, so mock libraries must be built
as shared libraries (without asneeded striping, and undefined
symbols allowed).

Signed-off-by: Scott Shambarger <scott-libvirt@xxxxxxxxxxxxxx>
---

Given that this is a MacOS specific limitation it's not a good idea
to force it on the common code for everyone else.

The 'meson.build' file in the project root uses, in line 1727:

host_machine.system() != 'darwin'

To identify the building system as MacOS. IMO you should use an
"if host_machine.system() != 'darwin'" inside the 'foreach' loop
below to do whatever it is necessary for MacOS, leave the rest as
is.


Thing is, we want our test suite to run on as many platforms as possible (ideally every supported one), because only then we can guarantee basic functionality of that platform. But client also contains some drivers and thus is worth testing. Therefore, I would rather see us running all tests possible.

Michal




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux