What Anonymous Monk is saying is that the first line is already enough:

use Exporter qw(import);
our @ISA = qw(Exporter); # don't add this line!
[download]

Thank you for this. Almost all of my distributions are Object Oriented, so I rarely ever use Exporter functionality.

I never use it. I am also in the habit of exporting nothing (, Lewbowski) and using the fully qualified package name. The only time I've see Exporter even remotely useful is a package was just a ton of constants and the combination of namespace and actual variable names was simply too much.