Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Startup error handling #816

Closed
knutov opened this issue Aug 12, 2012 · 3 comments
Closed

Startup error handling #816

knutov opened this issue Aug 12, 2012 · 3 comments

Comments

@knutov
Copy link

knutov commented Aug 12, 2012

$ ./app.pl
 core - engine - unknown template engine 'ctpp2', perhaps you need to install Dancer::Template::Ctpp2? at /usr/local/share/perl/5.14.2/Dancer/Template.pm line 17
 BEGIN failed--compilation aborted at ./app.pl line 2.

 $ perl -MDancer::Template::Ctpp2 -MData::Dumper -e 'print Dumper(\%INC)' | grep tpp
 'Dancer/Template/Ctpp2.pm' => '/usr/local/share/perl/5.14.2/Dancer/Template/Ctpp2.pm',

The reason was in missing module, which D:T:Ctpp2 depends, but it's always hard to find the problem with such error messages. In this case D:T:Ctpp2 exists, but was unable start without missing module. Error message tells completely different. May be it's better to change this behavior?

@xsawyerx
Copy link
Member

That's a good point, we should improve it. Any suggestions on how it would look like?

@knutov
Copy link
Author

knutov commented Aug 13, 2012

At least it's better to write "not installed or was unable to start [because of dependencies]" with current implementation.

But.. If script can not start at all - why Dancer shows his messages instead of perl native errors? It does not make anything better or simple.

@xsawyerx
Copy link
Member

Fixed, thank you. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants