All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [Bug 1777252] [NEW] tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris
@ 2018-06-16 15:13 WHR
  2018-06-16 19:29 ` [Qemu-devel] [Bug 1777252] " Thomas Huth
                   ` (5 more replies)
  0 siblings, 6 replies; 8+ messages in thread
From: WHR @ 2018-06-16 15:13 UTC (permalink / raw)
  To: qemu-devel

Public bug reported:

Building script 'tests/Makefile.include' contains following code
```
ifeq ($(CONFIG_POSIX),y)
LIBS += -lutil
endif
```

library -lutil is not available on Solaris, so the building will failed, like
```
ld: fatal: library -lutil: not found
make: *** [SOMEWHERE/src/qemu-2.12.0/rules.mak:121: qemu-nbd] Error 1
```

Commenting those code out fixed the error.

** Affects: qemu
     Importance: Undecided
         Status: New


** Tags: solaris

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

Title:
  tests/Makefile.include trying to add linking library '-lutil' that
  break the build on Solaris

Status in QEMU:
  New

Bug description:
  Building script 'tests/Makefile.include' contains following code
  ```
  ifeq ($(CONFIG_POSIX),y)
  LIBS += -lutil
  endif
  ```

  library -lutil is not available on Solaris, so the building will failed, like
  ```
  ld: fatal: library -lutil: not found
  make: *** [SOMEWHERE/src/qemu-2.12.0/rules.mak:121: qemu-nbd] Error 1
  ```

  Commenting those code out fixed the error.

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Qemu-devel] [Bug 1777252] Re: tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris
  2018-06-16 15:13 [Qemu-devel] [Bug 1777252] [NEW] tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris WHR
@ 2018-06-16 19:29 ` Thomas Huth
  2018-06-18  5:55   ` Markus Armbruster
  2019-01-15  8:56 ` Thomas Huth
                   ` (4 subsequent siblings)
  5 siblings, 1 reply; 8+ messages in thread
From: Thomas Huth @ 2018-06-16 19:29 UTC (permalink / raw)
  To: qemu-devel

I'm sorry, but Solaris is currently unsupported and might get removed in
a future release, see:

 https://wiki.qemu.org/ChangeLog/2.12#Warning:_unsupported_host_systems

So it would be great if you could contribute patches, or find someone
who's willing to maintain QEMU on Solaris.

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

Title:
  tests/Makefile.include trying to add linking library '-lutil' that
  break the build on Solaris

Status in QEMU:
  New

Bug description:
  Building script 'tests/Makefile.include' contains following code
  ```
  ifeq ($(CONFIG_POSIX),y)
  LIBS += -lutil
  endif
  ```

  library -lutil is not available on Solaris, so the building will failed, like
  ```
  ld: fatal: library -lutil: not found
  make: *** [SOMEWHERE/src/qemu-2.12.0/rules.mak:121: qemu-nbd] Error 1
  ```

  Commenting those code out fixed the error.

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Qemu-devel] [Bug 1777252] Re: tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris
  2018-06-16 19:29 ` [Qemu-devel] [Bug 1777252] " Thomas Huth
@ 2018-06-18  5:55   ` Markus Armbruster
  0 siblings, 0 replies; 8+ messages in thread
From: Markus Armbruster @ 2018-06-18  5:55 UTC (permalink / raw)
  To: Thomas Huth; +Cc: qemu-devel

Thomas Huth <1777252@bugs.launchpad.net> writes:

> I'm sorry, but Solaris is currently unsupported and might get removed in
> a future release, see:
>
>  https://wiki.qemu.org/ChangeLog/2.12#Warning:_unsupported_host_systems

Quote configure:

if test "$supported_os" = "no"; then
    echo
    echo "WARNING: SUPPORT FOR THIS HOST OS WILL GO AWAY IN FUTURE RELEASES!"
    echo
    echo "Host OS $targetos support is not currently maintained."
    echo "The QEMU project intends to remove support for this host OS in"
    echo "a future release if nobody volunteers to maintain it and to"
    echo "provide a build host for our continuous integration setup."
    echo "configure has succeeded and you can continue to build, but"
    echo "if you care about QEMU on this platform you should contact"
    echo "us upstream at qemu-devel@nongnu.org."
fi

> So it would be great if you could contribute patches, or find someone
> who's willing to maintain QEMU on Solaris.

