Hi,

On Mon, Sep 30, 2013 at 5:34 PM, Slavik Goltser <slavikg@gmail.com> wrote:
there is already a Dancer::Template::Mason, this plugin is intended to be a Dancer2 module.

Sure, I know, but I'm saying that the name should be Dancer2::Template::Mason not Dancer2::Template::Mason2, because the template system is called Mason, not Mason2.

 
data_dir ... I followed the original module closely, will take a look to see what happens when it isn't there.

I think in the situation less is more. The less you add as semantics to your layer, the less documentation one needs to know how D2::T::Mason will work, because you can say "just look at Mason defaults".

Besides, there is no good data_dir default to use. Your suggestion about using comp_root/data is not good because with shared perl installs you might not even have permissions to write there. The default data_dir doesn't have that problem.
 
thank you for taking a look.

No problem. I'm still using a git submodule to have my own Dancer2::Template::Mason module, so having this on CPAN will make my life easier.

Bye,
-- 
Pedro Melo
@pedromelo
http://www.simplicidade.org/
xmpp:melo@simplicidade.org
mailto:melo@simplicidade.org