Add missing files from win32 to Makefile.am

- Missing files were added in the Makefile.am to EXTRA_DIST
- The extra files will be included in the tarball
- See [rrd-users] message:
  https://lists.oetiker.ch/pipermail/rrd-users/2016-August/020443.html

Merge pull request #818 from sourcejedi/valgrind_tests

Clean up 2 compiler warnings

Merge pull request #819 from sourcejedi/rrd_list

2 fixes for rrdtool list

src/rrd_daemon.c: never compare uninitialized bytes

I don't think this caused the valgrind error in CI either.  It looks wrong
though.

src/rrd_list.c: fix errno

rrd_daemon expected zero errno iff there was not an error
(and we forgot to provide it).

  list = rrd_list_r(recursive, fullpath);

  if (list == NULL) {
    /* Empty directory listing */
    if (errno == 0) {
      goto out_send_response;
    }

src/rrd_daemon.c: "comparison between pointer and zero character constant"

rrd_daemon.c: In function ‘handle_request_list’:
rrd_daemon.c:2587:22: warning: comparison between pointer and zero character constant [-Wpointer-compare]
   } while (start_ptr != '\0');
                      ^~
rrd_daemon.c:2587:12: note: did you mean to dereference the pointer?
   } while (start_ptr != '\0');

I thought this might be related to what travis picked up with valgrind
here, but I couldn't reproduce it.  Also I don't think this commit fixes a
bug; all it's doing is cleaning up the compiler warning.

Starting rrdcached...
OK: empty directory ./list1_dir returns nothing
==23957== LEAK SUMMARY:
==23957==    definitely lost: 0 bytes in 0 blocks
==23957==    indirectly lost: 0 bytes in 0 blocks
==23957==      possibly lost: 0 bytes in 0 blocks
==23957==    still reachable: 14,712 bytes in 6 blocks
==23957==         suppressed: 0 bytes in 0 blocks
==23957==
==23957== For counts of detected and suppressed errors, rerun with: -v
==23957== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 2 from 2)
FAILED: (rc=valgrind error) single file /list1.rrd
FAIL: list1

src/rrd_daemon.c: remove unused local variable rc

rrd_daemon.c: In function ‘handle_request_pending’:
rrd_daemon.c:1479:15: warning: unused variable ‘rc’ [-Wunused-variable]
   int status, rc;
               ^~

