On Sun, Mar 25, 2012 at 01:01:55PM -0400, Dubrovskiy Viacheslav wrote: > >> - cmd_p->res.tmpl.ostmpl = strdup(tmpl->ostmpl); > >> + cmd_p->res.tmpl.ostmpl = strdup(basename(tmpl->ostmpl)); > > Ты теряешь информацию перед записью, а я предлагаю отбрасывать > > лишнюю при чтении. Тогда по мере доработки утилит ей можно > > будет воспользоваться, т.к. она будет сохранена. > Мне кажется ты усложняешь.
Возможно. Или же стоило сразу выписывать полную картину маслом, но уже засыпал. > Директории могут быть любые и вешать какой-то функционал на их > имя мне кажется не очень хорошей идеей. Если хламовник, то любые... мне показалось, что можно дать интересную функциональность тем, кто использует их с головой (при этом ничего не сломав тем, кто сваливает всё в кучу). > Достаточно того что уже есть привязка к имени образа. Удалённо (http://download.openvz.org/template/precreated/contrib/): contrib/altlinux-p6 contrib/debian-6.0 Локально (/var/lib/vz/template/cache/): altlinux/p6-kiev-x86_64 altlinux/p6-odessa-i586 opensuse/11.4-kiev-i586 А в светлом будущем(tm) ещё и управлять различиями этих образов, генерируемых из mkimage-profiles. Один тестовый из сузи 11.4 уже был сделан (но невоспроизводимо), а сейчас ещё и centos6 удалось запинать (и уже в базовой части воспроизводимо). При этом можно аккуратно различать test и prod, архитектуры, версии, дистрибутивы, назначение... > Да и какой функционал может быть повешен на имя директории? > Пример можешь привести? --- to get things done properly src/lib/dist.c::get_dist_conf_name() would need fixup too -- probably looking at basename and then at dirname would be fine (so that contrib/* would still work for generally available templates but e.g. centos/6.0-x86_64 or thatdepartment/debian-7.0-ns would be feasible too). --- http://bugzilla.openvz.org/show_bug.cgi?id=2222#c0 -- ---- WBR, Michael Shigorin <[email protected]> ------ Linux.Kiev http://www.linux.kiev.ua/ _______________________________________________ Sysadmins mailing list [email protected] https://lists.altlinux.org/mailman/listinfo/sysadmins
