Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Fri, 21 Oct 2011 15:05:57 +0300, Serdar Dalgıç ser...@pardus.org.tr wrote: On Friday 21 October 2011 14:45:26 Semen Cirit wrote: On Fri, 21 Oct 2011 14:36:50 +0300, Serdar Dalgıç ser...@pardus.org.tr aç kapa aç kapa yapmayalım diyosun, ok de, NEEDSREVIEW'u sadece bir kere kullanabilicez, ilk aldığı ACK'ten itibaren NEEDSREVIEW'u kaldırıyor ve kullanamıyoruz dediklerinden anladığıma göre (madem bi ACK aldı, o anahtarı kaldırıyoruz diyosun di mi?) Evet boşta kalmışsa feedback bekler durumda olacak. Ama ACKD almış, ACKS'de örneğin onay için feedback bekliyorsa zaten RESOLVED/FIXED commit'i ile feedback gelip gelmediği belli, anında ACKS verebilir diye düşünmüştüm. Dediğini anlamadım. 1) Hata kaydı açıldı, NEEDSREVIEW etiketi konuldu. 2) ACKS ya da ACKD aldı, bunların sırası önemli değil. ikinci ACK için bekliyor 3) ikinci ACK için beklerken herhangi bir KEYWORD kullanıyor muyuz? 4) İkinci ACK'i de aldığında paket depoya aktarılıp review hatası kapanıyor. Birinci ACK ile İkinci ACK arasında herhangi bir keyword kullanmıyoruz. Bu durumda hata ilk açıldığında, zaten herhangi bir ACK almadığı zaman bu pakete review gelmemiş diyebiliyoruz, bunun için ayrı bir KEYWORD kullanmamıza gerek yok diyorum. Ya hiç review edilmemiş, ya da review edilmiş ama feedback bekleniyor diyebiliyoruz. Ok ama farketmez daha önce de dediğim gibi :) İlk ACK'ini almış, ama ikinci ACK için review yapılıp da paket sahibinden eylem beklenen hata kayıtları için herhangi bir keyword'ümüz yok. Benim bahsettiğim REVIEWED keyword'ü burada işe yarıyor. Bu durum için de ACKD, REVIEWED - ACKS, REVIEWED gibi ikili anahtarlar olma durumu var ama olabilir isterseniz. Buna da ok :) Önemli olan paket sahibinden geri dönüş beklendiğinde bunu bir keyword ile belirtmemiz, kimse review yapmadıysa ya da bi review daha bekleniyorsa bunu zaten mevcut KEYWORDlerle anlayabiliyoruz, REVIEWED-ACK gibi bir keyword kullanmamıza gerek yok. Arada - yok , var, ACKD almış ama ACKS bir cevap bekliyorsa, ACKD, REVIEWED durumu oluşabilir. Aynı şekilde ACKS almış ama ACKD bir cevap bekliyorsa ACKS, REVIEWED durumu oluşabilir. -- Semen Cirit TUBITAK/BILGEM - Pardus GNU/Linux http://developer.pardus.org.tr/ ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Mon, Oct 24, 2011 at 09:20:47AM +0300, Semen Cirit wrote: Dediğini anlamadım. 1) Hata kaydı açıldı, NEEDSREVIEW etiketi konuldu. 2) ACKS ya da ACKD aldı, bunların sırası önemli değil. ikinci ACK için bekliyor 3) ikinci ACK için beklerken herhangi bir KEYWORD kullanıyor muyuz? 4) İkinci ACK'i de aldığında paket depoya aktarılıp review hatası kapanıyor. Birinci ACK ile İkinci ACK arasında herhangi bir keyword kullanmıyoruz. Bu durumda hata ilk açıldığında, zaten herhangi bir ACK almadığı zaman bu pakete review gelmemiş diyebiliyoruz, bunun için ayrı bir KEYWORD kullanmamıza gerek yok diyorum. +1 Ya hiç review edilmemiş, ya da review edilmiş ama feedback bekleniyor diyebiliyoruz. Ok ama farketmez daha önce de dediğim gibi :) Arada - yok , var, ACKD almış ama ACKS bir cevap bekliyorsa, ACKD, REVIEWED durumu oluşabilir. Aynı şekilde ACKS almış ama ACKD bir cevap bekliyorsa ACKS, REVIEWED durumu oluşabilir. Bir tane keyword alması, zaten onun ACKD/ACKS bekliyor olması anlamına gelmiyor mu ? Benim anlamadığım neden ayrı bir keyword ihtiyaçı duyuyoruz. İşleri gereksiz yere karmaşıklaştırıyor. Serdar da zaten belirtmiş bu durumu. Şu an düzgün bir şekilde işleyen bir Review süreçimiz var. ACKD ve ACKS keywordları bunları daha iyi bir şekle koydu. Ama NEEEDSREVIEW, REVIEWED gibi kavramlar bir şey katmıyor şu anki sisteme, aksine karmaşık hale getiriyor. -- Fatih Arslan ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Mon, 24 Oct 2011 09:50:45 +0300, Fatih Arslan fars...@pardus.org.tr wrote: On Mon, Oct 24, 2011 at 09:20:47AM +0300, Semen Cirit wrote: Dediğini anlamadım. 1) Hata kaydı açıldı, NEEDSREVIEW etiketi konuldu. 2) ACKS ya da ACKD aldı, bunların sırası önemli değil. ikinci ACK için bekliyor 3) ikinci ACK için beklerken herhangi bir KEYWORD kullanıyor muyuz? 4) İkinci ACK'i de aldığında paket depoya aktarılıp review hatası kapanıyor. Birinci ACK ile İkinci ACK arasında herhangi bir keyword kullanmıyoruz. Bu durumda hata ilk açıldığında, zaten herhangi bir ACK almadığı zaman bu pakete review gelmemiş diyebiliyoruz, bunun için ayrı bir KEYWORD kullanmamıza gerek yok diyorum. +1 Ya hiç review edilmemiş, ya da review edilmiş ama feedback bekleniyor diyebiliyoruz. Ok ama farketmez daha önce de dediğim gibi :) Arada - yok , var, ACKD almış ama ACKS bir cevap bekliyorsa, ACKD, REVIEWED durumu oluşabilir. Aynı şekilde ACKS almış ama ACKD bir cevap bekliyorsa ACKS, REVIEWED durumu oluşabilir. Bir tane keyword alması, zaten onun ACKD/ACKS bekliyor olması anlamına gelmiyor mu ? Benim anlamadığım neden ayrı bir keyword ihtiyaçı duyuyoruz. İşleri gereksiz yere karmaşıklaştırıyor. Serdar da zaten belirtmiş bu durumu. Şu an düzgün bir şekilde işleyen bir Review süreçimiz var. ACKD ve ACKS keywordları bunları daha iyi bir şekle koydu. Ama NEEEDSREVIEW, REVIEWED gibi kavramlar bir şey katmıyor şu anki sisteme, aksine karmaşık hale getiriyor. Bunu kullanmak istememizi Serdar da ben de anlatmıştık sanırım, tüm review hatalarına baktığımızda, hiç review edilmemişler ile review edilip feedback bekleyenler birbirinden ayrılamıyor. Sen işini zamanında, düzgün yapıyor, cevap geldiği gibi düzeltip, commit ediyor olabilirsin ama 3,4 aydır feedback bekleyen raporlar var ve bunlar yeni gelen review istekleri ile aynı havuz da toplanıyor. Bir birbirinden ayırmak güzel olur, ama bir süreliğine bu şekilde kalsın, ACKD, ACKS'ye alışalım sonra bakarız, fazla ısrar etmeyeyim :) -- Semen Cirit TUBITAK/BILGEM - Pardus GNU/Linux http://developer.pardus.org.tr/ ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Monday 24 October 2011 09:54:56 Semen Cirit wrote: On Mon, 24 Oct 2011 09:50:45 +0300, Fatih Arslan fars...@pardus.org.tr Bir tane keyword alması, zaten onun ACKD/ACKS bekliyor olması anlamına gelmiyor mu ? Benim anlamadığım neden ayrı bir keyword ihtiyaçı duyuyoruz. İşleri gereksiz yere karmaşıklaştırıyor. Serdar da zaten belirtmiş bu durumu. Şu an düzgün bir şekilde işleyen bir Review süreçimiz var. ACKD ve ACKS keywordları bunları daha iyi bir şekle koydu. Ama NEEEDSREVIEW, REVIEWED gibi kavramlar bir şey katmıyor şu anki sisteme, aksine karmaşık hale getiriyor. Bunu kullanmak istememizi Serdar da ben de anlatmıştık sanırım, tüm review hatalarına baktığımızda, hiç review edilmemişler ile review edilip feedback bekleyenler birbirinden ayrılamıyor. Sen işini zamanında, düzgün yapıyor, cevap geldiği gibi düzeltip, commit ediyor olabilirsin ama 3,4 aydır feedback bekleyen raporlar var ve bunlar yeni gelen review istekleri ile aynı havuz da toplanıyor. Bir birbirinden ayırmak güzel olur, ama bir süreliğine bu şekilde kalsın, ACKD, ACKS'ye alışalım sonra bakarız, fazla ısrar etmeyeyim :) Problem ısrar edip etmemek değil semen, kullanışlı olmayan bir şeyi öneriyorsun, Fatih'le ben de neden kullanışlı olmadığını anlatmaya çalışıyoruz. Hiç review edilmemiş paketle geliştiricisinden geri dönüş bekleyen paketler ayrı kavramlar. Hiç review edilmemiş paketler için keyword kullanmamızın hiç bir tutarlı yanı yok, herhangi bir keyword'ü olmayan paket hiç review edilmemiştir, bu kadar basit. Buna NEEDSREVIEW diye ek bir keyword getirmek gereksiz ve kullanışsız bir şey. Yeni review'a alınan paketin NEEDSREVIEW ile işaretlenmesi kimsenin işini kolaylaştırmayacak ya da kimseye yapması gereken bir işi hatırlatmayacak. Geliştiricisinden cevap bekleyen paketler için de elle dürtmeye devam edelim, Fatih'le aynı fikirdeyim. -- Serdar Dalgıç ser...@pardus.org.tr ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Mon, Oct 24, 2011 at 09:54:56AM +0300, Semen Cirit wrote: Bunu kullanmak istememizi Serdar da ben de anlatmıştık sanırım, tüm review hatalarına baktığımızda, hiç review edilmemişler ACKD/ACKS ibaresi olmayanlar hiç review edilmemiştir ile review edilip feedback bekleyenler birbirinden ayrılamıyor. ACKD/ACKS ikilisinden birini almış olan da review edilmiş, başka bir geri dönüşüm beklendiği anlamına geliyor. Sen işini zamanında, düzgün yapıyor, cevap geldiği gibi düzeltip, commit ediyor olabilirsin ama 3,4 aydır feedback bekleyen raporlar var ve bunlar yeni gelen review istekleri ile aynı havuz da toplanıyor. Evet haklısın, ama feedback bekliyor diye yeni bir keyword oluşturup bunu sisteme dahil etsen bile, insanlara bir önceki durumdan ne gibi bir farklılık getireceksin ki ? Yapmayan yine yapmayacak, yapan yine yapacak. Değişen bir şey yok. Bir birbirinden ayırmak güzel olur, ama bir süreliğine bu şekilde kalsın, ACKD, ACKS'ye alışalım sonra bakarız, fazla ısrar etmeyeyim :) Bence de, bu işleri yavaş yavaş yapalım. Bir süre sonra bir sürü ayrıntıda boğulup hiç bir iş yapılamaması çıkmazına girmek istemiyorum açıkcası. Zamanında Google tek bir arama kutusu koyduğunda bununla iş yapılmaz diyenler çıkmıştı, ama gayet de iyi iş yapılıyor. Tek istediğim sade ve işlevsel bir sistem :) Kolay gelsin -- Fatih Arslan ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Tue, 18 Oct 2011 09:40:02 +0300, Semen Cirit sci...@pardus.org.tr wrote: On Tue, 18 Oct 2011 09:38:33 +0300, Semen Cirit sci...@pardus.org.tr wrote: On Tue, 18 Oct 2011 08:55:19 +0300, Fatih Arslan fars...@pardus.org.tr wrote: On Tue, Oct 18, 2011 at 08:40:21AM +0300, Semen Cirit wrote: developer.pardus.org.tr'de güncelledim, bugzillaya da ekliyorum. ACKD ve ACKS yerine ACK-D ve ACK-S kullansak ? Okunması cidden zor oluyor diğer türlü. Çizgi sanırım olmuyormuş. ACKD ve ACKS olarak bıraktım. Bir de şu durum da sanırım açığa çıkarılabilir, review istemiş ama hiç feedback almamışlarla, almış ama cevap vermemiş olanları da ayırabiliriz. Review isteyen NEEDSREVIEW ekler, review eden bu anahtarı kaldırır, böylece geri dönüş bekleyenler ile hiç review işlemi başlatılmamış olanlar birbirinden ayrılabilir. Ping? Ne yapalım? -- Semen Cirit TUBITAK/BILGEM - Pardus GNU/Linux http://developer.pardus.org.tr/ ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Friday 21 October 2011 12:37:17 Semen Cirit wrote: Bir de şu durum da sanırım açığa çıkarılabilir, review istemiş ama hiç feedback almamışlarla, almış ama cevap vermemiş olanları da ayırabiliriz. Review isteyen NEEDSREVIEW ekler, review eden bu anahtarı kaldırır, böylece geri dönüş bekleyenler ile hiç review işlemi başlatılmamış olanlar birbirinden ayrılabilir. Ping? Ne yapalım? Bu bana biraz karışık geliyor, şimdi direk review'a koyduğumuz pakete NEEDSREVIEW mu ekleyeceğiz? Sonra iki ACK alması gerekiyor paketin depoya girebilmesi için, ilk ACK'i aldıktan sonra NEEDSREVIEW'u kaldırdık, ikinci ACK için de gözden geçirme bekleyecek, o sırada ne zaman NEEDSREVIEW'u koyacağız? Bence ACKS/ACKD mekanizması yeterli gözüküyor, hiç review almamışlarda hiç bi keyword olmaz, tek review alanda ACKS ya da ACKD keyword'ü olur, takibi bu şekilde yapılabilir. Benim bi önerim şu olacak, herhangi bir pakete review yapıldıktan sonra geliştiricisinin o review'a uygun değişiklikleri yapmasını beklerken REVIEWED, REVIEWDONE gibi bir keyword koyabiliriz, bu sayede o paketin review'unun yapıldığını ve ACK vermek için geliştiricinin gerekli değişiklikleri yapmasını beklediğimizi belirtebiliriz. -- Serdar Dalgıç ser...@pardus.org.tr ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Fri, 21 Oct 2011 13:30:17 +0300 Serdar Dalgıç ser...@pardus.org.tr wrote: Benim bi önerim şu olacak, herhangi bir pakete review yapıldıktan sonra geliştiricisinin o review'a uygun değişiklikleri yapmasını beklerken REVIEWED, REVIEWDONE gibi bir keyword koyabiliriz, bu sayede o paketin review'unun yapıldığını ve ACK vermek için geliştiricinin gerekli değişiklikleri yapmasını beklediğimizi belirtebiliriz. Review süreçinde yorum yapıldığından anlaşılmıyor mu zaten ? Doğal mekanizmalar varken (örneğin review yorumun yazılması, ping atılması,vs..) kendimize ayrı bir mekanizma yaratmanın bir avantajını göremiyorum. -- Fatih Arslan fars...@pardus.org.tr ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Friday 21 October 2011 13:41:46 Fatih Arslan wrote: On Fri, 21 Oct 2011 13:30:17 +0300 Serdar Dalgıç ser...@pardus.org.tr wrote: Benim bi önerim şu olacak, herhangi bir pakete review yapıldıktan sonra geliştiricisinin o review'a uygun değişiklikleri yapmasını beklerken REVIEWED, REVIEWDONE gibi bir keyword koyabiliriz, bu sayede o paketin review'unun yapıldığını ve ACK vermek için geliştiricinin gerekli değişiklikleri yapmasını beklediğimizi belirtebiliriz. Review süreçinde yorum yapıldığından anlaşılmıyor mu zaten ? Doğal mekanizmalar varken (örneğin review yorumun yazılması, ping atılması,vs..) kendimize ayrı bir mekanizma yaratmanın bir avantajını göremiyorum. Direk pakete baktığın zaman anlaşılıyor, ama review'daki paketlerin listesini çıkarıp keywordlere baktığında hangi paketin paketçiden, hangi paketin diğer geliştiricilerden bir eylem beklediğini anlayamıyorsun. Bu verdiğim örnek illa olsun diye demiyorum zaten, yararı olabilir, ama size mantıklı gelmiyorsa göz ardı edebiliriz, çok ısrarcı değilim :) -- Serdar Dalgıç ser...@pardus.org.tr ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Fri, 21 Oct 2011 13:30:17 +0300, Serdar Dalgıç ser...@pardus.org.tr wrote: On Friday 21 October 2011 12:37:17 Semen Cirit wrote: Bir de şu durum da sanırım açığa çıkarılabilir, review istemiş ama hiç feedback almamışlarla, almış ama cevap vermemiş olanları da ayırabiliriz. Review isteyen NEEDSREVIEW ekler, review eden bu anahtarı kaldırır, böylece geri dönüş bekleyenler ile hiç review işlemi başlatılmamış olanlar birbirinden ayrılabilir. Ping? Ne yapalım? Bu bana biraz karışık geliyor, şimdi direk review'a koyduğumuz pakete NEEDSREVIEW mu ekleyeceğiz? Sonra iki ACK alması gerekiyor paketin depoya girebilmesi için, ilk ACK'i aldıktan sonra NEEDSREVIEW'u kaldırdık, ikinci ACK için de gözden geçirme bekleyecek, o sırada ne zaman NEEDSREVIEW'u koyacağız? Bence ACKS/ACKD mekanizması yeterli gözüküyor, hiç review almamışlarda hiç bi keyword olmaz, tek review alanda ACKS ya da ACKD keyword'ü olur, takibi bu şekilde yapılabilir. Benim bi önerim şu olacak, herhangi bir pakete review yapıldıktan sonra geliştiricisinin o review'a uygun değişiklikleri yapmasını beklerken REVIEWED, REVIEWDONE gibi bir keyword koyabiliriz, bu sayede o paketin review'unun yapıldığını ve ACK vermek için geliştiricinin gerekli değişiklikleri yapmasını beklediğimizi belirtebiliriz. İşte benim de demek istediğim böyle bir sıkıntımızın olması idi, hiç review edilmemişlerle geliştiricisinden feedback bekleyenler belli olmuyor. Sen feedback bekleyenlere bir anahtar koyalım diyorsun, ben de hiç review edilmemişlere :) Benim söylediğimde hiç anahtarı olmayanlar geliştiricisinden feedback bekleyenler olacak, senin senaryonda da hiç anahtarı olmayanlar review bekleyenler olacak. Fakat seninki biraz daha anahtarı ekle koy, ekle koy şeklinde bir süreç yaratıyor, benim dediğim senaryoda sadece en başta bir kez NEEDSREVIEW denilip, ACKD almışsa ACKD, ACKS almışsa ACKS denniliyor, hiç anahtarı olmayanlar feedback bekler durumda oluyor. -- Semen Cirit TUBITAK/BILGEM - Pardus GNU/Linux http://developer.pardus.org.tr/ ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Friday 21 October 2011 14:04:03 Semen Cirit wrote: On Fri, 21 Oct 2011 13:30:17 +0300, Serdar Dalgıç ser...@pardus.org.tr İşte benim de demek istediğim böyle bir sıkıntımızın olması idi, hiç review edilmemişlerle geliştiricisinden feedback bekleyenler belli olmuyor. Sen feedback bekleyenlere bir anahtar koyalım diyorsun, ben de hiç review edilmemişlere :) Benim söylediğimde hiç anahtarı olmayanlar geliştiricisinden feedback bekleyenler olacak, senin senaryonda da hiç anahtarı olmayanlar review bekleyenler olacak. Fakat seninki biraz daha anahtarı ekle koy, ekle koy şeklinde bir süreç yaratıyor, benim dediğim senaryoda sadece en başta bir kez NEEDSREVIEW denilip, ACKD almışsa ACKD, ACKS almışsa ACKS denniliyor, hiç anahtarı olmayanlar feedback bekler durumda oluyor. Hiç review edilmemişlere niye anahtar koyuyoruz ki? ACKS ya da ACKD yoksa zaten review edilmemiştir. hata açtığım gibi anahtar koymak bana pek mantıklı gelmiyor. aç kapa aç kapa yapmayalım diyosun, ok de, NEEDSREVIEW'u sadece bir kere kullanabilicez, ilk aldığı ACK'ten itibaren NEEDSREVIEW'u kaldırıyor ve kullanamıyoruz dediklerinden anladığıma göre (madem bi ACK aldı, o anahtarı kaldırıyoruz diyosun di mi?) İlk ACK'ini almış, ama ikinci ACK için review yapılıp da paket sahibinden eylem beklenen hata kayıtları için herhangi bir keyword'ümüz yok. Benim bahsettiğim REVIEWED keyword'ü burada işe yarıyor. -- Serdar Dalgıç ser...@pardus.org.tr ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Fri, 21 Oct 2011 14:36:50 +0300, Serdar Dalgıç ser...@pardus.org.tr wrote: On Friday 21 October 2011 14:04:03 Semen Cirit wrote: On Fri, 21 Oct 2011 13:30:17 +0300, Serdar Dalgıç ser...@pardus.org.tr İşte benim de demek istediğim böyle bir sıkıntımızın olması idi, hiç review edilmemişlerle geliştiricisinden feedback bekleyenler belli olmuyor. Sen feedback bekleyenlere bir anahtar koyalım diyorsun, ben de hiç review edilmemişlere :) Benim söylediğimde hiç anahtarı olmayanlar geliştiricisinden feedback bekleyenler olacak, senin senaryonda da hiç anahtarı olmayanlar review bekleyenler olacak. Fakat seninki biraz daha anahtarı ekle koy, ekle koy şeklinde bir süreç yaratıyor, benim dediğim senaryoda sadece en başta bir kez NEEDSREVIEW denilip, ACKD almışsa ACKD, ACKS almışsa ACKS denniliyor, hiç anahtarı olmayanlar feedback bekler durumda oluyor. Hiç review edilmemişlere niye anahtar koyuyoruz ki? ACKS ya da ACKD yoksa zaten review edilmemiştir. hata açtığım gibi anahtar koymak bana pek mantıklı gelmiyor. aç kapa aç kapa yapmayalım diyosun, ok de, NEEDSREVIEW'u sadece bir kere kullanabilicez, ilk aldığı ACK'ten itibaren NEEDSREVIEW'u kaldırıyor ve kullanamıyoruz dediklerinden anladığıma göre (madem bi ACK aldı, o anahtarı kaldırıyoruz diyosun di mi?) Evet boşta kalmışsa feedback bekler durumda olacak. Ama ACKD almış, ACKS'de örneğin onay için feedback bekliyorsa zaten RESOLVED/FIXED commit'i ile feedback gelip gelmediği belli, anında ACKS verebilir diye düşünmüştüm. İlk ACK'ini almış, ama ikinci ACK için review yapılıp da paket sahibinden eylem beklenen hata kayıtları için herhangi bir keyword'ümüz yok. Benim bahsettiğim REVIEWED keyword'ü burada işe yarıyor. Bu durum için de ACKD, REVIEWED - ACKS, REVIEWED gibi ikili anahtarlar olma durumu var ama olabilir isterseniz. Buna da ok :) -- Semen Cirit TUBITAK/BILGEM - Pardus GNU/Linux http://developer.pardus.org.tr/ ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Friday 21 October 2011 14:45:26 Semen Cirit wrote: On Fri, 21 Oct 2011 14:36:50 +0300, Serdar Dalgıç ser...@pardus.org.tr aç kapa aç kapa yapmayalım diyosun, ok de, NEEDSREVIEW'u sadece bir kere kullanabilicez, ilk aldığı ACK'ten itibaren NEEDSREVIEW'u kaldırıyor ve kullanamıyoruz dediklerinden anladığıma göre (madem bi ACK aldı, o anahtarı kaldırıyoruz diyosun di mi?) Evet boşta kalmışsa feedback bekler durumda olacak. Ama ACKD almış, ACKS'de örneğin onay için feedback bekliyorsa zaten RESOLVED/FIXED commit'i ile feedback gelip gelmediği belli, anında ACKS verebilir diye düşünmüştüm. Dediğini anlamadım. 1) Hata kaydı açıldı, NEEDSREVIEW etiketi konuldu. 2) ACKS ya da ACKD aldı, bunların sırası önemli değil. ikinci ACK için bekliyor 3) ikinci ACK için beklerken herhangi bir KEYWORD kullanıyor muyuz? 4) İkinci ACK'i de aldığında paket depoya aktarılıp review hatası kapanıyor. Birinci ACK ile İkinci ACK arasında herhangi bir keyword kullanmıyoruz. Bu durumda hata ilk açıldığında, zaten herhangi bir ACK almadığı zaman bu pakete review gelmemiş diyebiliyoruz, bunun için ayrı bir KEYWORD kullanmamıza gerek yok diyorum. İlk ACK'ini almış, ama ikinci ACK için review yapılıp da paket sahibinden eylem beklenen hata kayıtları için herhangi bir keyword'ümüz yok. Benim bahsettiğim REVIEWED keyword'ü burada işe yarıyor. Bu durum için de ACKD, REVIEWED - ACKS, REVIEWED gibi ikili anahtarlar olma durumu var ama olabilir isterseniz. Buna da ok :) Önemli olan paket sahibinden geri dönüş beklendiğinde bunu bir keyword ile belirtmemiz, kimse review yapmadıysa ya da bi review daha bekleniyorsa bunu zaten mevcut KEYWORDlerle anlayabiliyoruz, REVIEWED-ACK gibi bir keyword kullanmamıza gerek yok. -- Serdar Dalgıç ser...@pardus.org.tr ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Tue, 18 Oct 2011 08:55:19 +0300, Fatih Arslan fars...@pardus.org.tr wrote: On Tue, Oct 18, 2011 at 08:40:21AM +0300, Semen Cirit wrote: developer.pardus.org.tr'de güncelledim, bugzillaya da ekliyorum. ACKD ve ACKS yerine ACK-D ve ACK-S kullansak ? Okunması cidden zor oluyor diğer türlü. Çizgi sanırım olmuyormuş. ACKD ve ACKS olarak bıraktım. Bir de şu durumda sanırım açığa çıkarılabilir, review istemiş ama hiç feedback almamışlarla, almış ama cevap vermemiş olanları da ayırabiliriz. Review isteyen NEEDSREVIEW ekler, review eden bu anahtarı kaldırır, böylece geri dönüş bekleyenler ile hiç review işlemi başlatılmamış olanlar birbirinden ayrılabilir. http://bugs.pardus.org.tr/buglist.cgi?keywords=ACKSquery_format=advancedkeywords_type=allwordsbug_status=NEWbug_status=ASSIGNEDbug_status=REOPENEDproduct=Paket%20G%C3%B6zden%20Ge%C3%A7irme%20%2F%20Package%20Reviewclassification=Da%C4%9F%C4%B1t%C4%B1m%20S%C3%BCre%C3%A7leri%20%2F%20Distribution%20Process http://bugs.pardus.org.tr/buglist.cgi?keywords=ACKDquery_format=advancedkeywords_type=allwordsbug_status=NEWbug_status=ASSIGNEDbug_status=REOPENEDproduct=Paket%20G%C3%B6zden%20Ge%C3%A7irme%20%2F%20Package%20Reviewclassification=Da%C4%9F%C4%B1t%C4%B1m%20S%C3%BCre%C3%A7leri%20%2F%20Distribution%20Process http://bugs.pardus.org.tr/buglist.cgi?keywords=ACKD%2CACKSquery_format=advancedkeywords_type=allwordsbug_status=NEWbug_status=ASSIGNEDbug_status=REOPENEDproduct=Paket%20G%C3%B6zden%20Ge%C3%A7irme%20%2F%20Package%20Reviewclassification=Da%C4%9F%C4%B1t%C4%B1m%20S%C3%BCre%C3%A7leri%20%2F%20Distribution%20Process -- Semen Cirit TUBITAK/BILGEM - Pardus GNU/Linux http://developer.pardus.org.tr/ ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Tue, 18 Oct 2011 09:38:33 +0300, Semen Cirit sci...@pardus.org.tr wrote: On Tue, 18 Oct 2011 08:55:19 +0300, Fatih Arslan fars...@pardus.org.tr wrote: On Tue, Oct 18, 2011 at 08:40:21AM +0300, Semen Cirit wrote: developer.pardus.org.tr'de güncelledim, bugzillaya da ekliyorum. ACKD ve ACKS yerine ACK-D ve ACK-S kullansak ? Okunması cidden zor oluyor diğer türlü. Çizgi sanırım olmuyormuş. ACKD ve ACKS olarak bıraktım. Bir de şu durumda sanırım açığa çıkarılabilir, review istemiş ama hiç feedback almamışlarla, almış ama cevap vermemiş olanları da ayırabiliriz. Review isteyen NEEDSREVIEW ekler, review eden bu anahtarı kaldırır, böylece geri dönüş bekleyenler ile hiç review işlemi başlatılmamış olanlar birbirinden ayrılabilir. http://bugs.pardus.org.tr/buglist.cgi?keywords=ACKSquery_format=advancedkeywords_type=allwordsbug_status=NEWbug_status=ASSIGNEDbug_status=REOPENEDproduct=Paket%20G%C3%B6zden%20Ge%C3%A7irme%20%2F%20Package%20Reviewclassification=Da%C4%9F%C4%B1t%C4%B1m%20S%C3%BCre%C3%A7leri%20%2F%20Distribution%20Process http://bugs.pardus.org.tr/buglist.cgi?keywords=ACKDquery_format=advancedkeywords_type=allwordsbug_status=NEWbug_status=ASSIGNEDbug_status=REOPENEDproduct=Paket%20G%C3%B6zden%20Ge%C3%A7irme%20%2F%20Package%20Reviewclassification=Da%C4%9F%C4%B1t%C4%B1m%20S%C3%BCre%C3%A7leri%20%2F%20Distribution%20Process http://bugs.pardus.org.tr/buglist.cgi?keywords=ACKD%2CACKSquery_format=advancedkeywords_type=allwordsbug_status=NEWbug_status=ASSIGNEDbug_status=REOPENEDproduct=Paket%20G%C3%B6zden%20Ge%C3%A7irme%20%2F%20Package%20Reviewclassification=Da%C4%9F%C4%B1t%C4%B1m%20S%C3%BCre%C3%A7leri%20%2F%20Distribution%20Process Söylemeyi unuttum, gözümden kaçmış olanlar olabilir, bunları fark ederseniz, düzeltebilirseniz sevinirim. -- Semen Cirit TUBITAK/BILGEM - Pardus GNU/Linux http://developer.pardus.org.tr/ ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Fri, 14 Oct 2011 15:05:03 +0300, Renan Cakirerk re...@pardus.org.tr wrote: On 10/14/2011 01:27 PM, Serdar Dalgic wrote: On Fri, 14 Oct 2011 08:47:02 +0300, Fatih Arslan fars...@pardus.org.tr wrote: On Fri, Oct 14, 2011 at 08:35:23AM +0300, Semen Cirit wrote: Suffix yerine Prefix kullanmamız ayırtetme kısmında yardımcı olur. Bol bol review okuyanlar bir süre sonra ACKS, ACKD, ACK gibi kelimelerin arasında boğulacaktır diye düşünüyorum. Bunun yerine ilk bakışta anlaşılacak kelimeler seçilmesi daha mantıklı. Örneğin S-Ack, Svisor-ack, Super-ack D-Ack, Dev-ack Önek kullanmaya benden de +1, son ekleri ayrıştırmak sıkıntı olabilir. Suffix daha iyi diyorum. Hem ilerisi için olası bir parsing açısından hem de okunaklılık açısından daha iyi diye düşünüyorum. Şöyle ki; The keyword is ACK, The type is S or D Ya da ACK of type D or S. gibi okuyorum. ACK-D ve ACK-S'e +1 diyorum o vakit. Ne diyelim? Bu gün içerisinde anahtarı ekleyebilir ve gözden geçirip hataları bu şekilde güncelleyebilirim. -- Semen Cirit TUBITAK/BILGEM - Pardus GNU/Linux http://developer.pardus.org.tr/ ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Monday 17 October 2011 14:49:39 Semen Cirit şunları yazmıştı: Suffix daha iyi diyorum. Hem ilerisi için olası bir parsing açısından hem de okunaklılık açısından daha iyi diye düşünüyorum. Şöyle ki; The keyword is ACK, The type is S or D Ya da ACK of type D or S. gibi okuyorum. ACK-D ve ACK-S'e +1 diyorum o vakit. Ne diyelim? Bu gün içerisinde anahtarı ekleyebilir ve gözden geçirip hataları bu şekilde güncelleyebilirim. ACK-D ya da ACKD,ACKS ikilisi iyi gibi gözüküyor, ben de Renan gibi okuyorum. Daha fazla üzerinde konuşmaya gerek yok, sen seç birini uygulayalım. -- Gökmen Göksel ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Mon, 17 Oct 2011 16:32:56 +0300, Gökmen Göksel gok...@pardus.org.tr wrote: On Monday 17 October 2011 14:49:39 Semen Cirit şunları yazmıştı: Suffix daha iyi diyorum. Hem ilerisi için olası bir parsing açısından hem de okunaklılık açısından daha iyi diye düşünüyorum. Şöyle ki; The keyword is ACK, The type is S or D Ya da ACK of type D or S. gibi okuyorum. ACK-D ve ACK-S'e +1 diyorum o vakit. Ne diyelim? Bu gün içerisinde anahtarı ekleyebilir ve gözden geçirip hataları bu şekilde güncelleyebilirim. ACK-D ya da ACKD,ACKS ikilisi iyi gibi gözüküyor, ben de Renan gibi okuyorum. Daha fazla üzerinde konuşmaya gerek yok, sen seç birini uygulayalım. developer.pardus.org.tr'de güncelledim, bugzillaya da ekliyorum. -- Semen Cirit TUBITAK/BILGEM - Pardus GNU/Linux http://developer.pardus.org.tr/ ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Tue, Oct 18, 2011 at 08:40:21AM +0300, Semen Cirit wrote: developer.pardus.org.tr'de güncelledim, bugzillaya da ekliyorum. ACKD ve ACKS yerine ACK-D ve ACK-S kullansak ? Okunması cidden zor oluyor diğer türlü. -- Fatih Arslan ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Fri, 14 Oct 2011 08:47:02 +0300, Fatih Arslan fars...@pardus.org.tr wrote: On Fri, Oct 14, 2011 at 08:35:23AM +0300, Semen Cirit wrote: Suffix yerine Prefix kullanmamız ayırtetme kısmında yardımcı olur. Bol bol review okuyanlar bir süre sonra ACKS, ACKD, ACK gibi kelimelerin arasında boğulacaktır diye düşünüyorum. Bunun yerine ilk bakışta anlaşılacak kelimeler seçilmesi daha mantıklı. Örneğin S-Ack, Svisor-ack, Super-ack D-Ack, Dev-ack Önek kullanmaya benden de +1, son ekleri ayrıştırmak sıkıntı olabilir. -- - Serdar Dalgic TUBITAK/BILGEM - Pardus GNU/Linux http://www.pardus.org.tr/eng ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On 10/14/2011 01:27 PM, Serdar Dalgic wrote: On Fri, 14 Oct 2011 08:47:02 +0300, Fatih Arslan fars...@pardus.org.tr wrote: On Fri, Oct 14, 2011 at 08:35:23AM +0300, Semen Cirit wrote: Suffix yerine Prefix kullanmamız ayırtetme kısmında yardımcı olur. Bol bol review okuyanlar bir süre sonra ACKS, ACKD, ACK gibi kelimelerin arasında boğulacaktır diye düşünüyorum. Bunun yerine ilk bakışta anlaşılacak kelimeler seçilmesi daha mantıklı. Örneğin S-Ack, Svisor-ack, Super-ack D-Ack, Dev-ack Önek kullanmaya benden de +1, son ekleri ayrıştırmak sıkıntı olabilir. Suffix daha iyi diyorum. Hem ilerisi için olası bir parsing açısından hem de okunaklılık açısından daha iyi diye düşünüyorum. Şöyle ki; The keyword is ACK, The type is S or D Ya da ACK of type D or S. gibi okuyorum. ACK-D ve ACK-S'e +1 diyorum o vakit. -- Renan Çakırerk | Pardus Developer ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici
Re: [Gelistirici] paket gözden geçirmede küçük değişiklik
On Fri, Oct 14, 2011 at 08:35:23AM +0300, Semen Cirit wrote: Selamlar, Merhaba, Paket gözden geçirmelerinde hangi paketler hem bileşen sorumlusundan, hem geliştiriciden ACK aldı, hangisi sadece herhangi birinden ACK aldı gibi aramaları kolayca yapamıyoruz, bunun için küçük bir değişiklik yapmanın iyi olabileceğini düşündüm. Bileşen sorumlusundan ACK alan ACK-S anahtarı, geliştiriciden ACK alan ise ACK-D anahtarı gibi bir anahtar verse, (Aynı anda iki keyword kullanılabiliyor) bunları takip edebiliyor olacağız. Aynı zamanda yanlışlıkla bileşen sorumlusu tarafından ACK alınmamış paketlerin de merge edilmesini de daha kolay takip ediyor olacağız. Ne dersiniz? Anahtarlar hakkında farklı fikirler de olabilir tabi :) Suffix yerine Prefix kullanmamız ayırtetme kısmında yardımcı olur. Bol bol review okuyanlar bir süre sonra ACKS, ACKD, ACK gibi kelimelerin arasında boğulacaktır diye düşünüyorum. Bunun yerine ilk bakışta anlaşılacak kelimeler seçilmesi daha mantıklı. Örneğin S-Ack, Svisor-ack, Super-ack D-Ack, Dev-ack gibi .. Yani prefix olsun da, gerisi güzel kelime bulmaya bakar :) -- Fatih Arslan ___ Gelistirici mailing list Gelistirici@pardus.org.tr http://liste.pardus.org.tr/mailman/listinfo/gelistirici