[PATCH 1/2] [packages] collectd: Update luci-statistics / collectd to 5.4.1 - packages
[PATCH 2/2] [luci] collectd: Update luci-statistics / collectd to 5.4.1 - Luci

Collectd, the backbone of Luci statistics module, has been recently updated already to 5.4.1, while Openwrt is still with 4.10.8.

5.0-5.4 branches have been released in 2011-2013 and the last Collectd release in January 2014 only contained 5.4.1. 4.10.x is being "phased out" and new commits have been made only rarely. No 4.10.x release for ages.

My ar71xx/wndr3700 build has been using 5.4.1 since March (and used 5.3.1 long before that).

I have compile-tested (with ar71xx) all collectd modules, and I have verified the real-life functionality of the following plugins: Conntrack, CPU/Processor, DNS, Interfaces, Memory, Ping, Processes, TCP Connections, System Load, Uptime, Wireless.


The version bump requires changes both to "collectd" in the packages feed and to "luci-statistics" in the luci feed, as some of the data items have changed name or file structure.

I am including two separate patches: one for collectd in packages and one for luci-statistics in Luci trunk.


Collectd:
Explanation of main changes in the patches for collectd in packages feed:

Patches to be removed:

- 120-fix_kernel_2.6.37.patch is unnecessary. The file to be patched has been removed upstream.

- 130-timeleft-max.patch is included upstream. Patch removed.
http://git.verplant.org/?p=collectd.git;a=commitdiff;h=d5b09f2bb673ac9396da52b85bcbd16c98b3d633

Changed patches:

- 003-remove-werror.patch: Quilt choked on some -Werror related changes, so I removed them.

- 900-add-iwinfo-plugin.patch: Changed to patch configure.ac instead of configure.in

New patches:

- 140-fix-fqdnlookup.patch : FQDNLookup option has been turned on by default since 5.1. If that option is enabled, it causes collectd to refuse to start in a basic Openwrt box as typically a fully qualified domain name does not exist for the router. (That error related can only be properly seen when console is used to start collectd. Otherwise Luci statistics collectd simply dies immediately after starting it.) That option should be false by default. I included a new patch 140-fix-fqdnlookup.patch to turn the option off by default.
http://git.verplant.org/?p=collectd.git;a=commit;h=e58f85bb68b4c45130bb6fdf40b51db95d2b71e5

- 150-fix-interface-af-link : a FreeBSD10-specific change broke build for me, so I reverted the change.
http://git.verplant.org/?p=collectd.git;a=commitdiff;h=645dadb3fcc466e8880fda4eb23b21ad433631fc

- 905-disable-sigrok-glib: libsigrok checks for a glib-related macro that breaks configure. Bypass the invalid macro. I have also added a disabling option in Makefile.


Luci:
Data presentation in luci-statistics requires three changes in the Luci sources:

-conntrack result data has changed name from "entropy" to "value".
-ping result data has changed name from "ping" to "value".
http://git.verplant.org/?p=collectd.git;a=commitdiff;h=77a6905147798210ec17173bd5e4410adcc3a112

-interface plugin has changed rrd data file structure so that the plugin instances (=interfaces) have separate data files. Luci needs to draw the graph for each plugin instance, not for each data instance. http://git.verplant.org/?p=collectd.git;a=commitdiff;h=cc3640ba512862cd5745446f1f1a997dd4344454



My previous patches for upgrading to earlier 5.x series can be marked as superseeded in Patchwork.
http://patchwork.openwrt.org/patch/2457/
http://patchwork.openwrt.org/patch/3300/
http://patchwork.openwrt.org/patch/4204/
http://patchwork.openwrt.org/patch/4205/


Signed-off-by: Hannu Nyman <hannu.ny...@iki.fi>

Index: packages/utils/collectd/Makefile
===================================================================
--- packages/utils/collectd/Makefile    (revision 40659)
+++ packages/utils/collectd/Makefile    (working copy)
@@ -8,12 +8,12 @@
 include $(TOPDIR)/rules.mk
 
 PKG_NAME:=collectd
-PKG_VERSION:=4.10.8
-PKG_RELEASE:=3
+PKG_VERSION:=5.4.1
+PKG_RELEASE:=1
 
 PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.bz2
 PKG_SOURCE_URL:=http://collectd.org/files/
-PKG_MD5SUM:=7a60495208b9383225493c6dd6abb2f0
+PKG_MD5SUM:=6f56c71c96573a7f4f7fb3bfab185974
 
 PKG_FIXUP:=autoreconf
 PKG_REMOVE_FILES:=aclocal.m4 libltdl/aclocal.m4