Fix valgrind usage in tests (#804)

* Fix leak of socket address in rrd_daemon

==11140== 59 bytes in 1 blocks are definitely lost in loss record 132
==11140==    at 0x4C2DB9D: malloc (vg_replace_malloc.c:299)
==11140==    by 0x6AD8CC9: strdup (in /usr/lib64/libc-2.24.so)
==11140==    by 0x40A8AF: read_options (rrd_daemon.c:4493)
==11140==    by 0x403114: main (rrd_daemon.c:4882)

* create a file in case the caller has difficulty testing the rc

e.g. because they pipe the output of rrdtool though something else.

Additional valgrind suppressions were needed in 20 out of 21 tests.

* tests: don't try to remove $VALGRIND_ERR_FILE when it doesn't exist

Fix non-fatal warning.

rm: cannot remove `/home/travis/build/oetiker/rrdtool-1.x/tests/
dump-restore-valgrind-err.tmp': No such file or directory

show 1.7

update infrastructure files

Build fixes tcl commands (#817)

* Link to gthread-2.0 when g_thread_init needs to be invoked

See https://developer.gnome.org/glib/stable/glib-Deprecated-Thread-APIs.html#g-thread-init:

> To use g_thread_init() in your program, you have to link
> with the libraries that the command `pkg-config --libs
> gthread-2.0` outputs. This is not the case for all the other
> thread-related functions of GLib. Those can be used without having to
> link with the thread libraries.

* Support building in a separate directory

* bindings/tcl/Makefile.am: use a relative path to the shlib in pkgIndex.tcl

* Add rrd_info support in Tcl bindings

* Add rrd_first support in Tcl bindings

* Fix memory leak in Rrd_Lastupdate

Merge pull request #815 from flumm/perl-documentation-fix

correct the note in the perl binding about the empty filename

correct the note in the perl binding about the empty filename

passing an empty filename to graphv does not work anymore, but the
documentation of the perl-shared binding still said so

correctly note that you have to pass '-'

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

fix spelling

Return success for rrdcached pending on empty (#809)

If we have no pending entries for a file return success with no entries instead of returning an error.

Merge pull request #813 from kinote/master

src/rrd_client.c: fix memory leak in rrd_client_info

src/rrd_client.c: fix memeory leak in rrd_client_info

A string is being duplicated with strdup and the copy was not being
freed.

Added missing rrdcached CREATE options to docs (#810)

Added -s <sourcefile> and -t <templatefile> to the docs for rrdcached CREATE.

Recently discovered fixes (#803)

* fix incorrect use of snprintf in rrd_cgi

DS_NAM_SIZE is used to size a buffer, but there is no relation to ds_nam.
Fortunately this lead to the buffer being over-sized.

Replace with code which does not rely on calculating the size of a
formatted string.  (If the error message is around the maximum possible
size supported by librrd, we might drop a closing ']'.  That won't cause
us a problem).

We don't have a shim for asprintf().  rrd_asprintf() and friends are for
writing e.g. XML etc without locale-specific formatting.

* refactor - header_len in rrd_open() is only valid for new files

this would be confusing and undesirable if it was used for anything other
than calculating newfile_size.  Fortunately it is not.  The variable can
easily be scoped to avoid confusion.

* rrd_open() should position "to the first cdp in the first rra"

but RRD_READVALUES was buggy.  It left the fd positioned at the end of the
file, but with rrd_file->pos still set to just after the header.
(rados, which doesn't use the fd, would be unafffected by this bug, leading
to differing behaviour).

The comment for rrd_open implies we should rrd_seek() back to just after
the header.  So let's resolve the inconsistency that way.

The old code with d_offset was completely pointless.  It was used to
restore rrd_file->pos, but rrd_file->pos was not changed, even by the
__rrd_read macro magic.

* rrd_open(rrd, ... RRD_CREAT) requires non-NULL rrd->stat_head

We're only going to do something horrible later involving `(size_t) -1`.
Just dereference NULL and trigger the undefined behaviour now, rather
than confuse readers about how RRD_CREAT works when rrd->stat_head is not
provided.

(But if anyone wants to expand argument checking and make it not
conditional on DEBUG, I have no objection to that either :).

* remove unecessary strdup/free in rrd_flushcached()

* rrd_get_error() does not return NULL when there is no message

* don't silently lose error messages to ENOMEM

If we run out of memory, that's a catastrophic error; we want to report it
in some way.  (Note there's a good chance the error message we lost was
caused by ENOMEM, directly or indirectly).

* rrd_client: request() never sets `res` on failure, don't bother checking it

When request() returns non-zero (failure), it does not set `ret_response`.
So there is no point trying to extract an error message from ret_response
when request() returns non-zero.

If rrdcached returned an error response (ret_response->status < 0), then
response_read() will already have passed it to rrd_set_error().

* rrd_client: add missing checks for error responses

add read locking through rrd_open() (#800)

* fix cleanup in rrd_open()

* rados
  * don't leak rrd_file->rados
  * don't mysteriously close STDIN
* !HAVE_MMAP
  * don't leak rrd->stat_head

I have no explanation for why we were chosing to leak rrd->stat_head
(and only rrd->stat_head).  If it _was_ required for some reason, there was
a bug anyway, as we did not leak it if we fail to allocate live_hdr.

If it's required by legacy ABI, then we should remove rrd_open() from the
public API (and bump the soname).  It was originally scheduled for removal
in v1.4.

* avoid unmapping pages that don't belong to us in rrd_close()

When a rados RRD is closed, don't `munmap(0, rrd_file->file_len)` !

Note munmap() does not return an error when there were no pages mapped, so
our callers wouldn't notice even if they check the result of rrd_close().

Also fix the code for munmap errors - so it doesn't format one message and
then immediately replace it with a less specific one.  (If both munmap()
and close() fails though, we will describe the later error only).

Also, there's no point rrd_close() setting rrd_file to NULL at the end,
it won't affect the variable used by the caller.

* Add read locking through rrd_open()

Read lock or write lock is taken according to whether RRD_READONLY or
RRD_READWRITE is passed.  For librrd compat, the lock is only taken
when explicitly requested by passing RRD_LOCK.  Similarly rrd_lock()
is still available, but it should not be needed by new code.

So `RRD_LOCK|RRD_READONLY` is intended to take a read lock, however I must
admit to limitations of this new code:

* On rados, our write lock expires after 2 seconds.  This write-lock is
  probably useful as a way to detect conflicts.  However I would not want
  to rely on a lock that works like this to guarantee consistent reads for
  a backup.  I did not add read locking on rados.

  If rados users need to e.g. obtain a consistent backup, they should
  create a rados snapshot.   (This is able to work because we use a single
  atomic rados operation when when we *write* to RRDs).

* On the Windows _locking() function, read locks are treated the same as
  write locks.

  https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/locking

  The Windows code required several changes, and has not been compiled
  as of this moment.

With Windows _locking(), we should have been unlocking before close().
Apparently the automatic unlock on close() is not guaranteed to occur
synchronously.  This isn't specific to _locking(), it's a limitation
of winapi LockFileEx().

https://docs.microsoft.com/cpp/c-runtime-library/reference/locking

https://msdn.microsoft.com/en-us/library/windows/desktop/aa365203(v=vs.85).aspx

To manually unlock, we need to know what we actually locked.
But on Windows, rrd_lock() was locking the bytes starting at the current
rrd_file->pos, instead of always starting from 0.

Also, I thought RRD_LOCK should lock the file immediately, to guarantee we
don't have conflicting writes.  But on Windows we only tried to lock
the bytes that already existed in the file... which would be 0 bytes for
RRD_CREATE.  Fortunately, even Windows recognizes this problem and allows
locking bytes that don't exist yet.  So we can just lock the maximum number
of bytes allowed.  Locking 1 byte would probably work equally well for us,
but using LONG_MAX seems cleaner to me.

Fix madvise (and fadvise) (#802)

* fadvise and madvise code in rrd_open can be kept together

There was no reason to break up fadvise and madvise.

Putting them together makes it _slightly_ clearer that the exising #ifdefs
are intended.  (Documentation on the effect of fadvise() on mmap() is
hard to find, so it was simpler to avoid doing that).

Also, setvbuf() clearly doesn't apply to fd's.  Buffering is a function of
stdio e.g. fread(), fwrite() etc., but rrd_open() does not (currently?)
use these, it uses fds directly.  Hence, we have no buffering already :).

There must be some reason people use fread() but disable buffering,
but I can't think why rrd_open() might want to do that in future.
Remove the commented code which uses setvbuf().

* rrd_open(... RRD_COPY) does not want FADV_RANDOM

fix the !HAVE_MMAP case, to match the USE_MADVISE code.

# Conflicts:
# src/rrd_open.c

* Passing RRD_READAHEAD to rrd_open() had no effect on current linux, fix it

MAP_NONBLOCK (since Linux 2.5.46)
       Only meaningful in conjunction with MAP_POPULATE.  Don't perform
       read-ahead:  create  page tables entries only for pages that are
       already present in RAM.  Since Linux 2.6.23,  this  flag  causes
       MAP_POPULATE  to  do  nothing.   One  day,  the  combination  of
       MAP_POPULATE and MAP_NONBLOCK may be reimplemented.

RRD_READAHEAD has been "broken" for a looong time.  But I don't think it's
a good idea to carry code for RRD_READAHEAD which does nothing, at the same
time as we disable the default readahead by passing MADV_RANDOM.

This solution would not necessarily be API-quality, as changing the kernel
access hint from RANDOM to SEQUENTIAL means it's allowed to do drop-behind.
I am crudely assuming the in-tree users wouldn't have a problem with this.

Test:

for i in $(seq 50); do
  dd if=dns.rrd of=x/$i.rrd bs=1M oflag=direct 2>/dev/null
done
/usr/bin/time sh -c 'for i in x/*.rrd; do
  rrdtool dump $i >/dev/null
done'

Before:

6.52user 0.74system 0:09.46elapsed 76%CPU
(0avgtext+0avgdata 9796maxresident)k
204800inputs+0outputs (25600major+25112minor)pagefaults 0swaps

After:

5.82user 0.27system 0:06.57elapsed 92%CPU
(0avgtext+0avgdata 9740maxresident)k
204800inputs+0outputs (600major+27020minor)pagefaults 0swaps

I do not add madvise() WILLNEED before or after SEQUENTIAL.
Counter-intuitively, the test shows this would decrease performance.
(WILLNEED was implemented simply as triggering kernel readahead.
I think the implementation fails to use the increased readahead window.
Compare the number of major page faults).

WILLNEED after SEQUENTIAL:

6.02user 0.30system 0:06.96elapsed 90%CPU
(0avgtext+0avgdata 9528maxresident)k
204800inputs+0outputs (900major+26648minor)pagefaults 0swaps

WILLNEED before SEQUENTIAL:

6.35user 0.27system 0:07.28elapsed 91%CPU
(0avgtext+0avgdata 9548maxresident)k
204800inputs+0outputs (900major+26864minor)pagefaults 0swaps

* rrd_open() does not want to enable drop-behind of memory-mapped header

MADV_SEQUENTIAL mentions drop-behind.  Override it for the header now
we've read it, in case anyone implemented drop-behind.

Linux does not currently implement drop-behind.  My performance test
confirmed this did not help, but it didn't hurt either.

Do *not* fall back to fadvise() for !HAVE_MMAP.  We've already copied
the header in that case.  Doing e.g. FADV_NORMAL on Linux (4.12) on
*any* region would negate the effect of previous FADV_SEQUENTIAL.

Merge pull request #801 from sourcejedi/sidefixes3

Fix noise encountered in testing

fix test list1

$ cd tests && ./list1
...
ERROR: rrdcached@unix:/home/alan/src/rrdtool-1.x/tests/list1-rrdcached.sock
: Usage: LIST [RECURSIVE] /[<path>]

fix wrong printf specifier

$ cd tests && ./list1
...
ERROR: rrdcached@����������������������������������������������������������
��������������������������������������������正1: Usage: LIST [RECURSIVE]
/[<path>]

under valgrind:

==16450== Conditional jump or move depends on uninitialised value(s)
==16450==    at 0x4C356A9: wcslen (vg_replace_strmem.c:1848)
==16450==    by 0x535C953: wcsrtombs (in /usr/lib64/libc-2.24.so)
==16450==    by 0x52FFFC4: vfprintf (in /usr/lib64/libc-2.24.so)
==16450==    by 0x5328448: vsnprintf (in /usr/lib64/libc-2.24.so)
==16450==    by 0x4E6C848: rrd_set_error (rrd_error.c:51)
==16450==    by 0x4E6E113: response_read.constprop.5 (rrd_client.c:577)
==16450==    by 0x4E6E251: request.constprop.3 (rrd_client.c:659)
==16450==    by 0x4E6F319: rrd_client_list (rrd_client.c:1354)
==16450==    by 0x4E6F567: rrdc_list (rrd_client.c:1412)
==16450==    by 0x4E485C8: rrd_list (rrd_list.c:308)
==16450==    by 0x402D07: HandleInputLine (rrd_tool.c:696)
==16450==    by 0x40158B: main (rrd_tool.c:551)
==16450==
==16450== Invalid read of size 4
==16450==    at 0x4C356A4: wcslen (vg_replace_strmem.c:1848)
==16450==    by 0x535C953: wcsrtombs (in /usr/lib64/libc-2.24.so)
==16450==    by 0x52FFFC4: vfprintf (in /usr/lib64/libc-2.24.so)
==16450==    by 0x5328448: vsnprintf (in /usr/lib64/libc-2.24.so)
==16450==    by 0x4E6C848: rrd_set_error (rrd_error.c:51)
==16450==    by 0x4E6E113: response_read.constprop.5 (rrd_client.c:577)
==16450==    by 0x4E6E251: request.constprop.3 (rrd_client.c:659)
==16450==    by 0x4E6F319: rrd_client_list (rrd_client.c:1354)
==16450==    by 0x4E6F567: rrdc_list (rrd_client.c:1412)
==16450==    by 0x4E485C8: rrd_list (rrd_list.c:308)
==16450==    by 0x402D07: HandleInputLine (rrd_tool.c:696)
==16450==    by 0x40158B: main (rrd_tool.c:551)
...

$ man 3 printf
...
Conversion specifiers
...
S      (Not in C99 or C11, but in SUSv2, SUSv3, and SUSv4.)  Synonym for
       ls.  Don't use.
...
s      If  an l modifier is present: The const wchar_t * argument is
       expected to be a pointer to an array of wide characters.

Merge branch 'master' of github.com:oetiker/rrdtool-1.x

include prebuilt makefile.in as well

Merge pull request #797 from bangert/add_daemon_to_error_message

include the daemon name in the error message

do not install systemd support automatically. fix for #798

include the daemon name in the error message

permissions are 644 not 622

the switch in disambiguating m[inute] and m[onth] in time spec is 6 not 25. fix for #796

Remove all occurances of umask ... this is NOT thread safe! Fix for #794.

regression caused by cd139a88f4994c3560b56d0a3166ebfc395e42b8 fix for #794

Merge branch 'master' of github.com:oetiker/rrdtool-1.x

Merge pull request #795 from rubenk/remove-unneeded-ifdef

Unconditionally include <stdint.h>

Unconditionally include <stdint.h>

Code including rrd_client.h should not have to define HAVE_STDINT_H.

I accidentally bumped into this while porting Collectd's build system
from autotools to meson.

Merge pull request #789 from svenpanne/no-init

Do not initialize Cairo/Pango on export.

Merge pull request #790 from svenpanne/add-prototype

Fixed declaration and export of rrd_dump_opt_r.

Improved buffering (#792)

* Encapsulate access to write buffer.

Fixed ssize_t vs. size_t confusion on the way.

* Improve wbuf_append's time complexity from linear to amortized constant.

This is done by the usual technique employed in many, may libraries out
there (C++'s vector, Java's ArrayList, Python's list, ...): Distinguish
between the size and the capacity of the underlying container.  The capacity
grows by some *factor* (2 in our case) if it is too small, amortizing the
needed allocations/copies over time.

In a nutshell: Adding a single character to the buffer can now be done in
constant amortized time.

* Simplify and improve handle_request_fetch.

Now that add_response_info is efficient (thanks to the improved
wbuf_append), we can vastly simplify handle_request_fetch and even remove
some arbitrary length restrictions on the way.

Merge pull request #793 from svenpanne/suspend-resume

Added SUSPEND/RESUME/SUSPENDALL/RESUMEALL commands.

Added SUSPEND/RESUME/SUSPENDALL/RESUMEALL commands.

Suspending/resuming is essential to make e.g. backups of RRDs while
rrdcached is running.

Merge pull request #791 from svenpanne/no-version-info

Do not specify -version-info for convenience libraries.

Do not specify -version-info for convenience libraries.

libtool warns about a useless -version-info flag for convenience libraries,
see:

   http://git.savannah.gnu.org/cgit/libtool.git/tree/build-aux/ltmain.in#n6882

Let's just remove it, it doesn't serve a purpose.

Fixed declaration and export of rrd_dump_opt_r.

The previous state of affairs regarding rrd_dump_opt_r was a bit
inconsistent: The dynamic library didn't export it, while the static one
did.  So the right way to fix it will be exporting it in both variants,
because removing it from the static library would require a major version
change.  Furthermore, rrd_dump_opt_r is mentioned in the documentation at
librrd.pod, so this is another hint for the fix.

In addition, a C prototype was wmissing, so GCC complained about it.

Do not initialize Cairo/Pango on export.

On a heavily loaded rrdcached, initializing lots of unnecessary Cairo/Pango
data structures (surfaces, font-related structures, etc.) is not a good
idea. Do this only when we really want to draw something.

Merge pull request #788 from svenpanne/log-to-file

Added -o option for logging into a file instead of using syslog.

Added -o option for logging into a file instead of using syslog.

For some use cases it is better to keep log output separated from the normal
syslog. To handle such scenarios, a new -o option has been added.

Fixed foreground logging on the way, too: There was a missing mutex for
avoiding intermingled output.

Merge pull request #787 from svenpanne/fix-thread-init

Added missing GLib thread initialization.

Added missing GLib thread initialization.

Older GLib versions (< 2.32, e.g. the ones shipped with CentOS 5 and 6,
SLES11, Debian 6) need an explicit initialization to make things
thread-safe, see:

  https://developer.gnome.org/glib/stable/glib-Deprecated-Thread-APIs.html#g-thread-init

RRDtools didn't do that, so you get random crashes in clients and the
rrdcached on those systems.  Now we make sure that GLib is properly
initialized before calling into it.

geting ready for 1.7.0

get ready for 1.7.0

bump library version since three additional functions have been added

export 3 extra functions rrd_reduce_data(), rrd_lcd() and rrd_cf_conv()

cleanup THREADs notes since the library is now threadsafe

Merge pull request #781 from Marek77/pr_coverity

Pr coverity

src/rrd_graph_helper.c: fix Coverity CID #32827 Copy into fixed size buffer

bindings/rrdtoolmodule.c: fix Coverity CIDs #32891, #32892 Resource leak

bindings/rrdtoolmodule.c: fix Coverity CID #32890 Resource leak

bindings/rrdtoolmodule.c: fix Coverity CIDs #32888, #32889 Resource leak

bindings/rrdtoolmodule.c: fix Coverity CID #32887 Explicit null dereferenced

src/rrd_tune.c: fix Coverity CID #32826 Resource leak

src/rrd_rados.c: fix Coverity CID #32884 Dereference after null check

Merge pull request #777 from urbalazs/hun

Added Hungarian translation

Added Hungarian translation

Merge pull request #775 from sonertari/sonertari-patch-issue#774

Fix oetiker/rrdtool-1.x#774

Fix oetiker/rrdtool-1.x#774

Merge pull request #772 from Marek77/pr_dist-hook

Makefile.am: Fix broken build on Fedora

Makefile.am: Fix broken build on Fedora

Fix the following build error:

make top_distdir="bxrrdtool-1.6.1" distdir="bxrrdtool-1.6.1" dist-hook
make[2]: Entering directory '/home/vficet/src/rrdtool-1.x'
cd bxrrdtool-1.6.1 && /usr/bin/perl -i -p -e 's/^\1ERSION.+/\1ERSION='1.6002';/' bindings/perl-*/*.pm
Reference to nonexistent group in regex; marked by <-- HERE in m/^\1 <-- HERE ERSION.+/ at -e line 1.
Makefile:933: recipe for target 'dist-hook' failed

Merge pull request #771 from tvestelind/systemd-logic-in-rpm-spec

use systemd macros for rrdcached in rpm spec file

use systemd macros for rrdcached in rpm spec file

I've added:
* a variable to control whether or not to build for systemd
* systemd-provided macros in the RPM spec-file according what Fedora
  recommends.

Signed-off-by: Tomas Vestelind <tomas.vestelind@gmail.com>

rrd_list: adds --recursive option (daemon: LIST [RECURSIVE] /<path..>) (#767)

* rrd_list: adds --recursive option (daemon: LIST [RECURSIVE] /<path..>)

* tests/list1: updates LIST tests

python-rrdtool as replacement for the upstream Python bindings (#755)

* Replaced Python bindings with python-rrdtool

* Updated Makefile.in contents

* Added missing import

* Path correction

* Python 3 compatible print statements in acinclude.m4

* Updated configure script

* Added LIST command to Python binding

* Added missing var decls

* Added callback support for Python binding and initial tests suite for the binding

* Fixed some gcc compiler errors in pre-C99 mode

* setuptools is required to build the Python bindings

* Directory traversal

update for #765

Merge branch 'master' of github.com:oetiker/rrdtool-1.x

explicitly linking against libdl is not necessary anymore. fix for #765

restart rrdcached on upgrade, start and stop in install/uninstall (#764)

* restart rrdcached on upgrade, start and stop in install/uninstall

Start rrdcached and enable autostart when installing
Restart rrdcached and enable autostart when upgrading
Stop rrdcached and disable autostart when uninstalling

Catch non-zero return codes and return 0 in scriptlets

Signed-off-by: Tomas Vestelind <tomas.vestelind@gmail.com>
* enable and restart on both install and upgrade just before finishing the rpm installation

Signed-off-by: Tomas Vestelind <tomas.vestelind@gmail.com>

better use the right number of arguments in the format string

attempt to fix #763

DEBUG should not be active and libraries should not exit anyway ... fix for #760

DEBUG should not be active and libraries should not exit anyway ... fix for #760

fix #760

revert  0eed0e5

fix #760

Merge pull request #756 from bangert/add_deamon_to_cache_key

Add deamon variable to cache key

Add deamon variable to cache key

This allows for two identically named rrd's on two different rrdcache daemons
to be graphed on the samen graph. Without this patch the following

rrdtool graph - --start=1478559600 --end=1478685600 --width=1000 --step=300 --imgformat=JSONTIME \
  DEF:rrd1=aname.rrd:ds_name:AVERAGE:daemon=rrdcache1 \
  DEF:rrd2=aname.rrd:ds_name:AVERAGE:daemon=rrdcache2

would not actually fetch any data from rrdcache2, but rrd2 would reuse the data from rrd1,
since the current logic determines them to be equal.

With this patch rrd2 data is actually fetched from rrdcache2.

rebuilt autoconf

master will become 1.7 ... so prepare

updated autotools

Merge pull request #752 from Marek77/pr_daemon_fetch_fix

src/rrd_daemon.c: fix incorrect line number reporting by rrdcached in fetch

src/rrd_daemon.c: fix incorrect line number reporting by rrdcached in fetch

  Example of error:
    rrdtool fetch --daemon 127.0.0.1:42218 <path to>.rrd --start -5min LAST
    ERROR: rrdc_fetch: Got 34 lines, expected 37

  While the same, without '--daemon <address>' works fine.

  The bug manifests itself when the last element of line to write falls
  within the last 10% of the length of 'char linebuf[1024]', ie between
  922 and 1023 bytes; in that case the '\n' at the end of the line is
  not written, and so not seen by count_lines(), wrong number of lines
  given in the daemon's response -> 'rrdtool fetch' errors out.

Merge pull request #750 from he32/unsigned-fix

Fix a signedness/unsignedness conversion bug in the expression

Fix a signedness/unsignedness conversion bug in the expression
calculating "now" before printout of the timestamp.  Negative values
are first cast to unsigned, it seems, causing overflow.

Improve the RRD client API (#742)

* RRD client: Add an interface allowing for multiple client connections.

Add a client object which resembles a persistent connection to a server.
Multiple such objects may be used in parallel.

The old, "simple" interface is a wrapper around the new interface now, using
a default client connection. As before, this can only be used for one
connection.

also parse the daemon parameter in DEF as specified in the documentation (#748)

Make sure all listen_fds have their own copy of the addr string. (#746)

Fixes double free on shutdown #745.

Read the last timestamp from the RRD file, instead of setting it to zero. (#744)

This fixes a dataloss problem when trying to insert old data. rrdcached
will accept the data at first, but then fail the whole batch when trying
to flush the data to disk.

The last timestamp is initially set to 0, and later updated to the
timestamp of the first update. However, if the RRD file already contains
newer data,  fails later on. And since RRD cache writes all updates
in one go, the whole batch of samples fails and is discarded (even though,
just the first few updates are too old).

By reading in the last timestamp from the RRD file, we already fail
correctly on the individual update commands (ie. on the protocol layer).

RRD  ----------------------|
RRDcached         |------------------| <- Flush
                           |---------| <- Dataloss

With this patch, the initial updates, that overlap with data in the RRD
file will fail indivdually (at submission time).

RRD  ----------------------|
RRDcached         |xxxxxxxx----------| <- Flush

 Improve messages for rrdcached daemon failed due to PID Exist (#743)

*  Improve messages for rrdcached daemon failed due to PID Exist
    New messages added in order to get clarity about the nature of problem

* Review Comment incorporated regarding indentation of error messages

Merge pull request #741 from sharuzzaman/patch-1

fixed typo for 100,000 second in example1

fixed typo for 100,000 second in example1

the example show heartbeat of 100,000 but the explanation show 10,000

Merge pull request #739 from oetiker/cb-fix

detect overlong keys in callbacks ... fix for #738