Merge branch 'master' into changes

Define mkdir() macro for Windows builds

- Windows: #define mkdir(A, B) mkdir(A)
  mkdir() has only got one argument under Windows

Call getuid() only if HAVE_GETUID is defined

Update FSF address and fix UTF8

- Update FSF address in COPYRIGHT
  old: 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA
  new: 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
- Update LICENSE file according to the current version from:
  https://www.gnu.org/licenses/gpl-2.0.txt
- Update bindings/python/COPYING according to the current version from:
  https://www.gnu.org/licenses/old-licenses/lgpl-2.1.txt
- Change encoding of CONTRIBUTORS from ISO 8859-1 to UTF-8
- Fixes rpmlint warnings and errors:
  W: file-not-utf8 CONTRIBUTORS
  E: incorrect-fsf-address COPYRIGHT
  E: incorrect-fsf-address LICENSE
  E: incorrect-fsf-address bindings/python/COPYING

add notes on update

add missing testfile

Typo

Thank lintian

add ghostscript to build env

add missing cd ..

consistant numbering for windows

generate NUMVERS from VERSION file

bump version

add releasee tests to travis and bump version number for rerelease with fixed makefiles

Fixed 'No rule to make target 'etc/rrdcached.socket'

Signed-off-by: Jaroslav Škarvada <jskarvad@redhat.com>

properly add etc files via EXTRA_DIST - fixes #956 some more

add missing etc/rrdcached-default-lsb.in to tarball - fixes #956

sync 1.7.1 release updates

update library version since new features were added

fix log

Update version to 1.7.1 for MSVC builds

ready for 1.7.1 release

Remove librrd-4.vcxproj.user

- Remove file from git source control, because
  .vcxproj.user files typically contain user-specific settings
  or overrides

Update issue templates

Fix warnings: different const qualifiers (MSVC)

- Cast affected variables from 'const char **' to 'char **'
- Fixes the following warning:
  C4090: 'function': different 'const' qualifiers
  in: rrd_create.c(248), rrd_modify.c(1312),
  rrd_modify.c(1329), rrd_modify.c(1457), rrd_modify.c(1463)

Define PATH_MAX as _MAX_PATH for MSVC builds

- The max. length of full pathname is 260 under Windows.
  _MAX_PATH is defined in stdlib.h, use:
  #define PATH_MAX _MAX_PATH

Define HAVE_ASPRINTF and HAVE_VASPRINTF (MSVC)

- asprintf.c and vasprintf-msvc.c are compiled in MSVC builds, so
  define HAVE_ASPRINTF and HAVE_VASPRINTF in win32/rrd_config.h

Use Unix line endings for win32-glob

- Restore original line endings for win32-glob.c and win32-glob.h
- Furthermore, RRDtool source files use Unix line endings in general

Remove win32/RCa02816, which is a temporary file

Fix _errno inconsistent dll linkage (Windows)

- Exclude in case of _WIN32 in rrd_snprintf.c:
  extern int errno;
- Fixes warnings:
  MSVC:
  rrd_snprintf.c(583): warning C4273:
  '_errno': inconsistent dll linkage
  errno.h(19): note: see previous definition of '_errno'
  MinGW-w64 (gcc):
  rrd_snprintf.c:583:12: warning: '_errno' redeclared without
  dllimport attribute: previous dllimport ignored [-Wattributes]

Fix rc compiler redefinition warnings (nmake)

- Surround code in win32/rrd_config.h that causes warnings by rc with
  #ifndef RC_INVOKED and #endif. Remark: _MSC_VER is unknown to rc
- Fixes the following warnings for .rc files, e.g.:
  rc /nologo /I./src /fo./win32/rrdtool.res ./win32\rrdtool.rc
  ./win32\rrd_config.h(130) : warning RC4005: 'isinf' : redefinition
  ./win32\rrd_config.h(131) : warning RC4005: 'isnan' : redefinition
- This concerns: librrd-4.rc, rrdtool.rc, rrdupdate.rc, rrdcgi.rc,
  when using the resource compiler (rc) via nmake under Windows

Initialize local variable junk

- Fixes the following compiler warnings (MSVC):
  rrd_modify.c(439) : warning C4700: uninitialized local variable
  'junk' used
  rrd_modify.c(1027) : warning C4700: uninitialized local variable
  'junk' used

Include winsdkver.h for _WIN32_MAXVER