The key phrase here is "provide a build host for our continuous
integration setup".  Once that's in place, continuing maintenance
shouldn't be hard.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Qemu-devel] [Bug 1777252] Re: tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris
  2018-06-16 15:13 [Qemu-devel] [Bug 1777252] [NEW] tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris WHR
  2018-06-16 19:29 ` [Qemu-devel] [Bug 1777252] " Thomas Huth
@ 2019-01-15  8:56 ` Thomas Huth
  2019-01-15 17:39 ` Michele
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Thomas Huth @ 2019-01-15  8:56 UTC (permalink / raw)
  To: qemu-devel

Does something like this work for you?

diff --git a/tests/Makefile.include b/tests/Makefile.include
--- a/tests/Makefile.include
+++ b/tests/Makefile.include
@@ -777,7 +777,7 @@ tests/migration/initrd-stress.img: tests/migration/stress$(EXESUF)
        rm $(INITRD_WORK_DIR)/init
        rmdir $(INITRD_WORK_DIR)
 
-ifeq ($(CONFIG_POSIX),y)
+ifeq ($(CONFIG_POSIX)$(call lnot,$(CONFIG_SOLARIS)),yy)
 LIBS += -lutil
 endif

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

Title:
  tests/Makefile.include trying to add linking library '-lutil' that
  break the build on Solaris

Status in QEMU:
  New

Bug description:
  Building script 'tests/Makefile.include' contains following code
  ```
  ifeq ($(CONFIG_POSIX),y)
  LIBS += -lutil
  endif
  ```

  library -lutil is not available on Solaris, so the building will failed, like
  ```
  ld: fatal: library -lutil: not found
  make: *** [SOMEWHERE/src/qemu-2.12.0/rules.mak:121: qemu-nbd] Error 1
  ```

  Commenting those code out fixed the error.

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Qemu-devel] [Bug 1777252] Re: tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris
  2018-06-16 15:13 [Qemu-devel] [Bug 1777252] [NEW] tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris WHR
  2018-06-16 19:29 ` [Qemu-devel] [Bug 1777252] " Thomas Huth
  2019-01-15  8:56 ` Thomas Huth
@ 2019-01-15 17:39 ` Michele
  2019-01-15 18:22 ` Peter Maydell
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Michele @ 2019-01-15 17:39 UTC (permalink / raw)
  To: qemu-devel

That's great.  I spent quite a while last summer looking for the source
of that error, without success.  I finally just created a stub library
named lutil figuring I would then find which function was missing on the
next build attempt.  But it worked fine so apparently QEMU does not even
use whatever is in lutil, at least on Solaris.

I'm still offering to help maintain this as I have some actual Solaris
hardware.

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

Title:
  tests/Makefile.include trying to add linking library '-lutil' that
  break the build on Solaris

Status in QEMU:
  New

Bug description:
  Building script 'tests/Makefile.include' contains following code
  ```
  ifeq ($(CONFIG_POSIX),y)
  LIBS += -lutil
  endif
  ```

  library -lutil is not available on Solaris, so the building will failed, like
  ```
  ld: fatal: library -lutil: not found
  make: *** [SOMEWHERE/src/qemu-2.12.0/rules.mak:121: qemu-nbd] Error 1
  ```

  Commenting those code out fixed the error.

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Qemu-devel] [Bug 1777252] Re: tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris
  2018-06-16 15:13 [Qemu-devel] [Bug 1777252] [NEW] tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris WHR
                   ` (2 preceding siblings ...)
  2019-01-15 17:39 ` Michele
@ 2019-01-15 18:22 ` Peter Maydell
  2019-01-16  4:11 ` WHR
  2019-04-24  5:22 ` Thomas Huth
  5 siblings, 0 replies; 8+ messages in thread
From: Peter Maydell @ 2019-01-15 18:22 UTC (permalink / raw)
  To: qemu-devel

The condition we use in configure to decide whether to add -lutil to
libs_softmmu is "not Darwin and not MinGW and not Solaris and not
Haiku"...

I think the thing we're trying to get from libutil is openpty(), which
is in that library for Linux and the BSDs. It's not in libutil for OSX
(though since OSX has a libutil anyway it's harmless that it gets linked
in). For Windows we avoid code paths that use openpty(), and Solaris
doesn't have openpty() at all so we have to hand-roll our own
implementation in util/qemu-openpty.c.

