[Paketler] [review] server/www/nginx

[Eren Türkay]

On 08 Jun 2008 Sun 02:14:51 Onur Küçük wrote:
> - /var/log/nginx işlerini niye package.py de yapıyorsun, ve hatta
> os.system le yapıyorsun ? Dizin dosya yaratma işlemleri paketin içinde
> olsa, sadece chown package.py de olsa daha iyi olur

OK. Dizin oluşturma ve touch işlemlerini install()'da hallettim. Package.py'de 
sadece "/usr/bin/chown" çağırılıyor. /usr/bin/chown da gereken yere linkli, 
2007/2008'de sorun olmaması gerek
>
>  - actions.py de değişkenleri tek tek yazmak yerine direkt dictionary
> olarak yazarsan hem daha az yer tutar hem de daha iyi takip edebilirsin

Daha okunaklı olabilir diye düşünmüştüm değişkenlere atayarak ama 
değiştirebilrim tabi. Sanki dict içerisinde olunca daha karışık geliyor :-)

>  - actions.py de statik perl sürümü yazıyor, get.curPERL()
> kullanmalısın

OK.

>  - AdditionalFiles da default pardus page dosyalarının yetkileri yanlış

Apache de 0755 kullandığı için default page dosyalarına 0755 verdim. Burada 
hem apache hem de nginxde hata var sanırım, doğru yetkinin ne olması 
gerekiyor?

>  - Yazılımın lisansı BSD diye biliyorum

Tarball içerisindeki LICENSE dosyasında as-is olarak geçiyor.