- _WIN32_MAXVER is defined in winsdkver.h
  The include is required in case of MinGW-w64 builds
- Fixes rrd_open.c:11:5: warning:
  "_WIN32_MAXVER" is not defined, evaluates to 0 [-Wundef]
  #if _WIN32_MAXVER >= 0x0602 /* _WIN32_WINNT_WIN8 */

Use _WIN32 instead of WIN32 preprocessor macro

- _WIN32 is the recommended preprocessor macro
- Substitute occurrences of WIN32 with _WIN32 using:
git grep -lw -e '#ifdef WIN32' -e '#ifndef WIN32' -e 'defined(WIN32)' \
-- '*.c' '*.h' '*.xs' | xargs \
sed -i -e 's/#ifdef WIN32/#ifdef _WIN32/g' \
-e 's/#ifndef WIN32/#ifndef _WIN32/g' \
-e 's/defined(WIN32)/defined(_WIN32)/g'

Fix: Cannot rename temporary file to final file

- Fixes the following failing tests under Windows:
  modify1, modify2, modify3, modify4, modify5
- Use CreateFileA() with FILE_SHARE_DELETE in rrd_open.c,
  so that the outfilename can later be replaced by tmpfilename using
  write_rrd() in rrd_create.c

Indent rrd_create.c, rrd_open.c before changes

- Using GNU indent 2.2.12

Define realpath for MinGW builds

- Define realpath the same way as in win32/rrd_config.h
- Fixes the following compiler warnings in rrd_client.c:
  115:11: warning: implicit declaration of function 'realpath';
  [-Wimplicit-function-declaration] ret = realpath(path, NULL);
  115:11: warning: nested extern declaration of 'realpath'
  [-Wnested-externs]
  115:9: warning: assignment to 'char *' from 'int' makes pointer from
  integer without a cast [-Wint-conversion] ret = realpath(path, NULL);
  132:22: warning: assignment to 'char *' from 'int' makes pointer from
  integer without a cast [-Wint-conversion]
  dir_path = realpath(dir, NULL);

Remove LINKTYPE from perl-piped/Makefile.PL

- Remove line:
  'linkext'   => {LINKTYPE => ''},
  Since version 5.00 of MakeMaker such a line can be deleted safely.
  MakeMaker recognizes when there's nothing to be linked.
  See: https://perldoc.perl.org/ExtUtils/MakeMaker.html
- Fixes Warning:
  LINKTYPE set to '', no longer necessary

Re-enable 0-width lines

Some people were using these as a base for stacks.

Use g_list_free_full instead of g_list_foreach

- Fixes the following compiler warnings in rrd_create.c:
  236:34: warning: cast between incompatible function types from
  'void (*)(void *)' to 'void (*)(void *, void *)'
  [-Wcast-function-type] g_list_foreach( sources, (GFunc)free, NULL );

  1022:44: warning: cast between incompatible function types from
  'void (*)(rrd_file_t *)' {aka 'void (*)(struct rrd_file_t *)'} to
  'void (*)(void *, void *)' [-Wcast-function-type]
  g_list_foreach(sources_rrd_files, (GFunc) cleanup_source_file, NULL);

Remove function substring from rrd_graph.c

- The function substring is not used any more. This was a leftover
  from https://github.com/oetiker/rrdtool-1.x/commit/cd4a728
- Fixes rrd_graph.c:6159:7: warning:
  no previous prototype for 'substring' [-Wmissing-prototypes]

Update doc/Makefile.am

- Add -Wbreak, as in:
  https://github.com/oetiker/rrdtool-1.x/commit/564aed3

Include rrd_pdpcalc.pod in Makefile.am also

Should really have been part of:

    commit 47e0f4391bccc0a3b12f75f567596425bd95b9ad
    Author: Peter Valdemar Mørch <peter@morch.com>
    Date:   Thu Jan 3 21:35:38 2019 +0100

        Added some documentation for the rrdxport output format

        See https://github.com/oetiker/rrdtool-1.x/issues/913 for a discussion

Rename PDP_calculation_explanation.pod -> rrd_pdpcalc.pod as pr. review comment in #931

Changed "[ timestamp - step ; timestamp [" as it was unclear as pr. review comment in #931

Spell rrdtool as RRDtool as pr. review comment in #931

Added some documentation for the rrdxport output format

