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)

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

Version 1.7.0 released on **2017**-05-16 not 2016-05-16

Merge pull request #858 from NHellFire/master

Add missing import in setup.py

Merge pull request #866 from matwey/fix/python_race

Fix/python race

bindings: python: Fix race conditions at arguments access

rrdtool_argv and rrdtool_argc have to be allocated at each function stack in
order to have copy of variables for each running thread. Note that we call
librrd functions with GIL unlocked, so previously there was simultaneous access
to global rrdtool_argv and rrdtool_argc variables.

Fixes: 8949308812a0 ("Unlock the global thread lock while processing rrd")
Signed-off-by: Matwey V. Kornilov <matwey.kornilov@gmail.com>

bindings: python: Don't use global variables in convert_args() and destroy_args()

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

Merge pull request #864 from c72578/2018-01-26_update_documentation

Update documentation

Update documentation

- Some typos corrected

exclude pango fontmap cache

Merge pull request #862 from yarda/yardas

Fixed configure --enable / --disable options

Fixed configure --enable / --disable options

E.g. now './configure --enable-libdbi' works.

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

Merge pull request #860 from c72578/2018-01-04_update_documentation

Update documentation

Update documentation

- Some typos corrected
- Minor updates and corrections

Add missing import in setup.py

Merge pull request #857 from c72578/2018-01-02_fix_warning_this_use_of_defined

Fix expansion-to-defined warnings (GCC 7)

Merge pull request #856 from c72578/2017-12-31_fix_time_t_fprintf_printf_msvc

Fix 64-bit time_t in fprintf, printf (MSVC)

Fix expansion-to-defined warnings (GCC 7)

- Fixed compiler warnings:
  gettext.h:176:6: warning: this use of "defined" may not be portable
  [-Wexpansion-to-defined] #if !_LIBGETTEXT_HAVE_VARIABLE_SIZE_ARRAYS
  gettext.h:200:5: warning: this use of "defined" may not be portable
  [-Wexpansion-to-defined] #if _LIBGETTEXT_HAVE_VARIABLE_SIZE_ARRAYS
  gettext.h:215:6: warning: this use of "defined" may not be portable
  [-Wexpansion-to-defined] #if !_LIBGETTEXT_HAVE_VARIABLE_SIZE_ARRAYS
  gettext.h:246:5: warning: this use of "defined" may not be portable
  [-Wexpansion-to-defined] #if _LIBGETTEXT_HAVE_VARIABLE_SIZE_ARRAYS
  gettext.h:261:6: warning: this use of "defined" may not be portable
  [-Wexpansion-to-defined] #if !_LIBGETTEXT_HAVE_VARIABLE_SIZE_ARRAYS

Fix 64-bit time_t in fprintf, printf (MSVC)

- In VS2005 and later, the default size for time_t is 64-bit, unless
  _USE_32BIT_TIME_T has been defined to use a 32-bit time_t ...

- Fixes MSVC compiler warnings:
  ./src\rrd_graph_helper.c(1465): warning C4477: 'fprintf' :
  format string '%li' requires an argument of type 'long',
  but variadic argument 1 has type 'time_t'

  ./src\rrd_lastupdate.c(83): warning C4477: 'printf' :
  format string '%10lu' requires an argument of type 'unsigned long',
  but variadic argument 1 has type 'time_t'

  ./src\rrd_tool.c(720): warning C4477: 'printf' : format string '%ld'
  requires an argument of type 'long', but variadic argument 1 has type
  'time_t'
  ./src\rrd_tool.c(724): warning C4477: 'printf' : format string '%ld'
  requires an argument of type 'long', but variadic argument 1 has type
  'time_t'
  ./src\rrd_tool.c(742): warning C4477: 'printf' : format string
  '%10lu' requires an argument of type 'unsigned long', but variadic
  argument 1 has type 'time_t'

Merge pull request #855 from c72578/2017-12-29_add_files_to_extra_dist

Add missing files to src/Makefile.am EXTRA_DIST

Add missing files to src/Makefile.am EXTRA_DIST

- src/Makefile.am, added to EXTRA_DIST:
  mkstemp.c
  mkstemp.h

not required anymore

Merge pull request #854 from c72578/2017-12-26_configure.ac_Makefile.am_win32-glob_mingw-w64

