[devel] [Sisyphus-cybertalk] E: incoming reject: alterator-services-1.0-alt1.src.rpm

[Michael Shigorin]

On Fri, May 05, 2006 at 09:30:24PM +0400, Upravdom Team Robot wrote:
> Dear Dmitri Kuzishchin !
> The alterator-services-1.0-alt1.src.rpm package has been rejected from incoming.

Дмитрий, спасибо, что взялись за оставшиеся 80% :-)
Чувствуется, что не первый модуль.

Но за diff in advance всё-таки был бы признателен.

> +LANG=POSIX

2 legion, inger: возможно, это стоит добавить в
/usr/share/alterator/build/backend3.sh? (вместе с LC_MESSAGES,
наверное)

Или в пример на wiki.

> -renew_services()

Вообще-то эта функция используется в обработке write, вызов там
остался. (переработку disabled/enabled прочитал, по итогам не
могу согласиться -- refresh там "виртуальный" всё равно выходит,
без синхронизации с `chkconfig`)

> +				status)
> +				start)
> +				stop)

Спасибо!

Кстати, просьба имеющим инитскрипты и возможность посмотреть -- 
проверить, что `service $your_service status` выдаёт что-то
вменяемое.  Возможно, здесь будет браться до 80 символов
включительно первой строки, чтобы не организовать UI DoS.

2 rider: hotplug выдаёт невменяемое, хотя чинить его скорее
поздно ;-)

PS: вообще статус планировалось попробовать выдавать в tooltip, 
а не отдельно... хотя... (подумалось про возможно общий на
фронтэнды alterator статус-бар)

> --- /home/mike/src/alterator-services/po/alterator-services.pot	1970-01-01 03:00:00 +0300
> +++ alterator-services-1.0-alt1/po/alterator-services.pot	2006-05-05 15:35:02 +0300

Тоже спасибо.

> --- /home/mike/src/alterator-services/po/ru.po	1970-01-01 03:00:00 +0300
> +++ alterator-services-1.0-alt1/po/ru.po	2006-05-05 15:40:40 +0300
> @@ -0,0 +1,72 @@
> +# SOME DESCRIPTIVE TITLE.
> +# Copyright (C) YEAR THE PACKAGE'S COPYRIGHT HOLDER
> +# This file is distributed under the same license as the PACKAGE package.
> +# FIRST AUTHOR <EMAIL на ADDRESS>, YEAR.

Только эти штуки лучше всё же заполнять.

> +#: ui/services.scm:13
> +msgid "Enabled"
> +msgstr "ВКЫЛ"

:-)

> -(/services file "@UIDIR на services.scm" acc-description "Services")
> +(/services file "@UIDIR на services.scm" acc-description "Services control center")

Не согласен, информативности ни грамма не добавляет, а за пределы
вылазит.  Лаконичнее -- лучше.

> +(document:envelop with-translation _ "alterator-hello")

Ой.  Кто-нить, поясните, откуда hello :-)

> -(label "Available Services")
> +(label (_ " Services control console"))

Кстати.  Стас и компания.  Давайте мож какие-то более-менее общие
рекомендации выработаем, пока модули не разбежались кто в лес,
кто по дрова?  Что написано на кнопках, как переводится "Quit",
какова длина названия модуля и надо ли что-то (и что лучше)
писать в его "заголовке" (вверху фрейма).

> +	 (layout-policy 100 70)

Вот с layout тоже спасибо, что взялись, но на 1600x1200 ситуаций,
когда пришлось растягивать окошко, получилось немало; в принципе,
полноэкранный alterator-standalone в 1024x768 должен помещаться.

Плюс в стартовое окошко у меня правый список не помешается
(при 133 DPI).

> +				  (woo-write '/services 'command 'off 'service (enabled text) )
> +				  (disabled rows (woo-list-names '/services 'kind 'disabled) )
> +				  (enabled rows (woo-list-names '/services 'kind 'enabled) ) 

Там было вполне по причине сделано именно так, как есть (и после
консультации со Стасом и ещё подумав, решил _не_ оптимизировать).

> +(label (_ "Choose runlevel 0-6:") )
> +(document:id CB (combobox 
> +    (items (append `("0" "1" "2" "3" "4" "5" "6"))) current "5" 

Ну вот, я runlevel добывал-добывал, а тут бам -- и "5" :-)
[надо завести http look]

Краткое обсуждение с коллегами привело (через "а где 'все'?")
к мысли о том, что ранлевелы в идеале надо регулировать стопкой
чекбоксов, которые формируют маску для [фильтра] chkconfig; 
плюс кнопки "все/никакие/обычные", где последнее -- [2345].
Когда-нибудь.

>  (frame:buttons-view
> -  (button "Quit" (on-click (document:end))))
> +    (button (_ "Help")
> +    	(on-click
> +	    (splash-message (_ "HELP: Choose runlevel and move service") )

[на заметку: вот сюда бы классно полный гипертекстовый help
подоткнуть, чтоб про ранлевелы сразу и почитать можно было...]

> +    (button (_ "Refresh")
> +	( on-click
> +	    (disabled rows (woo-list-names '/services 'kind 'disabled) )
> +	    (enabled rows (woo-list-names '/services 'kind 'enabled) )			
> +	) 
> +    )

А если chkconfig $service {on,off} по каким-либо причинам не
отработает?  Смысл такой педали?

-- 
 ---- WBR, Michael Shigorin <mike на altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/