See https://github.com/oetiker/rrdtool-1.x/issues/913 for a discussion

Fix two review comments from #932

Improve links to AT-STYLE TIME SPECIFICATION section in rrdfetch

Use pod links like L<rrdfetch/"AT-STYLE TIME SPECIFICATION">, which
creates e.g. HTML links like:

    <a href="./rrdfetch.html#AT-STYLE-TIME-SPECIFICATION">
        "AT-STYLE TIME SPECIFICATION" in rrdfetch
    </a>

Fixed some podchecker error/warnings

Removed trailing spaces from .pod documentation files

This fixed warnings like:

*** WARNING: line containing nothing but whitespace in paragraph at line 145 in file rrdfetch.pod

from podchecker

Fixed POD in PDP_calculation_explanation.pod

podchecker PDP_calculation_explanation.pod showed:

*** WARNING: line containing nothing but whitespace in paragraph at line 46 in file PDP_calculation_explanation.pod
*** ERROR: Apparent command =head1 not preceded by blank line at line 176 in file PDP_calculation_explanation.pod
*** WARNING: node 'https://gallery.technet.microsoft.com/scriptcenter/Sample-Script-to-Generate-59c80d4c =head1 AUTHOR' contains non-escaped | or / at line 175 in file PDP_calculation_explanation.pod
*** ERROR: unresolved internal link 'https://gallery.technet.microsoft.com/scriptcenter/Sample-Script-to-Generate-59c80d4c =head1 AUTHOR' at line 175 in file PDP_calculation_explanation.pod
PDP_calculation_explanation.pod has 2 pod syntax errors.

Removed "can't break line" warnings when making .txt files from .1 files

Multiple warnings were removed e.g. this one (chosen at random):

rrdgraph_data.1:141: warning [p 1, 1.3i]: can't break line

update tests including verification of timestamp alignement

update tests including verification of timestamp alignement

fix off by one issue #913

add graph test

Add -lws2_32 for MinGW builds by configure

- Add -lws2_32 to LIBS in case of MinGW or MinGW-w64 builds.
  This makes building Windows executables more straightforward using
  configure.
- Update README-MinGW-w64, remove LIBS='-lws2_32' from configure
  instructions.

Check for gobject-2.0 in configure

- Since pango version 1.43.0, -lgobject-2.0 is not added automatically
  any more during configure. Remark: pango.pc uses Requires.private
  now for gobject-2.0 instead of formerly Requires: gobject-2.0
- Check for gobject-2.0 in configure to make sure, that -lgobject-2.0
  is added to ALL_LIBS
- Fixes the following linker errors:
  .libs/librrd_la-rrd_graph.o: In function `im_free':
    src/rrd_graph.c:465: undefined reference to `g_object_unref'
  .libs/librrd_la-rrd_graph.o: In function `rrd_graph_init':
    src/rrd_graph.c:4840: undefined reference to `g_object_unref'

Sync librrd-4.def with librrd.sym

- Export the same functions in the dll from MSVC builds under Windows
  as listed in librrd.sym

Fix warning strncpy bound equals destination size

- Reduce num from 5 to 4 in strncpy and add null terminator.
  Remark: Version strings consist of 4 chars, e.g. RRD_VERSION5 "0005"

- Fixes the following GCC 8 warnings:
  rrd_create.c:960:9: warning:
    ‘strncpy’ specified bound 5 equals destination size
    [-Wstringop-truncation]
    strncpy(rrd.stat_head->version, require_version, 5);

  rrd_modify.c:1218:9: warning:
    ‘strncpy’ specified bound 5 equals destination size
    [-Wstringop-truncation]
    strncpy(out->stat_head->version, require_version, 5);

Fix tests to support Windows time zone name

- On Windows, names of times zones are different.
  Substitute 'W. Europe Standard Time' with 'CET' by function seddif
- Removes unnecessary diff output and fixes therefore failing tests

Disable Travis CI valgrind-logfile failing tests

- These failing tests are not directly coming from rrdtool
  and are related to external libraries
- Allows Travis CI builds to pass again

Fix loop, sizeof(delims)

- Update to https://github.com/oetiker/rrdtool-1.x/commit/2154d02

Update to PR #915

Add windows-default-manifest to README-MinGW-w64

- windows-default-manifest is available for Fedora in the meantime.
  This adds a default manifest to the compiled binaries and prevents
  unnecessary elevated privileges, e.g. for 32-bit rrdupdate.exe