@@ -22,11 +22,16 @@
 PKG_BUILD_PARALLEL:=1
 
 COLLECTD_PLUGINS_DISABLED:= \
+       amqp \
        apple_sensors \
+       aquaero \
        battery \
+       cgroups \
        cpufreq \
        curl_json \
+       curl_xml \
        entropy \
+       ethstat \
        genericjmx \
        gmond \
        hddtemp \
@@ -34,28 +39,41 @@
        ipvs \
        java \
        libvirt \
+       lpar \
        mbmon \
+       md \
        memcachec \
        memcached \
+       mic \
        monitorus \
        multimeter \
        netapp \
        nfs \
        notify_desktop \
        notify_email \
+       numa \
        openvz \
        oracle \
        perl \
+       pf \
        pinba \
        python \
+       redis \
        routeros \
        rrdcached \
        serial \
+       sigrok \
+       statsd \
        swap \
        tape \
        tokyotyrant \
        uuid \
+       varnish \
        vserver \
+       write_graphite \
+       write_mongodb \
+       write_redis \
+       write_riemann \
        xmms \
        zfs_arc \
 
@@ -104,11 +122,11 @@
        sensors \
        snmp \
        syslog \
+       table \
        tail \
-       table \
-       ted \
        tcpconns \
        teamspeak2 \
+       ted \
        thermal \
        unixsock \
        uptime \
@@ -151,6 +169,7 @@
        --enable-daemon \
        --enable-getifaddrs \
        --with-nan-emulation \
+       --with-libsigrok=no \
        --without-libgcrypt
 
 CONFIGURE_VARS+= \
Index: packages/utils/collectd/patches/003-remove-werror.patch
===================================================================
--- packages/utils/collectd/patches/003-remove-werror.patch     (revision 40659)
+++ packages/utils/collectd/patches/003-remove-werror.patch     (working copy)
@@ -1,6 +1,6 @@
 --- a/src/Makefile.am
 +++ b/src/Makefile.am
-@@ -7,7 +7,7 @@ SUBDIRS += liboconfig
+@@ -4,7 +4,7 @@ SUBDIRS += liboconfig
  endif
  
  if COMPILER_IS_GCC
@@ -9,17 +9,6 @@
  endif
  
  AM_CPPFLAGS = -DPREFIX='"${prefix}"'
---- a/src/Makefile.in
-+++ b/src/Makefile.in
-@@ -1833,7 +1833,7 @@ top_build_prefix = @top_build_prefix@
- top_builddir = @top_builddir@
- top_srcdir = @top_srcdir@
- SUBDIRS = libcollectdclient $(am__append_1) $(am__append_2)
--@COMPILER_IS_GCC_TRUE@AM_CFLAGS = -Wall -Werror
-+@COMPILER_IS_GCC_TRUE@AM_CFLAGS = -Wall
- AM_CPPFLAGS = -DPREFIX='"${prefix}"' \
-       -DCONFIGFILE='"${sysconfdir}/${PACKAGE_NAME}.conf"' \
-       -DLOCALSTATEDIR='"${localstatedir}"' \
 --- a/src/libcollectdclient/Makefile.am
 +++ b/src/libcollectdclient/Makefile.am
 @@ -1,7 +1,7 @@
@@ -30,37 +19,4 @@
 +AM_CFLAGS = -Wall
  endif
  
