| Summary: | [FR] some spec improvements | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Sisyphus | Reporter: | Vladimir V. Kamarzin <vvk> | ||||
| Component: | ss5 | Assignee: | Nobody's working on this, feel free to take it <nobody> | ||||
| Status: | CLOSED FIXED | QA Contact: | qa-sisyphus | ||||
| Severity: | enhancement | ||||||
| Priority: | P2 | ||||||
| Version: | unstable | ||||||
| Hardware: | all | ||||||
| OS: | Linux | ||||||
| Attachments: |
|
||||||
|
Description
Vladimir V. Kamarzin
2006-05-02 10:48:01 MSD
Created attachment 1474 [details]
spec diff
Гм. с %attr(1770,root,%ss5_group) он не работает:
open("/var/run/ss5/ss5.pid", O_WRONLY|O_CREAT|O_TRUNC, 0666) = -1 EACCES
(Permission denied)
write(2, "Can\'t create pid file /var/run/s"..., 43Can't create pid file
/var/run/ss5/ss5.pid
) = 43
close(4)
(In reply to comment #0) > понадобился мне тут сокс-прокси, решил бэкпортнуть ss5 :) > небольшие замечания по спеку: Тот ss5 кривой и косой был, сам по себе, падал в некоторых местах :( Сейчас уже есть свежий и более нормальный, нужно пересобрать просто. Там даже configure наконец-то появился :) > %__mkdir_p и %__install юзать не рекомендуют, т.к. это внутренние макросы rpm > (http://wiki.sisyphus.ru/devel/spectips?v=293). Вот уж не знал, думал, наоборот, так правильнее :( Будем исправлять. > > т.е. кошерно просто mkdir -p и install. > > Так же можно поправить > %_mandir/man1/ --> %_man1dir > %_mandir/man5/ --> %_man5dir Тогда их вроде бы не было еще. Или просто содрал где-то пример :) > косяк: > %attr(0750,%ss5_user,root) %dir %_var/run/%name > > согласно sec policy должно быть нечто вроде > %attr(1770,root,%ss5_group) %dir %_var/run/%name Да, я знаю, там проблема в том, что он делает себе seteuid и форкается в демона, соответственно, потом при выходе он не может удалить из этого каталога pid-файл. Нужно все-таки определиться, кто будет заниматься pid-файлом, стартовые скрипты или сама программа? Я ж для чего и патч ему написал, он pid-файл вообще не умел, да и вообще много гадости внутри содержал, типа hardcoded-пути к файлам. В общем, или сам пока возьми свежий архив у автора или я на днях дособеру его. Текущий ss5 наполовину рабочий. (In reply to comment #2) > Гм. с %attr(1770,root,%ss5_group) он не работает: > > open("/var/run/ss5/ss5.pid", O_WRONLY|O_CREAT|O_TRUNC, 0666) = -1 EACCES > (Permission denied) > write(2, "Can\'t create pid file /var/run/s"..., 43Can't create pid file > /var/run/ss5/ss5.pid > ) = 43 > close(4) Ну да, о чем я и говорил :) Уже не помню подробности, но в ходе экспериментов пришлось создавать файл до seteuid или чего-то подобного, иначе еще какие-то проблемы были. Надо будет перепроверить :) В общем, я уже не помню, про что этот баг, поэтому закрываю :) закрываем |