Created attachment 307
patch to fix the workrave-dump perl script

Hi,

I tried to use the workrave-dump from contrib in the git repository.

I encountered the following errors:
------------------------------------------------------------------------------
vincent@hal:/tmp/workrave/contrib/plot$ perl workrave-dump
"my" variable @breakstats masks earlier declaration in same scope at workrave-dump line 89.
"my" variable @DS masks earlier declaration in same scope at workrave-dump line 91.
"my" variable $key masks earlier declaration in same statement at workrave-dump line 91.
"my" variable $heartbeat masks earlier declaration in same scope at workrave-dump line 91.
"my" variable @miscstats masks earlier declaration in same scope at workrave-dump line 94.
"my" variable @DS masks earlier declaration in same scope at workrave-dump line 95.
syntax error at workrave-dump line 88, near "$breaktype qw(micro rest daily)"
syntax error at workrave-dump line 92, near "}"
Execution of workrave-dump aborted due to compilation errors.
------------------------------------------------------------------------------
------------------------------------------------------------------------------
revision tested: e71af7433710d9bbf032231aa50d021f21a71d2d
Debian GNU/Linux 8.2 (jessie)
------------------------------------------------------------------------------

Here is a patch to fix it.
I am going to set up a pull request on github if it is easier for you to incorporate into the repository.

Best regards.

My perl version:

vincent@hal:~$ perl --version

This is perl 5, version 20, subversion 2 (v5.20.2) built for x86_64-linux-gnu-thread-multi
(with 42 registered patches, see perl -V for more detail)

Pull request provided at:

https://github.com/rcaelers/workrave/pull/59

Best regards,

Thanks! pull request merged.

Great! Thanks!