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 или чего-то подобного, иначе еще какие-то проблемы были. Надо будет перепроверить :) В общем, я уже не помню, про что этот баг, поэтому закрываю :) закрываем |