Can anyone tell whether it is valid code

Oh, it's completely valid code. It just doesn't do what you want it to :-)

You need to add

require Exporter;
[download]

to one.pm.

It's all very well setting up the @EXPORT variable, but you need to load the Exporter code in order for it to make any difference.