Skip Menu |

This queue is for tickets about the RRDTool-OO CPAN distribution.

Report information
The Basics
Id: 32601
Status: resolved
Priority: 0/
Queue: RRDTool-OO

People
Owner: Nobody in particular
Requestors: ANDK [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)



Subject: rrdtool/install dialog
I saw this for the first time in 0.21: This module requires rrdtool 1.2.x and the RRDs module to be installed. They are available in the rrdtool distribution: http://oss.oetiker.ch/rrdtool/pub/rrdtool.tar.gz Do you want me to install it for you right now ([y]/n)? As far as I can see this dialog has three issues: - it doesn't use the MakeMaker::prompt() function: please use it because it has several clever properties like checking for terminal output, checking well established environment variables, printing a blank after the prompt; - the message does not explain why it asks this question at all. In my case I just cannot understand it because I have rrdtool installed; - the answer defaults to "y". This is clearly wrong because you should always err on the cautious side and not fool the user into doing the potentially harmful thing like downloading and installing third party software; Thanks & Regards,
Today I further investigated the case and found out that although I do have rrdtool installed, your Makefile.PL fails when searching for RRDs.pm. That's because I do not have RRDS.pm installed specifically for the current perl. This shows me into which direction a solution might go. It would still be nice if the Makefile.PL would provide the answers it finds before it suggests to download and build rrdtool. Maybe something like: Found rrdtool in '/usr/lib/rrdtool'. Good. rrdtool version is '1.2.27'. Good. Not found RRDs.pm. Needs to be installed. It's unfortunate that RRDs.pm doesn't seem available separately. I wonder if I could sneak it into current perl with a simple cp. Liebe Gruesse,
This was addressed in http://github.com/mschilli/rrdtool-oo-perl/tree/c4dcf884ae3e99977827d5bfe88e42217ec584af and will be released in 0.26. Thanks for your report! -- Mike