- pkginclude_HEADERS = client.h lcc_features.h
---- a/src/libcollectdclient/Makefile.in
-+++ b/src/libcollectdclient/Makefile.in
-@@ -327,7 +327,7 @@ top_build_prefix = @top_build_prefix@
- top_builddir = @top_builddir@
- top_srcdir = @top_srcdir@
- AUTOMAKE_OPTIONS = foreign no-dependencies
--@COMPILER_IS_GCC_TRUE@AM_CFLAGS = -Wall -Werror
-+@COMPILER_IS_GCC_TRUE@AM_CFLAGS = -Wall
- pkginclude_HEADERS = client.h lcc_features.h
- lib_LTLIBRARIES = libcollectdclient.la
- nodist_pkgconfig_DATA = libcollectdclient.pc
---- a/src/owniptc/Makefile.am
-+++ b/src/owniptc/Makefile.am
-@@ -3,7 +3,7 @@ AUTOMAKE_OPTIONS = foreign no-dependenci
- EXTRA_DIST = libiptc.c README.collectd
- 
- if COMPILER_IS_GCC
--AM_CFLAGS = -Wall -Werror
-+AM_CFLAGS = -Wall
- endif
- 
- noinst_LTLIBRARIES = libiptc.la
---- a/src/owniptc/Makefile.in
-+++ b/src/owniptc/Makefile.in
-@@ -296,7 +296,7 @@ top_builddir = @top_builddir@
- top_srcdir = @top_srcdir@
- AUTOMAKE_OPTIONS = foreign no-dependencies
- EXTRA_DIST = libiptc.c README.collectd
--@COMPILER_IS_GCC_TRUE@AM_CFLAGS = -Wall -Werror
-+@COMPILER_IS_GCC_TRUE@AM_CFLAGS = -Wall
- noinst_LTLIBRARIES = libiptc.la
- libiptc_la_SOURCES = libip4tc.c libip6tc.c \
-               ipt_kernel_headers.h libip6tc.h libiptc.h linux_list.h \
+ pkginclude_HEADERS = collectd/client.h collectd/network.h 
collectd/network_buffer.h collectd/lcc_features.h
Index: packages/utils/collectd/patches/100-rrdtool-add-rrasingle-option.patch
===================================================================
--- packages/utils/collectd/patches/100-rrdtool-add-rrasingle-option.patch      
(revision 40659)
+++ packages/utils/collectd/patches/100-rrdtool-add-rrasingle-option.patch      
(working copy)
@@ -1,6 +1,6 @@
 --- a/src/rrdtool.c
 +++ b/src/rrdtool.c
-@@ -80,6 +80,7 @@ static const char *config_keys[] =
+@@ -82,6 +82,7 @@ static const char *config_keys[] =
        "HeartBeat",
        "RRARows",
        "RRATimespan",
@@ -8,17 +8,17 @@
        "XFF",
        "WritesPerSecond",
        "RandomTimeout"
-@@ -101,6 +102,8 @@ static rrdcreate_config_t rrdcreate_conf
+@@ -103,6 +104,8 @@ static rrdcreate_config_t rrdcreate_conf
        /* timespans = */ NULL,
        /* timespans_num = */ 0,
  
 +      /* rrasingle = */ 0,
 +
        /* consolidation_functions = */ NULL,
-       /* consolidation_functions_num = */ 0
- };
-@@ -1027,6 +1030,14 @@ static int rrd_config (const char *key,
+       /* consolidation_functions_num = */ 0,
  
+@@ -1093,6 +1096,14 @@ static int rrd_config (const char *key,
+ 
                free (value_copy);
        }
 +      else if (strcasecmp ("RRASingle", key) == 0)
@@ -34,7 +34,7 @@
                double tmp = atof (value);
 --- a/src/utils_rrdcreate.c
 +++ b/src/utils_rrdcreate.c
-@@ -122,6 +122,9 @@ static int rra_get (char ***ret, const v
+@@ -211,6 +211,9 @@ static int rra_get (char ***ret, const v
      rts_num = rra_timespans_num;
    }
  
@@ -54,4 +54,4 @@
 +
    char **consolidation_functions;
    size_t consolidation_functions_num;
- };
+ 
Index: packages/utils/collectd/patches/120-fix_kernel_2.6.37.patch
===================================================================
--- packages/utils/collectd/patches/120-fix_kernel_2.6.37.patch (revision 40659)
+++ packages/utils/collectd/patches/120-fix_kernel_2.6.37.patch (working copy)
@@ -1,78 +0,0 @@
---- a/src/owniptc/libiptc.c
-+++ b/src/owniptc/libiptc.c
-@@ -81,11 +81,11 @@ static const char *hooknames[] = {
- };
- 
- /* Convenience structures */
--struct ipt_error_target
-+ struct ipt_error_target2
- {
-       STRUCT_ENTRY_TARGET t;
-       char error[TABLE_MAXNAMELEN];
--};
-+}; 
- 
- struct chain_head;
- struct rule_head;
-@@ -1007,10 +1007,10 @@ static int parse_table(TC_HANDLE_T h)
- /* Convenience structures */
- struct iptcb_chain_start{
-       STRUCT_ENTRY e;
--      struct ipt_error_target name;
-+      struct ipt_error_target2 name;
- };
- #define IPTCB_CHAIN_START_SIZE        (sizeof(STRUCT_ENTRY) +                 
\
--                               ALIGN(sizeof(struct ipt_error_target)))
-+                               ALIGN(sizeof(struct ipt_error_target2)))
- 
- struct iptcb_chain_foot {
-       STRUCT_ENTRY e;
-@@ -1021,10 +1021,10 @@ struct iptcb_chain_foot {
- 
- struct iptcb_chain_error {
-       STRUCT_ENTRY entry;
--      struct ipt_error_target target;
-+      struct ipt_error_target2 target;
- };
- #define IPTCB_CHAIN_ERROR_SIZE        (sizeof(STRUCT_ENTRY) +                 
\
--                               ALIGN(sizeof(struct ipt_error_target)))
-+                               ALIGN(sizeof(struct ipt_error_target2)))
- 
- 
- 
-@@ -1069,7 +1069,7 @@ static int iptcc_compile_chain(TC_HANDLE
-               head->e.next_offset = IPTCB_CHAIN_START_SIZE;
-               strcpy(head->name.t.u.user.name, ERROR_TARGET);
-               head->name.t.u.target_size = 
--                              ALIGN(sizeof(struct ipt_error_target));
-+                              ALIGN(sizeof(struct ipt_error_target2));
-               strcpy(head->name.error, c->name);
-       } else {
-               repl->hook_entry[c->hooknum-1] = c->head_offset;        
-@@ -1113,7 +1113,7 @@ static int iptcc_compile_chain_offsets(T
-       if (!iptcc_is_builtin(c))  {
-               /* Chain has header */
-               *offset += sizeof(STRUCT_ENTRY) 
--                           + ALIGN(sizeof(struct ipt_error_target));
-+                           + ALIGN(sizeof(struct ipt_error_target2));
-               (*num)++;
-       }
- 
-@@ -1153,7 +1153,7 @@ static int iptcc_compile_table_prep(TC_H
-       /* Append one error rule at end of chain */
-       num++;
-       offset += sizeof(STRUCT_ENTRY)
--                + ALIGN(sizeof(struct ipt_error_target));
-+                + ALIGN(sizeof(struct ipt_error_target2));
- 
-       /* ruleset size is now in offset */
-       *size = offset;
-@@ -1177,7 +1177,7 @@ static int iptcc_compile_table(TC_HANDLE
-       error->entry.target_offset = sizeof(STRUCT_ENTRY);
-       error->entry.next_offset = IPTCB_CHAIN_ERROR_SIZE;
-       error->target.t.u.user.target_size = 
--              ALIGN(sizeof(struct ipt_error_target));
-+              ALIGN(sizeof(struct ipt_error_target2));
-       strcpy((char *)&error->target.t.u.user.name, ERROR_TARGET);
-       strcpy((char *)&error->target.error, "ERROR");
- 
Index: packages/utils/collectd/patches/130-timeleft-max.patch
===================================================================
--- packages/utils/collectd/patches/130-timeleft-max.patch      (revision 40659)
+++ packages/utils/collectd/patches/130-timeleft-max.patch      (working copy)
@@ -1,14 +0,0 @@
-Derived from uptream commit d5b09f2b: http://octo.cx/d5b09f2
-
---- a/src/types.db
-+++ b/src/types.db
-@@ -152,7 +152,7 @@ tcp_connections            value:GAUGE:0:429496729
- temperature           value:GAUGE:-273.15:U
- threads                       value:GAUGE:0:U
- time_dispersion               seconds:GAUGE:-1000000:1000000
--timeleft              timeleft:GAUGE:0:3600
-+timeleft              timeleft:GAUGE:0:U
- time_offset           seconds:GAUGE:-1000000:1000000
- total_bytes           value:DERIVE:0:U
- total_requests                value:DERIVE:0:U
-
Index: packages/utils/collectd/patches/140-fix-fqdnlookup.patch
===================================================================
--- packages/utils/collectd/patches/140-fix-fqdnlookup.patch    (revision 0)
+++ packages/utils/collectd/patches/140-fix-fqdnlookup.patch    (working copy)
@@ -0,0 +1,11 @@
+--- a/src/configfile.c
++++ b/src/configfile.c
+@@ -105,7 +105,7 @@ static cf_global_option_t cf_global_opti
+       {"BaseDir",     NULL, PKGLOCALSTATEDIR},
+       {"PIDFile",     NULL, PIDFILE},
+       {"Hostname",    NULL, NULL},
+-      {"FQDNLookup",  NULL, "true"},
++      {"FQDNLookup",  NULL, "false"},
+       {"Interval",    NULL, NULL},
+       {"ReadThreads", NULL, "5"},
+       {"WriteThreads", NULL, "5"},
Index: packages/utils/collectd/patches/150-fix-interface-af-link
===================================================================
--- packages/utils/collectd/patches/150-fix-interface-af-link   (revision 0)
+++ packages/utils/collectd/patches/150-fix-interface-af-link   (working copy)
@@ -0,0 +1,21 @@
+--- a/src/interface.c
++++ b/src/interface.c
+@@ -244,8 +244,8 @@ struct net_device_stats {
+ 
+       for (if_ptr = if_list; if_ptr != NULL; if_ptr = if_ptr->ifa_next)
+       {
+-              if (if_ptr->ifa_addr != NULL && if_ptr->ifa_addr->sa_family == 
AF_LINK) {
+-                      if_data = (struct IFA_DATA *) if_ptr->ifa_data;
++              if ((if_data = (struct IFA_DATA *) if_ptr->ifa_data) == NULL)
++                      continue;
+ 
+                       if_submit (if_ptr->ifa_name, "if_octets",
+                               if_data->IFA_RX_BYTES,
+@@ -256,7 +256,6 @@ struct net_device_stats {
+                       if_submit (if_ptr->ifa_name, "if_errors",
+                               if_data->IFA_RX_ERROR,
+                               if_data->IFA_TX_ERROR);
+-              }
+       }
+ 
+       freeifaddrs (if_list);
Index: packages/utils/collectd/patches/200-fix-git-describe-error.patch
===================================================================
--- packages/utils/collectd/patches/200-fix-git-describe-error.patch    
(revision 40659)
+++ packages/utils/collectd/patches/200-fix-git-describe-error.patch    
(working copy)
@@ -2,7 +2,7 @@
 +++ b/version-gen.sh
 @@ -2,7 +2,7 @@
  
- DEFAULT_VERSION="4.10.8.git"
+ DEFAULT_VERSION="5.4.1.git"
  
 -VERSION="`git describe 2> /dev/null | sed -e 's/^collectd-//'`"
 +#VERSION="`git describe 2> /dev/null | sed -e 's/^collectd-//'`"
Index: packages/utils/collectd/patches/900-add-iwinfo-plugin.patch
===================================================================
--- packages/utils/collectd/patches/900-add-iwinfo-plugin.patch (revision 40659)
+++ packages/utils/collectd/patches/900-add-iwinfo-plugin.patch (working copy)
@@ -1,6 +1,6 @@
---- a/configure.in
-+++ b/configure.in
-@@ -490,6 +490,9 @@ AC_CHECK_HEADERS(netinet/if_ether.h, [],
+--- a/configure.ac
++++ b/configure.ac
+@@ -568,6 +568,9 @@ AC_CHECK_HEADERS(net/pfvar.h,
  have_termios_h="no"
  AC_CHECK_HEADERS(termios.h, [have_termios_h="yes"])
  
@@ -10,7 +10,7 @@
  #
  # Checks for typedefs, structures, and compiler characteristics.
  #
-@@ -4081,6 +4084,7 @@ plugin_interface="no"
+@@ -4819,6 +4822,7 @@ plugin_interface="no"
  plugin_ipmi="no"
  plugin_ipvs="no"
  plugin_irq="no"
@@ -18,7 +18,7 @@
  plugin_libvirt="no"
  plugin_load="no"
  plugin_memory="no"
-@@ -4388,6 +4392,7 @@ AC_PLUGIN([ipmi],        [$plugin_ipmi],
+@@ -5157,6 +5161,7 @@ AC_PLUGIN([ipmi],        [$plugin_ipmi],
  AC_PLUGIN([iptables],    [$with_libiptc],      [IPTables rule counters])
  AC_PLUGIN([ipvs],        [$plugin_ipvs],       [IPVS connection statistics])
  AC_PLUGIN([irq],         [$plugin_irq],        [IRQ statistics])
@@ -26,7 +26,7 @@
  AC_PLUGIN([java],        [$with_java],         [Embed the Java Virtual 
Machine])
  AC_PLUGIN([libvirt],     [$plugin_libvirt],    [Virtual machine statistics])
  AC_PLUGIN([load],        [$plugin_load],       [System load])
-@@ -4666,6 +4671,7 @@ Configuration:
+@@ -5458,6 +5463,7 @@ Configuration:
      protobuf-c  . . . . . $have_protoc_c
      oracle  . . . . . . . $with_oracle
      python  . . . . . . . $with_python
@@ -34,7 +34,7 @@
  
    Features:
      daemon mode . . . . . $enable_daemon
-@@ -4705,6 +4711,7 @@ Configuration:
+@@ -5502,6 +5508,7 @@ Configuration:
      iptables  . . . . . . $enable_iptables
      ipvs  . . . . . . . . $enable_ipvs
      irq . . . . . . . . . $enable_irq
@@ -44,7 +44,7 @@
      load  . . . . . . . . $enable_load
 --- a/src/collectd.conf.in
 +++ b/src/collectd.conf.in
-@@ -82,6 +82,7 @@ FQDNLookup   true
+@@ -109,6 +109,7 @@
  #@BUILD_PLUGIN_IPMI_TRUE@LoadPlugin ipmi
  #@BUILD_PLUGIN_IPVS_TRUE@LoadPlugin ipvs
  #@BUILD_PLUGIN_IRQ_TRUE@LoadPlugin irq
@@ -52,7 +52,7 @@
  #@BUILD_PLUGIN_JAVA_TRUE@LoadPlugin java
  #@BUILD_PLUGIN_LIBVIRT_TRUE@LoadPlugin libvirt
  @BUILD_PLUGIN_LOAD_TRUE@@BUILD_PLUGIN_LOAD_TRUE@LoadPlugin load
-@@ -376,6 +377,12 @@ FQDNLookup   true
+@@ -467,6 +468,12 @@
  #     IgnoreSelected true
  #</Plugin>
  
@@ -67,7 +67,7 @@
  #     JVMArg "-Djava.class.path=@prefix@/share/collectd/java/collectd-api.jar"
 --- a/src/collectd.conf.pod
 +++ b/src/collectd.conf.pod
-@@ -1478,6 +1478,27 @@ and all other interrupts are collected.
+@@ -2056,6 +2056,27 @@ and all other interrupts are collected.
  
  =back
  
@@ -250,7 +250,7 @@
 +}
 --- a/src/Makefile.am
 +++ b/src/Makefile.am
-@@ -453,6 +453,15 @@ collectd_LDADD += "-dlopen" irq.la
+@@ -530,6 +530,15 @@ collectd_LDADD += "-dlopen" irq.la
  collectd_DEPENDENCIES += irq.la
  endif
  
@@ -268,8 +268,12 @@
  java_la_SOURCES = java.c
 --- a/src/types.db
 +++ b/src/types.db
-@@ -171,3 +171,4 @@ voltage                    value:GAUGE:U:U
+@@ -195,7 +195,7 @@ voltage                    value:GAUGE:U:U
  vs_memory             value:GAUGE:0:9223372036854775807
  vs_processes          value:GAUGE:0:65535
  vs_threads            value:GAUGE:0:65535
+-
 +stations              value:GAUGE:0:256
+ #
+ # Legacy types
+ # (required for the v5 upgrade target)
Index: packages/utils/collectd/patches/905-disable-sigrok-glib
===================================================================
--- packages/utils/collectd/patches/905-disable-sigrok-glib     (revision 0)
+++ packages/utils/collectd/patches/905-disable-sigrok-glib     (working copy)
@@ -0,0 +1,22 @@
+--- a/configure.ac
++++ b/configure.ac
+@@ -3833,13 +3833,13 @@ AC_ARG_WITH(libsigrok, [AS_HELP_STRING([
+ # libsigrok has a glib dependency
+ if test "x$with_libsigrok" = "xyes"
+ then
+-      if test -z "m4_ifdef([AM_PATH_GLIB_2_0], [yes], [])"
+-      then
++#     if test -z "m4_ifdef([AM_PATH_GLIB_2_0], [yes], [])"
++#     then
+               with_libsigrok="no (glib not available)"
+-      else
+-              AM_PATH_GLIB_2_0([2.28.0],
+-                      [with_libsigrok_cflags="$with_libsigrok_cflags 
$GLIB_CFLAGS"; with_libsigrok_ldflags="$with_libsigrok_ldflags $GLIB_LIBS"])
+-      fi
++#     else
++#             AM_PATH_GLIB_2_0([2.28.0],
++#                     [with_libsigrok_cflags="$with_libsigrok_cflags 
$GLIB_CFLAGS"; with_libsigrok_ldflags="$with_libsigrok_ldflags $GLIB_LIBS"])
++#     fi
+ fi
+ 
+ # libsigrok headers
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to