[Gelistirici] fetcher.fetch_url(), file.File.download() ve Bütünlük Kontrolü

[Fatih Aşıcı]

 On Mon, 1 Nov 2010 09:43:59 +0200, Bahadır Kandemir 
 <bahadir at pardus.org.tr> wrote:
> Pazartesi 01 Kasım 2010 günü (saat 09:25:19) Fatih Aşıcı şunları 
> yazmıştı:
>>  Kodun geri kalanı, izlediğimiz kod stiline zaten uymadığı için
>>  şimdilik stil düzeltmelerini es geçiyorum.
>
> Kod stili derken? :) PEP-8 uyumu yok, "file" adında bir modülümüz 
> var,
> "os.path.join" kullanmayın kuralına rağmen ~150 yerde kullanılmış,
> stile uymak
> için ben de öyle yaptım :D

 Elim değdikçe düzeltiyorum modülleri; ama stil düzeltmelerini ayrı bir 
 commit'te
 yapmayı tercih ediyorum.
 
>> >          if sha1sum:
>> >              if (pisi.util.sha1_file(localfile) != newsha1):
>> > +                clean_temporary()
>> >
>> >                  raise Error(_("File integrity of %s 
>> compromised.") %
>> >
>> > uri)
>>
>>  fetcher exception verdiğinde de clean_temporary() çağrılacak
>>  mı/çağrılmalı mı
>>  acaba? Fetcher exception'ları yakalanmıyor download() içinde. Eğer
>>  exception
>>  alınırsa bu code path'e hiç girmeyecek.
>
> Hedef dizin olarak depo dizini kullanıldığı için, geçici dosyaların 
> orada
> durması hoşuma gitmedi. Başka bir dizin kullanılabilir belki bunun 
> için.

 ctx.config.tmp_dir() kullanılabilir belki.