[PATCH] Makefile: warn user when libxml and/or libgtk2 are not available

[PATCH] Makefile: warn user when libxml and/or libgtk2 are not available

[Florian Fainelli]

From: Florian Fainelli <f.fainelli@gmail.com>

Since sparse's c2xml is not always packaged by modern distributions, it can be
built as a host package in build systems, finding out why c2xml was not
compiled turned out to be easier if a warning was emitted, add two for libxml
and libgtk2 presence/absence.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
diff --git a/Makefile b/Makefile
index 74905df..e9ae26f 100644
--- a/Makefile
+++ b/Makefile
@@ -45,6 +45,8 @@ ifeq ($(HAVE_LIBXML),yes)
 PROGRAMS+=c2xml
 INST_PROGRAMS+=c2xml
 c2xml_EXTRA_OBJS = `pkg-config --libs libxml-2.0`
+else
+$(info Your system does not have libxml, disabling c2xml)
 endif

 ifeq ($(HAVE_GTK2),yes)
@@ -55,6 +57,8 @@ INST_PROGRAMS += test-inspect
 test-inspect_EXTRA_DEPS := ast-model.o ast-view.o ast-inspect.o
 test-inspect.o $(test-inspect_EXTRA_DEPS): CFLAGS += $(GTK2_CFLAGS)
 test-inspect_EXTRA_OBJS := $(GTK2_LIBS)
+else
+$(info Your system does not have libgtk2, disabling test-inspect)
 endif

 LIB_H=    token.h parse.h lib.h symbol.h scope.h expression.h target.h \
--

Re: [PATCH] Makefile: warn user when libxml and/or libgtk2 are not available

[Christopher Li]

On Thu, Mar 3, 2011 at 12:57 AM, Florian Fainelli <f.fainelli@gmail.com> wrote:
> From: Florian Fainelli <f.fainelli@gmail.com>
>
> Since sparse's c2xml is not always packaged by modern distributions, it can be
> built as a host package in build systems, finding out why c2xml was not
> compiled turned out to be easier if a warning was emitted, add two for libxml
> and libgtk2 presence/absence.

The change looks good. How about using $(warning ...) instead of $(info ..)?

Chris

Re: [PATCH] Makefile: warn user when libxml and/or libgtk2 are not available

[Florian Fainelli]

On Thursday 03 March 2011 23:52:25 Christopher Li wrote:
> On Thu, Mar 3, 2011 at 12:57 AM, Florian Fainelli <f.fainelli@gmail.com> wrote:
> > From: Florian Fainelli <f.fainelli@gmail.com>
> > 
> > Since sparse's c2xml is not always packaged by modern distributions, it
> > can be built as a host package in build systems, finding out why c2xml
> > was not compiled turned out to be easier if a warning was emitted, add
> > two for libxml and libgtk2 presence/absence.
> 
> The change looks good. How about using $(warning ...) instead of $(info
> ..)?

Indeed, looks better. Updated patch below:
--
From: Florian Fainelli <f.fainelli@gmail.com>

Since sparse's c2xml is not always packaged my modern distributions, it can be
built as a host package in build systems, finding out why c2xml was not
compiled turned out to be easier if a warning was emitted, add two for libxml
and libgtk2 presence/absence.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
Changes since v1:
- use $(warning ..) instead of $(info ..)

diff --git a/Makefile b/Makefile
index 74905df..701cdc9 100644
--- a/Makefile
+++ b/Makefile
@@ -45,6 +45,8 @@ ifeq ($(HAVE_LIBXML),yes)
 PROGRAMS+=c2xml
 INST_PROGRAMS+=c2xml
 c2xml_EXTRA_OBJS = `pkg-config --libs libxml-2.0`
+else
+$(warning Your system does not have libxml, disabling c2xml)
 endif

 ifeq ($(HAVE_GTK2),yes)
@@ -55,6 +57,8 @@ INST_PROGRAMS += test-inspect
 test-inspect_EXTRA_DEPS := ast-model.o ast-view.o ast-inspect.o
 test-inspect.o $(test-inspect_EXTRA_DEPS): CFLAGS += $(GTK2_CFLAGS)
 test-inspect_EXTRA_OBJS := $(GTK2_LIBS)
+else
+$(warning Your system does not have libgtk2, disabling test-inspect)
 endif

 LIB_H=    token.h parse.h lib.h symbol.h scope.h expression.h target.h \
--
1.7.1

Re: [PATCH] Makefile: warn user when libxml and/or libgtk2 are not available

[Florian Fainelli]

Hello Christopher,

On Friday 04 March 2011 11:03:37 Florian Fainelli wrote:
> On Thursday 03 March 2011 23:52:25 Christopher Li wrote:
> > On Thu, Mar 3, 2011 at 12:57 AM, Florian Fainelli <f.fainelli@gmail.com> wrote:
> > > From: Florian Fainelli <f.fainelli@gmail.com>
> > > 
> > > Since sparse's c2xml is not always packaged by modern distributions, it
> > > can be built as a host package in build systems, finding out why c2xml
> > > was not compiled turned out to be easier if a warning was emitted, add
> > > two for libxml and libgtk2 presence/absence.
> > 
> > The change looks good. How about using $(warning ...) instead of $(info
> > ..)?
> 
> Indeed, looks better. Updated patch below:
> --
> From: Florian Fainelli <f.fainelli@gmail.com>
> 
> Since sparse's c2xml is not always packaged my modern distributions, it can
> be built as a host package in build systems, finding out why c2xml was not
> compiled turned out to be easier if a warning was emitted, add two for
> libxml and libgtk2 presence/absence.
> 
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
> ---
> Changes since v1:
> - use $(warning ..) instead of $(info ..)
> 
> diff --git a/Makefile b/Makefile
> index 74905df..701cdc9 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -45,6 +45,8 @@ ifeq ($(HAVE_LIBXML),yes)
>  PROGRAMS+=c2xml
>  INST_PROGRAMS+=c2xml
>  c2xml_EXTRA_OBJS = `pkg-config --libs libxml-2.0`
> +else
> +$(warning Your system does not have libxml, disabling c2xml)
>  endif
> 
>  ifeq ($(HAVE_GTK2),yes)
> @@ -55,6 +57,8 @@ INST_PROGRAMS += test-inspect
>  test-inspect_EXTRA_DEPS := ast-model.o ast-view.o ast-inspect.o
>  test-inspect.o $(test-inspect_EXTRA_DEPS): CFLAGS += $(GTK2_CFLAGS)
>  test-inspect_EXTRA_OBJS := $(GTK2_LIBS)
> +else
> +$(warning Your system does not have libgtk2, disabling test-inspect)
>  endif
> 
>  LIB_H=    token.h parse.h lib.h symbol.h scope.h expression.h target.h \
> --

Is this version okay with you? Shall I re-submit the patch as a separate email?
-- 
Florian

Re: [PATCH] Makefile: warn user when libxml and/or libgtk2 are not available

[Christopher Li]

On Wed, Mar 9, 2011 at 6:06 AM, Florian Fainelli <f.fainelli@gmail.com> wrote:
>
> Is this version okay with you? Shall I re-submit the patch as a separate email?

This version is fine. I will apply it today.

Thanks for the patch.

Chris