* [lttng-dev] [PATCH babeltrace] Fix: glib >= 2.32 required for g_hash_table_contains() in lib/values.c
@ 2015-09-21 20:48 Michael Jeanson
2015-09-21 21:15 ` Jérémie Galarneau
0 siblings, 1 reply; 4+ messages in thread
From: Michael Jeanson @ 2015-09-21 20:48 UTC (permalink / raw)
---
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index dc23daa..a815274 100644
--- a/configure.ac
+++ b/configure.ac
@@ -54,7 +54,7 @@ if test ! -f "$srcdir/formats/ctf/metadata/ctf-lexer.c"; then
fi
-AM_PATH_GLIB_2_0(2.22.0, ,AC_MSG_ERROR([glib is required in order to compile BabelTrace - download it from ftp://ftp.gtk.org/pub/gtk]) , gmodule)
+AM_PATH_GLIB_2_0(2.32.0, ,AC_MSG_ERROR([glib is required in order to compile BabelTrace - download it from ftp://ftp.gtk.org/pub/gtk]) , gmodule)
# Checks for typedefs, structures, and compiler characteristics.
AC_C_INLINE
--
1.9.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* [lttng-dev] [PATCH babeltrace] Fix: glib >= 2.32 required for g_hash_table_contains() in lib/values.c
2015-09-21 20:48 [lttng-dev] [PATCH babeltrace] Fix: glib >= 2.32 required for g_hash_table_contains() in lib/values.c Michael Jeanson
@ 2015-09-21 21:15 ` Jérémie Galarneau
2015-09-21 21:42 ` Mathieu Desnoyers
2015-09-21 21:43 ` Mathieu Desnoyers
0 siblings, 2 replies; 4+ messages in thread
From: Jérémie Galarneau @ 2015-09-21 21:15 UTC (permalink / raw)
We could change the uses of g_hash_table_contains() to use
g_hash_table_find() (which was present in 2.4).
However, at this point I wouldn't feel too bad bumping the dependency
to 2.32 in BT 2.0 considering glib 2.32 was released in March 2012 and
was, AFAICT, shipped in Ubuntu 12.04... No strong opinion.
Mathieu, thoughts?
J?r?mie
On Mon, Sep 21, 2015 at 4:48 PM, Michael Jeanson <mjeanson at efficios.com> wrote:
> ---
> configure.ac | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configure.ac b/configure.ac
> index dc23daa..a815274 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -54,7 +54,7 @@ if test ! -f "$srcdir/formats/ctf/metadata/ctf-lexer.c"; then
> fi
>
>
> -AM_PATH_GLIB_2_0(2.22.0, ,AC_MSG_ERROR([glib is required in order to compile BabelTrace - download it from ftp://ftp.gtk.org/pub/gtk]) , gmodule)
> +AM_PATH_GLIB_2_0(2.32.0, ,AC_MSG_ERROR([glib is required in order to compile BabelTrace - download it from ftp://ftp.gtk.org/pub/gtk]) , gmodule)
>
> # Checks for typedefs, structures, and compiler characteristics.
> AC_C_INLINE
> --
> 1.9.1
>
--
J?r?mie Galarneau
EfficiOS Inc.
http://www.efficios.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [lttng-dev] [PATCH babeltrace] Fix: glib >= 2.32 required for g_hash_table_contains() in lib/values.c
2015-09-21 21:15 ` Jérémie Galarneau
@ 2015-09-21 21:42 ` Mathieu Desnoyers
2015-09-21 21:43 ` Mathieu Desnoyers
1 sibling, 0 replies; 4+ messages in thread
From: Mathieu Desnoyers @ 2015-09-21 21:42 UTC (permalink / raw)
----- On Sep 21, 2015, at 5:15 PM, Jeremie Galarneau jeremie.galarneau at efficios.com wrote:
> We could change the uses of g_hash_table_contains() to use
> g_hash_table_find() (which was present in 2.4).
>
> However, at this point I wouldn't feel too bad bumping the dependency
> to 2.32 in BT 2.0 considering glib 2.32 was released in March 2012 and
> was, AFAICT, shipped in Ubuntu 12.04... No strong opinion.
>
> Mathieu, thoughts?
SLES 11 sp3 still has glib 2.22
>
> J?r?mie
>
> On Mon, Sep 21, 2015 at 4:48 PM, Michael Jeanson <mjeanson at efficios.com> wrote:
>> ---
>> configure.ac | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/configure.ac b/configure.ac
>> index dc23daa..a815274 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -54,7 +54,7 @@ if test ! -f "$srcdir/formats/ctf/metadata/ctf-lexer.c"; then
>> fi
>>
>>
>> -AM_PATH_GLIB_2_0(2.22.0, ,AC_MSG_ERROR([glib is required in order to compile
>> BabelTrace - download it from ftp://ftp.gtk.org/pub/gtk]) , gmodule)
>> +AM_PATH_GLIB_2_0(2.32.0, ,AC_MSG_ERROR([glib is required in order to compile
>> BabelTrace - download it from ftp://ftp.gtk.org/pub/gtk]) , gmodule)
>>
>> # Checks for typedefs, structures, and compiler characteristics.
>> AC_C_INLINE
>> --
>> 1.9.1
>>
>
>
>
> --
> J?r?mie Galarneau
> EfficiOS Inc.
> http://www.efficios.com
--
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [lttng-dev] [PATCH babeltrace] Fix: glib >= 2.32 required for g_hash_table_contains() in lib/values.c
2015-09-21 21:15 ` Jérémie Galarneau
2015-09-21 21:42 ` Mathieu Desnoyers
@ 2015-09-21 21:43 ` Mathieu Desnoyers
1 sibling, 0 replies; 4+ messages in thread
From: Mathieu Desnoyers @ 2015-09-21 21:43 UTC (permalink / raw)
----- On Sep 21, 2015, at 5:15 PM, Jeremie Galarneau jeremie.galarneau at efficios.com wrote:
> We could change the uses of g_hash_table_contains() to use
> g_hash_table_find() (which was present in 2.4).
>
> However, at this point I wouldn't feel too bad bumping the dependency
> to 2.32 in BT 2.0 considering glib 2.32 was released in March 2012 and
> was, AFAICT, shipped in Ubuntu 12.04... No strong opinion.
>
> Mathieu, thoughts?
SLES 11 SP3 has 2.22.5:
https://www.suse.com/LinuxPackages/packageRouter.jsp?product=desktop&version=11&service_pack=sp3&architecture=i386&package_name=glib2-devel
I'd keep the current old dep if possible.
Thanks,
Mathieu
>
> J?r?mie
>
> On Mon, Sep 21, 2015 at 4:48 PM, Michael Jeanson <mjeanson at efficios.com> wrote:
>> ---
>> configure.ac | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/configure.ac b/configure.ac
>> index dc23daa..a815274 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -54,7 +54,7 @@ if test ! -f "$srcdir/formats/ctf/metadata/ctf-lexer.c"; then
>> fi
>>
>>
>> -AM_PATH_GLIB_2_0(2.22.0, ,AC_MSG_ERROR([glib is required in order to compile
>> BabelTrace - download it from ftp://ftp.gtk.org/pub/gtk]) , gmodule)
>> +AM_PATH_GLIB_2_0(2.32.0, ,AC_MSG_ERROR([glib is required in order to compile
>> BabelTrace - download it from ftp://ftp.gtk.org/pub/gtk]) , gmodule)
>>
>> # Checks for typedefs, structures, and compiler characteristics.
>> AC_C_INLINE
>> --
>> 1.9.1
>>
>
>
>
> --
> J?r?mie Galarneau
> EfficiOS Inc.
> http://www.efficios.com
--
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-09-21 21:43 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-21 20:48 [lttng-dev] [PATCH babeltrace] Fix: glib >= 2.32 required for g_hash_table_contains() in lib/values.c Michael Jeanson
2015-09-21 21:15 ` Jérémie Galarneau
2015-09-21 21:42 ` Mathieu Desnoyers
2015-09-21 21:43 ` Mathieu Desnoyers
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox