Return-Path: MIME-Version: 1.0 In-Reply-To: References: <1485282579-9172-1-git-send-email-brian.gix@intel.com> <1485282579-9172-3-git-send-email-brian.gix@intel.com> <6ea10c35-12e7-fbcc-8db8-cee61754e066@felipetonello.com> From: Szymon Janc Date: Tue, 24 Jan 2017 20:08:19 +0100 Message-ID: Subject: Re: [PATCH v2 2/2] Require GLIB v2.46 To: "Gix, Brian" Cc: Felipe Ferreri Tonello , "linux-bluetooth@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Brian, On 24 January 2017 at 20:04, Gix, Brian wrote: > Done. > > -----Original Message----- > From: linux-bluetooth-owner@vger.kernel.org [mailto:linux-bluetooth-owner@vger.kernel.org] On Behalf Of Felipe Ferreri Tonello > Sent: Tuesday, January 24, 2017 10:56 AM > To: Gix, Brian ; linux-bluetooth@vger.kernel.org > Subject: Re: [PATCH v2 2/2] Require GLIB v2.46 > > Hi Brian, > > Please, add the reason why this change is necessary in the commit message. > > On 24/01/17 18:29, Brian Gix wrote: >> --- >> configure.ac | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/configure.ac b/configure.ac index 91b4955..6d8bf9c 100644 >> --- a/configure.ac >> +++ b/configure.ac >> @@ -61,8 +61,8 @@ AC_CHECK_LIB(dl, dlopen, dummy=yes, >> >> AC_CHECK_HEADERS(linux/types.h linux/if_alg.h) >> >> -PKG_CHECK_MODULES(GLIB, glib-2.0 >= 2.28, dummy=yes, >> - AC_MSG_ERROR(GLib >= 2.28 is required)) >> +PKG_CHECK_MODULES(GLIB, glib-2.0 >= 2.46, dummy=yes, >> + AC_MSG_ERROR(GLib >= 2.46 is required)) >> AC_SUBST(GLIB_CFLAGS) >> AC_SUBST(GLIB_LIBS) I'd rather change test instead of bumping glib dependency just for this... But I'll let others to comment on this too. -- pozdrawiam Szymon K. Janc