cocaine/cocaine-plugins — pull request #1 by bogdad (2012-06-21 18:31:23+00:00)
Title: bug: perl engine tries to execute manifest.path which gets set to ../spool/app_name and is a directory

Hi,

There seems to be a bug in perl plugin,

perl.cpp:
boost::filesystem::path source(manifest.path);
if(boost::filesystem::is_directory(source)) {
throw configuration_error_t("malformed manifest, expected path to perl script, got a directory.");
}

it throws always, since manifest.path is set to ../spool/app_name in manifest_t::deploy() which is a directory.

proposed change:

add a 'script-file' required key to manifest.args for perl apps.
which should be set to the name of the perl script inside the app package.

Thanks!