Code

modbus: avoid enabling libmodbus's debug flag by default
authorMarc Fournier <marc.fournier@camptocamp.com>
Thu, 25 Feb 2016 10:22:18 +0000 (11:22 +0100)
committerMarc Fournier <marc.fournier@camptocamp.com>
Thu, 25 Feb 2016 10:22:18 +0000 (11:22 +0100)
Having this enabled makes collectd's stdout and stderr very noisy. From
now on, only activate this flag when collectd is built with
--enable-debug.

Thanks to Eric Sandeen for mentioning this problem on IRC.

src/modbus.c

index b5d1e251960534be10793aeb7c44000d16acb225..7349dc564976b2a91aa44b7bb760e8d33a9bf4e0 100644 (file)
@@ -288,7 +288,9 @@ static int mb_init_connection (mb_host_t *host) /* {{{ */
   if (host == NULL)
     return (EINVAL);
 
+#if COLLECT_DEBUG
   modbus_set_debug (&host->connection, 1);
+#endif
 
   /* We'll do the error handling ourselves. */
   modbus_set_error_handling (&host->connection, NOP_ON_ERROR);
@@ -341,7 +343,9 @@ static int mb_init_connection (mb_host_t *host) /* {{{ */
     return (-1);
   }
 
+#if COLLECT_DEBUG
   modbus_set_debug (host->connection, 1);
+#endif
 
   /* We'll do the error handling ourselves. */
   modbus_set_error_recovery (host->connection, 0);