[Gelistirici] PiSi: run_batch

[Eray Ozkural]

S.Çağlar Onur wrote:
> Selamlar;
>
> Buildfarm'ın arada bir verdiği "argument 2 to map() must support iteration" 
> tarzı hataları debug ederken run_batch ile ilgili bir problem farkettim. 
>
>     if p.returncode:
>         # Non-zero means failed.
>         ctx.ui.error(_('Failed command: %s\n%s') % (cmd, out))
>
> ile çalıştırdığı işlemlerini kendi kontrol ediyor ve dönüş değerine göre hata 
> mesajı veriyordu fakat bunun yapılması gereken yerin run_batch'in içi değil 
> run_batch'i çağıran kimse olmalı diye düşünüyorum, böylece hem çağrı yapan 
> yerlerde dönüş değerini almaya zorlamış (ve doğru bir iş yapmış, yoksa neden 
> return ediyoruz) hemde aynı bilgiyi ekrana/log'a v.s'ye 2 kere basmamış 
> oluyoruz.
>   
E, evet zaten o sekilde yazmistim. Baska turlu kullanilmasi yanlis olur,
eger bir hata olabilecegi bekleniyorsa return durumuna bakilmali.
Alternatif olarak exception atilabilirdi ama o zaman oyle yapmamistim,
cunku bu fonksiyon pratik amacli bir script'ten geliyor, muhtemelen
exception daha dogru olurdu...