minor adjustment to patch for #914

issue #914, rrdtool graph with --title segfaults and misparses single-line args

reworked graph_title_split() to not segfault,
to handle leading/trailing/duplicate delimiters, and to produce the
appropriate number of output lines.

code now works and properly handles nasties
like --title '\n\none\n\nfourtyseven\n'

#914, rrdtool graph with --title segfaults

Update perl-shared README for MSVC and ActivePerl

- Using MSVC, the last supported ActivePerl version is 5.16,
  e.g. ActivePerl-5.16.3.1604-MSWin32-x86-298023.msi
- Since 5.18, ActivePerl is compiled with GCC compilers, and Microsoft
  toolchains are *not* compatible
- See this FAQ for further details:
  https://community.activestate.com/faq/windows-compilers-perl-modules

lua: build failure when rrd graph is disabled. (#900)

Needs the same protection used elsewhere in the file.

Signed-off-by: Karl Palsson <karlp@etactica.com>

lua: don't include rrd_restore if it wasn't included (#901)

The lua module compiles ok, but fails to load.

> rrd=require("rrd")
error loading module 'rrd' from file '/usr/lib/lua/rrd.so':
Error relocating /usr/lib/lua/rrd.so: rrd_restore: symbol not found
stack traceback:
[C]: ?
[C]: in function 'require'

Signed-off-by: Karl Palsson <karlp@etactica.com>

we can not use libtool from within configure as it only gets generated at the end of the run ...

Allow multiple lines in graph titles (#897)

Merge pull request #894 from c72578/2018-07-14_32bit_time_issue_dump_MSVC

Fix lastupdate time in dump (32bit MSVC only)

Fix lastupdate time in dump (32bit MSVC only)

- So far, (null) was written into the lastupdate time string:
  e.g.: <lastupdate>1511170870</lastupdate> <!-- (null) -->
- The problem occurred only in 32bit MSVC builds
- Works for both, with or without _USE_32BIT_TIME_T

Merge pull request #893 from c72578/2018-07-13_HAVE_RRD_RESTORE_MSVC

Define HAVE_RRD_RESTORE (MSVC)

Define HAVE_RRD_RESTORE (MSVC)

- Enables restore functionality on Windows MSVC builds
- Fixes ERROR: the instance of rrdtool has been compiled without XML
  import functions

Merge pull request #891 from weyou/fix-rrdfile-write-issue

Fix rrdfile write mode issue on Windows

Added a empty line to keep the original style

Fixed the rrd file write mode on Windows

Merge pull request #882 from nirgal/patch-1

Fixed typo in error message

Fixed typo in error message

Merge pull request #881 from c72578/2018-03-25_Update_README-MinGW-w64

Update MinGW-w64 build instructions

Update MinGW-w64 build instructions

- Info added, where to find compiled binaries:
  src/.libs/
- Added perl-Pod-Html to Fedora dependencies
- Added base-devel to MSYS2 build instructions

Merge pull request #879 from Tomo59/master

Fix rrdtool.graph python binding

Fix rrdtool.graph python binding

It was failing with 'munmap_chunk(): invalid pointer' error in python 3.6.3

Merge pull request #877 from nirgal/french

French

French translation

Stop ignoring translation updates

Merge pull request #875 from matwey/fix/static-2

Do not use static variable in gfx_prep_text()

Do not use static variable in gfx_prep_text()

Move last_tabwidth to image_desc_t

Signed-off-by: Matwey V. Kornilov <matwey.kornilov@gmail.com>

Merge pull request #874 from nirgal/patch-2

Really enable translations

Merge pull request #873 from matwey/fix/static-1

Do not use static variables in xtr() and ytr()

Really enable translations

Fix for issue #872.

Do not use static variables in xtr() and ytr()

Using static variables for writting leads to race conditions. In order to keep
scale for axis we introduce x_pixie and y_pixie into image_desc_t.

Signed-off-by: Matwey V. Kornilov <matwey.kornilov@gmail.com>

Merge pull request #871 from nirgal/patch-1

Quiets warning

Quiets warning

gcc using -Wimplicit-fallthrough emits warning because of the typo

Merge pull request #870 from nirgal/spell

Spelling fixes

Spelling fixes

Merge pull request #869 from igit/patch-1

Update CHANGES : version 1.7.0 released on 2017 (not 2016)