9499 2006-05-02 10:48:00 +0400 [FR] some spec improvements 2006-08-23 16:23:54 +0400 1 1 4 Development Sisyphus ss5 unstable all Linux CLOSED FIXED P2 enhancement --- 1 vvk nobody qa-sisyphus oldest_to_newest 37768 0 vvk 2006-05-02 10:48:01 +0400 понадобился мне тут сокс-прокси, решил бэкпортнуть ss5 :) небольшие замечания по спеку: %__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 37769 1 1474 vvk 2006-05-02 10:49:03 +0400 Created attachment 1474 spec diff 37771 2 vvk 2006-05-02 11:20:09 +0400 Гм. с %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) 37775 3 hiddenman 2006-05-02 16:05:13 +0400 (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 наполовину рабочий. 37776 4 hiddenman 2006-05-02 16:10:03 +0400 (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 или чего-то подобного, иначе еще какие-то проблемы были. Надо будет перепроверить :) 39391 5 hiddenman 2006-08-05 19:09:36 +0400 В общем, я уже не помню, про что этот баг, поэтому закрываю :) 39736 6 vvk 2006-08-23 16:23:52 +0400 закрываем 1474 2006-05-02 10:49:03 +0400 2006-05-02 10:49:03 +0400 spec diff ss5.spec.diff text/plain 2185 vvk LS0tIHNzNS5zcGVjLm9yaWcJMjAwNi0wNS0wMiAxMjo0Njo0MyArMDYwMAorKysgc3M1LnNwZWMJ MjAwNi0wNS0wMiAxMjo0Njo0NyArMDYwMApAQCAtNDQsMjYgKzQ0LDI3IEBACiBtYWtlCiAKICVp bnN0YWxsCi0lX19ta2Rpcl9wICVidWlsZHJvb3QlX3N5c2NvbmZkaXIvJW5hbWUKLSVfX21rZGly X3AgJWJ1aWxkcm9vdCVfaW5pdGRpcgotJV9fbWtkaXJfcCAlYnVpbGRyb290JV9zeXNjb25mZGly L3tzeXNjb25maWcscGFtLmR9Ci0lX19ta2Rpcl9wICVidWlsZHJvb3QlX2xpYmRpci8lbmFtZQot JV9fbWtkaXJfcCAlYnVpbGRyb290JV9iaW5kaXIKLSVfX21rZGlyX3AgJWJ1aWxkcm9vdCVfdmFy L3J1bi8lbmFtZQotJV9fbWtkaXJfcCAlYnVpbGRyb290JV9tYW5kaXIve21hbjEsbWFuNX0KLQot ZmluZCAuIC10eXBlIGYgLW5hbWUgKi5zbyAtZXhlYyAlX19pbnN0YWxsIC1tNjQ0ICd7fScgJWJ1 aWxkcm9vdCVfbGliZGlyLyVuYW1lLyBcOwotJV9faW5zdGFsbCAtbTc1NSBzcmMvJW5hbWUgJWJ1 aWxkcm9vdCVfYmluZGlyLwotJV9faW5zdGFsbCAtbTc1NSAlU09VUkNFMSAlYnVpbGRyb290JV9p bml0ZGlyLyVuYW1lCi0lX19pbnN0YWxsIC1tNjQwICVTT1VSQ0UyICVidWlsZHJvb3QvJV9zeXNj b25mZGlyL3BhbS5kLyVuYW1lCi0lX19pbnN0YWxsIC1tNjQwICVTT1VSQ0UzICVidWlsZHJvb3Qv JV9zeXNjb25mZGlyL3N5c2NvbmZpZy8lbmFtZQotJV9faW5zdGFsbCAtbTY0MCAlU09VUkNFNCAl YnVpbGRyb290JV9zeXNjb25mZGlyLyVuYW1lLwotJV9faW5zdGFsbCAtbTY0NCBtYW4vbGludXgv KlwuMVwuKiAlYnVpbGRyb290JV9tYW5kaXIvbWFuMS8KLSVfX2luc3RhbGwgLW02NDQgbWFuL2xp bnV4LypcLjVcLiogJWJ1aWxkcm9vdCVfbWFuZGlyL21hbjUvCitta2RpciAtcCAlYnVpbGRyb290 JV9zeXNjb25mZGlyLyVuYW1lCitta2RpciAtcCAlYnVpbGRyb290JV9pbml0ZGlyCitta2RpciAt cCAlYnVpbGRyb290JV9zeXNjb25mZGlyL3tzeXNjb25maWcscGFtLmR9Citta2RpciAtcCAlYnVp bGRyb290JV9saWJkaXIvJW5hbWUKK21rZGlyIC1wICVidWlsZHJvb3QlX2JpbmRpcgorbWtkaXIg LXAgJWJ1aWxkcm9vdCVfdmFyL3J1bi8lbmFtZQorbWtkaXIgLXAgJWJ1aWxkcm9vdHslX21hbjFk aXIsJV9tYW41ZGlyfQorCitmaW5kIC4gLXR5cGUgZiAtbmFtZSAqLnNvIC1leGVjIGluc3RhbGwg LW02NDQgJ3t9JyAlYnVpbGRyb290JV9saWJkaXIvJW5hbWUvIFw7CitpbnN0YWxsIC1tNzU1IHNy Yy8lbmFtZSAlYnVpbGRyb290JV9iaW5kaXIvCitpbnN0YWxsIC1tNzU1ICVTT1VSQ0UxICVidWls ZHJvb3QlX2luaXRkaXIvJW5hbWUKK2luc3RhbGwgLW02NDAgJVNPVVJDRTIgJWJ1aWxkcm9vdC8l X3N5c2NvbmZkaXIvcGFtLmQvJW5hbWUKK2luc3RhbGwgLW02NDAgJVNPVVJDRTMgJWJ1aWxkcm9v dC8lX3N5c2NvbmZkaXIvc3lzY29uZmlnLyVuYW1lCitpbnN0YWxsIC1tNjQwICVTT1VSQ0U0ICVi dWlsZHJvb3QlX3N5c2NvbmZkaXIvJW5hbWUvCitpbnN0YWxsIC1tNjQ0IG1hbi9saW51eC8qXC4x XC4qICVidWlsZHJvb3QlX21hbjFkaXIvCitpbnN0YWxsIC1tNjQ0IG1hbi9saW51eC8qXC41XC4q ICVidWlsZHJvb3QlX21hbjVkaXIvCiAKICVwcmUKIC91c3Ivc2Jpbi9ncm91cGFkZCAtciAtZiAl c3M1X2dyb3VwCi0vdXNyL3NiaW4vdXNlcmFkZCAtciAtZyAlc3M1X2dyb3VwIC1kICVzczVfaG9t ZSAtTSAtcyAlc3M1X3NoZWxsIC1uIC1jICJTb2NrczUgc2VydmVyIiAlc3M1X3VzZXIgPi9kZXYv bnVsbCAyPiYxIHx8OgorL3Vzci9zYmluL3VzZXJhZGQgLXIgLWcgJXNzNV9ncm91cCAtZCAlc3M1 X2hvbWUgLU0gLXMgJXNzNV9zaGVsbCAtbiBcCisJLWMgIlNvY2tzNSBzZXJ2ZXIiICVzczVfdXNl ciA+L2Rldi9udWxsIDI+JjEgfHw6CiAKICVwb3N0CiAlcG9zdF9zZXJ2aWNlIHNzNQpAQCAtNzks OCArODAsOSBAQAogJWNvbmZpZyhub3JlcGxhY2UpICVfc3lzY29uZmRpci9zeXNjb25maWcvJW5h bWUKICVfaW5pdGRpci8qCiAlX2JpbmRpci8qCi0lYXR0cigwNzUwLCVzczVfdXNlcixyb290KSAl ZGlyICVfdmFyL3J1bi8lbmFtZQotJV9tYW5kaXIvbWFuPy8qCislYXR0cigxNzcwLHJvb3QsJXNz NV9ncm91cCkgJWRpciAlX3Zhci9ydW4vJW5hbWUKKyVfbWFuMWRpci8qCislX21hbjVkaXIvKgog JV9saWJkaXIvJW5hbWUKIAogCg==