ALT Linux Bugzilla – Full Text Bug Listing
| Summary: | Лишняя зависимость на Apache::Util | ||
|---|---|---|---|
| Product: | Sisyphus | Reporter: | Vladimir Lettiev <crux> |
| Component: | perl-Template | Assignee: | viy <viy> |
| Status: | CLOSED FIXED | QA Contact: | qa-sisyphus |
| Severity: | critical | ||
| Priority: | P2 | CC: | at, cas, crux, ender, lav, ldv, mike, qa_viy, shaba, viy |
| Version: | unstable | ||
| Hardware: | all | ||
| OS: | Linux | ||
Вытягиваются пакеты apache-base apache-mod_perl-base. Алексей, будете что-то делать? |
Модуль Template::Toolkit может использовать как Apache::Util, так и HTML::Entities в динамическом фильтре html. Т.о. достаточно только одного из них для работы данного функционала. Зависимость от Apache::Util порождает тяжёлую зависимость на веб-сервер Apache1 и модуль mod_perl к нему. Причём для использования Template::Toolkit никаких веб-серверов в принципе и не нужно. Ещё хуже ситуация если используется другой веб-сервер и наличие лишнего веб-сервера на системе неприятно по многим причинам. Наличие этих модулей определяется в момент исполнения с использованием конструкции eval{}, но в блоке eval содержится не вызовы require, а процедура, которая уже делает require. Т.о. при определении зависимостей оба модуля попадают в жёсткие зависимости пакета perl-Template. Как вариант обхода проблемы с лишней зависимостью, можно перетащить eval внутрь процедуры определения наличия модуля, чтобы perl.req не захватывал эту зависимость: --- a/lib/Template/Filters.pm +++ b/lib/Template/Filters.pm @@ -407,9 +407,11 @@ sub use_html_entities { } sub use_apache_util { - require Apache::Util; - Apache::Util::escape_html(''); # TODO: explain this - return ($AVAILABLE->{ HTML_ENTITY } = \&Apache::Util::escape_html); + eval { + require Apache::Util; + Apache::Util::escape_html(''); # TODO: explain this + return ($AVAILABLE->{ HTML_ENTITY } = \&Apache::Util::escape_html); + } } sub html_entity_filter_factory { @@ -418,7 +420,7 @@ sub html_entity_filter_factory { # if Apache::Util is installed then we use escape_html $haz = $AVAILABLE->{ HTML_ENTITY } - || eval { use_apache_util() } + || use_apache_util() || eval { use_html_entities() } || -1; # we use -1 for "not available" because it's a true value Идеальный вариант - наличие возможности отфильтровывать perl-зависимости в спеке (как, например, можно в python модулях).