comment-talk

pisi/install.py

@@ -26,6 +26,15 @@ class InstallError(Exception):
 ## 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)
 def remove(package_name):
     """Remove a goddamn package"""
     ui.info('Removing package ' + package_name)
@@ -43,6 +52,16 @@ class PisiInstall:
         self.ctx = installcontext
         ## bu context fikri multi package installation fikriyle
         ## uyusmuyor ne yazik ki. gitmesi gerek.
+        #
+        # 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.metadata = self.ctx.metadata
         self.files = self.ctx.files
         self.package = self.ctx.package