I think the cleanest fix to this problem is to have a configure test
that checks "does openpty() require -lutil?", and then use the answer to
that both to decide whether to add -lutil to libs_softmmu and to decide
whether to add -lutil to the link line for this test binary.

PS: Thanks for the offer of Solaris hardware access. We may be able to
take you up on that but it would also require some time from somebody on
our end to set things up to use it. The other possible option is to have
one of the free-solaris-equivalents set up as a VM to run tests in in
the same way we do for the BSDs in tests/vm.

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

Title:
  tests/Makefile.include trying to add linking library '-lutil' that
  break the build on Solaris

Status in QEMU:
  New

Bug description:
  Building script 'tests/Makefile.include' contains following code
  ```
  ifeq ($(CONFIG_POSIX),y)
  LIBS += -lutil
  endif
  ```

  library -lutil is not available on Solaris, so the building will failed, like
  ```
  ld: fatal: library -lutil: not found
  make: *** [SOMEWHERE/src/qemu-2.12.0/rules.mak:121: qemu-nbd] Error 1
  ```

  Commenting those code out fixed the error.

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Qemu-devel] [Bug 1777252] Re: tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris
  2018-06-16 15:13 [Qemu-devel] [Bug 1777252] [NEW] tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris WHR
                   ` (3 preceding siblings ...)
  2019-01-15 18:22 ` Peter Maydell
@ 2019-01-16  4:11 ` WHR
  2019-04-24  5:22 ` Thomas Huth
  5 siblings, 0 replies; 8+ messages in thread
From: WHR @ 2019-01-16  4:11 UTC (permalink / raw)
  To: qemu-devel

>>> Does something like this work for you?

Yes, this fix works; but I don't think this is a clean fix, since
libutil may also missing in other OS.

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

Title:
  tests/Makefile.include trying to add linking library '-lutil' that
  break the build on Solaris

Status in QEMU:
  New

Bug description:
  Building script 'tests/Makefile.include' contains following code
  ```
  ifeq ($(CONFIG_POSIX),y)
  LIBS += -lutil
  endif
  ```

  library -lutil is not available on Solaris, so the building will failed, like
  ```
  ld: fatal: library -lutil: not found
  make: *** [SOMEWHERE/src/qemu-2.12.0/rules.mak:121: qemu-nbd] Error 1
  ```

  Commenting those code out fixed the error.

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Qemu-devel] [Bug 1777252] Re: tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris
  2018-06-16 15:13 [Qemu-devel] [Bug 1777252] [NEW] tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris WHR
                   ` (4 preceding siblings ...)
  2019-01-16  4:11 ` WHR
@ 2019-04-24  5:22 ` Thomas Huth
  5 siblings, 0 replies; 8+ messages in thread
From: Thomas Huth @ 2019-04-24  5:22 UTC (permalink / raw)
  To: qemu-devel

Problem has been fixed here:
https://git.qemu.org/?p=qemu.git;a=commitdiff;h=d99e97e6912d90a55e9

** Changed in: qemu
       Status: New => Fix Released

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

Title:
  tests/Makefile.include trying to add linking library '-lutil' that
  break the build on Solaris

Status in QEMU:
  Fix Released

Bug description:
  Building script 'tests/Makefile.include' contains following code
  ```
  ifeq ($(CONFIG_POSIX),y)
  LIBS += -lutil
  endif
  ```

  library -lutil is not available on Solaris, so the building will failed, like
  ```
  ld: fatal: library -lutil: not found
  make: *** [SOMEWHERE/src/qemu-2.12.0/rules.mak:121: qemu-nbd] Error 1
  ```

  Commenting those code out fixed the error.

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2019-04-24  5:30 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-16 15:13 [Qemu-devel] [Bug 1777252] [NEW] tests/Makefile.include trying to add linking library '-lutil' that break the build on Solaris WHR
2018-06-16 19:29 ` [Qemu-devel] [Bug 1777252] " Thomas Huth
2018-06-18  5:55   ` Markus Armbruster
2019-01-15  8:56 ` Thomas Huth
2019-01-15 17:39 ` Michele
2019-01-15 18:22 ` Peter Maydell
2019-01-16  4:11 ` WHR
2019-04-24  5:22 ` Thomas Huth

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.