From 32b356e2e25fcfe4b5b2d08d746883936a2268e2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Eray=20=C3=96zkural?= Date: Mon, 18 Jul 2005 10:30:55 +0000 Subject: [PATCH] * cok code'lamak mi cok konusmak mi :P --- pisi/install.py | 25 ------------------------- 1 file changed, 25 deletions(-) diff --git a/pisi/install.py b/pisi/install.py index 11b342eb..4731693a 100644 --- a/pisi/install.py +++ b/pisi/install.py @@ -21,34 +21,9 @@ import comariface class InstallError(Exception): pass -# bu fonksiyonun burada işi olmamalı. Silmeyi doğrudan install.py'nin -# içinde yapmamalıyız! -## evet module'un ismi yanlis. ozunde install/upgrade/remove -## islemlerinin cok buyuk farklari yok. hepsine operations.py denip -## su class gereksizliginden kurtulacagim. -# -# operations.py'yi ayrıca yapalım. Tek paket kurulumunu ayrı bir modül -# ile kotarmak daha temiz bir implementation'a götürecektir -# bizi. PisiInstall'un sınıf olması ise bence gereksiz değil, install -# sırasında kullanılan ortak veriyi aşağıda görüyoruz zaten. Ayrı ayrı -# fonksiyonlar ve ilk parametre olarak "package_fn" gibi bir değerin -# gönderilmesi gereksiz bence. Dahası bu PisiBuild ile de uyumlu bir -# implemantation olduğu için daha rahat anlaşılır bir kod tabanı -# oluşturuyor. (baris) - - class Installer: "Installer class, provides install rutines for pisi packages" def __init__(self, package_fname): - # Ben buna karşıyım. Multi-package işini kotarmak yalnızca - # install'da değil hemen hemen tüm işlemlerde halletmemiz - # gereken bir iş. Build, install, remove, upgrade, vs. hemen - # hepsinde multi-package ile ne yapacağımıza karar vermemiz - # gerekiyor. Bu yüzden katman katman düşünelim - # işleri. install.py tek paketi kuran bir modül, build.py'nin - # tek pspec.xml dosyasından peket derlediği gibi. Biz bu - # modülleri kullanarak üst seviye modüller ile çoklu paketler - # işini halletmeliyiz. (baris) self.package = Package(package_fname) self.package.read() self.metadata = self.package.metadata