[Bug 1896096] Re: Git version: Build process is broken in block_curl.c.o

[Peter Maydell]

Huh. I thought we ignored the environment LDFLAGS/CFLAGS from much
earlier than 5.0 and was about to ask if honouring it was a change
introduced with the Meson switch! My (not necessarily very considered)
opinion is that generally it's better not to honour the environment
CFLAGS and LDFLAGS because they can have all sorts of random stuff in
them that just doesn't work in complicated cases like QEMU, as in this
example where they included "-pie". --extra-ldflags has the semantics of
"add these on top of whatever the standard set of QEMU build flags is"
so the user doesn't need to try to specify absolutely everything, and
it's clear from the configure line that extra non-default options are
being added.

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1896096

Title:
  Git version: Build process is broken in block_curl.c.o

Status in QEMU:
  Invalid

Bug description:
  Gcc version: 10.2.0
  Glusterfs: 8.1
  Libguestfs: 1.42

  Configure options used:

  configure \
      --prefix=/usr \
      --sysconfdir=/etc \
      --localstatedir=/var \
      --libexecdir=/usr/lib/qemu \
      --extra-ldflags="$LDFLAGS" \
      --smbd=/usr/bin/smbd \
      --enable-modules \
      --enable-sdl \
      --disable-werror \
      --enable-slirp=system \
      --enable-xfsctl \
      --audio-drv-list="pa alsa sdl"
      
  Error log attached. Here is the beginning:

  /usr/bin/ld: /usr/lib/gcc/x86_64-pc-linux-gnu/10.2.0/../../../../lib/Scrt1.o: 
in function `_start':
  (.text+0x24): undefined reference to `main'
  /usr/bin/ld: libblock-curl.a(block_curl.c.o): in function `curl_block_init':

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1896096/+subscriptions