Re: [devel] [SCM] packages/zabbix: heads/master

Re: [devel] [SCM] packages/zabbix: heads/master

[Vladimir V. Kamarzin]

>>>>> On 06 Aug 2010 at 22:22 "AA" == Aleksey Avdeev writes:

AA> Update of /people/solo/packages/zabbix.git

AA> commit 409a3805cd321c58db45cdc258920d42ce0fc62c
AA> Author: Aleksey Avdeev <solo@altlinux.ru>
AA> Date:   Fri Aug 6 17:47:54 2010 +0400

AA>      - [ZBX-416,ZBX-986] fixed problem of implicit cast with PostgreSQL 8.3, thanks to Fernando Ike de Oliveira (solo)
    
AA>     Signed-off-by: Aleksey Avdeev <solo@altlinux.ru>


AA> commit c141437c4aa2ba030670fc9d84e3bd51e29fe29b
AA> Author: Aleksey Avdeev <solo@altlinux.ru>
AA> Date:   Fri Jul 23 23:24:04 2010 +0400

AA>      - [ZBXNEXT-451] added ability to use items from the children node in the calculated items on the master node (solo)
    
AA>     Signed-off-by: Aleksey Avdeev <solo@altlinux.ru>

AA> commit 030707ce2084ee80bcb8ca6fa65aa1de8320532b
AA> Author: Aleksey Avdeev <solo@altlinux.ru>
AA> Date:   Fri Jul 23 19:40:32 2010 +0400

AA>      - [ZBX-2751] fixed return the empty string as the hostname when parsing (solo)
    
AA>     Signed-off-by: Aleksey Avdeev <solo@altlinux.ru>

Не стоит писать собственные изменения в файл ChangeLog апстрима не состоя в
этом самом апстриме. Для этого есть changelog пакета.

-- 
vvk

Re: [devel] [SCM] packages/zabbix: heads/master

[Aleksey Avdeev]

[-- Attachment #1: Type: text/plain, Size: 2477 bytes --]

22.10.2010 13:59, Vladimir V. Kamarzin пишет:
>>>>>> On 06 Aug 2010 at 22:22 "AA" == Aleksey Avdeev writes:
> 
> AA> Update of /people/solo/packages/zabbix.git
> 
> AA> commit 409a3805cd321c58db45cdc258920d42ce0fc62c
> AA> Author: Aleksey Avdeev <solo@altlinux.ru>
> AA> Date:   Fri Aug 6 17:47:54 2010 +0400
> 
> AA>      - [ZBX-416,ZBX-986] fixed problem of implicit cast with PostgreSQL 8.3, thanks to Fernando Ike de Oliveira (solo)
>     
> AA>     Signed-off-by: Aleksey Avdeev <solo@altlinux.ru>
> 
> 
> AA> commit c141437c4aa2ba030670fc9d84e3bd51e29fe29b
> AA> Author: Aleksey Avdeev <solo@altlinux.ru>
> AA> Date:   Fri Jul 23 23:24:04 2010 +0400
> 
> AA>      - [ZBXNEXT-451] added ability to use items from the children node in the calculated items on the master node (solo)
>     
> AA>     Signed-off-by: Aleksey Avdeev <solo@altlinux.ru>
> 
> AA> commit 030707ce2084ee80bcb8ca6fa65aa1de8320532b
> AA> Author: Aleksey Avdeev <solo@altlinux.ru>
> AA> Date:   Fri Jul 23 19:40:32 2010 +0400
> 
> AA>      - [ZBX-2751] fixed return the empty string as the hostname when parsing (solo)
>     
> AA>     Signed-off-by: Aleksey Avdeev <solo@altlinux.ru>
> 
> Не стоит писать собственные изменения в файл ChangeLog апстрима не состоя в
> этом самом апстриме. Для этого есть changelog пакета.
> 

  В апстримном репозитории все что касается вещей зарегистрированных в
трекере (имеющим номер ZBX/ZBXNEXT) -- отмечено в ChangeLog`е. Т. к. все
обсуждаемые патчи (т. е. те что добавлены в ChangeLog) были предложены
мной апстриму (в соответствующих записях трекира), я старался их
оформить аналогично: с выделением отдельной ветки, содержащий патч, и
описанием сути патча в ChangeLog`е... (Темболие, что есть пара патчей
расширяющих синтакс, и конфигурации данные расширения использующие на
непатченом zabbix`работать не будут).

  Если апстрим не для принятия патча не требует описания его сути а
ChangeLog`е -- правки ChangeLog`а нужно удалить (они оформлены
отдельными коммитами, если я правильно помню).  (Оно мне же легче, т. к
позволит реже свитить мой кривой инглиш. ;-)) Но при этом желательно
придумать механизм который позволит администратору смотреть изменения
наряду с апстримными. ИМХО, changelog`а пакета для этого не достаточно:
там упомянуты только наши (ALT) изменения, изменения а апстрима
переведены без подробностей (в виде ссвлки на версию и/или номера svn
коммита).

-- 

С уважением. Алексей.



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 554 bytes --]

Re: [devel] [SCM] packages/zabbix: heads/master

[Vladimir V. Kamarzin]

>>>>> On 23 Oct 2010 at 16:32 "AA" == Aleksey Avdeev writes:

AA>   В апстримном репозитории все что касается вещей зарегистрированных в
AA> трекере (имеющим номер ZBX/ZBXNEXT) -- отмечено в ChangeLog`е. Т. к. все
AA> обсуждаемые патчи (т. е. те что добавлены в ChangeLog) были предложены
AA> мной апстриму (в соответствующих записях трекира), я старался их
AA> оформить аналогично: с выделением отдельной ветки, содержащий патч, и
AA> описанием сути патча в ChangeLog`е... (Темболие, что есть пара патчей
AA> расширяющих синтакс, и конфигурации данные расширения использующие на
AA> непатченом zabbix`работать не будут).

AA>   Если апстрим не для принятия патча не требует описания его сути а
AA> ChangeLog`е -- правки ChangeLog`а нужно удалить (они оформлены
AA> отдельными коммитами, если я правильно помню). 

Удалил. Это были не отдельные коммиты, так что без revert-а.

AA> (Оно мне же легче, т. к
AA> позволит реже свитить мой кривой инглиш. ;-)) Но при этом желательно
AA> придумать механизм который позволит администратору смотреть изменения
AA> наряду с апстримными. ИМХО, changelog`а пакета для этого не достаточно:
AA> там упомянуты только наши (ALT) изменения, изменения а апстрима
AA> переведены без подробностей (в виде ссвлки на версию и/или номера svn
AA> коммита).

Апстримные изменения публикуются в release notes
http://www.zabbix.com/rn1.8.3.php каждой версии. А вот что ChangeLog не
упакован, возможно дейтсвительно не очень хорошо, так как для svn-сборок
release notes на сайте нету. Запакую его в doc.

Почему changelog-а пакета вдруг недостаточно, я не понимаю: для всех остальных
пакетов его достаточно, а для zabbix вдруг стало мало.

-- 
vvk