Check for MinGW-w64 build, add win32-glob files

Remove win32-glob.c, win32-glob.h from EXTRA_DIST

- Makefile.am: win32/win32-glob.c and win32/win32-glob.h are mentioned
  in src/Makefile.am now. Removed from EXTRA_DIST

Check for MinGW-w64 build, add win32-glob files

- configure.ac: Add check for MinWG-w64 build during configure.
  Pass the conditional MINGW_W64 to automake
- src/Makefile.am: Add win32-glob.c to RRD_C_FILES and
  win32-glob.h to noinst_HEADERS in case of MINGW_W64
- Enables out of the box configure and make for MinGW-w64 builds

Merge pull request #853 from c72578/2017-12-25_README-MinGW-w64

Add documentation for building under MinGW-w64

Add documentation for building under MinGW-w64

Merge pull request #851 from c72578/2017-12-14_fix_compiler_warning_llu_lli_64bit_mingw-w64

Fix compiler warnings, llu lli 64bit (MinGW-w64)

Fix compiler warnings, llu lli 64bit (MinGW-w64)

- time_t is of type 'long long int', when compiled for 64 bit Windows
  (x86_64-w64-mingw32). Use %10llu instead of %10lu in printf and
  %lli instead of %li in dprintf. Requires __USE_MINGW_ANSI_STDIO 1

- Fixes: rrd_lastupdate.c: In function 'rrd_lastupdate':
  rrd_lastupdate.c:76:18:
  warning: format '%lu' expects argument of type 'long unsigned int',
  but argument 2 has type 'time_t {aka long long int}' [-Wformat=]

- Fixes: rrd_graph_helper.c: In function 'parse_shift':
  rrd_graph_helper.c:1458:13:
  warning: format '%li' expects argument of type 'long int',
  but argument 3 has type 'time_t {aka long long int}' [-Wformat=]
  dprintf("SHIFTBY : %li\n",gdp->shval);

Merge pull request #849 from c72578/2017-12-07_fix_further_build_errors_mingw-w64

Fix ctime_r configure error (MinGW-w64)

Fix ctime_r configure error (MinGW-w64)

- configure.ac: include pthread.h inside AC_LANG_PROGRAM
  for ctime_r check if __MINGW32__ is defined
  Fixes:
  checking if ctime_r need special care to act posixly correct...
  configure: error: Can't figure how to compile ctime_r

Merge pull request #848 from c72578/2017-12-06_fix_further_build_errors_mingw-w64

Fix further build errors (MinGW-w64)

Fix further build errors (MinGW-w64)

- rrd_list.c: MinGW-w64 does not provide glob.h (yet?)
  include win32/win32-glob.h using HAVE_GLOB_H. Fixes:
  rrd_list.c:9:10: fatal error: glob.h: No such file or directory
    #include <glob.h>
- rrd_update.c: MinGW has gettimeofday() in time.h. Fixes:
  rrd_update.c:46:12: error: conflicting types for 'gettimeofday'
    static int gettimeofday(
- rrd_tool.c: MinGW has not got getuid and mkdir only one argument.
  Fixes:
  rrd_tool.c:585:17: warning: nested extern declaration of 'getuid'
  rrd_tool.c:632:16: error: too many arguments to function 'mkdir'
    if(mkdir(argv[2], 0777)!=0){
- rrd_tool.h:
  MinGW-w64 provides localtime_r, ctime_r, gmtime_r, strtok_r
- rrd_graph.c: MinGW has strftime and tzname.
  Due to the resulting later include of time.h, fixes also:
  rrd_graph.c:1558: undefined reference to `localtime_r'
- rrd_client.c: MinGW does not provide realpath, "is_unix" excluded.
  Fixes:
  rrd_client.c:113: undefined reference to `realpath'
  rrd_client.c:130: undefined reference to `realpath'
- rrd_rpncalc.c: include pthread.h. Fixes e.g.:
  rrd_rpncalc.c:708: undefined reference to `localtime_r'

Merge pull request #847 from c72578/2017-12-01_add_glob.h_to_configure.ac_AC_CHECK_HEADERS

Add glob.h to configure.ac AC_CHECK_HEADERS

Add glob.h to configure.ac AC_CHECK_HEADERS

- Adds check for glob.h
- Provides HAVE_GLOB_H