Bug 18463

Summary: Дублирование пакета spawn-fcgi
Product: Sisyphus Reporter: enp <enp>
Component: trac-spawn-fcgiAssignee: Nobody's working on this, feel free to take it <nobody>
Status: ASSIGNED --- QA Contact: qa-sisyphus
Severity: normal    
Priority: P2 CC: ns, peet, vvk
Version: unstable   
Hardware: all   
OS: Linux   
Attachments:
Description Flags
drop prilileges patch
none
trac initscript none

Description enp 2009-01-08 23:48:41 MSK
# rpm -qf /var/run/trac-spawn-fcgi/trac-spawn-fcgi.pid
trac-spawn-fcgi-0.2-alt1

подкаталог trac-spawn-fcgi - это тоже перебор

и вообще перепилить бы этот пакет, ибо ничего trac-специфичного в бинарнике нет ...
Comment 1 Grigory Batalov 2009-01-09 01:45:19 MSK
pid-файла в пакете нет, он упакован как %ghost.
Подкаталог для разделения полномочий: в него можно разрешить запись псевдопользователю, не имеющему права писать в /var/run .
Пока этим пакетом обслуживается только trac, его название выглядит осмысленным. В дальнейшем можно переименовать.
Comment 2 enp 2009-01-09 23:45:33 MSK
(In reply to comment #1)
> pid-файла в пакете нет, он упакован как %ghost.

спасибо, я не знал о такой фиче rpm :)

> Подкаталог для разделения полномочий: в него можно разрешить запись
> псевдопользователю, не имеющему права писать в /var/run.

полномочия рута сбрасываются после создания pid-файла, поэтому такая фича не нужна, не реализуется же она для большинства других сервисов 

> Пока этим пакетом обслуживается только trac, его название выглядит
> осмысленным. В дальнейшем можно переименовать.

Так ведь есть пакет spawn-fcgi :)

Я допускаю, что когда-то его не было, но сейчас это дублирование. Пакет с бинарником и маном должен быть общим, а spawn-fcgi-trac и spawn-fcgi-php - дополнительными конфигурационными пакетами (с инитскриптом и т.д.) с зависимостью на общий spawn-fcgi. 

Оставшуюся после реорганизации часть обсуждаемого пакета было бы правильнее перенести в trac-fcgi и собирать из одного src.rpm с самим trac. Впрочем, в перспективе trac-fcgi тоже не нужен, т.к. fcgi планируется сделать одной из опций запуска tracd - http://trac.edgewall.org/ticket/6224. Может нам уже приложить этот патч для уменьшения лишних сущностей?

Сам tracd почти можно использовать в инитскрипте, разве что добавить -u/-g ;)

Переоткрываю и добавляю в cc: возможных заинтересованных лиц
Comment 3 Grigory Batalov 2009-01-09 23:55:09 MSK
А, ну значит я отстал от жизни =).
Самое правильное - пропатчить tracd, раз он уже сам умеет форкаться. Правда, я не уверен, что предлагаемый патч создаёт pid-файл. Это нужно проверить.
Comment 4 enp 2009-01-10 00:29:46 MSK
Created attachment 3201 [details]
drop prilileges patch
Comment 5 enp 2009-01-10 00:30:24 MSK
Created attachment 3202 [details]
trac initscript
Comment 6 enp 2009-01-10 00:35:53 MSK
(In reply to comment #3)
> А, ну значит я отстал от жизни =).
> Самое правильное - пропатчить tracd, раз он уже сам умеет форкаться. Правда, я
> не уверен, что предлагаемый патч создаёт pid-файл. Это нужно проверить.

tracd умеет создавать pid-файл даже без патча :)

Я приложил свой патч для снижения полномочий и инитскрипт - у меня они работают. Сможешь (ты или ns@ как последний заливавший) по мотивам этих патчей обновить trac? Я бы допилил у себя в git, но ваши репо неактуальные. Брать из git.alt:/archive ?
Comment 7 Ivan Fedorov 2009-01-10 10:53:59 MSK
(In reply to comment #6)
> Я приложил свой патч для снижения полномочий и инитскрипт - у меня они
> работают. Сможешь (ты или ns@ как последний заливавший) по мотивам этих патчей
> обновить trac? Я бы допилил у себя в git, но ваши репо неактуальные. Брать из
> git.alt:/archive ?

Я залил свой git. Пакет могу подарить любому желающему им заниматься...
Comment 8 enp 2009-01-11 00:01:43 MSK
> Я залил свой git. Пакет могу подарить любому желающему им заниматься...
 
Боюсь, я буду плохим хозяином, т.к. wsgi/fcgi/cgi меня пока слабо интересуют и в порядок я их не приведу :(

То, что мне сейчас нужно, реализовано в http://git.altlinux.org/people/enp/packages/trac.git (особенно крив там патч, может таки исправлю ;) ) + см. http://admdev.blogspot.com/2009/01/trac-quickstart-with-alt-linux.html в качестве use case

Впрочем, если тебе все-таки внапряг вливать это в incoming, то куда мне идти с заявлением о вступлении в ряды Python Development Team?
Comment 9 Ivan Fedorov 2009-01-11 15:44:03 MSK
ну в @python я тебя включил... хотя вообще думаю о её роспуске как несостоятельной сущности...
Comment 10 enp 2009-01-13 13:11:06 MSK
новый trac ушел в incoming, а проблема, ради которой заводился баг, так и осталась ...
Comment 11 Ivan Fedorov 2009-01-13 13:19:15 MSK
ну так реши её, раз уж взялся :)
Comment 12 enp 2009-01-13 20:23:45 MSK
(In reply to comment #11)
> ну так реши её, раз уж взялся :)

она сама собой решится в 0.12 :)

если, конечно, никого не припечет раньше - меня пока отпустило ;)

еще остались вопросы к spawn-fcgi от vvk@, для того он и в cc: - но опять-таки жить это пока никому не мешает
Comment 13 Vladimir V. Kamarzin 2009-01-14 09:44:57 MSK
Вопросы можно в devel@, spawn-fcgi был сделан по инициативе thresh-a
Comment 14 enp 2009-01-14 10:39:47 MSK
(In reply to comment #13)
> Вопросы можно в devel@, spawn-fcgi был сделан по инициативе thresh-a

Да нет, пожалуй, вопросов до тех пор, пока не потребуется сделать что-то еще, кроме trac, работающим из коробки с spawn-fcgi.