qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] pixman: require 0.18.4 or newer
@ 2012-11-27 12:19 Gerd Hoffmann
  2012-12-01 14:36 ` Blue Swirl
  2012-12-08 17:53 ` Blue Swirl
  0 siblings, 2 replies; 3+ messages in thread
From: Gerd Hoffmann @ 2012-11-27 12:19 UTC (permalink / raw)
  To: qemu-devel; +Cc: agraf, Gerd Hoffmann

When older versions are found the internal pixman version is prefered.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 configure |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/configure b/configure
index 780b19a..1a3458f 100755
--- a/configure
+++ b/configure
@@ -2103,7 +2103,7 @@ fi
 # pixman support probe
 
 if test "$pixman" = ""; then
-  if $pkg_config pixman-1 > /dev/null 2>&1; then
+  if $pkg_config --atleast-version=0.18.4 pixman-1 > /dev/null 2>&1; then
     pixman="system"
   else
     pixman="internal"
@@ -2114,7 +2114,7 @@ if test "$pixman" = "system"; then
   pixman_libs=`$pkg_config --libs pixman-1 2>/dev/null`
 else
   if test ! -d ${source_path}/pixman/pixman; then
-    echo "ERROR: pixman not present. Your options:"
+    echo "ERROR: pixman not present (or older than 0.18.4). Your options:"
     echo "  (1) Prefered: Install the pixman devel package (any recent"
     echo "      distro should have packages as Xorg needs pixman too)."
     echo "  (2) Fetch the pixman submodule, using:"
-- 
1.7.1

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

* Re: [Qemu-devel] [PATCH] pixman: require 0.18.4 or newer
  2012-11-27 12:19 [Qemu-devel] [PATCH] pixman: require 0.18.4 or newer Gerd Hoffmann
@ 2012-12-01 14:36 ` Blue Swirl
  2012-12-08 17:53 ` Blue Swirl
  1 sibling, 0 replies; 3+ messages in thread
From: Blue Swirl @ 2012-12-01 14:36 UTC (permalink / raw)
  To: Gerd Hoffmann; +Cc: qemu-devel, agraf

On Tue, Nov 27, 2012 at 12:19 PM, Gerd Hoffmann <kraxel@redhat.com> wrote:
> When older versions are found the internal pixman version is prefered.

Isn't this 1.3 material?

>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>  configure |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index 780b19a..1a3458f 100755
> --- a/configure
> +++ b/configure
> @@ -2103,7 +2103,7 @@ fi
>  # pixman support probe
>
>  if test "$pixman" = ""; then
> -  if $pkg_config pixman-1 > /dev/null 2>&1; then
> +  if $pkg_config --atleast-version=0.18.4 pixman-1 > /dev/null 2>&1; then
>      pixman="system"
>    else
>      pixman="internal"
> @@ -2114,7 +2114,7 @@ if test "$pixman" = "system"; then
>    pixman_libs=`$pkg_config --libs pixman-1 2>/dev/null`
>  else
>    if test ! -d ${source_path}/pixman/pixman; then
> -    echo "ERROR: pixman not present. Your options:"
> +    echo "ERROR: pixman not present (or older than 0.18.4). Your options:"
>      echo "  (1) Prefered: Install the pixman devel package (any recent"
>      echo "      distro should have packages as Xorg needs pixman too)."
>      echo "  (2) Fetch the pixman submodule, using:"
> --
> 1.7.1
>
>

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

* Re: [Qemu-devel] [PATCH] pixman: require 0.18.4 or newer
  2012-11-27 12:19 [Qemu-devel] [PATCH] pixman: require 0.18.4 or newer Gerd Hoffmann
  2012-12-01 14:36 ` Blue Swirl
@ 2012-12-08 17:53 ` Blue Swirl
  1 sibling, 0 replies; 3+ messages in thread
From: Blue Swirl @ 2012-12-08 17:53 UTC (permalink / raw)
  To: Gerd Hoffmann; +Cc: qemu-devel, agraf

Thanks, applied.

On Tue, Nov 27, 2012 at 12:19 PM, Gerd Hoffmann <kraxel@redhat.com> wrote:
> When older versions are found the internal pixman version is prefered.
>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>  configure |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index 780b19a..1a3458f 100755
> --- a/configure
> +++ b/configure
> @@ -2103,7 +2103,7 @@ fi
>  # pixman support probe
>
>  if test "$pixman" = ""; then
> -  if $pkg_config pixman-1 > /dev/null 2>&1; then
> +  if $pkg_config --atleast-version=0.18.4 pixman-1 > /dev/null 2>&1; then
>      pixman="system"
>    else
>      pixman="internal"
> @@ -2114,7 +2114,7 @@ if test "$pixman" = "system"; then
>    pixman_libs=`$pkg_config --libs pixman-1 2>/dev/null`
>  else
>    if test ! -d ${source_path}/pixman/pixman; then
> -    echo "ERROR: pixman not present. Your options:"
> +    echo "ERROR: pixman not present (or older than 0.18.4). Your options:"
>      echo "  (1) Prefered: Install the pixman devel package (any recent"
>      echo "      distro should have packages as Xorg needs pixman too)."
>      echo "  (2) Fetch the pixman submodule, using:"
> --
> 1.7.1
>
>

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

end of thread, other threads:[~2012-12-08 17:53 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-27 12:19 [Qemu-devel] [PATCH] pixman: require 0.18.4 or newer Gerd Hoffmann
2012-12-01 14:36 ` Blue Swirl
2012-12-08 17:53 ` Blue Swirl

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).