Re: [PATCH v1 03/15] meson: move libmpathpersist test

[Alex Bennée]

Paolo Bonzini <pbonzini@redhat.com> writes:

> On 25/09/20 17:40, Alex Bennée wrote:
>> From: Paolo Bonzini <pbonzini@redhat.com>
>> 
>> This is the first compiler/linker test that has been moved to Meson.
>> Add more section headings to keep things clearer.
>> 
>> [thuth: Add check for mpathpersist.found() before showing 
>> mpathpersist_new_api]
>> 
>> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> Message-Id: <20200918103430.297167-3-thuth@redhat.com>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
<snip>
>
> This is not the latest version of the series, can you drop patches 2
> and 3?

Hmm so now I'm seeing failures in the bionic builds thanks to libmpath:

  Linking target qemu-pr-helper
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `free_config'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `dlog'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `strlcpy'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `mpath_connect'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `mpath_disconnect'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `select_reservation_key'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `free_pathvec'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `dm_get_maps'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `select_all_tg_pt'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `find_mp_by_alias'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `send_packet'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `cleanup_prio'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `pathinfo'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `disassemble_map'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `dm_mapname'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `recv_packet'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `vector_alloc'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `dm_get_status'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `pathcount'
  /usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `cleanup_checkers'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `set_max_fds'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `disassemble_status'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `dm_map_present'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `dm_is_mpath'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `dm_lib_release'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `get_udev_device'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `dm_get_map'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `dm_lib_exit'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `free_multipathvec'
  /usr/bin/ld: 
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../libmpathpersist.so: undefined 
reference to `load_config'
  clang: error: linker command failed with exit code 1 (use -v to see 
invocation)
  make: *** [Makefile.ninja:2150: qemu-pr-helper] Error 1
  make: *** Waiting for unfinished jobs....

Is this detection ordering issue? It happens on master as well but isn't
currently picked up by CI because nothing is running Focal yet.

-- 
Alex Bennée