[Gelistirici] depo commit mesajları ve merge istekleri hakkında

[Ozan Çağlayan]

Selamlar,

devel depolarında yapılan commitlerde, yapılan değişikliğin ne olduğunun 
anlaşılmasını sağlayacak şekilde ayrıntılı bilgi vermemiz gerekiyor. Bu 
hem diğer geliştiriciler için hem de sürüm yöneticileri için önemli bir 
detay.

Güzel örnek:

Bump to new upsream release.

Includes many bug fixes and new features. These are some of them.

* Fixed an OTB coder bug.
* Fix pixel cache race condition.
* Fix memory leak when XMP profile is corrupt.
* Add support for inline images.

Kötü örnek:

- Version bump.
- Stable release.
- Fixes #3495.


--

Aynı detayın Merge isteği esnasında atılan e-posta'da da tekrarlanması 
yine sürüm yöneticilerinin gidip, commit loglarına bakıp "bu pakette ne 
değişmiş, depoya almalı mıyım yoksa almamalı mıyım, bir şeyi kırıyor mu 
yoksa ufak bir bugfix mi?" gibi sorulara kendi başlarına cevap aramaları 
ihtiyacını ortadan kaldıracak.

Yine bu bilgilerin pspec içindeki <History> bloğunda da düzgünce ifade 
edilmesi, pspec'i açan birinin N tane "Version Bump" yorumu görmesini 
engelleyecektir. pspec.xml'de çoklu satırlı yorum kullanabilirsiniz.

Kötü örnek:

<Update release="27">
   <Date>2008-10-15</Date>
   <Version>0.9.8</Version>
   <Comment>Stable update</Comment>
   <Name>Ozan Çağlayan</Name>
   <Email>ozan at pardus.org.tr</Email>
</Update>

Güzel örnek:

<Update release="55">
   <Date>2008-10-08</Date>
   <Version>7.3</Version>
   <Comment>
   Add fixes for intel driver:
   	* Disable FBC on 945GM and 965GM
   	* Fix ordering of VGA vs. plane disable
   </Comment>
   <Name>Fatih Aşıcı</Name>
   <Email>fatih at pardus.org.tr</Email>
</Update>

--

Son olarak bugzilla'daki bir hatayı kapatan commitlerde hatadan 
bahsedilmesine de özen gösterelim:

Kötü örnek:

<Update release="27">
   <Date>2008-10-15</Date>
   <Version>0.9.8</Version>
   <Comment>Fixes #3945</Comment>
   <Name>Ozan Çağlayan</Name>
   <Email>ozan at pardus.org.tr</Email>
</Update>

Güzel örnek:

..
   <Comment>Remove unneeded symbolic link /var/hede/hodo.so (Fixes 
#3945)</Comment>
..


Saygılar, sevgiler.

-- 

Ozan Çağlayan