[Gelistirici] hal/hal-info

[Ismail Dönmez]

On Wednesday 20 June 2007 22:29:48 Faik Uygur wrote:
> On Wednesday 20 June 2007 22:05:16 Ismail Dönmez wrote:
> > > İsmail branch daki pisi ile dener misin? commit 14400 bu sorunu çözüyor
> > > olmalı. Onur'un dediğine göre sanırım kdelibs sorunu ile aynı. Eski hal
> > > versiyonunu kurup, hal_info'yu eski hal ile Conflict ettirdiğimizde
> > > pisi up ile gelen hal ve hal_info ile sorun olmamalı.
> >
> > Branch pisi'deyim zaten ama hal-info'da Conflicts eksik.
>
> Bir sorunum var ve fikrinizi almak isterim. İki paket birbirleri ile dosya
> değiştirdiğinde doğal olarak ikisinde de verison bump ediliyor.
>
> 14400 nolu commit ile birisine Conflicts ekleyerek çok da yanlış olmayan
> bir şekilde ve basitçe bu sorunu çözdüğümüzü düşünüyorum.
>
> Oradaki mantık şu, eğer kurulum akışında kurulacak/yükseltilecek paketler
> birbirleri ile conflict etmeme testini geçip ardından paketlerden birinin
> kurulumunda bir dosya çakışması ortaya çıkarsa really_conflicts ile bunun
> paket sahibi tarafından bilinerek işaretlenip işaretlenmediği kontrol
> ediliyor. graph koduna conflicts karmaşasını sokmadan, order'da conflicts'i
> dikkate almadan böylece aşabiliyoruz.
>
> Ama paket tek olarak seçilirse, yani kurulu sistemde conflict ettiği ama
> depo da conflict'i çözülmüş paket yükletilmezse. O zaman sistemde Conflict
> olduğu ve kaldırmak gerektiği uyarısını verecekti. Bununla 2 gündür
> uğraşıyorum ve gerçekten gereğinden fazla kod değişikliği getirdiğini
> düşünüyorum.
>
> Düşünce bu conflict'i resolve edecek depo'da bir paket varsa onu sen indir
> denilmese de yine indir demekti. Bu iş çok noktada değişiklik gerektiriyor;
> paket yöneticisinin kullandığı, seçili olan paketlerle gelecek ek paketleri
> hesaplama kodundan, conflict uyarıları veren koda aslında bunlar depodan
> çözülüyor sen bunun için uyarma dedirtmeye kadar çok fazla değişiklik
> gerektiriyor. Uyarma dediğin ama conflict ediyor dediğin değişiklikleri
> tekrar plan_upgrade'den çıkan order listesine ekleyip ignore_dependency
> denilmediyse yine bi plan_upgrade'e soka kadar.... filan filan...
>
> Sorunu çözdüğümüz çok basit bir yamanın üstüne, 200-250 satır kod
> değişikliğini yapmama taraftarıyım. Kullanıcı'nın karar verme işini
> basitleştirmeye çalışırken kodda ciddi değişiklikler yapmak gerekiyor.
> Kod karmaşık bir hal alıyor. Şu haliyle yanlış bir davranış içermediğini
> düşünüyorum.

Aslında ben stable'da bu tür hareketlerin yapılmaması taraftarıyım, 
yapılıyorsa da en temiz çözüm Conflicts eklemek, Gürer'in dediği gibi yakında 
really_conflicts_i_mean_it gibi değişkenlerimiz olacak. Kodu temiz tutmak en 
iyisi bence.

/ismail

-- 
Perfect is the enemy of good
-------------- sonraki bölüm --------------
A non-text attachment was scrubbed...
Name: kullanılamıyor
Type: application/pgp-signature
Size: 827 bytes
Desc: This is a digitally signed message part.
URL: <http://liste.pardus.org.tr/gelistirici/attachments/20070620/924c11c1/attachment-0002.pgp>